clazz.newInstance() -> clazz.getDeclaredConstructor().newInstance()

Vladimir Kotal · Vladimir Kotal · commit a586fed388f8 · 2018-12-07T22:02:44.000+01:00
diff --git a/opengrok-indexer/src/test/java/org/opengrok/indexer/authorization/AuthorizationPluginClassLoaderTest.java b/opengrok-indexer/src/test/java/org/opengrok/indexer/authorization/AuthorizationPluginClassLoaderTest.java
@@ -191,7 +191,7 @@ public void testTruePlugin() {
     private IAuthorizationPlugin getNewInstance(Class c) {
         IAuthorizationPlugin plugin = null;
         try {
-            plugin = (IAuthorizationPlugin) c.newInstance();
+            plugin = (IAuthorizationPlugin) c.getDeclaredConstructor().newInstance();
         } catch (InstantiationException ex) {
             Assert.fail("Should not produce InstantiationException");
         } catch (IllegalAccessException ex) {
diff --git a/opengrok-indexer/src/test/java/org/opengrok/indexer/condition/ConditionalRunRule.java b/opengrok-indexer/src/test/java/org/opengrok/indexer/condition/ConditionalRunRule.java
@@ -155,7 +155,7 @@ private RunCondition createCondition() throws Exception {
                     result = new CompositeCondition();
                 }
                 if (isConditionTypeStandalone(clazz)) {
-                    result.add(clazz.newInstance());
+                    result.add(clazz.getDeclaredConstructor().newInstance());
                 } else {
                     result.add(clazz.getDeclaredConstructor(mTestClass).newInstance(mTestClass));
                 }

Original file line number	Diff line number	Diff line change
`@@ -155,7 +155,7 @@ private RunCondition createCondition() throws Exception {`
`155`	`155`	`result = new CompositeCondition();`
`156`	`156`	`}`
`157`	`157`	`if (isConditionTypeStandalone(clazz)) {`
`158`		`- result.add(clazz.newInstance());`
	`158`	`+ result.add(clazz.getDeclaredConstructor().newInstance());`
`159`	`159`	`} else {`
`160`	`160`	`result.add(clazz.getDeclaredConstructor(mTestClass).newInstance(mTestClass));`
`161`	`161`	`}`