There are quite a few test cases that have code like:
public void testSomething(){
try {
something();
fail("an exception should have been caught");
} catch (EstimationException ee) {
// expected behavior
} catch (Exception e) {
fail("wrong exception type caught");
}
}
This is unnecessary code; worse, the actual Exception is lost.
I propose to fix these by converting them to:
public void testSomething() throws Exception {
try {
something();
fail("Expecting EstimationException ");
} catch (EstimationException ee) {
// expected behavior
}
Any objections?
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]