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

commit d67eefa4e2c9b6a9db41fb9f2edd137f494de233
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sat May 11 11:28:09 2024 -0400

    Sort imports
    
    Whitespace
---
 src/main/java/org/apache/commons/logging/LogSource.java     |  2 +-
 .../apache/commons/logging/avalon/AvalonLoggerTestCase.java |  6 +++---
 .../commons/logging/config/FirstPriorityConfigTestCase.java |  6 +++---
 .../commons/logging/config/PriorityConfigTestCase.java      |  6 +++---
 .../commons/logging/jdk14/CustomConfigAPITestCase.java      |  4 ++--
 .../commons/logging/jdk14/CustomConfigFullTestCase.java     |  4 ++--
 .../apache/commons/logging/jdk14/CustomConfigTestCase.java  |  4 ++--
 .../apache/commons/logging/jdk14/DefaultConfigTestCase.java |  6 +++---
 .../org/apache/commons/logging/log4j/StandardTests.java     |  4 ++--
 .../logging/log4j/log4j12/ApiClasspathStandardTestCase.java |  6 +++---
 .../logging/log4j/log4j12/AppClasspathStandardTestCase.java |  6 +++---
 .../log4j/log4j12/ChildClasspathStandardTestCase.java       |  6 +++---
 .../log4j/log4j12/ParentClasspathStandardTestCase.java      |  6 +++---
 .../commons/logging/log4j2/CallerInformationTestCase.java   |  8 +++-----
 .../org/apache/commons/logging/logkit/StandardTestCase.java |  4 ++--
 .../apache/commons/logging/pathable/ChildFirstTestCase.java |  6 +++---
 .../apache/commons/logging/pathable/GeneralTestCase.java    |  6 +++---
 .../commons/logging/pathable/ParentFirstTestCase.java       |  6 +++---
 .../commons/logging/security/SecurityAllowedTestCase.java   |  6 +++---
 .../commons/logging/security/SecurityForbiddenTestCase.java |  6 +++---
 .../logging/serviceloader/ServiceLoaderTestCase.java        |  4 ++--
 .../commons/logging/servlet/BasicServletTestCase.java       |  6 +++---
 .../apache/commons/logging/simple/CustomConfigTestCase.java |  4 ++--
 .../logging/simple/DateTimeCustomConfigTestCase.java        |  4 ++--
 .../commons/logging/simple/DefaultConfigTestCase.java       |  6 +++---
 .../commons/logging/slf4j/CallerInformationTestCase.java    | 13 +++++--------
 .../org/apache/commons/logging/tccl/BadTCCLTestCase.java    |  6 +++---
 .../org/apache/commons/logging/tccl/NullTCCLTestCase.java   |  6 +++---
 .../commons/logging/tccl/log/TcclDisabledTestCase.java      |  6 +++---
 .../commons/logging/tccl/log/TcclEnabledTestCase.java       |  6 +++---
 .../logging/tccl/logfactory/TcclDisabledTestCase.java       |  6 +++---
 .../logging/tccl/logfactory/TcclEnabledTestCase.java        |  6 +++---
 32 files changed, 88 insertions(+), 93 deletions(-)

