Question

直到昨天，我一直使用MVP模式和Android Annotations library成功组建了一个非常易读的Android项目。

但是昨天当我开始为我的LoginPresenter编写unittest时，问题就出现了。

来自我的LoginPresenter的第一段代码。

...
@EBean
public class LoginPresenterImpl implements LoginPresenter, LoginInteractor.OnLoginFinishedListener {

  @RootContext
  protected LoginActivity loginView;

  @Bean(LoginInteractorImpl.class)
  LoginInteractor loginInteractor;

  @Override public void validateCredentials(String username, String password) {
    if (loginView != null) {
        loginView.showProgress();
    }

    if (TextUtils.isEmpty(username)) {
        // Check that username isn't empty
        onUsernameError();
    }
    if (TextUtils.isEmpty(password)){
        // Check that password isn't empty
        onPasswordError();
        // No reason to continue to do login
    } else {

    }
}

  @UiThread(propagation = UiThread.Propagation.REUSE)
  @Override public void onUsernameError() {
    if (loginView != null) {
        loginView.setUsernameError();
        loginView.hideProgress();
    }
}

...

我的测试：

@RunWith(MockitoJUnitRunner.class)
public class LoginPresenterImplTest {

  private LoginPresenter loginPresenter;

  @Mock
  private LoginPresenter.View loginView;

  @Before
  public void setUp() {
      // mock or create a Context object
      Context context = new MockContext();

      loginPresenter = LoginPresenterImpl_.getInstance_(context);

      MockitoAnnotations.initMocks(this);
  }

  @After
  public void tearDown() throws Exception {
      loginPresenter = null;

  }

