JUnit测试构造器测试

Question

伙计们，我是JUnit测试的新手，并试图很好地掌握它，现在我正在为一个构造函数（用于创建有向图的有向图类）编写JUnit测试，该构造函数在读取负int值时抛出IllegalArgumentExctive，如果一切正常（节点值的数量）大于零，则创建一个图。

有向图类：

 In in = new In();
 public Digraph(In in) {
  try {
    this.nodes = in.readInt();
    System.out.println("Total nodes in graph: "+ nodes);
    if (nodes < 0) throw new IllegalArgumentException("Number of vertices must be > 0);
    int E = in.readInt();
    if (E < 0) throw new IllegalArgumentException("Number of edges must be >0);
  }catch (NoSuchElementException e) {
     throw new InputMismatchException("Invalid input format in Digraph constructor");
  }

下面是我试图写的测试：

@Rule
  public ExpectedException exception = ExpectedException.none();  

@Test(expected = IllegalArgumentException.class)
public void DigraphIn() {

    Digraph G = new Digraph(in.readInt());

    exception.expect(IllegalArgumentException.class);
    exception.expectMessage("Vertices can't be nagative");
    exception.expectMessage("Invalid input format in Digraph constructor");
    exception.expectMessage("Number of edges in a Digraph must be nonnegative");
try{
}catch (AssertionError e){
    }
}

我应该如何使用一个（或两个）测试用例来测试这两个用例？如果“in”没有检测到-ve值，我就得到java。否则测试通过。提前谢谢

Answer 1

当你测试一个方法时，你实际上调用了它，这将使它执行。每个测试只能验证一个异常，因为该异常将取消方法的其余处理。因此，您需要对每个可能引发异常的位置进行一次测试。

当验证单元测试引发异常时，基本上有三种方法：

试抓块：

@Test
public void myTest() {
    try {
        myClass.myMethod(42);
        fail();
    } catch(final IllegalArgumentException e) {
        assertEquals("something went wrong", e.getMessage());
    }
}

@test-注释的预期-属性：

@Test(expected=IllegalArgumentException.class)
public void myTest() {
    myClass.myMethod(42);
}

和ExpectedException：

@Rule
public ExpectedException expectedException = ExpectedException.none();

@Test
public void myTest() {
    expectedException.expect(IllegalArgument.class);
    expectedException.expectMessage("something went wrong");

    myClass.myMethod(42);
}

在你的例子中，你试图使用这三个。

如果我们比较测试异常的方法，只有第一种和第三种方法实际上能够对抛出的异常进行验证，这使得它们更适合测试可以从多个位置抛出相同类型异常的方法，然后您可以使用异常消息，以验证异常是否从您将其放置的位置引发。

第二个是迄今为止最可读的，但不允许您区分被测试方法引发的异常，在大多数情况下，它提供的值不如其他两个。

在第一和第三本书中，第三本是迄今为止最具可读性的，也是我个人的最爱。

由于被测试的方法每次执行只能引发一个异常，因此对于可能引发异常的每个位置，它都应该有一个测试方法：

public class DiagraphTest {

    @Rule
    public ExpectedException expectedException = ExpectedException.none();

    private Diagraph diagraph;
    private In in;

    @Before
    public void setup() {
        in = mock(In.class); 
    }

    @Test
    public void constructorShouldThrowExceptionWhenNumberOfVerticesIsLessThanOne() {
         expectedException.expect(IllegalArgumentException.class);
         expectedException.expectMessage("vertices must be > 0"); //expectMessage only needs a substring of the exception-message

         doReturn(-1).when(in).readInt();

         new Diagraph(in);
    }

    @Test
    public void constructorShouldThrowExceptionWhenNumberOfEdgesIsLessThanOne() {
         expectedException.expect(IllegalArgumentException.class);
         expectedException.expectMessage("edges must be > 0"); 

         when(in.readInt()).thenReturn(42, -1);

         new Diagraph(in);
    }

    //as to the last exception, I really can't see that it will ever be thrown in that try-block, but here's a test for that as well..
    @Test
    public void constructorShouldThrowInputMismatchExceptionIfReceivedNoSuchElementException() {
         expectedException.expect(InputMismatchException.class);
         expectedException.expectMessage("Invalid input format); 

         doThrow(new NoSuchElementException("phail")).when(in).readInt();

         new Diagraph(in);
    }

}

Answer 2

你应该有很多测试用例。每个异常都有一个是很好的。

您执行的每个测试都是不同的，应该以不同的方式对待。

一本很好的参考书是：Junit Cookbook。

事实上，我在你的代码中看到了一个错误。在你的测试案例中，你可以像下面这样嘲笑合作者。我做了一个模拟，在每次调用时使用“mockito”模拟库返回不同的值。

你基本上需要这样的东西：

@Test(expected = IllegalArgumentException.class)
public void DigraphInThatThrowsExceptionForVertices() {
    In in = Mockito.mock(In.class);
    when(in.readInt()).thenReturn(-1);
    Digraph G = new Digraph(in);
    fail();
}

@Test(expected = IllegalArgumentException.class)
public void DigraphInThatThrowsExceptionForEdges() {
    In in = Mockito.mock(In.class);
    when(in.readInt()).thenReturn(10).thenReturn(-1);
    Digraph G = new Digraph(in);
    fail();
}

@Test
public void DigraphInThatDoesNotThrowException() {
    In in = Mockito.mock(In.class);
    when(in.readInt()).thenReturn(10).thenReturn(15);
    Digraph G = new Digraph(in.readInt());
}

这种方式测试代码更干净，易于阅读。