Mockito, gdy(). ThenReturn wywołuje metodę niepotrzebnie

Question

Pracuję trochę nad odziedziczonym kodem. pisałem test, który ma złapać NullPointerException (bo próbuje wywołać metodę od null object)

@Test(expected=NullPointerException.class) 
public void checkXRequirement_NullProduct_AddAction_ShouldThrowNullPointerException() throws CustomException { 
    Site site = mock(Site.class); 
    Product product = null; 
    when(BasketHelper.getAction(request)).thenReturn(0); 
    when(BasketHelper.getActionProduct(site, request)).thenReturn(product); 
    BasketHelper.requiresX(request, site); 

} 

odpowiednich metod i zmiennych:

public static final int ACTION_ADD = 0; 
public static final int ACTION_DELETE = 1; 

protected static int getAction(HttpServletRequest a_request) { 
    String sBuyProduct = a_request.getParameter(ATTRIBUTE_NAME_BUY_PRODUCT); 
    String sBuyProduct = a_request.getParameter(ATTRIBUTE_NAME_BUY_PRODUCT); 

    if (sBuyProduct != null) iAction = ACTION_ADD; 
    else (sDelProduct != null) iAction = ACTION_DELETE; 

    return iBasketAction 
} 

protected static Product getActionProduct(Site a_site, HttpServletRequest a_request) { 

    String sBuyProduct = a_request.getParameter(ATTRIBUTE_NAME_BUY_PRODUCT); 
    String sDelProduct = a_request.getParameter(ATTRIBUTE_NAME_DEL_PRODUCT); 
    String sProduct = null; 

    switch (getBasketAction(a_request)) { 
     case BASKET_ACTION_ADD: 
     sProduct = sBuyProduct; 
    break; 
     case BASKET_ACTION_DELETE: 
     sProduct = sDelProduct; 
    break; 
    } 

    int iProductId; 
    try { 
     iProductId = Integer.parseInt(sProduct); 
    } catch (NumberFormatException nbrEx) { 
     return null; 
    } 

    Product prod = getProductById(iProductId); 

    if (prod.isMasterProduct()) { 
     prod = getChildProduct(prod, a_site, a_request); 
    } 

    return prod; 
} 


public static boolean requiresX(HttpServletRequest request, Site site) throws CustomException { 
    try{ 
    if (getAction(request) == ACTION_ADD) { 
    Product prod = getActionProduct(site, request); 
    return prod.getType().isRequiredX(); 
    } 
    } catch(NullPointerException exception) { 
    log.error("Error Message", exception); 
    } 
    return false; 
} 

Wynikiem jUnit od biegania test jest porażką z ślad stosu:

java.lang.Exception: Unexpected exception, expected<java.lang.NullPointerException> but was<org.mockito.exceptions.misusing.WrongTypeOfReturnValue> 
Caused by: org.mockito.exceptions.misusing.WrongTypeOfReturnValue: 
Integer cannot be returned by getParameter() 
getParameter() should return String# 

Czy ja źle zinterpretować jak gdy() thenReturn ma pracować on. re? Chcę tylko, aby getAction zwracało 0, a getActionProduct zwracało wartość null, gdy jest wywoływane. Najwyraźniej funkcja getParameter() jest wywoływana i nie wiem, dlaczego dokładnie.

Answer 1

Mockito nie może symulować metody statycznej. Twoje, gdy sprawdzenie jest nieważne:

when(BasketHelper.getAction(request)).thenReturn(0); 
    when(BasketHelper.getActionProduct(site, request)).thenReturn(product); 

To kolejny powód, dla którego chcemy ograniczyć stosowanie metody statycznej, ponieważ trudno jest kpić.

Nie ma łatwiejszego sposobu na wyśmiewanie się z zachowania, jeśli twoja klasa pozostaje w ten sposób. Jednak jeśli chcesz zmienić swój projekt i uczynić obie metody niestatycznymi. Prawidłowym sposobem użycia "kiedy" jest zastosowanie sprawdzenia na wyśmiewanym obiekcie. Na przykład:

BasketHelper basketHelper = mock(BasketHelper.class); 
    when(basketHelper.getAction(request)).thenReturn(0); 
    when(basketHelper.getActionProduct(site, request)).thenReturn(product); 

Ale znowu, to tylko praca, jeśli ponownie zaprojektowany getAction i getProduct metodę swojej klasy, aby mieć charakteru statycznego.

Pamiętam, że istnieją pewne inne ramy testowe, które wspierają szyderstwo metodą statyczną.

Answer 2

Możesz użyć PowerMock. Najpierw utworzyć mock klasy, na którym jest wywoływanie metod statycznych -

mockStatic(BasketHelper.class); 

Następnie zdefiniować odcinki -

when(BasketHelper.getAction(request)).thenReturn(0); 
when(BasketHelper.getActionProduct(site, request)).thenReturn(product); 

Answer 3

To może pomóc innym, którzy używają adnotacji. Jeśli używasz adnotacji, może być konieczne użycie @Mock zamiast @InjectMocks. Ponieważ @InjectMocks działa jako @Spy i @Mock razem. A @Spy śledzi ostatnio wykonane metody i możesz poczuć, że nieprawidłowe dane są zwracane/podporządkowane. Sprawdź te dwa:

https://groups.google.com/forum/?fromgroups#!topic/mockito/9WUvkhZUy90

http://code.google.com/p/mockito/issues/detail?id=127

Answer 4

natknąłem tego wątku, starając się rozwiązać ten sam problem w moich testów.

W przypadku, gdy inni widzą ten problem i kończą tutaj ... W moim przypadku było to spowodowane brakiem adnotacji @PrepareForTest dla klasy wsparcia.