Merge changes from topic "revert-1601635-AIQYZOHWTP"

* changes:
  Revert "Upgrade external/junit to 4.13.2"
  Revert "Extra generic type information to aid certain javacs."
  Revert "Remove DisableOnDebug (new in 4.12) as it is not support..."
  Revert "Remove support for stuck threads"

1 files changed, 3 insertions, 38 deletions

diff --git a/src/main/java/org/junit/runner/notification/RunNotifier.java b/src/main/java/org/junit/runner/notification/RunNotifier.java
index 752fa3b..6875f76 100644
--- a/src/main/java/org/junit/runner/notification/RunNotifier.java
+++ b/src/main/java/org/junit/runner/notification/RunNotifier.java
@@ -65,8 +65,8 @@ public class RunNotifier {
 
         void run() {
             int capacity = currentListeners.size();
-            List<RunListener> safeListeners = new ArrayList<RunListener>(capacity);
-            List<Failure> failures = new ArrayList<Failure>(capacity);
+            ArrayList<RunListener> safeListeners = new ArrayList<RunListener>(capacity);
+            ArrayList<Failure> failures = new ArrayList<Failure>(capacity);
             for (RunListener listener : currentListeners) {
                 try {
                     notifyListener(listener);
@@ -78,7 +78,7 @@ public class RunNotifier {
             fireTestFailures(safeListeners, failures);
         }
 
-        protected abstract void notifyListener(RunListener each) throws Exception;
+        abstract protected void notifyListener(RunListener each) throws Exception;
     }
 
     /**
@@ -106,41 +106,6 @@ public class RunNotifier {
     }
 
     /**
-     * Invoke to tell listeners that a test suite is about to start. Runners are strongly
-     * encouraged--but not required--to call this method. If this method is called for
-     * a given {@link Description} then {@link #fireTestSuiteFinished(Description)} MUST
-     * be called for the same {@code Description}.
-     *
-     * @param description the description of the suite test (generally a class name)
-     * @since 4.13
-     */
-    public void fireTestSuiteStarted(final Description description) {
-        new SafeNotifier() {
-            @Override
-            protected void notifyListener(RunListener each) throws Exception {
-                each.testSuiteStarted(description);
-            }
-        }.run();
-    }
-
-    /**
-     * Invoke to tell listeners that a test suite is about to finish. Always invoke
-     * this method if you invoke {@link #fireTestSuiteStarted(Description)}
-     * as listeners are likely to expect them to come in pairs.
-     *
-     * @param description the description of the suite test (generally a class name)
-     * @since 4.13
-     */
-    public void fireTestSuiteFinished(final Description description) {
-        new SafeNotifier() {
-            @Override
-            protected void notifyListener(RunListener each) throws Exception {
-                each.testSuiteFinished(description);
-            }
-        }.run();
-    }
-
-    /**
      * Invoke to tell listeners that an atomic test is about to start.
      *
      * @param description the description of the atomic test (generally a class and method name)