Dlaczego Mockito pomija inicjalizację zmiennej składowej mojej abstrakcyjnej klasy

Question

Próbuję przetestować klasę abstrakcyjną, a Mockito nie inicjuje moich zmiennych składowych. Oto prosty przykład, aby pokazać ci mój problem.

Jest to klasa abstrakcyjna, która inicjuje jego „pole” członek:

import java.util.ArrayList; 
import java.util.Collection; 

public abstract class Foo { 
    private final Collection field = new ArrayList(); 

    protected Foo() { 
     System.out.println("In constructor"); 
    } 

    public boolean isNull(Object o) { 
     field.add(o); 

     return o == null; 
    } 

    abstract void someAbstractMethod(); 
} 

Tutaj klasa Test:

import org.junit.Assert; 
import org.junit.Test; 
import org.mockito.Mockito; 

public class FooTest { 
    @Test 
    public void testSomething() { 
     final Foo foo = Mockito.mock(Foo.class); 

     Mockito.when(foo.isNull(Mockito.anyObject())).thenCallRealMethod(); 

     Assert.assertFalse(foo.isNull("baaba")); 
    } 
} 

Kiedy test jest go uruchomić rzuca NPE ponieważ pole zmiennej” "nie jest zainicjowany!

Co robię źle?

Answer 1

Jest to oczekiwane zachowanie, gdy kpisz z czegoś, co stworzona instancja jest kompletną próbą, więc nie ma sensu inicjowanie pól jako domyślnego ustawienia .

Oprócz tego, pola mogą być inicjowane przez konstruktor w klasach betonowych lub abstrakcyjnych, ponieważ tworzenie makr omija konstruktora po prostu dlatego, że jest to makiety, jeszcze bardziej irracjonalne jest ich inicjowanie.

Próba wywoływania prawdziwej metody jest zwykle niewłaściwa podczas korzystania z mocks. Zamiast tego należy zaprzestać zachowania fałszywego.

Mockito.when(foo.isNull(Mockito.anyObject())).thenReturn(false); 
Assert.assertFalse(foo.isNull("baaba")); // assertion always passing 

nie wiem rzeczywisty przypadek użycia, ale może chcesz częściową makiety, z spy. Mimo że jest to nadal uważane za złą praktykę, ponieważ zazwyczaj oznacza to, że musisz zmienić kod, aby użyć kompozycji.

Answer 2

Wygląda na to, że kpisz z klasy, którą faktycznie próbujesz sprawdzić. To nie jest naprawdę kpina. Kpisz z zajęć, które nie mieszczą się w zakresie twojego testu, i pozostawiasz przedmiot, który testujesz, niezmieniony.

W takim przypadku powinieneś po prostu utworzyć Foo, z fałszywą implementacją someAbstractMethod i przetestować to bezpośrednio. Nie ma potrzeby na żadne kpiny, o ile widzę.