Модульное тестирование с 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() возвращает некоторый объект, и вы не можете использовать

делатьНичего(). Когда(recipeApiSpy).getAllRecipes();

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

вариант правильный:

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

Вы смешиваете Spy (частичные макеты) и Mocks (полные макеты). В этом нет необходимости — Spy позволяет вам смешивать насмешливые и реальные вызовы методов, но вам не нужны какие-либо частичные насмешки. В вашем случае вы либо полностью издеваетесь, либо не издеваетесь. документация от Mockito содержит больше информация о насмешках и шпионаже.

В вашем первом примере ошибка заключается в том, что вы пытаетесь doNothing использовать метод, который что-то возвращает. Мокито не позволяет этого. То, что вы делали во втором примере, было почти правильно.

Для вашего второго примера проблема заключается в том, что вы настроили 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();
}