當使用Mockito.spy()模擬一個方法被另一個方法呼叫時，為什么我會得到一個UnnecessaryStubbingException？-有解無憂

我有以下課程-

public class A {
    int a;

    public A() {
        this.a = 67;
    }
    
    public int get_a() {
        return this.a;
    }
    
    public boolean is_a_even() {
        if(this.get_a() % 2 == 0) {
            return true;

        } else{
            return false;
        }
    }
}

我想模擬 get_a() 函式，以便它回傳我想要的值，所以我撰寫了以下測驗 -

@ExtendWith(MockitoExtension.class)
class ATest {

    @Test
    public void testing_if_call_from_another_function_is_mocked() {
        A obj = new A();

        A mock = Mockito.spy(obj);

        Mockito.doReturn(2).when(mock).get_a();

        assertTrue(obj.is_a_even());

    }
}

但是當我運行它時，我得到以下例外 -

org.mockito.exceptions.misusing.UnnecessaryStubbingException: 
Unnecessary stubbings detected.
Clean & maintainable test code requires zero unnecessary code.
Following stubbings are unnecessary (click to navigate to relevant line of code):
  1. -> at com.example.demo.ATest.testing_if_call_from_another_function_is_mocked(ATest.java:19)
Please remove unnecessary stubbings or use 'lenient' strictness. More info: javadoc for UnnecessaryStubbingException class.

    at org.mockito.junit.jupiter.MockitoExtension.afterEach(MockitoExtension.java:186)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.lambda$invokeAfterEachCallbacks$11(TestMethodTestDescriptor.java:253)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.lambda$invokeAllAfterMethodsOrCallbacks$12(TestMethodTestDescriptor.java:269)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.lambda$invokeAllAfterMethodsOrCallbacks$13(TestMethodTestDescriptor.java:269)
    at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.invokeAllAfterMethodsOrCallbacks(TestMethodTestDescriptor.java:268)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.invokeAfterEachCallbacks(TestMethodTestDescriptor.java:252)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.execute(TestMethodTestDescriptor.java:137)
    at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.execute(TestMethodTestDescriptor.java:65)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$5(NodeTestTask.java:139)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$7(NodeTestTask.java:129)
    at org.junit.platform.engine.support.hierarchical.Node.around(Node.java:137)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$8(NodeTestTask.java:127)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:126)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:84)
    at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
    at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.invokeAll(SameThreadHierarchicalTestExecutorService.java:38)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$5(NodeTestTask.java:143)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$7(NodeTestTask.java:129)
    at org.junit.platform.engine.support.hierarchical.Node.around(Node.java:137)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$8(NodeTestTask.java:127)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:126)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:84)
    at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
    at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.invokeAll(SameThreadHierarchicalTestExecutorService.java:38)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$5(NodeTestTask.java:143)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$7(NodeTestTask.java:129)
    at org.junit.platform.engine.support.hierarchical.Node.around(Node.java:137)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$8(NodeTestTask.java:127)
    at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:126)
    at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:84)
    at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.submit(SameThreadHierarchicalTestExecutorService.java:32)
    at org.junit.platform.engine.support.hierarchical.HierarchicalTestExecutor.execute(HierarchicalTestExecutor.java:57)
    at org.junit.platform.engine.support.hierarchical.HierarchicalTestEngine.execute(HierarchicalTestEngine.java:51)
    at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:108)
    at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:88)
    at org.junit.platform.launcher.core.EngineExecutionOrchestrator.lambda$execute$0(EngineExecutionOrchestrator.java:54)
    at org.junit.platform.launcher.core.EngineExecutionOrchestrator.withInterceptedStreams(EngineExecutionOrchestrator.java:67)
    at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:52)
    at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:96)
    at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:75)
    at com.intellij.junit5.JUnit5IdeaTestRunner.startRunnerWithArgs(JUnit5IdeaTestRunner.java:71)
    at com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33)
    at com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221)
    at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54)

這是否意味著對 get_a() 的方法呼叫沒有被模擬，如果是這樣，我如何模擬將從另一個方法呼叫的方法？

uj5u.com熱心網友回復：

讓我們更深入地了解正在發生的事情和原因。

當我們打電話

A mock = Mockito.spy(obj);

mockito 創建一個包裝物件，我們可以用它來監視一個物件。我們使用這個間諜來模擬我們想要模擬的呼叫：

Mockito.doReturn(2).when(mock).get_a();

請注意，原始內容obj保持不變。特別是，它的行為保持不變。如果我們使用obj，它將表現得好像我們從未創建過間諜一樣。

因此，如果我們呼叫

obj.is_a_even()

我們的模擬將不會被使用。要獲得所需的行為，我們需要呼叫

mock.is_a_even()

然而，正確的解決方案是使類可測驗，例如通過提供一個允許設定的值的建構式a，例如：

public class A {
    private final int a;

    public A(int a) {
        this.a = a;
    }

    public A() {
        this(67);
    }

    ...
}

通過這種設定，我們可以撰寫相同的測驗而不需要任何模擬。

三個備注：

我們應該尊重封裝，即設定屬性private并使其只能通過訪問器方法訪問。
我們應該贊成
```
Mockito.when(mock.get_a()).thenReturn(2);
```
超過
```
Mockto.doReturn(2).when(mock).get_a();
```
由于前者保證編譯時的型別安全，后者不保證。
在 Java 中，方法名應該寫成lowerCamelCase而不是snake_case( get_a()-> getA(), is_a_even()-> isAEven())。我還建議為該領域a以及這兩種方法找到更好的名稱。

uj5u.com熱心網友回復：

您會得到 UnnecessaryStubbingException ，因為mock.get_a()它從未被呼叫過。

這是因為你打電話

assertTrue(obj.is_a_even());

代替：

assertTrue(mock.is_a_even());

因此在is_a_even方法中this指向obj，而不是mock

其他備注：

堅持 JavaBean 命名約定 -getA()而不是get_a(). 這很普遍，很多工具都依賴它。
替換被測系統的一部分是一種高級工具 - 謹慎使用它。如果可能，模擬 SUT 的合作者，而不是 SUT 本身。

轉載請註明出處，本文鏈接：https://www.uj5u.com/caozuo/406798.html

標籤：

上一篇：單元測驗回呼流

下一篇：MockK：如何模擬在另一個被測方法中呼叫的已測驗方法（Kotlin）