Fixes the test failure. Junit on cmd-line was failing because one of the
classes had a name ending in Test (even though eclipse's junit did not have a
problem).

Plus, some sort & format. 

Patch by: amitmanjhi



git-svn-id: https://google-web-toolkit.googlecode.com/svn/releases/1.6@4516 8db76d5a-ed1c-0410-87a9-c151d255dfc7
diff --git a/tools/api-checker/test/com/google/gwt/tools/apichecker/ApiCompatibilityUnitTest.java b/tools/api-checker/test/com/google/gwt/tools/apichecker/ApiCompatibilityUnitTest.java
index d17a8eb..073b3ae 100644
--- a/tools/api-checker/test/com/google/gwt/tools/apichecker/ApiCompatibilityUnitTest.java
+++ b/tools/api-checker/test/com/google/gwt/tools/apichecker/ApiCompatibilityUnitTest.java
@@ -59,7 +59,7 @@
     }
   }
 
-  private static class FinalKeywordRefactoring extends ApiCompatibilityUnitTest {
+  private static class FinalKeywordRefactoring {
     private static String getFirstApiSourceForObject() {
       StringBuffer sb = new StringBuffer();
       sb.append("package java.lang;\n");
@@ -88,9 +88,9 @@
 
       // firstApi is the reference Api
       Collection<ApiChange> apiChanges = getApiChanges(firstApi, secondApi);
-      assertEquals(
-          Arrays.asList(new ApiChange[] {new ApiChange(new MockApiElement(
-              "java.lang.Object::foo"), ApiChange.Status.FINAL_ADDED),}), apiChanges);
+      assertEquals(Arrays.asList(new ApiChange[] {new ApiChange(
+          new MockApiElement("java.lang.Object::foo"),
+          ApiChange.Status.FINAL_ADDED),}), apiChanges);
 
       // secondApi is the reference Api
       apiChanges = getApiChanges(secondApi, firstApi);
@@ -109,7 +109,7 @@
    * However, if foo(Object ..) is added, it should be okay since JLS matches
    * from the most specific to the least specific.
    */
-  private static class MethodOverloadingTest extends ApiCompatibilityUnitTest {
+  private static class OverloadedMethodRefactoring {
     private static String getFirstApiSourceForObject() {
       StringBuffer sb = new StringBuffer();
       sb.append("package java.lang;\n");
@@ -170,7 +170,7 @@
    * to a super class.
    * 
    */
-  private static class SuperClassRefactoring extends ApiCompatibilityUnitTest {
+  private static class SuperClassRefactoring {
     private static String getFirstApiSourceForObject() {
       StringBuffer sb = new StringBuffer();
       sb.append("package java.lang;\n");
@@ -289,7 +289,7 @@
 
   public void testMethodOverloading() throws NotFoundException,
       UnableToCompleteException {
-    new MethodOverloadingTest().testBothWays();
+    new OverloadedMethodRefactoring().testBothWays();
   }
 
   public void testSuperClassRefactoring() throws NotFoundException,