diff --git a/src/main/java/org/apache/commons/logging/LogSource.java 
b/src/main/java/org/apache/commons/logging/LogSource.java
index 586573f..49084ec 100644
--- a/src/main/java/org/apache/commons/logging/LogSource.java
+++ b/src/main/java/org/apache/commons/logging/LogSource.java
@@ -61,7 +61,7 @@ public class LogSource {
     /** Is Log4j available (in the current classpath) */
     static protected boolean log4jIsAvailable;
 
-    /** 
+    /**
      * Is JDK 1.4 logging available, always true.
      *
      * @deprecated Java 8 is the baseline and includes JUL.
diff --git 
a/src/test/java/org/apache/commons/logging/avalon/AvalonLoggerTestCase.java 
b/src/test/java/org/apache/commons/logging/avalon/AvalonLoggerTestCase.java
index 9156cc4..15047e4 100644
--- a/src/test/java/org/apache/commons/logging/avalon/AvalonLoggerTestCase.java
+++ b/src/test/java/org/apache/commons/logging/avalon/AvalonLoggerTestCase.java
@@ -16,14 +16,14 @@
  */
 package org.apache.commons.logging.avalon;
 
+import junit.framework.Test;
+import junit.framework.TestSuite;
+
 import org.apache.avalon.framework.logger.NullLogger;
 import org.apache.commons.logging.AbstractLogTest;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.impl.AvalonLogger;
 
-import junit.framework.Test;
-import junit.framework.TestSuite;
-
 /**
  */
 public class AvalonLoggerTestCase extends AbstractLogTest {
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 47f3803..aeb2ccd 100644
--- 
a/src/test/java/org/apache/commons/logging/config/FirstPriorityConfigTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/config/FirstPriorityConfigTestCase.java
@@ -19,13 +19,13 @@ package org.apache.commons.logging.config;
 
 import java.net.URL;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests that verify that the process of configuring logging on startup
  * works correctly by selecting the file with the highest priority.
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 440ba00..f7d2e6e 100644
--- 
a/src/test/java/org/apache/commons/logging/config/PriorityConfigTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/config/PriorityConfigTestCase.java
@@ -19,13 +19,13 @@ package org.apache.commons.logging.config;
 
 import java.net.URL;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests that verify that the process of configuring logging on startup
  * works correctly by selecting the file with the highest priority.
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 4287446..2af4eb7 100644
--- 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigAPITestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigAPITestCase.java
@@ -17,11 +17,11 @@
 
 package org.apache.commons.logging.jdk14;
 
+import junit.framework.Test;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-
 /**
  * TestCase for Jdk14 logging when the commons-logging-api jar file is in
  * the parent classpath and commons-logging.jar is in the 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 4fd703e..f1f4dff 100644
--- 
a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigFullTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigFullTestCase.java
@@ -17,11 +17,11 @@
 
 package org.apache.commons.logging.jdk14;
 
+import junit.framework.Test;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-
 /**
  * TestCase for Jdk14 logging when the commons-logging jar file is in
  * the parent classpath.
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 4781907..1d08137 100644
--- a/src/test/java/org/apache/commons/logging/jdk14/CustomConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/jdk14/CustomConfigTestCase.java
@@ -27,12 +27,12 @@ import java.util.logging.LogManager;
 import java.util.logging.LogRecord;
 import java.util.logging.Logger;
 
+import junit.framework.Test;
+
 import org.apache.commons.logging.DummyException;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-
 /**
  * <p>TestCase for JDK 1.4 logging when running on a JDK 1.4 system with
  * custom configuration, so that JDK 1.4 should be selected and an appropriate
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 70436ad..680d9c7 100644
--- a/src/test/java/org/apache/commons/logging/jdk14/DefaultConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/jdk14/DefaultConfigTestCase.java
@@ -22,14 +22,14 @@ import java.io.ByteArrayOutputStream;
 import java.io.ObjectInputStream;
 import java.io.ObjectOutputStream;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * <p>TestCase for JDK 1.4 logging when running on a JDK 1.4 system with
  * zero configuration, and with Log4J not present (so JDK 1.4 logging
diff --git a/src/test/java/org/apache/commons/logging/log4j/StandardTests.java 
b/src/test/java/org/apache/commons/logging/log4j/StandardTests.java
index 7bb6922..b9f66be 100644
--- a/src/test/java/org/apache/commons/logging/log4j/StandardTests.java
+++ b/src/test/java/org/apache/commons/logging/log4j/StandardTests.java
@@ -24,12 +24,12 @@ import java.io.ObjectOutputStream;
 import java.util.ArrayList;
 import java.util.List;
 
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.DummyException;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
-import junit.framework.TestCase;
-
 /**
  * Abstract set of tests that can be executed with various classpaths set.
  * <p>
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 9d555a5..33e02d7 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
@@ -17,14 +17,14 @@
 
 package org.apache.commons.logging.log4j.log4j12;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.Log4JLogger;
 import org.apache.commons.logging.impl.LogFactoryImpl;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for Log4J logging that emulate a webapp running within
  * a container where the commons-logging-api jar file is in
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 cd9d402..a8230bd 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
@@ -17,14 +17,14 @@
 
 package org.apache.commons.logging.log4j.log4j12;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.Log4JLogger;
 import org.apache.commons.logging.impl.LogFactoryImpl;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for Log4J logging when there is only one class loader and everything
  * is in it, as would be the situation for a standalone application.
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 600e1ce..6c66cf7 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
@@ -17,14 +17,14 @@
 
 package org.apache.commons.logging.log4j.log4j12;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.Log4JLogger;
 import org.apache.commons.logging.impl.LogFactoryImpl;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for Log4J logging that emulate a webapp running within
  * a container where all the necessary libs are in the 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 8343d10..76c1b31 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
@@ -17,14 +17,14 @@
 
 package org.apache.commons.logging.log4j.log4j12;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.Log4JLogger;
 import org.apache.commons.logging.impl.LogFactoryImpl;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for Log4J logging that emulate a webapp running within
  * a container where all the necessary libs are in the parent.
diff --git 
a/src/test/java/org/apache/commons/logging/log4j2/CallerInformationTestCase.java
 
b/src/test/java/org/apache/commons/logging/log4j2/CallerInformationTestCase.java
index 9bad253..23b1277 100644
--- 
a/src/test/java/org/apache/commons/logging/log4j2/CallerInformationTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/log4j2/CallerInformationTestCase.java
@@ -18,6 +18,8 @@ package org.apache.commons.logging.log4j2;
 
 import java.util.List;
 
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.impl.Log4jApiLogFactory;
@@ -31,8 +33,6 @@ import 
org.apache.logging.log4j.core.test.appender.ListAppender;
 import org.apache.logging.log4j.message.ObjectMessage;
 import org.apache.logging.log4j.message.SimpleMessage;
 
-import junit.framework.TestCase;
-
 public class CallerInformationTestCase extends TestCase {
 
     private static final Object OBJ = new Object();
@@ -90,9 +90,7 @@ public class CallerInformationTestCase extends TestCase {
                 assertEquals("Correct source file.", 
"CallerInformationTestCase.java", location.getFileName());
                 assertEquals("Correct method name.", "testLocationInfo", 
location.getMethodName());
                 assertEquals("Correct location class.", getClass().getName(), 
location.getClassName());
-                assertEquals("Correct location line.",
-                        currentLineNumber + 2 * lev + hasThrowable + 1,
-                        location.getLineNumber());
+                assertEquals("Correct location line.", currentLineNumber + 2 * 
lev + hasThrowable + 1, location.getLineNumber());
                 assertEquals("Correct exception", hasThrowable > 0 ? T : null, 
event.getThrown());
             }
         }
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 7feb9d4..12dbb1c 100644
--- a/src/test/java/org/apache/commons/logging/logkit/StandardTestCase.java
+++ b/src/test/java/org/apache/commons/logging/logkit/StandardTestCase.java
@@ -22,6 +22,8 @@ import java.io.ByteArrayOutputStream;
 import java.io.ObjectInputStream;
 import java.io.ObjectOutputStream;
 
+import junit.framework.Test;
+
 import org.apache.commons.logging.AbstractLogTest;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
@@ -29,8 +31,6 @@ import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.LogKitLogger;
 
-import junit.framework.Test;
-
 /**
  * Basic tests for Avalon LogKit logger adapter.
  */
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 a0a85ca..49262e3 100644
--- a/src/test/java/org/apache/commons/logging/pathable/ChildFirstTestCase.java
+++ b/src/test/java/org/apache/commons/logging/pathable/ChildFirstTestCase.java
@@ -23,13 +23,13 @@ import java.util.Enumeration;
 import java.util.HashSet;
 import java.util.Set;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Artifacts;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for the PathableTestSuite and PathableClassLoader functionality,
  * where lookup order for the PathableClassLoader is child-first.
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 ed75cdd..1d99932 100644
--- a/src/test/java/org/apache/commons/logging/pathable/GeneralTestCase.java
+++ b/src/test/java/org/apache/commons/logging/pathable/GeneralTestCase.java
@@ -19,12 +19,12 @@ package org.apache.commons.logging.pathable;
 import java.net.URL;
 import java.net.URLClassLoader;
 
-import org.apache.commons.logging.PathableClassLoader;
-import org.apache.commons.logging.PathableTestSuite;
-
 import junit.framework.Test;
 import junit.framework.TestCase;
 
+import org.apache.commons.logging.PathableClassLoader;
+import org.apache.commons.logging.PathableTestSuite;
+
 /**
  * Tests for the PathableTestSuite class.
  */
diff --git 
a/src/test/java/org/apache/commons/logging/pathable/ParentFirstTestCase.java 
b/src/test/java/org/apache/commons/logging/pathable/ParentFirstTestCase.java
index a7a8983..77d7f67 100644
--- a/src/test/java/org/apache/commons/logging/pathable/ParentFirstTestCase.java
+++ b/src/test/java/org/apache/commons/logging/pathable/ParentFirstTestCase.java
@@ -25,13 +25,13 @@ import java.util.Enumeration;
 import java.util.HashSet;
 import java.util.Set;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Artifacts;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for the PathableTestSuite and PathableClassLoader functionality,
  * where lookup order for the PathableClassLoader is parent-first.
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 a71a42a..fe4e5ff 100644
--- 
a/src/test/java/org/apache/commons/logging/security/SecurityAllowedTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/security/SecurityAllowedTestCase.java
@@ -24,14 +24,14 @@ import java.lang.reflect.Method;
 import java.security.AllPermission;
 import java.util.Hashtable;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for logging with a security policy that allows JCL access to 
everything.
  * <p>
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 04a96c1..ee1afb8 100644
--- 
a/src/test/java/org/apache/commons/logging/security/SecurityForbiddenTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/security/SecurityForbiddenTestCase.java
@@ -25,14 +25,14 @@ import java.lang.reflect.Field;
 import java.lang.reflect.Method;
 import java.util.Hashtable;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for logging with a security policy that forbids JCL access to 
anything.
  * <p>
diff --git 
a/src/test/java/org/apache/commons/logging/serviceloader/ServiceLoaderTestCase.java
 
b/src/test/java/org/apache/commons/logging/serviceloader/ServiceLoaderTestCase.java
index 7ac7c4f..24b561d 100644
--- 
a/src/test/java/org/apache/commons/logging/serviceloader/ServiceLoaderTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/serviceloader/ServiceLoaderTestCase.java
@@ -16,11 +16,11 @@
  */
 package org.apache.commons.logging.serviceloader;
 
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.serviceloader.internal.DummyLogFactory;
 
-import junit.framework.TestCase;
-
 public class ServiceLoaderTestCase extends TestCase {
 
     public void testServiceLoader() {
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 9b39508..4157946 100644
--- a/src/test/java/org/apache/commons/logging/servlet/BasicServletTestCase.java
+++ b/src/test/java/org/apache/commons/logging/servlet/BasicServletTestCase.java
@@ -17,13 +17,13 @@
 
 package org.apache.commons.logging.servlet;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.ServletContextCleaner;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Tests for ServletContextCleaner utility class.
  */
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 a11a5b2..9b45fe4 100644
--- a/src/test/java/org/apache/commons/logging/simple/CustomConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/simple/CustomConfigTestCase.java
@@ -21,14 +21,14 @@ import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
 
+import junit.framework.Test;
+
 import org.apache.commons.logging.DummyException;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.SimpleLog;
 
-import junit.framework.Test;
-
 /**
  * <p>TestCase for simple logging when running with custom configuration
  * properties.</p>
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 cc80518..f29abef 100644
--- 
a/src/test/java/org/apache/commons/logging/simple/DateTimeCustomConfigTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/simple/DateTimeCustomConfigTestCase.java
@@ -21,11 +21,11 @@ import java.text.DateFormat;
 import java.text.SimpleDateFormat;
 import java.util.Date;
 
+import junit.framework.Test;
+
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-
 /**
  * Tests custom date time format configuration
  */
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 2ac9e8a..60562dc 100644
--- a/src/test/java/org/apache/commons/logging/simple/DefaultConfigTestCase.java
+++ b/src/test/java/org/apache/commons/logging/simple/DefaultConfigTestCase.java
@@ -22,15 +22,15 @@ import java.io.ByteArrayOutputStream;
 import java.io.ObjectInputStream;
 import java.io.ObjectOutputStream;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 import org.apache.commons.logging.impl.SimpleLog;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * <p>TestCase for simple logging when running with zero configuration
  * other than selecting the SimpleLog implementation.</p>
diff --git 
a/src/test/java/org/apache/commons/logging/slf4j/CallerInformationTestCase.java 
b/src/test/java/org/apache/commons/logging/slf4j/CallerInformationTestCase.java
index fd36345..eebe98f 100644
--- 
a/src/test/java/org/apache/commons/logging/slf4j/CallerInformationTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/slf4j/CallerInformationTestCase.java
@@ -20,6 +20,8 @@ import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.impl.Slf4jLogFactory;
@@ -35,7 +37,6 @@ import ch.qos.logback.classic.spi.ThrowableProxy;
 import ch.qos.logback.core.filter.Filter;
 import ch.qos.logback.core.read.ListAppender;
 import ch.qos.logback.core.spi.FilterReply;
-import junit.framework.TestCase;
 
 public class CallerInformationTestCase extends TestCase {
 
@@ -75,7 +76,7 @@ public class CallerInformationTestCase extends TestCase {
     public void testLocationInfo() {
         appender.list.clear();
         // The following value must match the line number
-        final int currentLineNumber = 78;
+        final int currentLineNumber = 79;
         log.fatal(STRING);
         log.fatal(STRING, T);
         log.error(STRING);
@@ -100,13 +101,9 @@ public class CallerInformationTestCase extends TestCase {
                 assertTrue("Has location", callerData != null && 
callerData.length > 0);
                 final StackTraceElement location = callerData[0];
                 assertEquals("Correct location class.", getClass().getName(), 
location.getClassName());
-                assertEquals("Correct location line.",
-                        currentLineNumber + 2 * lev + hasThrowable + 1,
-                        location.getLineNumber());
+                assertEquals("Correct location line.", currentLineNumber + 2 * 
lev + hasThrowable + 1, location.getLineNumber());
                 final ThrowableProxy throwableProxy = (ThrowableProxy) 
event.getThrowableProxy();
-                assertEquals("Correct exception",
-                        hasThrowable > 0 ? T : null,
-                        throwableProxy != null ? throwableProxy.getThrowable() 
: null);
+                assertEquals("Correct exception", hasThrowable > 0 ? T : null, 
throwableProxy != null ? throwableProxy.getThrowable() : null);
             }
         }
     }
diff --git a/src/test/java/org/apache/commons/logging/tccl/BadTCCLTestCase.java 
b/src/test/java/org/apache/commons/logging/tccl/BadTCCLTestCase.java
index f2d8dcb..4c426f4 100644
--- a/src/test/java/org/apache/commons/logging/tccl/BadTCCLTestCase.java
+++ b/src/test/java/org/apache/commons/logging/tccl/BadTCCLTestCase.java
@@ -16,14 +16,14 @@
  */
 package org.apache.commons.logging.tccl;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Simulates the case when TCCL is badly set and cannot load JCL.
  */
diff --git 
a/src/test/java/org/apache/commons/logging/tccl/NullTCCLTestCase.java 
b/src/test/java/org/apache/commons/logging/tccl/NullTCCLTestCase.java
index 75ca5cf..8590d5d 100644
--- a/src/test/java/org/apache/commons/logging/tccl/NullTCCLTestCase.java
+++ b/src/test/java/org/apache/commons/logging/tccl/NullTCCLTestCase.java
@@ -16,13 +16,13 @@
  */
 package org.apache.commons.logging.tccl;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Simulates the case when TCCL is set to NULL.
  */
diff --git 
a/src/test/java/org/apache/commons/logging/tccl/log/TcclDisabledTestCase.java 
b/src/test/java/org/apache/commons/logging/tccl/log/TcclDisabledTestCase.java
index 8a787fe..9a2798e 100644
--- 
a/src/test/java/org/apache/commons/logging/tccl/log/TcclDisabledTestCase.java
+++ 
b/src/test/java/org/apache/commons/logging/tccl/log/TcclDisabledTestCase.java
@@ -19,15 +19,15 @@ package org.apache.commons.logging.tccl.log;
 
 import java.net.URL;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogConfigurationException;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Verify that by default LogFactoryImpl is loaded from the tccl class loader.
  */
diff --git 
a/src/test/java/org/apache/commons/logging/tccl/log/TcclEnabledTestCase.java 
b/src/test/java/org/apache/commons/logging/tccl/log/TcclEnabledTestCase.java
index 3aea5e5..4a49113 100644
--- a/src/test/java/org/apache/commons/logging/tccl/log/TcclEnabledTestCase.java
+++ b/src/test/java/org/apache/commons/logging/tccl/log/TcclEnabledTestCase.java
@@ -19,14 +19,14 @@ package org.apache.commons.logging.tccl.log;
 
 import java.net.URL;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Verify that by default the standard LogFactoryImpl class loads a
  * custom Log implementation via the TCCL.
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 e8f2e88..5273668 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
@@ -19,13 +19,13 @@ package org.apache.commons.logging.tccl.logfactory;
 
 import java.net.URL;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Verify that a commons-logging.properties file can prevent a custom
  * LogFactoryImpl being loaded from the 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 0088070..990d05f 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
@@ -19,13 +19,13 @@ package org.apache.commons.logging.tccl.logfactory;
 
 import java.net.URL;
 
+import junit.framework.Test;
+import junit.framework.TestCase;
+
 import org.apache.commons.logging.LogFactory;
 import org.apache.commons.logging.PathableClassLoader;
 import org.apache.commons.logging.PathableTestSuite;
 
-import junit.framework.Test;
-import junit.framework.TestCase;
-
 /**
  * Verify that by default a custom LogFactoryImpl is loaded from the
  * tccl class loader.

Reply via email to