为两个方法创建测试用例，其中一个仅在第一个返回true时调用

Question

我有一个类，在不深入两个细节的情况下，它包含了方法

1. 公共布尔validateData（Person[]p）；
2. 公共无效进程数据（Person[]p）；

通常使用如下：

if(validateData(myData)) {
  processData(myData);
}

使用JUnit并为上述两种方法创建测试用例的最佳方法是什么？我不确定在这种情况下使用之前的@是否正确。我想让每个测试首先调用validateData（myData），然后如果它没有错误地完成并返回true，那么测试应该使用相同的myData调用processData（myData），但这会在测试用例中引入编程逻辑，我认为这不太好。

Answer 1

您在评论中提到您担心测试中的冗余。总的来说，这是一个好主意，但是您应该明白“生产”和“测试”代码之间存在细微的区别。

你看，测试代码的主要目的是

a）快速指出故障

b）让您轻松修复它们。

换句话说：如果你的测试类有3个公共方法，那么你的测试类也至少有3个测试方法。但最有可能的是，您的测试类将为您可以通过生产方法进行的每个和任何路径提供测试方法。

是的，这导致了很多测试方法。但关键是：如果其中一个测试失败，你可能只需要30秒就能弄清楚到底发生了什么-而不是：当你整个测试只有一种测试方法时；你必须花相当长的时间来理解你过长的测试中哪一部分实际上失败了。当然，我们需要在“我的测试很容易理解”和“我在到处复制代码”之间取得平衡。这个播客为进一步思考这个话题提供了一些很好的起点。

最后，从可靠的角度来看，也可以考虑拆分生产代码。类应该只负责一件事——仅方法的名称就表明类似乎负责做不止一件事（显然：验证和处理）。请注意，拥有这两个方法也会使您的界面不清楚-该类的用户是否希望在处理之前调用validate；或者“过程”是否也包括验证？

Answer 2

由于这两种方法都是公共的，因此它们应该各自有自己的单元测试。假设是这样，您可以检查Person[]上的某些内容，以确保processData执行了某些操作。

您可以做的另一件事是重构代码以使用对象而不仅仅是方法，然后您可以模拟对象，并确保根据特定条件调用它们：

public interface IDataProcessor {
    void processData(Data data);
}

public class Foo{
    public Foo(IDataProcessor processor) {
        _processor = processor;
    }

    public void bar(Data data){
        if(validateData(data)){
            _processor.processData(data);
        }
    }

    private boolean validateData(Data data){
        return data != null; // or whatever you want
    }

    private IDataProcessor _processor;
}

public class MockProcessor implements IDataProcessor {
    public MockProcessor(){}

    public void processData(Data data){
        CallFlag = true;
    }

    public bool CallFlag = false;
}

然后，对于单元测试，您可以这样做：

MockProcessor mock = new MockProcessor();
Foo foo = new Foo(mock);
Data data = new Data();
foo.bar(data);
Assert.True(mock.CallFlag);

在C语言中，有一个很好的库称为Moq，它可以很容易地在单元测试中创建接口的模拟实现，而无需自己手动定义接口。Java中可能有类似的功能。