Как исправить `UnfinishedStubbingException`?

Question

У меня есть следующее:

public class EnvWebEndpointExtensionEnvironmentPostProcessorTests {
  @Rule
  public ExpectedException thrown = ExpectedException.none();

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

  final EnvWebEndpointExtensionEnvironmentPostProcessor postProcessor = new EnvWebEndpointExtensionEnvironmentPostProcessor();

  @Mock
  ConfigurableEnvironment environmentMock;

  @Mock
  MutablePropertySources propertySourcesMock;

  @Test
  public void shouldAddPropertySource() {
    final MutablePropertySources propertySources = new MutablePropertySources();

    doReturn(propertySources) // line 40
        .when(environmentMock).getPropertySources();

    postProcessor.postProcessEnvironment(environmentMock, null);

    assertNotNull(propertySources.get("actuators-defaults"));
  }

  @Test
  public void shouldThrowExceptionOnFailingToAddLaptopPropertySource() {
    thrown.expect(RuntimeException.class);

    final MutablePropertySources propertySourcesReal = new MutablePropertySources();

    doReturn(propertySourcesReal)
        .when(environmentMock).getPropertySources();

    doReturn(true)
        .when(environmentMock).acceptsProfiles("laptop");
    doReturn(propertySourcesMock)
        .when(environmentMock).getPropertySources();
    doThrow(IOException.class) // line 61
        .when(propertySourcesMock).addBefore("actuators-defaults", any(ResourcePropertySource.class));

    postProcessor.postProcessEnvironment(environmentMock, null);
  }
}

Когда тесты запускаются индивидуально, они проходят, но когда они оба выполняются, shouldAddPropertySource завершается неудачно с:

org.mockito.exceptions.misusing.UnfinishedStubbingException: 
Unfinished stubbing detected here:
-> at com.netflix.springboot.actuators.EnvWebEndpointExtensionEnvironmentPostProcessorTests.shouldThrowExceptionOnFailingToAddLaptopPropertySource(EnvWebEndpointExtensionEnvironmentPostProcessorTests.java:61)

E.g. thenReturn() may be missing.
Examples of correct stubbing:
    when(mock.isOk()).thenReturn(true);
    when(mock.isOk()).thenThrow(exception);
    doThrow(exception).when(mock).someVoidMethod();
Hints:
 1. missing thenReturn()
 2. you are trying to stub a final method, which is not supported
 3: you are stubbing the behaviour of another mock inside before 'thenReturn' instruction if completed


    at com.netflix.springboot.actuators.EnvWebEndpointExtensionEnvironmentPostProcessorTests.shouldAddPropertySource(EnvWebEndpointExtensionEnvironmentPostProcessorTests.java:40)
    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.mockito.internal.junit.JUnitRule$1.evaluateSafely(JUnitRule.java:52)
    at org.mockito.internal.junit.JUnitRule$1.evaluate(JUnitRule.java:43)
    at org.junit.rules.ExpectedException$ExpectedExceptionStatement.evaluate(ExpectedException.java:239)
    at org.junit.rules.RunRules.evaluate(RunRules.java:20)
    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.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)

УчитываяИнформация, которую я нашел, и поведение выше, Mockito хранит некоторое статическое состояние, но уровень моего понимания недостаточно глубок, чтобы найти решение для вышеупомянутого.Что такое правильное исправление и, кроме того, объяснение исправления?

Answer 1

Дж. Б. Низет правильно диагностировал основную причину в своем ответе : Когда вы используете совпадения для одного аргумента, вы должны использовать совпадения для всех аргументов.

Моя догадка заключается в том, что Mockito правильно генерирует InvalidUseOfMatchersException , которое происходит от RuntimeException, поэтому ваш тест по ошибке проходит без проверки вашей тестируемой системы. Это важная причина, чтобы не перехватывать RuntimeException без разбора, особенно в верхней части тестового метода. Это также может быть причиной использования assertThrows или идиомы try { methodUnderTest(); fail(); } catch (YourSpecificException expected) {}.

Если это так, то вы видите это конкретное исключение, потому что ваш тестовый исполнитель вызывает ваши тесты в порядке shouldThrow, shouldAdd на той же виртуальной машине, а Mockito сохраняет состояние соответствия в static ThreadLocal , который может выжить между тестами. Если эта теория верна, то InvalidUseOfMatchersException происходит до того, как Mockito сможет сохранить ожидание из строки 61, оставляя заглушку в строке 61 технически незавершенной. Поскольку Mockito не знает, когда заканчивается один тест, а другой начинается, он не может сбросить свое состояние, поэтому Mockito может обнаружить этот случай только при следующем взаимодействии с Mockito (в строке 40).

Вы можете улучшить свой опыт, вызвав Mockito.validateMockitoUsage() методом @After (или используя MockitoRule или MockitoJUnitRunner ).

Answer 2

Если вы используете совпадение в качестве аргумента вызова метода-заглушки, все аргументы должны быть совпадениями. Поэтому вам нужно заменить "actuators-defaults" на eq("actuators-defaults").

Я не уверен, почему это вызывает это исключение.