diff options
author | gjoranv <gv@oath.com> | 2018-04-30 15:18:27 +0200 |
---|---|---|
committer | gjoranv <gv@oath.com> | 2018-04-30 15:24:29 +0200 |
commit | 667b92cdcb8c06d5a8285ac59141a890ae9f0f3c (patch) | |
tree | 8f3d340903aad65d754a5eaf2b24a5083de98e7f /logserver | |
parent | c57701c0cefd8c0198844d97d40c037698de39cf (diff) |
Fix usage of apis deprecated from Java 9.
+ rewrite if to switch-case.
Diffstat (limited to 'logserver')
-rw-r--r-- | logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java | 5 | ||||
-rw-r--r-- | logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java | 37 |
2 files changed, 26 insertions, 16 deletions
diff --git a/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java b/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java index 619643a32b6..db2ed6b0b83 100644 --- a/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java +++ b/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java @@ -5,6 +5,7 @@ import com.yahoo.log.LogLevel; import com.yahoo.plugin.Plugin; import com.yahoo.plugin.SystemPropertyConfig; +import java.lang.reflect.InvocationTargetException; import java.util.logging.Logger; /** @@ -20,8 +21,8 @@ public abstract class AbstractPluginLoader implements PluginLoader { protected void loadFromClass(Class<? extends Plugin> pluginClass) { Plugin plugin; try { - plugin = (Plugin) pluginClass.newInstance(); - } catch (InstantiationException | IllegalAccessException e) { + plugin = pluginClass.getDeclaredConstructor().newInstance(); + } catch (InstantiationException | IllegalAccessException | NoSuchMethodException | InvocationTargetException e) { log.log(LogLevel.ERROR, pluginClass.getName() + ": load failed: " + e); throw new RuntimeException(e); } 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(); |