Major Description
We've got bunch of (negative) tests on the current codebase expecting a thrown XYZException however they don't realize if the expected exception is not thrown, the typical pattern for this is:
try { doSomething_With_Expectation_Of_A_Thrown_XYZException(); } catch (XYZException e) { // expected }
Which correctly should be:
try { doSomething_With_Expectation_Of_A_Thrown_XYZException(); fail("Should have thrown an exception"); } catch (XYZException e) { // expected }