Improve tests with AssumptionViolatedExceptions

The expectations that the test are failing and not being skipped is now
explicit.

Author: stefanbirkner

src/test/java/org/junit/rules/ErrorCollectorTest.java

@@ -36,7 +36,7 @@ public static Object[][] testsWithEventMatcher() {
                         hasNumberOfFailures(2)},
                 {
                     AddInternalAssumptionViolatedException.class,
-                        hasSingleFailure()},
+                        allOf(hasSingleFailure(), hasNoAssumptionFailure())},
                 {
                     CheckMatcherThatDoesNotFailWithoutProvidedReason.class,
                         everyTestRunSuccessful()},
@@ -65,10 +65,10 @@ public static Object[][] testsWithEventMatcher() {
                         hasNumberOfFailures(2)},
                 {
                     CheckCallableThatThrowsInternalAssumptionViolatedException.class,
-                        hasSingleFailure()},
+                        allOf(hasSingleFailure(), hasNoAssumptionFailure())},
                 {
                     CheckCallableWithFailingAssumption.class,
-                        hasSingleFailure()},
+                        allOf(hasSingleFailure(), hasNoAssumptionFailure())},
                 {
                     CheckCallableThatDoesNotThrowAnException.class,
                         everyTestRunSuccessful()},