Mockito - переопределение метода, который принимает примитивные параметры

Question

У меня есть класс, который является подклассом Context. Я тестирую единицы другого класса, которые имеют отношение к этому классу, и поэтому я издевался над ним. Тем не менее, мне нужны некоторые методы, чтобы действовать как их первоначальное поведение, поэтому я собираюсь «развязать» их.

Один из них getAssets(), так что я написал это, и она работает правильно:

Mockito.doReturn(this.getContext().getAssets()).when(keyboard).getAssets(); 

keyboard является издевались экземпляр класса упоминается.

Поскольку этот метод не принимает параметров, переопределение его было довольно простым.

Мне также необходимо переопределить Context.getString(int). Параметр усложняет ситуацию и является примитивным, что еще более затрудняет задачу.

Я взял this advice и еще один, и попытался писать этот код:

Mockito.when(keyboard.getString(Mockito.anyInt())).thenAnswer(new Answer<String>(){ 
    @Override 
    public String answer(InvocationOnMock invocation) throws Throwable 
     Integer arg = (Integer)invocation.getArguments()[0]; 
     return OuterClass.this.getContext().getString(arg.intValue()); 
    } 
}); 

Это компилируется и выполняется, но дал следующее исключение:

org.mockito.exceptions.misusing.InvalidUseOfMatchersException: 
Invalid use of argument matchers! 
0 matchers expected, 1 recorded: 
-> at [...] <The code above> 

This exception may occur if matchers are combined with raw values: 
//incorrect: 
someMethod(anyObject(), "raw String"); 
When using matchers, all arguments have to be provided by matchers. 
For example: 
//correct: 
someMethod(anyObject(), eq("String by matcher")); 

For more info see javadoc for Matchers class. 

at android.content.Context.getString(Context.java:282) 
at [...] 
at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:169) 
at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:154) 
at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:545) 
at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1551) 

Так что главный вопрос заключается в том, чтобы переопределить методы в Мокито, которые имеют примитивные параметры?

Заранее спасибо

Answer 1

Вы не можете заглушить getString, потому что это окончательный. Мокито не может заглушить окончательные методы. Просто оставьте его ненакрашенным, и вы получите его первоначальную реализацию. Это то, чего ты хочешь, да?

Answer 2

Не должна ли она быть Answer<String>?

Mockito.when(keyboard.getString(Mockito.anyInt())).thenAnswer(new Answer<String>(){ 
    @Override 
    public String answer(InvocationOnMock invocation) throws Throwable { 
     return "foo"; 
    } 
}); 

И вы могли бы переименовать клавиатуру -> mockContext, это было бы более ясно?

Answer 3

Как David Wallace, указанным в his answer, метод Context.getString() является окончательным, и мы знаем, что mockito не может высмеять окончательные методы. Так что переопределение это тоже плохая идея, и она не сработает.

"0 matchers expected, 1 recorded" информация о описании InvalidUseOfMatchersException, является смутным способом сказать, что метод final, и мы не должны пытаться его переопределить.

Однако оставить его как есть не поможет, потому что объект все еще является издеваемым объектом, и метод не покажет ожидаемое поведение, которое мы хотим от обычного объекта; поэтому этот параметр также находится вне таблицы.

Наконец-то я решил изменить и улучшить свой дизайн. Я использовал Interface segregation principle и разделил обязанности Context и Keyboard (Я разрабатываю службу IME). В самой заявке я передал один и тот же объект keyboard для этих двух, но в тесте я высмеивал Keyboard без каких-либо плохих побочных эффектов и использовал TestCase.getContext() как Context, что устранило необходимость «разблокировать» его методы.