This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-logging.git


The following commit(s) were added to refs/heads/master by this push:
     new 1e987dc  Fix generics warnings in tests
1e987dc is described below

commit 1e987dcacebd4f37bef2f63ff3e0e1a18c9a7b62
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Aug 14 08:30:16 2024 -0400

    Fix generics warnings in tests
    
    Javadoc in test
---
 .../org/apache/commons/logging/AltHashtableTestCase.java |  4 ++--
 .../java/org/apache/commons/logging/LoadTestCase.java    |  8 ++++----
 .../org/apache/commons/logging/PathableClassLoader.java  |  5 ++---
 .../org/apache/commons/logging/PathableTestSuite.java    |  2 +-
 .../logging/config/FirstPriorityConfigTestCase.java      |  4 ++--
 .../commons/logging/config/PriorityConfigTestCase.java   |  4 ++--
 .../commons/logging/jdk14/CustomConfigAPITestCase.java   |  2 +-
 .../commons/logging/jdk14/CustomConfigFullTestCase.java  |  2 +-
 .../commons/logging/jdk14/CustomConfigTestCase.java      |  4 ++--
 .../commons/logging/jdk14/DefaultConfigTestCase.java     |  2 +-
 .../log4j/log4j12/ApiClasspathStandardTestCase.java      |  2 +-
 .../log4j/log4j12/AppClasspathStandardTestCase.java      |  2 +-
 .../log4j/log4j12/ChildClasspathStandardTestCase.java    |  2 +-
 .../log4j/log4j12/ParentClasspathStandardTestCase.java   |  2 +-
 .../apache/commons/logging/logkit/StandardTestCase.java  |  4 ++--
 .../commons/logging/pathable/ChildFirstTestCase.java     | 16 ++++++++--------
 .../apache/commons/logging/pathable/GeneralTestCase.java |  4 ++--
 .../logging/security/SecurityAllowedTestCase.java        |  4 ++--
 .../logging/security/SecurityForbiddenTestCase.java      |  6 +++---
 .../logging/serviceloader/internal/DummyLogFactory.java  |  2 +-
 .../commons/logging/servlet/BasicServletTestCase.java    |  2 +-
 .../commons/logging/simple/CustomConfigTestCase.java     |  4 ++--
 .../logging/simple/DateTimeCustomConfigTestCase.java     |  4 ++--
 .../commons/logging/simple/DefaultConfigTestCase.java    |  4 ++--
 .../logging/tccl/logfactory/TcclDisabledTestCase.java    |  8 ++++----
 .../logging/tccl/logfactory/TcclEnabledTestCase.java     |  8 ++++----
 26 files changed, 55 insertions(+), 56 deletions(-)

