diff options
author | gjoranv <gjoranv@gmail.com> | 2018-05-02 12:23:17 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-05-02 12:23:17 +0200 |
commit | 8e80aede65732210a72dbdfe6e7863ff502bd5ac (patch) | |
tree | f8b7d0da3fc9e28a7cfed66c6154ee603ba7c60f /logserver/src/test | |
parent | bbb5bcd8fa10bde6d3275e68c54b0a83c2ab94ad (diff) |
Revert "Revert "Gjoranv/java9 prep 05""
Diffstat (limited to 'logserver/src/test')
-rw-r--r-- | logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java | 37 |
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(); |