java - getter() 的 Mockito/Junit5 org.opentest4j.AssertionFailedError

Question

如果我使用以下方法进行 JUnit Mockito 测试，则会失败并显示 org.opentest4j.AssertionFailedError: Expected: not null。但是之前使用 verify() 和 Assertions.assertNotNull() 进行的 Mockito-(Parameter-)Test 是可以的。我做错了什么或者我不明白什么？这是我的代码:

JUnit5/模拟测试:

@Mock
private MyentityAServiceImpl myentityAServiceImplmock;
@Captor
private ArgumentCaptor<MyentityA> myentityAArgument;
@Mock
private MyentityBdao myentityBdaomock;
@Mock
private MyentityB myentityBmock;
@Mock
private Logger loggermock;
@InjectMocks
private MyentityBServiceImpl teServiceImpl;

@Test
public void setMyentityAOfTeIfNullByLanr17() {
    myentityBmock.setLanr7( "1234567" );
    final MyentityA myentityA = new MyentityA();
    mockito.when( myentityAServiceImplmock.findMyentityAByLanr17( myentityBmock.getLanr7() ) ).thenReturn( myentityA );
    mockito.when( myentityBdaomock.save( myentityBmock ) ).thenReturn( myentityBmock );
    myentityBmock = teServiceImpl.setMyentityAOfTeIfNullByLanr17( myentityBmock );
    mockito.verify( myentityBmock ).setMyentityA( myentityAArgument.capture() );
    Assertions.assertNotNull( myentityAArgument );
    Assertions.assertNotNull( myentityBmock.getMyentityA() );  // --> org.opentest4j.AssertionFailedError: expected: not <null>
}

测试方法:

  public MyentityB setMyentityAOfTeIfNullByLanr17( final MyentityB entity ) {
    MyentityA myentityA = entity.getMyentityA();
    if ( myentityA != null ) {
        return entity;
    }
    final String lanr17 = entity.getLanr7();
    myentityA = myentityAServiceImpl.findMyentityAByLanr17( lanr17 );
    if ( myentityA != null ) {
        entity.setMyentityA( myentityA );
        entity.setModuser( "root" );
        return myentityBdao.save( entity );
    }
    return entity;
}

Answer 1

抱歉，您的测试代码实在是太复杂了。
它对被测试方法中操作的对象的调用流程描述得太精细。
除此之外，您还模拟了很多东西:依赖项、被测试方法的参数。
最后，将被测试方法的返回值分配给引用模拟参数的变量。这让事情变得很不清楚。

测试必须是可以直接理解的，但事实并非如此。
我花了大约 500 万时间才理解错误原因。对于如此简单的代码来说，需要做的事情太多了。

此断言失败:

Assertions.assertNotNull( myentityBmock.getMyentityA() );  // --> org.opentest4j.AssertionFailedError: expected: not <null>

是因为在测试方法中getMynEntityA()只能为null，因为myentityBmock是一个模拟并设置一个字段正如您在此处所做的那样，不会对实际字段和相关的 getter 行为产生影响:

if ( myentityA != null ) {
    entity.setMyentityA( myentityA ); // here you invoke a mocked method.
    entity.setModuser( "root" );
    return myentityBdao.save( entity );
}

事实上，你的测试方式主要描述了被测试方法的调用流程，这就足够了:

mockito.verify( myentityBmock ).setMyentityA( myentityAArgument.capture() );

因为您无法测试 setMyentityA() 的副作用。

但我真的强烈建议您对您的方法进行单元测试，而不要模拟被测试方法的参数。
它可能看起来像:

@Test
public void setMyentityAOfTeIfNullByLanr17() {
    MyentityB entityB = new MyentityB(...) ;
    entityB.setLanr7( "1234567" );
    final MyentityA myentityA = new MyentityA();
    mockito.when( myentityAServiceImplmock.findMyentityAByLanr17( entityB.getLanr7() ) ).thenReturn( myentityA );
    mockito.when( myentityBdaomock.save(entityB) ).thenReturn(entityB);

    // action
    MyentityB entityActualB = teServiceImpl.setMyentityAOfTeIfNullByLanr17(entityB);

    // Perform content/logic assertion and no flow assertion :
    Assertions.assertEquals(myEntityA, entityActualB.getMyEntityA());
    Assertions.assertEquals("root", entityActualB.getModuser());
}

根本没有经过测试的代码，但它应该可以帮助您理解我的意图。