Tidy up tests for TestWatcher

- Make it easier to understand TestWatcher's behavior based on the tests. - Improve expressiveness of the test. - Add missing tests.
junit-team · Feb 10, 2018 · 9cacd0b · 9cacd0b
1 parent 1b4ac07
commit 9cacd0b
Show file tree

Hide file tree

Showing 3 changed files with 281 additions and 280 deletions.
diff --git a/src/test/java/org/junit/rules/LoggingTestWatcher.java b/src/test/java/org/junit/rules/LoggingTestWatcher.java
@@ -25,6 +25,11 @@ protected void skipped(AssumptionViolatedException e, Description description) {
         log.append("skipped ");
     }
 
+    @Override
+    protected void skipped(org.junit.internal.AssumptionViolatedException e, Description description) {
+        log.append("deprecated skipped ");
+    }
+
     @Override
     protected void starting(Description description) {
         log.append("starting ");

diff --git a/src/test/java/org/junit/rules/TestRuleTest.java b/src/test/java/org/junit/rules/TestRuleTest.java
@@ -148,64 +148,6 @@ public void ignoreNonRules() {
 
     private static String log;
 
-    public static class OnFailureTest {
-        @Rule
-        public TestRule watcher = new TestWatcher() {
-            @Override
-            protected void failed(Throwable e, Description description) {
-                log += description + " " + e.getClass().getSimpleName();
-            }
-        };
-
-        @Test
-        public void nothing() {
-            fail();
-        }
-    }
-
-    @Test
-    public void onFailure() {
-        log = "";
-        Result result = JUnitCore.runClasses(OnFailureTest.class);
-        assertEquals(String.format("nothing(%s) AssertionError", OnFailureTest.class.getName()), log);
-        assertEquals(1, result.getFailureCount());
-    }
-
-    public static class WatchmanTest {
-        private static String watchedLog;
-
-        @Rule
-        public TestRule watcher = new TestWatcher() {
-            @Override
-            protected void failed(Throwable e, Description description) {
-                watchedLog += description + " "
-                        + e.getClass().getSimpleName() + "\n";
-            }
-
-            @Override
-            protected void succeeded(Description description) {
-                watchedLog += description + " " + "success!\n";
-            }
-        };
-
-        @Test
-        public void fails() {
-            fail();
-        }
-
-        @Test
-        public void succeeds() {
-        }
-    }
-
-    @Test
-    public void succeeded() {
-        WatchmanTest.watchedLog = "";
-        JUnitCore.runClasses(WatchmanTest.class);
-        assertThat(WatchmanTest.watchedLog, containsString(String.format("fails(%s) AssertionError", WatchmanTest.class.getName())));
-        assertThat(WatchmanTest.watchedLog, containsString(String.format("succeeds(%s) success!", WatchmanTest.class.getName())));
-    }
-
     public static class BeforesAndAfters {
         private static StringBuilder watchedLog = new StringBuilder();
 
@@ -437,101 +379,6 @@ public void methodIgnoreNonRules() {
         assertEquals(0, result.getFailureCount());
     }
 
-    public static class MethodOnFailureTest {
-        private TestRule watchman = new TestWatcher() {
-            @Override
-            protected void failed(Throwable e, Description description) {
-                log += description + " " + e.getClass().getSimpleName();
-            }
-        };
-
-        @Rule
-        public TestRule getWatchman() {
-            return watchman;
-        }
-
-        @Test
-        public void nothing() {
-            fail();
-        }
-    }
-
-    @Test
-    public void methodOnFailure() {
-        log = "";
-        Result result = JUnitCore.runClasses(MethodOnFailureTest.class);
-        assertEquals(String.format("nothing(%s) AssertionError", MethodOnFailureTest.class.getName()), log);
-        assertEquals(1, result.getFailureCount());
-    }
-
-    public static class MethodOnSkippedTest {
-        private TestRule watchman = new TestWatcher() {
-            @Override
-            protected void skipped(AssumptionViolatedException e, Description description) {
-                log += description + " " + e.getClass().getSimpleName();
-            }
-        };
-
-        @Rule
-        public TestRule getWatchman() {
-            return watchman;
-        }
-
-        @Test
-        public void nothing() {
-            Assume.assumeTrue(false);
-        }
-    }
-
-    @Test
-    public void methodOnSkipped() {
-        log = "";
-        Result result = JUnitCore.runClasses(MethodOnSkippedTest.class);
-        assertEquals(String.format("nothing(%s) AssumptionViolatedException", MethodOnSkippedTest.class.getName()), log);
-        assertEquals(0, result.getFailureCount());
-        assertEquals(1, result.getRunCount());
-    }
-
-    public static class MethodWatchmanTest {
-        @SuppressWarnings("unused")
-        private static String watchedLog;
-
-        private TestRule watchman = new TestWatcher() {
-            @Override
-            protected void failed(Throwable e, Description description) {
-                watchedLog += description + " "
-                        + e.getClass().getSimpleName() + "\n";
-            }
-
-            @Override
-            protected void succeeded(Description description) {
-                watchedLog += description + " " + "success!\n";
-            }
-        };
-
-        @Rule
-        public TestRule getWatchman() {
-            return watchman;
-        }
-
-        @Test
-        public void fails() {
-            fail();
-        }
-
-        @Test
-        public void succeeds() {
-        }
-    }
-
-    @Test
-    public void methodSucceeded() {
-        WatchmanTest.watchedLog = "";
-        JUnitCore.runClasses(WatchmanTest.class);
-        assertThat(WatchmanTest.watchedLog, containsString(String.format("fails(%s) AssertionError", WatchmanTest.class.getName())));
-        assertThat(WatchmanTest.watchedLog, containsString(String.format("succeeds(%s) success!", WatchmanTest.class.getName())));
-    }
-
     public static class BeforesAndAftersAreEnclosedByRule {
         private static StringBuilder log;