Модульное тестирование MVP с использованием mockito со слушателями событий

Это сводится к разнице между историей пользователя и прецедентом. В этом случае, у вас есть 1 пользовательская история (например, "как пользователь, я хочу войти, так что я предоставить свой логин и пароль"), но на самом деле существует как минимум 3 Варианты использования: правильный логин/пароль, логин правильный/неправильный пароль, неверный логин/пароль, и т. д. В качестве общей передовой практики вы хотите, чтобы тесты соответствовали 1: 1 с вариантами использования, поэтому я бы рекомендовал что-то вроде этого:

 @Test
 public void shouldCompleteWithValidCredentials() {
    mMockModelContract.login("steve", "1234", 
                              mMockOnLoginCompletedListener);

    verify(mMockOnLoginCompletedListener, times(1)).onSuccess();     
 }

 @Test
 public void shouldNotCompleteWithInvalidUser() {
    mMockModelContract.login("wrong_user", "1234",
                               mMockOnLoginCompletedListener);
    verify(mMockOnLoginCompletedListener, 
                            times(1)).onUsernameError();      
 }

@Test
public void shouldNotCompleteWithInvalidPassword() {
    mMockModelContract.login("steve", "wrong_password", 
                         mMockOnLoginCompletedListener);
    verify(mMockOnLoginCompletedListener, times(1)).onPasswordError();
}

In другими словами, Для теста 1 вы пытаетесь положительно проверить, что после завершения имени пользователя и пароля вызывается успех. Для теста 2 вы проверяете условия для вызова onUsernameError, а для 3-условия для onPasswordError. Все три являются допустимыми вещами для тестирования, и вы правы, что хотите проверить, что они называются, но вам нужно рассматривать их как разные варианты использования.

для полноты я бы проверил, что происходит на Wrong_User/Wrong_Password, а также проверьте, что происходит, если есть условие Wrong_Password N раз (вам нужна блокировка учетной записи?).

надеюсь, что это помогает. Удача.

Я думаю, потому что вы издеваетесь над LoginModelContract и OnLoginCompletedListener вы не можете утверждать, что onUsernameError, onPasswordError и onSuccess на самом деле называются, потому что, издеваясь LoginModelContract" реальный " метод входа в систему (который должен вызывать эти методы) не будет выполнен, но будет вызван только издевательский метод. Вы можете запустить эти методы с чем-то вроде:

Mockito.doAnswer(new Answer<Void>() {
    @Override
    public Void answer(InvocationOnMock invocation) throws Throwable {
        Object[] args = invocation.getArguments();
        OnLoginCompletedListener listener = (OnLoginCompletedListener) args[2];
        listener.onUsernameError();
        return null;
    }
}).when(mMockModelContract).login(anyString(), anyString(), any(OnLoginCompletedListener.class)).thenAnswer();

но причина такого теста не имеет смысла, потому что вы явно вызываете то, что вы пытаетесь тест.

на мой взгляд, было бы разумнее просто проверить LoginModelContract без LoginFragmentViewContract и LoginPresenterContract. Что-то вроде:

public class LoginPresenterImpTest {
    private LoginModelContract mMockModelContract;
    private LoginModelContract.OnLoginCompletedListener mMockOnLoginCompletedListener;

    @Before
    public void setUp() throws Exception {
        mMockOnLoginCompletedListener = Mockito.mock(LoginModelContract.OnLoginCompletedListener.class);
        mMockModelContract = new LoginModelContract();
    }

    @Test
    public void shouldSuccessWithValidCredentials() {
        mMockModelContract.login("steve", "1234", mMockOnLoginCompletedListener);

        verify(mMockOnLoginCompletedListener, times(1)).onSuccess();

        verify(mMockOnLoginCompletedListener, never()).onPasswordError();
        verify(mMockOnLoginCompletedListener, never()).onUsernameError();
    }
}

Я мог бы пропустить вашу точку зрения, но вы пробовали использовать PowerMock?

вам понадобятся следующие зависимости:

• testCompile " org.powermock: powermock-модуль-junit4: 1.6.5"
• testCompile " org.powermock: powermock-модуль-junit4-правило: 1.6.5"
• testCompile " org.powermock: powermock-api-mockito: 1.6.5"
• testCompile " org.powermock:powermock-classloading-xstream: 1.6.5"

и затем использовать его таким образом:

@PowerMockIgnore({ "org.mockito.*", "android.*" })
@PrepareForTest(DownloadPresenterContract.Events.class)
public class DownloadModelTest {

    @Rule
    public PowerMockRule rule = new PowerMockRule();

    private DownloadPresenterContract.Events mockEvents;

    @Before
    public void setUp() throws Exception {
         this.mockEvents = PowerMockito.spy(new DownloadPresenterContract.Events());

         PowerMockito.whenNew(DownloadPresenterContract.Events.class)
                     .withNoArguments()
                     .thenReturn(this.mockEvents); 
    }

    @Test
    public void testStaticMocking() {

         //Do your logic, which should trigger mockEvents actions

         Mockito.verify(this.mockEvents, Mockito.times(1)).onDownloadSuccess();
         //Or use this:
         //PowerMockito.verifyPrivate(this.mockEvents, times(1)).invoke("onDownloadSuccess", "someParam");
}

}