我有以下的JUnit测试。我正在测试的方法非常简单,只是接收一个数字并返回带有除数的List。我不想重复测试代码很多次,所以我创建了一个附配的方法,testDivisorsAux
:我应该在JUnit中拆分重用方法吗?
@Test
public final void testDivisors() {
testDivisorsAux(1, new int[] { 1 });
testDivisorsAux(6, new int[] { 1, 2, 3, 6 });
testDivisorsAux(0, new int[] { });
...
}
private final void testDivisorsAux(final int number, final int[] expected) {
List<Integer> divisors = Util.divisors(number);
assertSame(divisors.size(), expected.length);
for (int i : expected) {
assertTrue(divisors.contains(i));
}
}
一切正常,我只是想知道...... 这是一个不好的做法?我应该以不同的方式编写测试吗?也许保留“@Test
方法”中的所有代码?
PMD是告诉我,JUnit测试应包括断言()或失败()(对于第一方法),并且执行测试JUnit 4测试应该使用@Test注解(用于第二个) 。我知道PMD正在使用一个正则表达式(呃,实际上它是XPath)来确定我正在打破哪个规则 ...所以我倾向于认为这只是一个“误报”警告。但无论如何,我想知道我是否做错了什么。 (公寓从写作测试4倍的时间比被测试的方法:)
虽然我在寻找类似于这个问题,我发现了一种叫做参数化测试 ...但它似乎是面向更大的场景,不是吗?
非常感谢您的回答@Adam。我并不是很担心PMD的警告,但无论如何都是避免它们的好方法! :)不仅是一个很好的窍门,我认为你提出的名字也比我的更具描述性...... – AJPerez 2010-09-07 06:05:25