Delete Redundant RoutingServiceTests (#37750)

* This test compleletly overrode the `reroute` method and hence did nothing put test the override itself * Removed the test since it tests nothing and simplified `reroute` accordingly
elastic · Jan 23, 2019 · 2439f68 · 2439f68
1 parent 6a98383
commit 2439f68
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 67 deletions.
diff --git a/server/src/main/java/org/elasticsearch/cluster/routing/RoutingService.java b/server/src/main/java/org/elasticsearch/cluster/routing/RoutingService.java
@@ -77,11 +77,6 @@ protected void doClose() {
      * Initiates a reroute.
      */
     public final void reroute(String reason) {
-        performReroute(reason);
-    }
-
-    // visible for testing
-    protected void performReroute(String reason) {
         try {
             if (lifecycle.stopped()) {
                 return;

diff --git a/server/src/test/java/org/elasticsearch/cluster/routing/RoutingServiceTests.java b/server/src/test/java/org/elasticsearch/cluster/routing/RoutingServiceTests.java