[SQOOP-3090] Normalize test cases where expect an exception

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.4.6
• Fix Version/s: 1.4.7
• Assignee: Boglarka Egyed
• Reporter: Boglarka Egyed
• Created: 22/Dec/16 12:52
• Updated: 16/Mar/17 21:06
• Resolved: 16/Mar/17 21:06

Description

Many test cases uses the following logic to test if an exception has been thrown:

try {
      <do something>
      fail("Expected exception");
    } catch (someException e) {
      // expected
    }
  }

This form is not self-explanatory enough and could be replaced by a one line ExpectedException check which would be more clean.