Update of /cvsroot/pmd/pmd/regress/test/net/sourceforge/pmd/rules
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv7231/regress/test/net/sourceforge/pmd/rules
Modified Files:
MissingBreakInSwitchTest.java
Log Message:
Fixed bug 1417106 - MissingBreakInSwitch no longer flags stmts where every case has either a return or a throw.
Index: MissingBreakInSwitchTest.java
===================================================================
RCS file: /cvsroot/pmd/pmd/regress/test/net/sourceforge/pmd/rules/MissingBreakInSwitchTest.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -C2 -d -r1.6 -r1.7
*** MissingBreakInSwitchTest.java 10 Feb 2006 14:26:25 -0000 1.6
--- MissingBreakInSwitchTest.java 12 Jun 2006 12:22:20 -0000 1.7
***************
*** 23,26 ****
--- 23,27 ----
new TestDescriptor(TEST3, "one break", 0, rule),
new TestDescriptor(TEST4, "each case stmt has a return", 0, rule),
+ new TestDescriptor(TEST5, "all cases have return or throw", 0, rule),
});
}
***************
*** 70,73 ****
--- 71,88 ----
"}";
+ private static final String TEST5 =
+ "public class Foo {" + PMD.EOL +
+ " int main() {" + PMD.EOL +
+ " switch(i) {" + PMD.EOL +
+ " case '1':" + PMD.EOL +
+ " return 1;" + PMD.EOL +
+ " case '2':" + PMD.EOL +
+ " return 2;" + PMD.EOL +
+ " default:" + PMD.EOL +
+ " throw new Exception(\"bad number\");" + PMD.EOL +
+ " }" + PMD.EOL +
+ " }" + PMD.EOL +
+ "}";
+
}
|