diff --git a/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java 
b/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java
index 726a7f3..c232e9a 100644
--- a/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java
+++ b/src/test/java/org/apache/commons/logging/AltHashtableTestCase.java
@@ -28,7 +28,7 @@ import junit.framework.TestCase;
 public class AltHashtableTestCase extends TestCase {
 
     public static Test suite() throws Exception {
-        final Class thisClass = AltHashtableTestCase.class;
+        final Class<AltHashtableTestCase> thisClass = 
AltHashtableTestCase.class;
         final ClassLoader thisClassLoader = thisClass.getClassLoader();
 
         final PathableClassLoader loader = new PathableClassLoader(null);
@@ -36,7 +36,7 @@ public class AltHashtableTestCase extends TestCase {
         loader.addLogicalLib("testclasses");
         loader.addLogicalLib("commons-logging");
 
-        final Class testClass = loader.loadClass(thisClass.getName());
+        final Class<?> testClass = loader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, loader);
     }
 
diff --git a/src/test/java/org/apache/commons/logging/LoadTestCase.java 
b/src/test/java/org/apache/commons/logging/LoadTestCase.java
index a2eda91..b882f39 100644
--- a/src/test/java/org/apache/commons/logging/LoadTestCase.java
+++ b/src/test/java/org/apache/commons/logging/LoadTestCase.java
@@ -77,7 +77,7 @@ public class LoadTestCase extends TestCase {
         // not very trivial to emulate we must implement "findClass",
         // but it will delegate to JUnit class loader first
         @Override
-        public Class loadClass(final String name) throws 
ClassNotFoundException {
+        public Class<?> loadClass(final String name) throws 
ClassNotFoundException {
             // isolates all logging classes, application in the same class 
loader too.
             // filters exceptions to simplify handling in test
             for (final String element : LOG_PCKG) {
@@ -96,7 +96,7 @@ public class LoadTestCase extends TestCase {
 
     private ClassLoader origContextClassLoader;
 
-    private void execute(final Class cls) throws Exception {
+    private void execute(final Class<?> cls) throws Exception {
         cls.getConstructor().newInstance();
     }
 
@@ -104,8 +104,8 @@ public class LoadTestCase extends TestCase {
      * Load class UserClass via a temporary class loader which is a child of
      * the class loader used to load this test class.
      */
-    private Class reload() throws Exception {
-        Class testObjCls = null;
+    private Class<?> reload() throws Exception {
+        Class<?> testObjCls = null;
         final AppClassLoader appLoader = new 
AppClassLoader(this.getClass().getClassLoader());
         try {
 
diff --git a/src/test/java/org/apache/commons/logging/PathableClassLoader.java 
b/src/test/java/org/apache/commons/logging/PathableClassLoader.java
index 57916f0..2247451 100644
--- a/src/test/java/org/apache/commons/logging/PathableClassLoader.java
+++ b/src/test/java/org/apache/commons/logging/PathableClassLoader.java
@@ -304,7 +304,7 @@ public class PathableClassLoader extends URLClassLoader {
      * that entries' class loader.
      */
     @Override
-    protected Class loadClass(final String name, final boolean resolve)
+    protected Class<?> loadClass(final String name, final boolean resolve)
     throws ClassNotFoundException {
         // just for performance, check java and javax
         if (name.startsWith("java.") || name.startsWith("javax.")) {
@@ -326,7 +326,7 @@ public class PathableClassLoader extends URLClassLoader {
             return super.loadClass(name, resolve);
         }
         try {
-            Class clazz = findLoadedClass(name);
+            Class<?> clazz = findLoadedClass(name);
             if (clazz == null) {
                 clazz = super.findClass(name);
             }
@@ -360,7 +360,6 @@ public class PathableClassLoader extends URLClassLoader {
     }
 
     /**
-     *
      * Clean implementation of list function of
      * {@link java.util.Collection} added in JDK 1.4
      * @param en {@code Enumeration}, possibly null
diff --git a/src/test/java/org/apache/commons/logging/PathableTestSuite.java 
b/src/test/java/org/apache/commons/logging/PathableTestSuite.java
index d2a2287..9c681dd 100644
--- a/src/test/java/org/apache/commons/logging/PathableTestSuite.java
+++ b/src/test/java/org/apache/commons/logging/PathableTestSuite.java
@@ -120,7 +120,7 @@ public class PathableTestSuite extends TestSuite {
      * calls to Thread.currentThread.getContextClassLoader from test methods
      * (or any method called by test methods).
      */
-    public PathableTestSuite(final Class testClass, final ClassLoader 
contextClassLoader) {
+    public PathableTestSuite(final Class<?> testClass, final ClassLoader 
contextClassLoader) {
         super(testClass);
         contextLoader = contextClassLoader;
     }
diff --git 
a/src/test/java/org/apache/commons/logging/config/FirstPriorityConfigTestCase.java
 
b/src/test/java/org/apache/commons/logging/config/FirstPriorityConfigTestCase.java
index aeb2ccd..bbc2f7f 100644
--- 
a/src/test/java/org/apache/commons/logging/config/FirstPriorityConfigTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/config/FirstPriorityConfigTestCase.java
@@ -43,7 +43,7 @@ public class FirstPriorityConfigTestCase extends TestCase {
      * Return the tests included in this test suite.
      */
     public static Test suite() throws Exception {
-        final Class thisClass = FirstPriorityConfigTestCase.class;
+        final Class<FirstPriorityConfigTestCase> thisClass = 
FirstPriorityConfigTestCase.class;
 
         // Determine the URL to this .class file, so that we can then
         // append the priority dirs to it. For tidiness, load this
@@ -75,7 +75,7 @@ public class FirstPriorityConfigTestCase extends TestCase {
 
         // load the test class via webapp loader, and use the webapp loader
         // as the tccl loader too.
-        final Class testClass = webappLoader.loadClass(thisClass.getName());
+        final Class<?> testClass = webappLoader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, webappLoader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/config/PriorityConfigTestCase.java 
b/src/test/java/org/apache/commons/logging/config/PriorityConfigTestCase.java
index f7d2e6e..824654d 100644
--- 
a/src/test/java/org/apache/commons/logging/config/PriorityConfigTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/config/PriorityConfigTestCase.java
@@ -50,7 +50,7 @@ public class PriorityConfigTestCase extends TestCase {
      * Return the tests included in this test suite.
      */
     public static Test suite() throws Exception {
-        final Class thisClass = PriorityConfigTestCase.class;
+        final Class<PriorityConfigTestCase> thisClass = 
PriorityConfigTestCase.class;
 
         // Determine the URL to this .class file, so that we can then
         // append the priority dirs to it. For tidiness, load this
@@ -93,7 +93,7 @@ public class PriorityConfigTestCase extends TestCase {
 
         // load the test class via webapp loader, and use the webapp loader
         // as the tccl loader too.
-        final Class testClass = webappLoader.loadClass(thisClass.getName());
+        final Class<?> testClass = webappLoader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, webappLoader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigAPITestCase.java 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigAPITestCase.java
index 2af4eb7..b374e64 100644
--- 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigAPITestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigAPITestCase.java
@@ -50,7 +50,7 @@ public class CustomConfigAPITestCase extends 
CustomConfigTestCase {
         child.addLogicalLib("testclasses");
         child.addLogicalLib("commons-logging");
 
-        final Class testClass = 
child.loadClass(CustomConfigAPITestCase.class.getName());
+        final Class<?> testClass = 
child.loadClass(CustomConfigAPITestCase.class.getName());
         return new PathableTestSuite(testClass, child);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigFullTestCase.java 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigFullTestCase.java
index f1f4dff..33f1337 100644
--- 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigFullTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigFullTestCase.java
@@ -49,7 +49,7 @@ public class CustomConfigFullTestCase extends 
CustomConfigTestCase {
         final PathableClassLoader child = new PathableClassLoader(parent);
         child.addLogicalLib("testclasses");
 
-        final Class testClass = 
child.loadClass(CustomConfigFullTestCase.class.getName());
+        final Class<?> testClass = 
child.loadClass(CustomConfigFullTestCase.class.getName());
         return new PathableTestSuite(testClass, child);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigTestCase.java 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigTestCase.java
index 1d08137..b6374c7 100644
--- a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigTestCase.java
@@ -62,7 +62,7 @@ public class CustomConfigTestCase extends 
DefaultConfigTestCase {
             final ClassLoader srcCL = 
CustomConfigAPITestCase.class.getClassLoader();
             final byte[] classData = readClass(className, srcCL);
 
-            final Class[] params = { String.class, classData.getClass(), 
Integer.TYPE, Integer.TYPE };
+            final Class<?>[] params = { String.class, classData.getClass(), 
Integer.TYPE, Integer.TYPE };
             final Method m = 
ClassLoader.class.getDeclaredMethod("defineClass", params);
 
             final Object[] args = new Object[4];
@@ -118,7 +118,7 @@ public class CustomConfigTestCase extends 
DefaultConfigTestCase {
         cl.addLogicalLib("commons-logging");
         cl.addLogicalLib("testclasses");
 
-        final Class testClass = 
cl.loadClass(CustomConfigTestCase.class.getName());
+        final Class<?> testClass = 
cl.loadClass(CustomConfigTestCase.class.getName());
         return new PathableTestSuite(testClass, cl);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/jdk14/DefaultConfigTestCase.java 
b/src/test/java/org/apache/commons/logging/jdk14/DefaultConfigTestCase.java
index 680d9c7..509517e 100644
--- a/src/test/java/org/apache/commons/logging/jdk14/DefaultConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/jdk14/DefaultConfigTestCase.java
@@ -46,7 +46,7 @@ public class DefaultConfigTestCase extends TestCase {
         loader.addLogicalLib("testclasses");
         loader.addLogicalLib("commons-logging");
 
-        final Class testClass = 
loader.loadClass(DefaultConfigTestCase.class.getName());
+        final Class<?> testClass = 
loader.loadClass(DefaultConfigTestCase.class.getName());
         return new PathableTestSuite(testClass, loader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/ApiClasspathStandardTestCase.java
 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/ApiClasspathStandardTestCase.java
index 33e02d7..10f290b 100644
--- 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/ApiClasspathStandardTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/ApiClasspathStandardTestCase.java
@@ -50,7 +50,7 @@ public class ApiClasspathStandardTestCase extends TestCase {
         child.addLogicalLib("commons-logging");
         child.addLogicalLib("testclasses");
 
-        final Class testClass = child.loadClass(
+        final Class<?> testClass = child.loadClass(
             "org.apache.commons.logging.log4j.log4j12.Log4j12StandardTests");
         return new PathableTestSuite(testClass, child);
     }
diff --git 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/AppClasspathStandardTestCase.java
 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/AppClasspathStandardTestCase.java
index a8230bd..1d79b79 100644
--- 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/AppClasspathStandardTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/AppClasspathStandardTestCase.java
@@ -46,7 +46,7 @@ public class AppClasspathStandardTestCase extends TestCase {
         loader.addLogicalLib("log4j2-core");
         loader.addLogicalLib("commons-logging");
 
-        final Class testClass = loader.loadClass(
+        final Class<?> testClass = loader.loadClass(
             "org.apache.commons.logging.log4j.log4j12.Log4j12StandardTests");
         return new PathableTestSuite(testClass, loader);
     }
diff --git 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/ChildClasspathStandardTestCase.java
 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/ChildClasspathStandardTestCase.java
index 6c66cf7..0024e9d 100644
--- 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/ChildClasspathStandardTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/ChildClasspathStandardTestCase.java
@@ -48,7 +48,7 @@ public class ChildClasspathStandardTestCase extends TestCase {
         child.addLogicalLib("log4j2-core");
         child.addLogicalLib("commons-logging");
 
-        final Class testClass = child.loadClass(
+        final Class<?> testClass = child.loadClass(
             "org.apache.commons.logging.log4j.log4j12.Log4j12StandardTests");
         return new PathableTestSuite(testClass, child);
     }
diff --git 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/ParentClasspathStandardTestCase.java
 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/ParentClasspathStandardTestCase.java
index 76c1b31..c9918ad 100644
--- 
a/src/test/java/org/apache/commons/logging/log4j/log4j12/ParentClasspathStandardTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/log4j/log4j12/ParentClasspathStandardTestCase.java
@@ -48,7 +48,7 @@ public class ParentClasspathStandardTestCase extends TestCase 
{
         final PathableClassLoader child = new PathableClassLoader(parent);
         child.addLogicalLib("testclasses");
 
-        final Class testClass = child.loadClass(
+        final Class<?> testClass = child.loadClass(
             "org.apache.commons.logging.log4j.log4j12.Log4j12StandardTests");
         return new PathableTestSuite(testClass, child);
     }
diff --git 
a/src/test/java/org/apache/commons/logging/logkit/StandardTestCase.java 
b/src/test/java/org/apache/commons/logging/logkit/StandardTestCase.java
index 12dbb1c..597cbfd 100644
--- a/src/test/java/org/apache/commons/logging/logkit/StandardTestCase.java
+++ b/src/test/java/org/apache/commons/logging/logkit/StandardTestCase.java
@@ -41,7 +41,7 @@ public class StandardTestCase extends AbstractLogTest {
      * Return the tests included in this test suite.
      */
     public static Test suite() throws Exception {
-        final Class thisClass = StandardTestCase.class;
+        final Class<StandardTestCase> thisClass = StandardTestCase.class;
 
         final PathableClassLoader loader = new PathableClassLoader(null);
         loader.useExplicitLoader("junit.", Test.class.getClassLoader());
@@ -49,7 +49,7 @@ public class StandardTestCase extends AbstractLogTest {
         loader.addLogicalLib("commons-logging");
         loader.addLogicalLib("logkit");
 
-        final Class testClass = loader.loadClass(thisClass.getName());
+        final Class<?> testClass = loader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, loader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/pathable/ChildFirstTestCase.java 
b/src/test/java/org/apache/commons/logging/pathable/ChildFirstTestCase.java
index 4f2e6f8..35f0c12 100644
--- a/src/test/java/org/apache/commons/logging/pathable/ChildFirstTestCase.java
+++ b/src/test/java/org/apache/commons/logging/pathable/ChildFirstTestCase.java
@@ -53,7 +53,7 @@ public class ChildFirstTestCase extends TestCase {
      * </ul>
      */
     public static Test suite() throws Exception {
-        final Class thisClass = ChildFirstTestCase.class;
+        final Class<ChildFirstTestCase> thisClass = ChildFirstTestCase.class;
         final ClassLoader thisClassLoader = thisClass.getClassLoader();
 
         // Make the parent a direct child of the bootloader to hide all
@@ -86,7 +86,7 @@ public class ChildFirstTestCase extends TestCase {
         context.setParentFirst(false);
 
         // reload this class via the child class loader
-        final Class testClass = child.loadClass(thisClass.getName());
+        final Class<?> testClass = child.loadClass(thisClass.getName());
 
         // and return our custom TestSuite class
         return new PathableTestSuite(testClass, context);
@@ -167,30 +167,30 @@ public class ChildFirstTestCase extends TestCase {
         // junit classes should be visible; their class loader is not
         // in the hierarchy of parent class loaders for this class,
         // though it is accessible due to trickery in the PathableClassLoader.
-        final Class junitTest = 
contextLoader.loadClass("junit.framework.Test");
+        final Class<?> junitTest = 
contextLoader.loadClass("junit.framework.Test");
         final Set<ClassLoader> ancestorCLs = getAncestorCLs();
         assertFalse("Junit not loaded by ancestor class loader",
                 ancestorCLs.contains(junitTest.getClassLoader()));
 
         // jcl api classes should be visible only via the parent
-        final Class logClass = 
contextLoader.loadClass("org.apache.commons.logging.Log");
+        final Class<?> logClass = 
contextLoader.loadClass("org.apache.commons.logging.Log");
         assertSame("Log class not loaded via parent",
                 logClass.getClassLoader(), parentLoader);
 
         // jcl adapter classes should be visible via both parent and child. 
However
         // as the class loaders are child-first we should see the child one.
-        final Class log4jClass = 
contextLoader.loadClass("org.apache.commons.logging.impl.Log4JLogger");
+        final Class<?> log4jClass = 
contextLoader.loadClass("org.apache.commons.logging.impl.Log4JLogger");
         assertSame("Log4JLogger not loaded via child",
                 log4jClass.getClassLoader(), thisLoader);
 
         // test classes should be visible via the child only
-        final Class testClass = 
contextLoader.loadClass("org.apache.commons.logging.PathableTestSuite");
+        final Class<?> testClass = 
contextLoader.loadClass("org.apache.commons.logging.PathableTestSuite");
         assertSame("PathableTestSuite not loaded via child",
                 testClass.getClassLoader(), thisLoader);
 
         // test loading of class that is not available
         try {
-            final Class noSuchClass = contextLoader.loadClass("no.such.class");
+            final Class<?> noSuchClass = 
contextLoader.loadClass("no.such.class");
             fail("Class no.such.class is unexpectedly available");
             assertNotNull(noSuchClass); // silence warning about unused var
         } catch (final ClassNotFoundException ex) {
@@ -198,7 +198,7 @@ public class ChildFirstTestCase extends TestCase {
         }
 
         // String class class loader is null
-        final Class stringClass = contextLoader.loadClass("java.lang.String");
+        final Class<?> stringClass = 
contextLoader.loadClass("java.lang.String");
         assertNull("String class class loader is not null!",
                 stringClass.getClassLoader());
     }
diff --git 
a/src/test/java/org/apache/commons/logging/pathable/GeneralTestCase.java 
b/src/test/java/org/apache/commons/logging/pathable/GeneralTestCase.java
index 1d99932..734edaa 100644
--- a/src/test/java/org/apache/commons/logging/pathable/GeneralTestCase.java
+++ b/src/test/java/org/apache/commons/logging/pathable/GeneralTestCase.java
@@ -60,7 +60,7 @@ public class GeneralTestCase extends TestCase {
      * Sets up a custom class loader hierarchy for this test case.
      */
     public static Test suite() throws Exception {
-        final Class thisClass = GeneralTestCase.class;
+        final Class<GeneralTestCase> thisClass = GeneralTestCase.class;
         final ClassLoader thisClassLoader = thisClass.getClassLoader();
 
         final PathableClassLoader loader = new PathableClassLoader(null);
@@ -68,7 +68,7 @@ public class GeneralTestCase extends TestCase {
         loader.addLogicalLib("testclasses");
 
         // reload this class via the child class loader
-        final Class testClass = loader.loadClass(thisClass.getName());
+        final Class<?> testClass = loader.loadClass(thisClass.getName());
 
         // and return our custom TestSuite class
         return new PathableTestSuite(testClass, loader);
diff --git 
a/src/test/java/org/apache/commons/logging/security/SecurityAllowedTestCase.java
 
b/src/test/java/org/apache/commons/logging/security/SecurityAllowedTestCase.java
index 28c0ba4..247bd8d 100644
--- 
a/src/test/java/org/apache/commons/logging/security/SecurityAllowedTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/security/SecurityAllowedTestCase.java
@@ -62,7 +62,7 @@ public class SecurityAllowedTestCase extends TestCase {
         parent.addLogicalLib("commons-logging");
         parent.addLogicalLib("testclasses");
 
-        final Class testClass = parent.loadClass(
+        final Class<?> testClass = parent.loadClass(
             "org.apache.commons.logging.security.SecurityAllowedTestCase");
         return new PathableTestSuite(testClass, parent);
     }
@@ -101,7 +101,7 @@ public class SecurityAllowedTestCase extends TestCase {
         try {
             // Use reflection so that we can control exactly when the static
             // initializer for the LogFactory class is executed.
-            final Class c = this.getClass().getClassLoader().loadClass(
+            final Class<?> c = this.getClass().getClassLoader().loadClass(
                     "org.apache.commons.logging.LogFactory");
             final Method m = c.getMethod("getLog", Class.class);
             final Log log = (Log) m.invoke(null, this.getClass());
diff --git 
a/src/test/java/org/apache/commons/logging/security/SecurityForbiddenTestCase.java
 
b/src/test/java/org/apache/commons/logging/security/SecurityForbiddenTestCase.java
index a940ec0..9176f2b 100644
--- 
a/src/test/java/org/apache/commons/logging/security/SecurityForbiddenTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/security/SecurityForbiddenTestCase.java
@@ -71,7 +71,7 @@ public class SecurityForbiddenTestCase extends TestCase {
         parent.addLogicalLib("commons-logging");
         parent.addLogicalLib("testclasses");
 
-        final Class testClass = parent.loadClass(
+        final Class<?> testClass = parent.loadClass(
             "org.apache.commons.logging.security.SecurityForbiddenTestCase");
         return new PathableTestSuite(testClass, parent);
     }
@@ -85,7 +85,7 @@ public class SecurityForbiddenTestCase extends TestCase {
      */
     private Object loadClass(final String name, final ClassLoader classLoader) 
{
         try {
-            final Class clazz = classLoader.loadClass(name);
+            final Class<?> clazz = classLoader.loadClass(name);
             return clazz.getConstructor().newInstance();
         } catch (final Exception e) {
             final StringWriter sw = new StringWriter();
@@ -135,7 +135,7 @@ public class SecurityForbiddenTestCase extends TestCase {
         try {
             // Use reflection so that we can control exactly when the static
             // initializer for the LogFactory class is executed.
-            final Class c = this.getClass().getClassLoader().loadClass(
+            final Class<?> c = this.getClass().getClassLoader().loadClass(
                     "org.apache.commons.logging.LogFactory");
             final Method m = c.getMethod("getLog", Class.class);
             final Log log = (Log) m.invoke(null, this.getClass());
diff --git 
a/src/test/java/org/apache/commons/logging/serviceloader/internal/DummyLogFactory.java
 
b/src/test/java/org/apache/commons/logging/serviceloader/internal/DummyLogFactory.java
index 409e63b..2c169a8 100644
--- 
a/src/test/java/org/apache/commons/logging/serviceloader/internal/DummyLogFactory.java
+++ 
b/src/test/java/org/apache/commons/logging/serviceloader/internal/DummyLogFactory.java
@@ -32,7 +32,7 @@ public class DummyLogFactory extends LogFactory {
     }
 
     @Override
-    public Log getInstance(final Class clazz) throws LogConfigurationException 
{
+    public Log getInstance(final Class<?> clazz) throws 
LogConfigurationException {
         return null;
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/servlet/BasicServletTestCase.java 
b/src/test/java/org/apache/commons/logging/servlet/BasicServletTestCase.java
index 4157946..18dbbfd 100644
--- a/src/test/java/org/apache/commons/logging/servlet/BasicServletTestCase.java
+++ b/src/test/java/org/apache/commons/logging/servlet/BasicServletTestCase.java
@@ -56,7 +56,7 @@ public class BasicServletTestCase extends TestCase {
         tccl.setParentFirst(false);
         tccl.addLogicalLib("commons-logging");
 
-        final Class testClass = 
child.loadClass(BasicServletTestCase.class.getName());
+        final Class<?> testClass = 
child.loadClass(BasicServletTestCase.class.getName());
         return new PathableTestSuite(testClass, tccl);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/simple/CustomConfigTestCase.java 
b/src/test/java/org/apache/commons/logging/simple/CustomConfigTestCase.java
index 1d159a3..5e1a1ba 100644
--- a/src/test/java/org/apache/commons/logging/simple/CustomConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/simple/CustomConfigTestCase.java
@@ -48,14 +48,14 @@ public class CustomConfigTestCase extends 
DefaultConfigTestCase {
      * Or we could fix SimpleLog to be sane...
      */
     public static Test suite() throws Exception {
-        final Class thisClass = CustomConfigTestCase.class;
+        final Class<CustomConfigTestCase> thisClass = 
CustomConfigTestCase.class;
 
         final PathableClassLoader loader = new PathableClassLoader(null);
         loader.useExplicitLoader("junit.", Test.class.getClassLoader());
         loader.addLogicalLib("testclasses");
         loader.addLogicalLib("commons-logging");
 
-        final Class testClass = loader.loadClass(thisClass.getName());
+        final Class<?> testClass = loader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, loader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/simple/DateTimeCustomConfigTestCase.java
 
b/src/test/java/org/apache/commons/logging/simple/DateTimeCustomConfigTestCase.java
index f29abef..fca58fb 100644
--- 
a/src/test/java/org/apache/commons/logging/simple/DateTimeCustomConfigTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/simple/DateTimeCustomConfigTestCase.java
@@ -44,14 +44,14 @@ public class DateTimeCustomConfigTestCase extends 
CustomConfigTestCase {
      * Or we could fix SimpleLog to be sane...
      */
     public static Test suite() throws Exception {
-        final Class thisClass = DateTimeCustomConfigTestCase.class;
+        final Class<DateTimeCustomConfigTestCase> thisClass = 
DateTimeCustomConfigTestCase.class;
 
         final PathableClassLoader loader = new PathableClassLoader(null);
         loader.useExplicitLoader("junit.", Test.class.getClassLoader());
         loader.addLogicalLib("testclasses");
         loader.addLogicalLib("commons-logging");
 
-        final Class testClass = loader.loadClass(thisClass.getName());
+        final Class<?> testClass = loader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, loader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/simple/DefaultConfigTestCase.java 
b/src/test/java/org/apache/commons/logging/simple/DefaultConfigTestCase.java
index 60562dc..74b2042 100644
--- a/src/test/java/org/apache/commons/logging/simple/DefaultConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/simple/DefaultConfigTestCase.java
@@ -51,14 +51,14 @@ public class DefaultConfigTestCase extends TestCase {
      * Or we could fix SimpleLog to be sane...
      */
     public static Test suite() throws Exception {
-        final Class thisClass = DefaultConfigTestCase.class;
+        final Class<DefaultConfigTestCase> thisClass = 
DefaultConfigTestCase.class;
 
         final PathableClassLoader loader = new PathableClassLoader(null);
         loader.useExplicitLoader("junit.", Test.class.getClassLoader());
         loader.addLogicalLib("testclasses");
         loader.addLogicalLib("commons-logging");
 
-        final Class testClass = loader.loadClass(thisClass.getName());
+        final Class<?> testClass = loader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, loader);
     }
 
diff --git 
a/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclDisabledTestCase.java
 
b/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclDisabledTestCase.java
index bc3c795..299cb6d 100644
--- 
a/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclDisabledTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclDisabledTestCase.java
@@ -42,7 +42,7 @@ public class TcclDisabledTestCase extends TestCase {
      * Returns the tests included in this test suite.
      */
     public static Test suite() throws Exception {
-        final Class thisClass = TcclDisabledTestCase.class;
+        final Class<TcclDisabledTestCase> thisClass = 
TcclDisabledTestCase.class;
 
         // Determine the URL to this .class file, so that we can then
         // append the priority dirs to it. For tidiness, load this
@@ -80,7 +80,7 @@ public class TcclDisabledTestCase extends TestCase {
         final PathableClassLoader tcclLoader = new 
PathableClassLoader(parentLoader);
         tcclLoader.addLogicalLib("testclasses");
 
-        final Class testClass = parentLoader.loadClass(thisClass.getName());
+        final Class<?> testClass = parentLoader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, tcclLoader);
     }
 
@@ -113,7 +113,7 @@ public class TcclDisabledTestCase extends TestCase {
 
         // MyLogFactoryImpl should not be loadable via parent loader
         try {
-            final Class clazz = thisClassLoader.loadClass(MY_LOG_FACTORY_IMPL);
+            final Class<?> clazz = 
thisClassLoader.loadClass(MY_LOG_FACTORY_IMPL);
             fail("Unexpectedly able to load MyLogFactoryImpl via test class 
class loader");
             assertNotNull(clazz); // silence warning about unused var
         } catch (final ClassNotFoundException ex) {
@@ -122,7 +122,7 @@ public class TcclDisabledTestCase extends TestCase {
 
         // MyLogFactoryImpl should be loadable via TCCL loader
         try {
-            final Class clazz = tcclLoader.loadClass(MY_LOG_FACTORY_IMPL);
+            final Class<?> clazz = tcclLoader.loadClass(MY_LOG_FACTORY_IMPL);
             assertNotNull(clazz);
         } catch (final ClassNotFoundException ex) {
             fail("Unexpectedly unable to load MyLogFactoryImpl via TCCL class 
loader");
diff --git 
a/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclEnabledTestCase.java
 
b/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclEnabledTestCase.java
index a0c5b7c..14a5878 100644
--- 
a/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclEnabledTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/tccl/logfactory/TcclEnabledTestCase.java
@@ -36,7 +36,7 @@ public class TcclEnabledTestCase extends TestCase {
      * Return the tests included in this test suite.
      */
     public static Test suite() throws Exception {
-        final Class thisClass = TcclEnabledTestCase.class;
+        final Class<TcclEnabledTestCase> thisClass = TcclEnabledTestCase.class;
 
         // Determine the URL to this .class file, so that we can then
         // append the priority dirs to it. For tidiness, load this
@@ -74,7 +74,7 @@ public class TcclEnabledTestCase extends TestCase {
         final PathableClassLoader tcclLoader = new 
PathableClassLoader(parentLoader);
         tcclLoader.addLogicalLib("testclasses");
 
-        final Class testClass = parentLoader.loadClass(thisClass.getName());
+        final Class<?> testClass = parentLoader.loadClass(thisClass.getName());
         return new PathableTestSuite(testClass, tcclLoader);
     }
 
@@ -107,7 +107,7 @@ public class TcclEnabledTestCase extends TestCase {
 
         // MyLogFactoryImpl should not be loadable via parent loader
         try {
-            final Class clazz = thisClassLoader.loadClass(
+            final Class<?> clazz = thisClassLoader.loadClass(
                 "org.apache.commons.logging.tccl.custom.MyLogFactoryImpl");
             fail("Unexpectedly able to load MyLogFactoryImpl via test class 
class loader");
             assertNotNull(clazz); // silence warning about unused var
@@ -117,7 +117,7 @@ public class TcclEnabledTestCase extends TestCase {
 
         // MyLogFactoryImpl should be loadable via TCCL loader
         try {
-            final Class clazz = tcclLoader.loadClass(
+            final Class<?> clazz = tcclLoader.loadClass(
                 "org.apache.commons.logging.tccl.custom.MyLogFactoryImpl");
             assertNotNull(clazz);
         } catch (final ClassNotFoundException ex) {


Reply via email to