fix toString for empty sentinel stats

Netflix · Apr 27, 2023 · 555f741 · 555f741
1 parent 811ffb0
commit 555f741
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/zuul-discovery/src/main/java/com/netflix/zuul/discovery/DiscoveryResult.java b/zuul-discovery/src/main/java/com/netflix/zuul/discovery/DiscoveryResult.java
@@ -61,7 +61,12 @@ public DiscoveryResult(DiscoveryEnabledServer server, LoadBalancerStats lbStats)
      */
     public DiscoveryResult(DiscoveryEnabledServer server) {
         this.server = server;
-        this.serverStats = new ServerStats();
+        this.serverStats = new ServerStats() {
+            @Override
+            public String toString() {
+                return "no stats configured for server";
+            }
+        };
     }
 
     /**

diff --git a/zuul-discovery/src/test/java/com/netflix/zuul/discovery/DiscoveryResultTest.java b/zuul-discovery/src/test/java/com/netflix/zuul/discovery/DiscoveryResultTest.java
@@ -39,6 +39,11 @@ void hashCodeForNull() {
         assertEquals(0, discoveryResult.hashCode());
     }
 
+    @Test
+    void serverStatsForEmptySentinel() {
+        Truth.assertThat(DiscoveryResult.EMPTY.getServerStats().toString()).isEqualTo("no stats configured for server");
+    }
+
     @Test
     void hostAndPortForNullServer() {
         final DiscoveryResult discoveryResult = new DiscoveryResult(null);