-
Notifications
You must be signed in to change notification settings - Fork 11
Closed
Labels
Low PrioritycodeDoneThe issue is rechecked for development branchThe issue is rechecked for development branchcodeReadyForRecheckThe issue is fixed/implemented AND merged into development, but needs to be rechecked thereThe issue is fixed/implemented AND merged into development, but needs to be rechecked thereimprovement
Milestone
Description
Exchange the expected parameter of the Test annotation with Assert.assertThrows within our test suite.
This reduces the risk of masking an error because the same error was thrown accidentally. Create a PR afterwards.
before:
@Test(expected = UIAssertionError.class
after:
// code before the Throwable is ecpected
Assert.assertThrows(UIAssertionError.class, () -> {
// code that will produce the Throwable
});
Metadata
Metadata
Assignees
Labels
Low PrioritycodeDoneThe issue is rechecked for development branchThe issue is rechecked for development branchcodeReadyForRecheckThe issue is fixed/implemented AND merged into development, but needs to be rechecked thereThe issue is fixed/implemented AND merged into development, but needs to be rechecked thereimprovement