Rename utils-test testXxx() methods

Fixed sonar complaints about the testXxx_Yyy() methods in the
XxxTester classes.  In most cases, they were renamed to testYyy().
The top-level method, test(), was not renamed so-as not to break
existing tests that already depend on it.

Renamed testStringConstuctor() to testString().

Change-Id: I00e8993c71fa9d9cb83bea82276d9706dd0e7c45
Issue-ID: POLICY-246
Signed-off-by: Jim Hahn <jrh3@att.com>

3 files changed, 39 insertions, 39 deletions

diff --git a/utils-test/src/main/java/org/onap/policy/common/utils/test/ErrorsTester.java b/utils-test/src/main/java/org/onap/policy/common/utils/test/ErrorsTester.java
index 1a69e58c..46814c96 100644
--- a/utils-test/src/main/java/org/onap/policy/common/utils/test/ErrorsTester.java
+++ b/utils-test/src/main/java/org/onap/policy/common/utils/test/ErrorsTester.java
@@ -40,7 +40,7 @@ public class ErrorsTester extends ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Error> int testError(Class<T> claz) {
-		return testThrowable(claz);
+	public <T extends Error> int testAllError(Class<T> claz) {
+		return testAllThrowable(claz);
 	}
 }
diff --git a/utils-test/src/main/java/org/onap/policy/common/utils/test/ExceptionsTester.java b/utils-test/src/main/java/org/onap/policy/common/utils/test/ExceptionsTester.java
index b4091080..3a43b943 100644
--- a/utils-test/src/main/java/org/onap/policy/common/utils/test/ExceptionsTester.java
+++ b/utils-test/src/main/java/org/onap/policy/common/utils/test/ExceptionsTester.java
@@ -46,7 +46,7 @@ public class ExceptionsTester extends ThrowablesTester {
 	 *             if the constructed objects fail to pass various tests
 	 */
 	public <T extends Exception> int test(Class<T> claz) {
-		int ncons = testException(claz);
+		int ncons = testAllException(claz);
 
 		assertTrue(ncons > 0);
 
@@ -67,13 +67,13 @@ public class ExceptionsTester extends ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Exception> int testException(Class<T> claz) {
+	public <T extends Exception> int testAllException(Class<T> claz) {
 		int ncons = 0;
 
-		ncons += testThrowable(claz);
-		ncons += testException_Exception(claz);
-		ncons += testException_StringException(claz);
-		ncons += testException_StringExceptionBooleanBoolean(claz);
+		ncons += testAllThrowable(claz);
+		ncons += testException(claz);
+		ncons += testStringException(claz);
+		ncons += testStringExceptionBooleanBoolean(claz);
 
 		return ncons;
 	}
@@ -100,7 +100,7 @@ public class ExceptionsTester extends ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Exception> int testException_Exception(Class<T> claz) {
+	public <T extends Exception> int testException(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "exception", Exception.class);
 		if (cons == null) {
 			return 0;
@@ -137,7 +137,7 @@ public class ExceptionsTester extends ThrowablesTester {
 	 * @throws ConstructionError
 	 *             if the Exception subclass cannot be constructed
 	 */
-	public <T extends Exception> int testException_StringException(Class<T> claz) {
+	public <T extends Exception> int testStringException(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "string-exception", String.class, Exception.class);
 		if (cons == null) {
 			return 0;
@@ -176,7 +176,7 @@ public class ExceptionsTester extends ThrowablesTester {
 	 * @throws ConstructionError
 	 *             if the Exception subclass cannot be constructed
 	 */
-	public <T extends Exception> int testException_StringExceptionBooleanBoolean(Class<T> claz) {
+	public <T extends Exception> int testStringExceptionBooleanBoolean(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "string-exception-flags", String.class, Exception.class,
 				Boolean.TYPE, Boolean.TYPE);
 		if (cons == null) {
@@ -184,10 +184,10 @@ public class ExceptionsTester extends ThrowablesTester {
 		}
 
 		// test each combination of "message" and "cause"
-		testThrowable_MessageCauseCombos(cons);
+		testMessageCauseCombos(cons);
 
 		// test each combination of the boolean flags
-		testThrowable_FlagCombos(cons);
+		testFlagCombos(cons);
 
 		return 1;
 	}
diff --git a/utils-test/src/main/java/org/onap/policy/common/utils/test/ThrowablesTester.java b/utils-test/src/main/java/org/onap/policy/common/utils/test/ThrowablesTester.java
index 8c997d56..4a8da3b4 100644
--- a/utils-test/src/main/java/org/onap/policy/common/utils/test/ThrowablesTester.java
+++ b/utils-test/src/main/java/org/onap/policy/common/utils/test/ThrowablesTester.java
@@ -66,14 +66,14 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> int testThrowable(Class<T> claz) {
+	public final <T extends Throwable> int testAllThrowable(Class<T> claz) {
 		int ncons = 0;
 
-		ncons += testThrowable_Default(claz);
-		ncons += testThrowable_StringConstuctor(claz);
-		ncons += testThrowable_Throwable(claz);
-		ncons += testThrowable_StringThrowable(claz);
-		ncons += testThrowable_StringThrowableBooleanBoolean(claz);
+		ncons += testDefault(claz);
+		ncons += testString(claz);
+		ncons += testThrowable(claz);
+		ncons += testStringThrowable(claz);
+		ncons += testStringThrowableBooleanBoolean(claz);
 
 		return ncons;
 	}
@@ -99,7 +99,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> int testThrowable_Default(Class<T> claz) {
+	public final <T extends Throwable> int testDefault(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "default");
 		if (cons == null) {
 			return 0;
@@ -136,7 +136,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> int testThrowable_StringConstuctor(Class<T> claz) {
+	public final <T extends Throwable> int testString(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "string", String.class);
 		if (cons == null) {
 			return 0;
@@ -173,7 +173,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> int testThrowable_Throwable(Class<T> claz) {
+	public final <T extends Throwable> int testThrowable(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "throwable", Throwable.class);
 		if (cons == null) {
 			return 0;
@@ -211,7 +211,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> int testThrowable_StringThrowable(Class<T> claz) {
+	public final <T extends Throwable> int testStringThrowable(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "string-throwable", String.class, Throwable.class);
 		if (cons == null) {
 			return 0;
@@ -251,7 +251,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> int testThrowable_StringThrowableBooleanBoolean(Class<T> claz) {
+	public final <T extends Throwable> int testStringThrowableBooleanBoolean(Class<T> claz) {
 		Constructor<T> cons = getConstructor(claz, "string-throwable-flags", String.class, Throwable.class,
 				Boolean.TYPE, Boolean.TYPE);
 		if (cons == null) {
@@ -259,13 +259,13 @@ public class ThrowablesTester {
 		}
 
 		// test each combination of "message" and "cause"
-		testThrowable_MessageCauseCombos(cons);
+		testMessageCauseCombos(cons);
 
 		// test each combination of the boolean flags
-		testThrowable_SuppressStack(cons);
-		testThrowable_SuppressNoStack(cons);
-		testThrowable_NoSuppressStack(cons);
-		testThrowable_NoSuppressNoStack(cons);
+		testSuppressStack(cons);
+		testSuppressNoStack(cons);
+		testNoSuppressStack(cons);
+		testNoSuppressNoStack(cons);
 
 		return 1;
 	}
@@ -284,7 +284,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> void testThrowable_MessageCauseCombos(Constructor<T> cons) {
+	public final <T extends Throwable> void testMessageCauseCombos(Constructor<T> cons) {
 		T ex;
 
 		ex = newInstance(cons, null, null, true, true);
@@ -322,11 +322,11 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> void testThrowable_FlagCombos(Constructor<T> cons) {
-		testThrowable_SuppressStack(cons);
-		testThrowable_SuppressNoStack(cons);
-		testThrowable_NoSuppressStack(cons);
-		testThrowable_NoSuppressNoStack(cons);
+	public final <T extends Throwable> void testFlagCombos(Constructor<T> cons) {
+		testSuppressStack(cons);
+		testSuppressNoStack(cons);
+		testNoSuppressStack(cons);
+		testNoSuppressNoStack(cons);
 	}
 
 	/**
@@ -349,7 +349,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> void testThrowable_SuppressStack(Constructor<T> cons) {
+	public final <T extends Throwable> void testSuppressStack(Constructor<T> cons) {
 		T ex = newInstance(cons, "yes,yes", CAUSE, true, true);
 
 		ex.addSuppressed(SUPPRESSED);
@@ -384,7 +384,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> void testThrowable_SuppressNoStack(Constructor<T> cons) {
+	public final <T extends Throwable> void testSuppressNoStack(Constructor<T> cons) {
 		T ex = newInstance(cons, "yes,no", CAUSE, true, false);
 
 		ex.addSuppressed(SUPPRESSED);
@@ -419,7 +419,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> void testThrowable_NoSuppressStack(Constructor<T> cons) {
+	public final <T extends Throwable> void testNoSuppressStack(Constructor<T> cons) {
 		T ex = newInstance(cons, "no,yes", CAUSE, false, true);
 
 		ex.addSuppressed(SUPPRESSED);
@@ -452,7 +452,7 @@ public class ThrowablesTester {
 	 * @throws AssertionError
 	 *             if the constructed objects fail to pass various tests
 	 */
-	public <T extends Throwable> void testThrowable_NoSuppressNoStack(Constructor<T> cons) {
+	public final <T extends Throwable> void testNoSuppressNoStack(Constructor<T> cons) {
 		T ex = newInstance(cons, "no,no", CAUSE, false, false);
 
 		ex.addSuppressed(SUPPRESSED);