JUNIT测试void方法

Question

我有一个充满void方法的java类,我想进行一些单元测试以获得最大的代码覆盖率.

例如,我有这个方法:

protected static void checkifValidElements(int arg1,  int arg2) {
    method1(arg1);
    method2(arg1);
    method3(arg1, arg2);
    method4(arg1, arg2);
    method5(arg1);
    method6(arg2);
    method7();
}

由于我翻译代码以便更好地理解,因此它的名字很差.每个方法都以某种方式验证参数是否有效并且写得很好.

示例:

private static void method1(arg1) {
    if (arg1.indexOf("$") == -1) {

        //Add an error message 
        ErrorFile.errorMessages.add("There is a dollar sign in the specified parameter");
    }
}

我的单元测试正在覆盖小方法,因为我要求他们检查ErrorFile是否包含错误消息,但我不知道如何测试我的方法checkIfValidElements,它什么都不返回或什么都不改变.当我使用Maven运行代码覆盖时,它告诉我单元测试会覆盖我课程的这一部分.

我看到的唯一方法是更改​​此方法以返回int或bollean值,如下所示:

protected static int checkifValidElements(int arg1,  int arg2) {
    method1(arg1);
    method2(arg1);
    method3(arg1, arg2);
    method4(arg1, arg2);
    method5(arg1);
    method6(arg2);
    method7();
    return 0;
}

使用这种方法,我能够做一个断言等于,但在我看来这样做是徒劳的.问题是我有几个类似这样的类,它降低了我的单元测试覆盖率%.

Answer 1

我想进行一些单元测试以获得最大的代码覆盖率

代码覆盖率永远不应该是编写单元测试的目标.您应该编写单元测试来证明您的代码是正确的,或者帮助您更好地设计它,或者帮助其他人理解代码的意图.

但是我没看到我如何测试我的方法checkIfValidElements,它什么都不返回或什么都没改变.

那么你应该给出一些测试,它们之间检查所有7个方法都被正确调用 - 无论是参数无效还是有效参数,都要检查ErrorFile每次的结果.

例如,假设某人删除了以下呼叫:

method4(arg1, arg2);

...或意外更改了参数顺序:

method4(arg2, arg1);

你会怎么注意到这些问题？从那里开始,设计测试来证明它.

Answer 2

如果你的方法没有副作用,并且没有返回任何东西,那么它没有做任何事情.

如果您的方法进行了一些计算并返回该计算的结果,那么显然可以断言返回的结果是正确的.

如果您的代码没有返回任何但有副作用,您可以调用代码然后断言正确的副作用已经发生.副作用将决定您如何进行检查.

在您的示例中,您从非返回函数调用静态方法,这使得它很棘手,除非您可以检查所有这些静态方法的结果是否正确.从测试的角度来看,更好的方法是在调用方法时注入实际对象.然后,您可以使用EasyMock或Mockito之类的东西在单元测试中创建Mock对象,并将模拟对象注入到类中.然后,Mock对象允许您断言正确的函数被调用,具有正确的值并且顺序正确.

例如:

private ErrorFile errorFile;

public void setErrorFile(ErrorFile errorFile) {
    this.errorFile = errorFile;
}

private void method1(arg1) {
    if (arg1.indexOf("$") == -1) {

        //Add an error message 
        errorFile.addErrorMessage("There is a dollar sign in the specified parameter");
    }
}

然后在你的测试中你可以写:

public void testMethod1() {
    ErrorFile errorFile = EasyMock.createMock(ErrorFile.class);
    errorFile.addErrorMessage("There is a dollar sign in the specified parameter");
    EasyMock.expectLastCall(errorFile);
    EasyMock.replay(errorFile);

    ClassToTest classToTest = new ClassToTest();
    classToTest.setErrorFile(errorFile);
    classToTest.method1("a$b");

    EasyMock.verify(errorFile); // This will fail the test if the required addErrorMessage call didn't happen
}

Answer 3

你可以学习一些叫做“mocking”的东西。例如，您可以使用它来检查是否： - 调用了函数 - 调用了函数 x 次 - 函数至少调用了 x 次 - 使用一组特定参数调用了函数。例如，在您的情况下，您可以使用模拟来检查 method3 是否被调用一次，无论您作为 arg1 和 arg2 传递什么。

看看这些： https://code.google.com/p/mockito/ https://code.google.com/p/powermock/

Answer 4

您仍然可以通过断言它具有适当的副作用来对 void 方法进行单元测试。在您的method1示例中，您的单元测试可能类似于：

public void checkIfValidElementsWithDollarSign() {
    checkIfValidElement("$",19);
    assert ErrorFile.errorMessages.contains("There is a dollar sign in the specified parameter");
}