Skip to content

Commit

Permalink
It turns out that Java creates an inner class for switch blocks, for …
Browse files Browse the repository at this point in the history
…which we do not require testing; therefore, this additional excemption

Signed-off-by: Rajarshi  Guha <rajarshi.guha@gmail.com>
  • Loading branch information
egonw authored and rajarshi committed May 18, 2012
1 parent 7c6506c commit 2026a25
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/test/org/openscience/cdk/coverage/CoverageTest.java
Expand Up @@ -140,7 +140,8 @@ private int checkClass(String className) {
}
// replace '$' with '_'
testMethod = replaceFunnyCharacters(testMethod);
if (!testMethod.equals("testClass$_String")) {
if (!testMethod.equals("testClass$_String") &&
!testMethod.startsWith("test_SWITCH_TABLE")) {
if (!testMethodNames.contains(testMethod)) {
System.out.println(removePackage(coreClass.getName()) + ": missing the expected test method: " + testMethod);
missingTestsCount++;
Expand Down

0 comments on commit 2026a25

Please sign in to comment.