Fix usage of apis deprecated from Java 9.

+ rewrite if to switch-case.

1 files changed, 23 insertions, 14 deletions

diff --git a/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java b/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java
index 2e0b3d67c49..434131878c0 100644
--- a/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java
+++ b/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java
@@ -83,20 +83,29 @@ public class LogMetricsTestCase {
         assertEquals(levelCount.entrySet().size(), 5);
         for (Map.Entry<String, Long> entry : levelCount.entrySet()) {
             String key = entry.getKey();
-            if (key.equals("config")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("info")) {
-                assertEquals(entry.getValue(), new Long(4));
-            } else if (key.equals("warning")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("severe")) {
-                assertEquals(entry.getValue(), new Long(0));
-            } else if (key.equals("error")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("fatal")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("debug")) {
-                assertEquals(entry.getValue(), new Long(0));  // always 0
+            switch (key) {
+                case "config":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "info":
+                    assertEquals(entry.getValue(), Long.valueOf(4));
+                    break;
+                case "warning":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "severe":
+                    assertEquals(entry.getValue(), Long.valueOf(0));
+                    break;
+                case "error":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "fatal":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "debug":
+                    assertEquals(entry.getValue(), Long.valueOf(0));  // always 0
+
+                    break;
             }
         }
         a.close();