Единично тестване с mockito и шпиониране, причиняващо грешка

Проблемът в кода е тази част: subscribeOn(Schedulers.io()). Само ако можехме да елиминираме това, тогава ще бъде добре да върнем тестови данни от recipesAPI и да проверим дали тези данни са били обработени правилно от recipeGetAllListener.

Така че трябва по някакъв начин да създадем шев: ако това е производствен код - тогава използвайте Schedulers.io()/AndroidSchedulers.mainThread(), ако това е тестов код - тогава използвайте някакъв конкретен планировчик.

Нека декларираме интерфейс, който ще предостави Schedulers:

    interface SchedulersProvider {
        Scheduler getWorkerScheduler();
        Scheduler getUiScheduler();
    }

Сега нека накараме RecipeListModelImp да има зависимост от SchedulersProvider:

    public class RecipeListModelImp implements RecipeListModelContract {

        ...
        private SchedulersProvider schedulersProvider;

        @Inject
        public RecipeListModelImp(@NonNull RecipesAPI recipesAPI, 
                                  @NonNull SchedulersProvider schedulerProvider) {
            ...
            this.schedulersProvider = schedulersProvider;
        }

        ...
    }

Сега ще заменим планировчиците:

    @Override
    public void getRecipesFromAPI(final RecipeGetAllListener recipeGetAllListener) {
        subscription = recipesAPI.getAllRecipes()
                                 .subscribeOn(schedulersProvider.getWorkerScheduler())
                                 .observeOn(schedulersProvider.getUiScheduler())
                                 ...
    }

Време е да се погрижим за предоставянето на SchedulerProvider:

    @Module
    public class MyModule {
        ...
        @Provides public SchedulerProvider provideSchedulerProvider() {
            return new SchedulerProvider() {
                @Override
                Scheduler getWorkerScheduler() {
                    return Schedulers.io();
                }

                @Override
                Scheduler getUiScheduler() {
                    return AndroidSchedulers.mainThread();
                }
            }
        }
    }

Сега нека създадем друг модул - TestModule, който ще предостави зависимости за тестови класове. TestModule ще разшири MyModule и ще замени метода, предоставил SchedulerProvider:

    public class TestModule extends MyModule {
        @Override public SchedulerProvider provideSchedulerProvider() {
            return new SchedulerProvider() {
                @Override
                Scheduler getScheduler() {
                    return Schedulers.trampoline();
                }

                @Override
                Scheduler getUiScheduler() {
                    return Schedulers.trampoline();
                }
            }
        }
    }

Schedulers.trampoline() ще изпълнява задачи в текущата нишка.

Време е да създадете тестов компонент:

    @Component(modules = MyModule.class)
    public interface TestComponent extends MyComponent {
        void inject(RecipeListModelImpTest test);
    }

Сега в тестовия клас:

    public class RecipeListModelImpTest {

        @Mock RecipesAPI recipesAPI;
        @Mock RecipeListModelContract.RecipeGetAllListener recipeGetAllListener;

        @Inject SchedulerProvider schedulerProvider;

        private RecipeListModelContract recipeListModel;

        @Before
        public void setup() {
            TestComponent component = DaggerTestComponent.builder()
                                                         .myModule(new TestModule())
                                                         .build();
            component.inject(this);

            MockitoAnnotations.initMocks(this);
            recipeListModel = new RecipeListModelImp(recipesAPI, schedulerProvider);
        }
        ...
    }

И същинската тестова част:

    private static final List<Recipe> TEST_RECIPES = new ArrayList<Recipe>() {
        {
            add(new Recipe(1)),
            add(new Recipe(2))
        }
    };

    @Test
    public void testGetRecipesFromAPI() {
        when(recipeAPI.getAllRecipes())
            .thenReturn(Observable.fromIterable(TEST_RECIPES));

        recipeListModel.getRecipesFromAPI(recipeGetAllListener);

        // verifying, that `recipeAPI.getAllRecipes()` has been called once
        verify(recipeAPI).getAllRecipes();

        // verifying, that we received correct result
        verify(recipeGetAllListener).onRecipeGetAllSuccess(TEST_RECIPES);
    }

както си написал

абонамент = recipesAPI.getAllRecipes().subscribeOn(Schedulers.io())

тогава методът getAllRecipes() връща някакъв обект и не можете да използвате

doNothing().when(recipeApiSpy).getAllRecipes();

doNothing() - това е за метод, който връща void.

вариантът е правилен:

doReturn(doReturn(Observable.just(Subscription.class)).when(recipeApiSpy).getAllRecipes()

Вие смесвате Spy (частични подигравки) и Mocks (пълни подигравки). Това не е необходимо - Spy ви позволява да смесвате подигравателни и реални извиквания на методи, но не се нуждаете от частично подиграване. В твоя случай или се подиграваш напълно, или не се подиграваш. документация на Mockito има повече информация за подигравки и шпиониране.

В първия ви пример грешката е, че се опитвате да doNothing на метод, който връща нещо. Mockito не позволява това. Това, което правехте във втория си пример, беше почти правилно.

За вашия втори пример, проблемът е, че сте настроили getAllRecipes() да връща Observable.just(Subscription.class), но все още имате цялата верига от методи, които се извикват за това в тествания модул: subscribeOn, observeOn и subscribe. Трябва да се подиграете и на тези извиквания, за да върнете подигравани обекти, с които можете да работите, или тези извиквания с хвърляне на NullPointerException.

@Test
public void testGetRecipesFromAPI() { 
    //recipesAPI.getAllRecipes() needs to be mocked to return something (likely a mock)
    // so subscribeOn can be called.
    //That needs to be mocked to return something so observeOn can be called
    //etc.

    recipeListModel.getRecipesFromAPI(recipeGetAllListener);

    verify(recipesAPI, times(1)).getAllRecipes();
}