  @Test
  public void whenUserNameIsEmptyShowUsernameError() throws Exception {
      loginPresenter.validateCredentials("", "testtest");

      // verify(loginPresenter).onUsernameError();
      verify(loginView).setUsernameError();
  }
}

问题是我没有使用过使用MVP模式的标准方法，而是尝试使用Android Annotations来使代码更具可读性。所以我没有使用attachView（） - 或detachView（） - 将我的演示者附加到我的LoginActivity（视图）的方法。这意味着我无法模仿我的“观点”。有人知道这个问题的解决方法。我在运行测试时不断收到消息：

Wanted but not invoked:
loginView.setUsernameError();
-> at      com.conhea.smartgfr.login.LoginPresenterImplTest.whenUserNameIsEmptyShowUsernameError(LoginPresenterImplTest.java:48)
Actually, there were zero interactions with this mock.

解决方案（我不再使用@RootContext）：

主讲人：

@EBean
public class LoginPresenterImpl extends AbstractPresenter<LoginPresenter.View>
        implements LoginPresenter, LoginInteractor.OnLoginFinishedListener {

    private static final String TAG = LoginPresenterImpl.class.getSimpleName();

    @StringRes(R.string.activity_login_authenticating)
    String mAuthenticatingString;

    @StringRes(R.string.activity_login_aborting)
    String mAbortingString;

    @StringRes(R.string.activity_login_invalid_login)
    String mInvalidCredentialsString;

    @StringRes(R.string.activity_login_aborted)
    String mAbortedString;

    @Inject
    LoginInteractor mLoginInteractor;

    @Override
    protected void initializeDagger() {
        Log.d(TAG, "Initializing Dagger injection");
        Log.d(TAG, "Application is :" + getApp().getClass().getSimpleName());
        Log.d(TAG, "Component is: " + getApp().getComponent().getClass().getSimpleName());
        Log.d(TAG, "UserRepo is: " + getApp().getComponent().userRepository().toString());
        mLoginInteractor = getApp().getComponent().loginInteractor();
        Log.d(TAG, "LoginInteractor is: " + mLoginInteractor.getClass().getSimpleName());
    }

    @Override
    public void validateCredentials(String username, String password) {
        boolean error = false;
        if (!isConnected()) {
            noNetworkFailure();
            error = true;
        }
        if (TextUtils.isEmpty(username.trim())) {
            // Check that username isn't empty
            onUsernameError();
            error = true;
        }
        if (TextUtils.isEmpty(password.trim())) {
            // Check that password isn't empty
            onPasswordError();
            error = true;
        }
        if (!error) {
            getView().showProgress(mAuthenticatingString);
            mLoginInteractor.login(username, password, this);
        }
    }
...

我的测试（其中一些）：

@RunWith(AppRobolectricRunner.class)
@Config(constants = BuildConfig.class)
public class LoginPresenterImplTest {

@Rule
public MockitoRule mMockitoRule = MockitoJUnit.rule();

private LoginPresenterImpl_ mLoginPresenter;

@Mock
private LoginPresenter.View mLoginViewMock;

@Mock
private LoginInteractor mLoginInteractorMock;

@Captor
private ArgumentCaptor<LoginInteractor.OnLoginFinishedListener> mCaptor;

@Before
public void setUp() {

    mLoginPresenter = LoginPresenterImpl_.getInstance_(RuntimeEnvironment.application);
    mLoginPresenter.attachView(mLoginViewMock);
    mLoginPresenter.mLoginInteractor = mLoginInteractorMock;
}

@After
public void tearDown() throws Exception {
    mLoginPresenter.detachView();
    mLoginPresenter = null;

}

@Test
public void whenUsernameAndPasswordIsValid_shouldLogin() throws Exception {

    String authToken = "Success";

    mLoginPresenter.validateCredentials("test", "testtest");

    verify(mLoginInteractorMock, times(1)).login(
            anyString(),
            anyString(),
            mCaptor.capture());
    mCaptor.getValue().onSuccess(authToken);
    verify(mLoginViewMock, times(1)).loginSuccess(authToken);
    verify(mLoginViewMock, times(1)).hideProgress();
}

@Test
public void whenUsernameIsEmpty_shouldShowUsernameError() throws Exception {

    mLoginPresenter.validateCredentials("", "testtest");


    verify(mLoginViewMock, times(1)).setUsernameError();
    verify(mLoginViewMock, never()).setPasswordError();
    verify(mLoginViewMock, never()).hideProgress();
}
...

Answer 1

作为一种解决方法，您可以拥有：

public class LoginPresenterImpl ... {

    ...

    @VisibleForTesting
    public void setLoginPresenter(LoginPresenter.View loginView) {
        this.loginView = loginView;
    }

}

在测试课程中：

@Before
public void setUp() {
    ...

    MockitoAnnotations.initMocks(this);
    loginPresenter.setLoginPresenter(loginView);
}

但是，根据经验，当您看到@VisibleForTesting注释时，这意味着您的架构不正确。最好重构你的项目。

Answer 2

向需要在其项目中使用Android注释的开发人员进行操作。在编写单元测试时，请注意您的代码不能访问Android API。 Android Annotations的底层实现在很大程度上依赖于Android API。因此，自动生成的代码可能依赖于此并且难以编写单元测试。

永远记住Android Annotations用最后一个类替换你的类，该类在它的类名末尾加了_。在这个生成的类中，许多样板代码是自动生成的，具体取决于原始类的注释方式。在我的情况下，问题是我正在开发一个Android项目，并希望我的演示者能够在UI线程上运行很多方法。这是使用Android注释使用@UIThread注释实现的。但这意味着我的方法实际上是用另一个调用超类的方法包装的：

@Override
public void onUsernameError() {
    if (Thread.currentThread() == Looper.getMainLooper().getThread()) {
        LoginPresenterImpl_.super.onUsernameError();
        return;
    }
    UiThreadExecutor.runTask("", new Runnable() {

        @Override
        public void run() {
            LoginPresenterImpl_.super.onUsernameError();
        }
    }
    , 0L);
}

我的测试用例无法越过界限：

...
if (Thread.currentThread() == Looper.getMainLooper().getThread()) {
...

这当然是因为我们无法通过简单的单元测试访问Android API。所以存在问题。

结论：在为使用Android Annotations的项目编写单元测试时，您必须非常小心，自动生成的代码不依赖于Android相关的API。

使用androids TextUtil-class时会出现同样的问题。

使用Android Annotations，Mockito和MVP模式进行单元测试

2 个答案: