重构代码以不抛出 Mockito 的 UnneedStubbingException

Question

>我有一个简单的视图，带有按钮，该按钮Intent.ACTION_PICK开始以获取结果，然后在屏幕上显示选定的联系人。为此，必须采取以下步骤：

• 检查是否授予android.permission.READ_CONTACTS
• 开放联系人活动
• 选择联系人并返回应用
• 再次检查android.permission.READ_CONTACTS
• 按给定 URI 查找联系人
• 在屏幕上显示联系人

我想测试一个打开联系人而不是撤销权限并返回具有选定联系人的应用程序的情况。预期结果是不调用通过其 uri 查找联系人的方法。

不幸的是，当前的实现抛出了：

org.mockito.exceptions.misusing.UnneedStubbingException：

为：

whenever(interactor.getContact(any())).thenReturn(Maybe.just(Contact()).doOnSuccess { find = true })

我知道我可以用Silent替换StrictStubs，但我正在寻找重构当前代码的更好解决方案。

所有必要的课程和测试：

class Contact
interface View {
val contactClicks: Observable<Any>
fun setContact(contact: Contact)
}
interface Interactor {
fun getContact(uri: String): Maybe<Contact>
}
interface Router {
fun goToContacts(): Maybe<String>
}
interface Permissioner {
fun requestReadContacts(): Single<Boolean>
}
class Presenter(
private val view: View,
private val interactor: Interactor,
private val router: Router,
private val permissioner: Permissioner
) {
private val disposables: CompositeDisposable = CompositeDisposable()
fun bindView() {
view.contactClicks
.flatMapSingle { permissioner.requestReadContacts() } //ask first time before opening contacts
.filter { it }
.flatMapMaybe { router.goToContacts() }
.flatMapMaybe {
permissioner.requestReadContacts() //ask second time before using ContentResolver
.filter { granted -> granted }
.flatMap { _ -> interactor.getContact(it) }
}
.subscribeBy { view.setContact(it) }
.addTo(disposables)
}
}
@RunWith(MockitoJUnitRunner.StrictStubs::class)
class PresenterTest {
@Mock
lateinit var view: View
@Mock
lateinit var router: Router
@Mock
lateinit var permissioner: Permissioner
@Mock
lateinit var interactor: Interactor
@InjectMocks
lateinit var presenter: Presenter
private val contactClickSubject = PublishSubject.create<Any>()
@Before
fun setUp() {
whenever(view.contactClicks).thenReturn(contactClickSubject)
}
@Test
fun shouldNotFindContactWhenReturnedWithUriAndPermissionNotGrantedSecondTime() {
var firstTimeAsk = true
whenever(permissioner.requestReadContacts()).thenReturn(Single.fromCallable {
if (firstTimeAsk) {
firstTimeAsk = false
return@fromCallable true
} else {
return@fromCallable false
}
})
whenever(router.goToContacts()).thenReturn(Maybe.just("contact"))
var find = false
whenever(interactor.getContact(any())).thenReturn(Maybe.just(Contact()).doOnSuccess { find = true })
presenter.bindView()
contactClickSubject.onNext(Any())
assertFalse(find)
}
}

Answer 1

>UnnecessaryStubbingException意味着你正在存根某些东西，但并没有真正使用它。这是正确的，在您的情况下interactor.getContact永远不应该在测试中调用 - 这是期望的行为。所以把它存根是没有意义的。

最简单的解决方案是删除不必要的变量var find = false和存根 - 在测试结束时用断言替换它们：

verify(interactor, never()).getContact(any())

这等效于您当前的解决方案，但比使用帮助程序变量更简单。