Блог

Главная - Вопросы по программированию - NPE при втором вызове Mockito.при использовании пользовательских средств сопоставления аргументов

NPE при втором вызове Mockito.при использовании пользовательских средств сопоставления аргументов

#java #unit-testing #mockito #junit4

#java #модульное тестирование #mockito #junit4

Вопрос:

Добрый день.

Предположим, у меня есть интерфейс. Интерфейс имеет метод, который принимает один или два экземпляра некоторых классов. В классе модульного тестирования я издеваюсь над этим интерфейсом и хочу издеваться над ним с помощью подобной логики:

  1. Если аргумент соответствует шаблону1, то возвращается значение1
  2. Если аргумент совпадает с template2, тогда верните значение2

Когда я имитирую только первый случай, тест выполняется гладко. Когда я добавляю второй случай (наряду с первым), тест завершается сбоем с помощью NPE, например:

 java.lang.NullPointerException
at org.some.pkg.TestTest$ArgMatcher.matches(TestTest.java:54)
at org.some.pkg.TestTest$ArgMatcher.matches(TestTest.java:43)
at org.mockito.internal.invocation.TypeSafeMatching.apply(TypeSafeMatching.java:24)
at org.mockito.internal.invocation.MatcherApplicationStrategy.forEachMatcherAndArgument(MatcherApplicationStrategy.java:83)
at org.mockito.internal.invocation.InvocationMatcher.argumentsMatch(InvocationMatcher.java:152)
at org.mockito.internal.invocation.InvocationMatcher.matches(InvocationMatcher.java:81)
at org.mockito.internal.stubbing.InvocationContainerImpl.findAnswerFor(InvocationContainerImpl.java:91)
at org.mockito.internal.handler.MockHandlerImpl.handle(MockHandlerImpl.java:87)
at org.mockito.internal.handler.NullResultGuardian.handle(NullResultGuardian.java:29)
at org.mockito.internal.handler.InvocationNotifierHandler.handle(InvocationNotifierHandler.java:35)
at org.mockito.internal.creation.bytebuddy.MockMethodInterceptor.doIntercept(MockMethodInterceptor.java:63)
at org.mockito.internal.creation.bytebuddy.MockMethodInterceptor.doIntercept(MockMethodInterceptor.java:49)
at org.mockito.internal.creation.bytebuddy.MockMethodInterceptor$DispatcherDefaultingToRealMethod.interceptAbstract(MockMethodInterceptor.java:128)
at org.some.pkg.Interface$MockitoMock$1549778145.callMe(Unknown Source)
at org.some.pkg.TestTest.test(TestTest.java:29)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.mockito.internal.runners.DefaultInternalRunner$1.run(DefaultInternalRunner.java:79)
at org.mockito.internal.runners.DefaultInternalRunner.run(DefaultInternalRunner.java:85)
at org.mockito.internal.runners.StrictRunner.run(StrictRunner.java:39)
at org.mockito.junit.MockitoJUnitRunner.run(MockitoJUnitRunner.java:163)
at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:68)
at com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:47)
at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:242)
at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70)

[MockitoHint] TestTest.test (see javadoc for MockitoHint):
[MockitoHint] 1. Unused... -> at org.some.pkg.TestTest.test(TestTest.java:24)
[MockitoHint]  ...args ok? -> at org.some.pkg.TestTest.test(TestTest.java:29)


Process finished with exit code 255

Ниже приведен пример настройки.

Файл: Interface.java — интерфейс для макетирования

 package org.some.pkg;

public interface Interface {
    public class Arg {
        private String str;
        private int v;

        public Arg(String str, int v) {
            this.str = str;
            this.v = v;
        }

        public String getStr() {
            return str;
        }

        public int getV() {
            return v;
        }
    }

    int callMe(Arg a1, Arg a2);
}

Файл: TestedClass.java — некоторый класс делегата (фактически протестированный)

 package org.some.pkg;

import org.springframework.beans.factory.annotation.Autowired;

public class TestedClass {
    @Autowired
    Interface delegate;

    int callMe(Interface.Arg a1, Interface.Arg a2) {
        int x = delegate.callMe(a1, a2);

        return x * 2;
    }
}

Файл: TestTest.java — тест

 package org.some.pkg;

import org.junit.Assert;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.ArgumentMatcher;
import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.junit.MockitoJUnitRunner;

import static org.mockito.ArgumentMatchers.argThat;
import static org.mockito.Mockito.when;

@RunWith(MockitoJUnitRunner.class)
public class TestTest {
    @Mock
    private Interface iface;

    @InjectMocks
    private TestedClass testedClass;

