【问题标题】:Creating tests with many mock objects?使用许多模拟对象创建测试?
【发布时间】:2018-11-14 09:14:19
【问题描述】:

我怀疑是否应该创建包含许多模拟对象的测试。

我最近阅读了When should I mock?,我感到很困惑。

看看我有的一个方法(只是为了说明问题)

@Override
protected void validate() throws WTException {
    Either<ImportError, RootFinderResult> rootPart = getDataValidator().getRootPart();
    if (rootPart.isLeft()) {
        addValidationMessage(ROOT_PART_NOT_FOUND);
    } else if (rootPart.isRight()) {
        getObjectsToValidate().forEach(Lambda.uncheckedBiConsumer((part, epmDocuments) -> {
            LocalizableMessage rootRevision = getRevision(part);

            Optional<EPMDocument> wrongRevisionEPM = epmDocuments.stream()
                    .filter(epmDocument -> !isSameRevision(rootRevision, epmDocument))
                    .findAny();

            wrongRevisionEPM.ifPresent(epmDocument -> addValidationMessage("blabla"));
        }));
    }
}

以下所有方法都需要连接到服务器才能工作,否则会抛出错误

getDataValidator().getRootPart();
getRevision(part)
!isSameRevision(rootRevision, epmDocument))

此外,我无法创建零件或 epm 文档的“真实”对象。这也需要连接到服务器。


所以此时,我真正要测试的其实是这部分代码的逻辑

    Optional<EPMDocument> wrongRevisionEPM = epmDocuments.stream()
            .filter(epmDocument -> !isSameRevision(rootRevision, epmDocument))
            .findAny();

    wrongRevisionEPM.ifPresent(epmDocument -> addValidationMessage("blabla"));

但要测试它,我需要模拟很多对象

@Spy
@InjectMocks
private SameRevision sameRevision;
@Mock
private WTPartRelatedObjectDataValidator wTPartRelatedObjectDataValidator;
@Mock
private ValidationEntry validationEntry;
@Mock
private WTPart rootPart1, rootPart2;
@Mock
private EPMDocument epmDocument1, epmDocument2, epmDocument3;
@Mock
private Either<ImportError, RootFinderResult> rootPart;
@Mock
private LocalizableMessage rootPartRevisionOne, rootPartRevisionTwo;

所以我终于可以测试逻辑了:

@Test
@DisplayName("Should contain error message when part -> epms revisions are not the same")
void shoulHaveErrorMessagesWhenDifferentRevisions() throws Exception {
    doReturn(getMockObjectsToValidate()).when(sameRevision).getObjectsToValidate();

    doReturn(rootPart).when(liebherrWTPartRelatedObjectDataValidator).getRootPart();
    doReturn(false).when(rootPart).isLeft();
    doReturn(true).when(rootPart).isRight();

    doReturn(rootPartRevisionOne).when(sameRevision).getRevision(rootPart1);
    doReturn(rootPartRevisionTwo).when(sameRevision).getRevision(rootPart2);

    doReturn(true).when(sameRevision).isSameRevision(rootPartRevisionOne, epmDocument1);
    doReturn(false).when(sameRevision).isSameRevision(rootPartRevisionOne, epmDocument2);
    doReturn(true).when(sameRevision).isSameRevision(rootPartRevisionTwo, epmDocument3);

    validationEntry = sameRevision.call();

    assertEquals(1, validationEntry.getValidationMessageSet().size());
}

在哪里

    doReturn(rootPart).when(liebherrWTPartRelatedObjectDataValidator).getRootPart();
    doReturn(false).when(rootPart).isLeft();
    doReturn(true).when(rootPart).isRight();

    doReturn(rootPartRevisionOne).when(sameRevision).getRevision(rootPart1);
    doReturn(rootPartRevisionTwo).when(sameRevision).getRevision(rootPart2);

可以移动到@BeforeEach。


最后,我进行了测试,并且成功了。它验证了我想要验证的内容,但为了达到这一点,我必须付出很多努力来完成需要与服务器交互的整个 API。

你们怎么看,创建这样的测试值得吗?我想这是一个广泛开放的话题,因为许多尝试进入“测试世界”的新手都会遇到类似的问题,所以请不要因为基于意见的判断而关闭该话题,并就该话题提供您的反馈。

【问题讨论】:

    标签: java testing junit mocking ptc-windchill


    【解决方案1】:

    你是对的。模拟所有这些依赖项是一项巨大的努力。让我谈谈可能会使事情更清楚的几点:

    • 将编写测试视为投资:所以是的,有时编写测试比编写实际代码更费力。但是,稍后当您引入错误并且测试可以捕获它时,您会感谢自己。拥有良好的测试可以让您在修改代码时确信您没有破坏任何东西,如果您这样做了,您的测试会发现问题。随着时间的推移,它会得到回报。

    • 让您的测试专注于特定的课程。模拟其余部分: 当您模拟除被测类之外的所有内容时,您可以确定当问题发生时,它来自被测类,而不是其依赖项之一。这使故障排除变得更加容易。

    • 在编写新代码时考虑可测试性: 有时可能不可避免地会出现难以测试的复杂代码。但是,通常可以通过将依赖项的数量保持在最低限度并编写可测试的代码来避免这种情况。例如,如果一个方法需要 5 或 6 个以上的依赖项来完成它的工作,那么该方法可能做得太多并且可能被分解。同样的事情可以在类级别、模块等上说。

    【讨论】:

      【解决方案2】:

      您应该模拟要测试的类所依赖的其他依赖项,并设置您需要的行为。 需要这样做以测试您的方法隔离而不依赖于第三方类 您可以编写可以包含您的模拟行为并在测试中使用它们的私有 void 方法, 在 @BeforeEach 带注释的方法中,您可以模拟所有测试中相同的行为,或者模拟所有测试中相同的模拟行为

      在您的无效方法中,您可以拥有可以验证的间谍对象,如果它们被称为 Mockito.verify()

      【讨论】:

        【解决方案3】:

        是的,当你不得不模拟这么多事情时,这是相当的投资时间。 在我看来,如果你在测试某样东西的时候增加了一些价值,那是值得测试的,问题当然可能是你会消耗多少时间。

        在您的具体情况下,我会在不同的“层”上进行测试。

        例如,方法: getDataValidator().getRootPart(); 获取修订(部分) !isSameRevision(rootRevision, epmDocument))

        它们可以被独立测试,在你的情况下只是模拟它们的结果,这意味着你并不真正关心那里的参数,你只关心在某个返回值的情况下会发生什么。

        因此,在一层您真正测试功能,在下一层您只需模拟您需要的结果,以便测试其他功能。

        我希望现在更清楚......

        【讨论】:

          猜你喜欢
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 1970-01-01
          • 2022-01-23
          • 1970-01-01
          • 1970-01-01
          相关资源
          最近更新 更多