    @Test
    public void test() {
        // when-1
        when(iface.callMe(
                argThat(new ArgMatcher("start1", 10)),
                argThat(new ArgMatcher("start2", 5))
        )).thenReturn(10);

        // when-2
        when(iface.callMe(
                argThat(new ArgMatcher("start3", 5)),
                argThat(new ArgMatcher("start4", 2))
        )).thenReturn(3);

        // test
        int v = testedClass.callMe(
                new Interface.Arg("start1 suffix", 3),
                new Interface.Arg("start2, suffix", 4));

        System.out.println("V = "   v);

        Assert.assertEquals(20, v);
    }

    class ArgMatcher implements ArgumentMatcher<Interface.Arg> {
        private final String start;
        private final int threshold;

        public ArgMatcher(String start, int threshold) {
            this.start = start;
            this.threshold = threshold;
        }

        @Override
        public boolean matches(Interface.Arg argument) {
            return argument.getV() < threshold amp;amp; argument.getStr().startsWith(start);
        }
    }
}

Если я закомментирую when-2 блок, тест выполняется нормально и гладко. В противном случае я вижу NPE без какой-либо очевидной причины для этого.

Mockito имеет версию 2.24.0, JUnit — версию 4.

ОБНОВЛЕНИЕ: борьба за второй вызов Mockito.when заключается в некотором знании того, что издевательский метод может вызываться несколько раз в течение одного модульного теста.

Комментарии:

1. NullPointer — это позор, используйте необязательный везде, где это необходимо.

2. Ну, я делаю это там, где это возможно. Тем не менее, я не могу найти никакой причины, по которой mockito выдает и NPE при втором вызове when . Это большой отстой…

3. ОБНОВЛЕНИЕ : борьба за второй вызов Mockito.when заключается в некотором знании того, что издевательский метод может вызываться несколько раз в течение одного модульного теста.

4. Я не откопал исходные тексты, чтобы действительно понять, почему, и я не нашел многого в документах , но в основном это то же самое, что и здесь . Szczepan (автор Mockito) также не дает никакого представления. Короче говоря, измените свой второй when на doReturn(3).when(iface).callMe(...); , и все будет хорошо. Пожалуйста, обратите внимание, что если вы на самом деле не используете вторую блокировку, ваш тест пройдет успешно, но в конце вы получите UnnecessaryStubbingException

5. @Morfic Это определенно должно быть ответом здесь…

Ответ №1:

С Mockito 2.28.2 и 3.1.0:

Столкнулся с аналогичной проблемой, и решение состояло в том, чтобы пойти по пути, предложенному @Morfic, то есть переписать (предполагая, что Mockito статически импортирован):

 when(someMock.someMethod(argThat(...))).thenReturn(someValue);

с:

 doReturn(someValue).when(someMock).someMethod(argThat(...));

Однако я понятия не имею или объяснения, почему 1-й сбой, а второй работает нормально…

Тестовый пример JUnit 4, показывающий сбой и рабочий вариант:

 class MyBean{
    public Integer id;
    public String name;

    public MyBean(int id, String name) {
        this.id = id;
        this.name = name;
    }
}

interface MyService{
    Integer measure(MyBean myBean);
}

    @Test
    public void testThatFailsWithNPE(){

        when(myService.measure(argThat((mb) -> Objects.equals(mb.id, 123) amp;amp; Objects.equals(mb.name, "123")))).thenReturn(123);
        when(myService.measure(argThat((mb) -> Objects.equals(mb.id, 321) amp;amp; Objects.equals(mb.name, "321")))).thenReturn(321);

        assertEquals(Integer.valueOf(123), myService.measure(new MyBean(123, "123")));
        assertEquals(Integer.valueOf(321), myService.measure(new MyBean(321, "321")));
        assertEquals(Integer.valueOf(0), myService.measure(new MyBean(777, "777")));
    }

    @Test
    public void testThatPasses(){

        doReturn(123).when(myService).measure(argThat((mb) -> Objects.equals(mb.id, 123) amp;amp; Objects.equals(mb.name, "123")));
        doReturn(321).when(myService).measure(argThat((mb) -> Objects.equals(mb.id, 321) amp;amp; Objects.equals(mb.name, "321")));

        assertEquals(Integer.valueOf(123), myService.measure(new MyBean(123, "123")));
        assertEquals(Integer.valueOf(321), myService.measure(new MyBean(321, "321")));
        assertEquals(Integer.valueOf(0), myService.measure(new MyBean(777, "777")));
    }

Ответ №2:

Что ж, я нашел обходной путь, используя when(mock.callMe(any(Arg.class), any(Arg.class))).thenAnswer(new Answer<Integer> { ...});