aries-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From djen...@apache.org
Subject svn commit: r1235095 [4/5] - in /aries/trunk/versioning: ./ versioning-checker/ versioning-checker/src/main/java/org/apache/aries/versioning/check/ versioning-checker/src/main/java/org/apache/aries/versioning/utils/ versioning-checker/src/test/java/org...
Date Tue, 24 Jan 2012 00:53:26 GMT
Modified: aries/trunk/versioning/versioning-checker/src/test/java/org/apache/aries/versioning/tests/BinaryCompatibilityTest.java
URL: http://svn.apache.org/viewvc/aries/trunk/versioning/versioning-checker/src/test/java/org/apache/aries/versioning/tests/BinaryCompatibilityTest.java?rev=1235095&r1=1235094&r2=1235095&view=diff
==============================================================================
--- aries/trunk/versioning/versioning-checker/src/test/java/org/apache/aries/versioning/tests/BinaryCompatibilityTest.java (original)
+++ aries/trunk/versioning/versioning-checker/src/test/java/org/apache/aries/versioning/tests/BinaryCompatibilityTest.java Tue Jan 24 00:53:25 2012
@@ -1,24 +1,23 @@
-
-
 /*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements.  See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership.  The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License.  You may obtain a copy of the License at
- *
- *   http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied.  See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
+* Licensed to the Apache Software Foundation (ASF) under one
+* or more contributor license agreements.  See the NOTICE file
+* distributed with this work for additional information
+* regarding copyright ownership.  The ASF licenses this file
+* to you under the Apache License, Version 2.0 (the
+* "License"); you may not use this file except in compliance
+* with the License.  You may obtain a copy of the License at
+*
+*   http://www.apache.org/licenses/LICENSE-2.0
+*
+* Unless required by applicable law or agreed to in writing,
+* software distributed under the License is distributed on an
+* "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+* KIND, either express or implied.  See the License for the
+* specific language governing permissions and limitations
+* under the License.
+*/
 package org.apache.aries.versioning.tests;
+
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNull;
@@ -45,8 +44,10 @@ import java.util.HashSet;
 import org.apache.aries.versioning.utils.BinaryCompatibilityStatus;
 import org.apache.aries.versioning.utils.MethodDeclaration;
 import org.apache.aries.versioning.utils.SemanticVersioningClassVisitor;
+
 import static org.apache.aries.versioning.utils.SemanticVersioningUtils.htmlOneLineBreak;
 import static org.apache.aries.versioning.utils.SemanticVersioningUtils.htmlTwoLineBreaks;
+
 import org.junit.BeforeClass;
 import org.junit.Test;
 import org.objectweb.asm.ClassReader;
@@ -55,1666 +56,1614 @@ import org.objectweb.asm.ClassWriter;
 
 /**
  * Test the jdk chap 13 -binary compatibility implementation
- * 
- * @author emily
  *
+ * @author emily
  */
-public class BinaryCompatibilityTest
-{
+public class BinaryCompatibilityTest {
+
+    private static final int ACC_ABASTRACT = 0;
+    private static URLClassLoader loader = null;
+
+    @BeforeClass
+    public static void setup() throws Exception {
+        Collection<URL> urls = new HashSet<URL>();
+        urls.add(new File("unitest/").toURI().toURL());
+        loader = new URLClassLoader(urls.toArray(new URL[0]));
+    }
+
+    /**
+     * Test of binary incompatibility where a class was not abstract is changed to be declared abstract,
+     */
+    @Test
+    public void test_jdk_chap13_4_1_1() {
+        // construct original class
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC + ACC_ABSTRACT, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertFalse(
+                "When a class is changed from non abstract to abstract, this should break binary compatibility.",
+                bcs.isCompatible());
+
+        assertEquals(" The class pkg/Test was not abstract but is changed to be abstract.", bcs.getReason());
+    }
+
+    /**
+     * Test of binary compatibility where a class was abstract is changed to be declared non-abstract,
+     */
+    @Test
+    public void test_jdk_chap13_4_1_2() {
+        // construct original class
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC + ACC_ABSTRACT, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertTrue(
+                "When a class is changed from static to non-static, this should not break binary compatibility.",
+                bcs.isCompatible());
+
+
+    }
+
+    /**
+     * Test a binary incompatibility where a class was not final is changed to be final.
+     */
+    @Test
+    public void test_jdk_chap13_4_2_1() {
+        // construct original class
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertFalse(
+                "When a class is changed from non final to final, this should break binary compatibility.",
+                bcs.isCompatible());
+
+        assertEquals(" The class pkg/Test was not final but is changed to be final.", bcs.getReason());
+    }
+
+    /**
+     * Test a binary compatibility where a class was final is changed to not final.
+     */
+    @Test
+    public void test_jdk_chap13_4_2_2() {
+        // construct original class
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertTrue(
+                "When a class is changed from final to not final, this should not break binary compatibility.",
+                bcs.isCompatible());
+
+    }
+
+    /**
+     * Test a binary incompatibility where a class was public is changed to not public.
+     */
+    @Test
+    public void test_jdk_chap13_4_3_1() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertNull(
+                "When a class is changed from public to non-public, this should break binary compatibility.",
+                newCV.getClassDeclaration());
+
+    }
+
+    /**
+     * Test a binary incompatibility where a class was not public is changed to public.
+     */
+    @Test
+    public void test_jdk_chap13_4_3_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertTrue(
+                "When a class is changed from non-public to public, this should break binary compatibility.",
+                newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
+    }
+
+    /**
+     * Changing the direct superclass or the set of direct superinterfaces of a class type will not break compatibility
+     * with pre-existing binaries, provided that the total set of superclasses or superinterfaces, respectively, of the class type loses no members.
+     */
+    @Test
+    public void test_jdk_chap13_4_4_1() throws IOException {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", new String[]{"versioning/java/files/TestB"});
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestBChild", null);
+
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertTrue(
+                "Changing the direct superclass or the set of direct superinterfaces of a class type will not breake binary compatibility if not losing any members.",
+                newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
+    }
+
+    /**
+     * Changing field type breaks binary compatibility
+     * with pre-existing binaries, provided that the total set of superclasses or superinterfaces, respectively, of the class type loses no members but the one of the fields has changed type.
+     */
+    @Test
+    public void test_jdk_chap13_4_7_4() throws IOException {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", new String[]{"versioning/java/files/TestB"});
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertFalse(
+                "Changing the direct superclass or the set of direct superinterfaces of a class type results fields changes. This should breake binary compatibility if not losing any members.",
+                bcs.isCompatible());
+        assertEquals("&#13;&#10;The public field bar was static but is changed to be non static or vice versa.&#13;&#10;The public field bar has changed its type.", bcs.getReason());
+    }
+
+    /**
+     * Change the signature of the field from Colllection<AA> to Collection<String>
+     */
+    @Test
+    public void test_jdk_chap13_4_7_5() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitField(ACC_PUBLIC, "more", "Ljava/util/Collection;", "Lcom/bim/AA;", null).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitField(ACC_PROTECTED, "more", "Ljava/util/Collection;", "Ljava/lang/String;", null).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The public field more has changed its type.&#13;&#10;The public field more becomes less accessible.", bcs.getReason());
+        assertFalse(
+                "Changing the declared access of a field to permit less access  , this should break binary compatibility.",
+                bcs.isCompatible());
+    }
+
+    /**
+     * If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_4_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", new String[]{"versioning/java/files/TestB"});
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertFalse(
+                "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
+                bcs.isCompatible());
+        assertEquals("&#13;&#10;The method int getFooLen(java.lang.String) has been deleted or its return type or parameter list has changed.&#13;&#10;The method java.lang.String getFoo() has changed from non abstract to abstract. &#13;&#10;The method int getBarLen(java.lang.String) has been deleted or its return type or parameter list has changed.&#13;&#10;The method int getBooLen(java.lang.String) has been deleted or its return type or parameter list has changed.&#13;&#10;&#13;&#10;The superclasses or superinterfaces have stopped being super: [versioning/java/files/TestC, versioning/java/files/TestA].&#13;&#10;The protcted field c has been deleted.&#13;&#10;The public field bar was not final but has been changed to be final.&#13;&#10;The public field bar was static but is changed to be non static or vice versa.&#13;&#10;The public field bar has changed its type.", bcs.getReason());
+    }
+
+    /**
+     * Test deleting a class member or constructor that is not declared private/default breaks binary compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_5_1() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitMethod(ACC_PROTECTED + ACC_ABSTRACT, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The method int convert(java.lang.Object) has been deleted or its return type or parameter list has changed.", bcs.getReason());
+        assertFalse(
+                "deleting a class member or constructor that is not declared private breaks binary compatibility.",
+                bcs.isCompatible());
+
+    }
+
+    /**
+     * Test deleting a class member or constructor that is declared private should not break binary compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_5_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitMethod(ACC_PRIVATE, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertTrue(
+                "Deleting a class member or constructor that is declared private should not break binary compatibility.",
+                newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
+    }
+
+    /**
+     * Test adding a class member or constructor should not break binary compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_5_3() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitMethod(ACC_PROTECTED, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertTrue(
+                "Adding a class member or constructor should not break binary compatibility.",
+                newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
+    }
+
+
+    /**
+     * Test changing the declared access of a method to permit less access, this should break compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_6_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitMethod(ACC_PUBLIC + ACC_STATIC, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitMethod(ACC_STATIC, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The method int convert(java.lang.Object) is less accessible.", bcs.getReason());
+        assertFalse(
+                "Changing the declared access of a member or contructor to permit less access  , this should break binary compatibility.",
+                bcs.isCompatible());
+    }
+
+    @Test
+    public void test_jdk_chap13_4_6_1() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitField(ACC_PUBLIC, "lESS", "I", null, new Integer(-1)).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitField(ACC_PROTECTED, "lESS", "I", null, new Integer(-1)).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The public field lESS becomes less accessible.", bcs.getReason());
+        assertFalse(
+                "Changing the declared access of a field to permit less access  , this should break binary compatibility.",
+                bcs.isCompatible());
+    }
+
+    /**
+     * Test deleting a private/default field, this should not break compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_7_1() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitField(ACC_PRIVATE, "lESS", "I", null, new Integer(-1)).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
+        cw.visitMethod(ACC_PROTECTED, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertTrue(
+                "Deleting a private field should not break binary compatibility.",
+                newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
+    }
+
+    /**
+     * If a field is added but is less accessible than the old one or change to static from non-static or from non-static to static.
+     */
+    @Test
+    public void test_jdk_chap13_4_7_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PROTECTED, "bar", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The public field bar becomes less accessible.", bcs.getReason());
+        assertFalse(
+                "The new field conflicts with a field in the super class. Check chapter 13.4.7 java spec for more info.",
+                bcs.isCompatible());
+    }
+
+    /**
+     * If a field was not final is changed to be final, then it can break compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_8_1() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC + ACC_FINAL, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The public field aa was not final but has been changed to be final.", bcs.getReason());
+        assertFalse(
+                "Change that a public or protected field was final but is changed to be not final will break binary compatibility.",
+                bcs.isCompatible());
+    }
+
+    /**
+     * If a field was final is changed to be non-final, then it does not break compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_8_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC + ACC_FINAL, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        assertTrue(
+                "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
+                newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
+    }
+
+    /**
+     * If a field was static is changed to be non-static, then it will break compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_9_1() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC + ACC_STATIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The public field aa was static but is changed to be non static or vice versa.", bcs.getReason());
+        assertFalse(
+                "If a field was static is changed to be non-static, then it will break compatibility.",
+                bcs.isCompatible());
+    }
+
+    /**
+     * If a field was non-static is changed to be static, then it will break compatibility.
+     */
+    @Test
+    public void test_jdk_chap13_4_9_2() {
+        ClassWriter cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] oldBytes = cw.toByteArray();
+
+
+        cw = new ClassWriter(0);
+        cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
+        cw.visitField(ACC_PUBLIC + ACC_STATIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
+        cw.visitEnd();
+        byte[] newBytes = cw.toByteArray();
+        SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
+        SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
+        ClassReader newCR = new ClassReader(newBytes);
+        ClassReader oldCR = new ClassReader(oldBytes);
+
+        newCR.accept(newCV, 0);
+        oldCR.accept(oldCV, 0);
+        BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
+        assertEquals(htmlOneLineBreak + "The public field aa was static but is changed to be non static or vice versa.", bcs.getReason());
+        assertFalse(
+                "If a field was non-static is changed to be static, then it will break compatibility.",
+                bcs.isCompatible());
+    }
 
-  private static final int ACC_ABASTRACT = 0;
-  private static URLClassLoader loader = null;
-  @BeforeClass
-  public  static void setup() throws Exception{
-    Collection<URL> urls = new HashSet<URL>();
-    urls.add(new File("unitest/").toURI().toURL());
-    loader = new URLClassLoader(urls.toArray(new URL[0]));
-  } 
-
-  /**
-   * Test of binary incompatibility where a class was not abstract is changed to be declared abstract, 
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_1_1()
-  {
-    // construct original class
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC + ACC_ABSTRACT, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertFalse(
-        "When a class is changed from non abstract to abstract, this should break binary compatibility.",
-        bcs.isCompatible());
-
-    assertEquals(" The class pkg/Test was not abstract but is changed to be abstract.", bcs.getReason());
-  }
-
-  /**
-   * Test of binary compatibility where a class was abstract is changed to be declared non-abstract, 
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_1_2()
-  {
-    // construct original class
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC + ACC_ABSTRACT, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertTrue(
-        "When a class is changed from static to non-static, this should not break binary compatibility.",
-        bcs.isCompatible());
-   
-    
-
-  }
-
-  /**
-   * Test a binary incompatibility where a class was not final is changed to be final.
-   */
-  @Test
-  public void test_jdk_chap13_4_2_1()
-  {
-    // construct original class
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertFalse(
-        "When a class is changed from non final to final, this should break binary compatibility.",
-        bcs.isCompatible());
-
-    assertEquals(" The class pkg/Test was not final but is changed to be final.", bcs.getReason());
-  }
-
-  /**
-   * Test a binary compatibility where a class was final is changed to not final.
-   */
-  @Test
-  public void test_jdk_chap13_4_2_2()
-  {
-    // construct original class
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertTrue(
-        "When a class is changed from final to not final, this should not break binary compatibility.",
-        bcs.isCompatible());
-    
-  }
-
-  /**
-   * Test a binary incompatibility where a class was public is changed to not public.
-   */
-  @Test
-  public void test_jdk_chap13_4_3_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertNull(
-        "When a class is changed from public to non-public, this should break binary compatibility.",
-        newCV.getClassDeclaration());
-
-  }
-
-  /**
-   * Test a binary incompatibility where a class was not public is changed to public.
-   */
-  @Test
-  public void test_jdk_chap13_4_3_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC + ACC_FINAL, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "When a class is changed from non-public to public, this should break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * Changing the direct superclass or the set of direct superinterfaces of a class type will not break compatibility 
-   * with pre-existing binaries, provided that the total set of superclasses or superinterfaces, respectively, of the class type loses no members.
-   */
-  @Test
-  public void test_jdk_chap13_4_4_1() throws IOException
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", new String[] { "versioning/java/files/TestB" });
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestBChild", null);
-
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Changing the direct superclass or the set of direct superinterfaces of a class type will not breake binary compatibility if not losing any members.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-  
-  /**
-   * Changing field type breaks binary compatibility 
-   * with pre-existing binaries, provided that the total set of superclasses or superinterfaces, respectively, of the class type loses no members but the one of the fields has changed type.
-   */
-  @Test
-  public void test_jdk_chap13_4_7_4() throws IOException
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", new String[] { "versioning/java/files/TestB" });
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertFalse(
-        "Changing the direct superclass or the set of direct superinterfaces of a class type results fields changes. This should breake binary compatibility if not losing any members.",
-        bcs.isCompatible());
-    assertEquals("&#13;&#10;The public field bar was static but is changed to be non static or vice versa.&#13;&#10;The public field bar has changed its type.", bcs.getReason());
-  }
-
-  /**
-   * Change the signature of the field from Colllection<AA> to Collection<String>
-   */
-  @Test
-  public void test_jdk_chap13_4_7_5() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitField(ACC_PUBLIC, "more", "Ljava/util/Collection;", "Lcom/bim/AA;", null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitField(ACC_PROTECTED, "more", "Ljava/util/Collection;", "Ljava/lang/String;", null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The public field more has changed its type.&#13;&#10;The public field more becomes less accessible.", bcs.getReason());
-    assertFalse(
-        "Changing the declared access of a field to permit less access  , this should break binary compatibility.",
-        bcs.isCompatible());
-  }
-  /**
-   * If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_4_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", new String[] { "versioning/java/files/TestB" });
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertFalse(
-        "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
-        bcs.isCompatible());
-   assertEquals("&#13;&#10;The method int getFooLen(java.lang.String) has been deleted or its return type or parameter list has changed.&#13;&#10;The method java.lang.String getFoo() has changed from non abstract to abstract. &#13;&#10;The method int getBarLen(java.lang.String) has been deleted or its return type or parameter list has changed.&#13;&#10;The method int getBooLen(java.lang.String) has been deleted or its return type or parameter list has changed.&#13;&#10;&#13;&#10;The superclasses or superinterfaces have stopped being super: [versioning/java/files/TestC, versioning/java/files/TestA].&#13;&#10;The protcted field c has been deleted.&#13;&#10;The public field bar was not final but has been changed to be final.&#13;&#10;The public field bar was static but is changed to be non static or vice versa.&#13;&#10;The public field bar has changed its type.", bcs.getReason());
-  }
-
-  /**
-   * Test deleting a class member or constructor that is not declared private/default breaks binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_5_1() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitMethod(ACC_PROTECTED+ ACC_ABSTRACT, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));    
-    assertEquals(htmlOneLineBreak  +"The method int convert(java.lang.Object) has been deleted or its return type or parameter list has changed.", bcs.getReason());
-    assertFalse(
-        "deleting a class member or constructor that is not declared private breaks binary compatibility.",
-        bcs.isCompatible());
-    
-  }
-
-  /**
-   * Test deleting a class member or constructor that is declared private should not break binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_5_2() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitMethod(ACC_PRIVATE, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Deleting a class member or constructor that is declared private should not break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * Test adding a class member or constructor should not break binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_5_3() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitMethod(ACC_PROTECTED, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Adding a class member or constructor should not break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-
-  /**
-   * Test changing the declared access of a method to permit less access, this should break compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_6_2() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitMethod(ACC_PUBLIC + ACC_STATIC, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitMethod( ACC_STATIC, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int convert(java.lang.Object) is less accessible.", bcs.getReason());
-    assertFalse(
-        "Changing the declared access of a member or contructor to permit less access  , this should break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  @Test
-  public void test_jdk_chap13_4_6_1() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitField(ACC_PUBLIC, "lESS", "I", null, new Integer(-1)).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitField(ACC_PROTECTED, "lESS", "I", null, new Integer(-1)).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The public field lESS becomes less accessible.", bcs.getReason());
-    assertFalse(
-        "Changing the declared access of a field to permit less access  , this should break binary compatibility.",
-        bcs.isCompatible());
-  }
-  /**
-   * Test deleting a private/default field, this should not break compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_7_1() {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitField(ACC_PRIVATE, "lESS", "I", null, new Integer(-1)).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "java/lang/Object", null);
-    cw.visitMethod( ACC_PROTECTED, "convert", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Deleting a private field should not break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * If a field is added but is less accessible than the old one or change to static from non-static or from non-static to static.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_7_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PROTECTED, "bar", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The public field bar becomes less accessible.", bcs.getReason());
-    assertFalse(
-        "The new field conflicts with a field in the super class. Check chapter 13.4.7 java spec for more info.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * If a field was not final is changed to be final, then it can break compatibility.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_8_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_FINAL, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The public field aa was not final but has been changed to be final.", bcs.getReason());
-    assertFalse(
-        "Change that a public or protected field was final but is changed to be not final will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * If a field was final is changed to be non-final, then it does not break compatibility.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_8_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_FINAL, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC , "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * If a field was static is changed to be non-static, then it will break compatibility.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_9_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_STATIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC , "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The public field aa was static but is changed to be non static or vice versa.", bcs.getReason());
-    assertFalse(
-        "If a field was static is changed to be non-static, then it will break compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * If a field was non-static is changed to be static, then it will break compatibility.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_9_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC , "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_STATIC, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The public field aa was static but is changed to be non static or vice versa.", bcs.getReason());
-    assertFalse(
-         "If a field was non-static is changed to be static, then it will break compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * If a field was transient is changed to non-transient, then it will not break compatibility.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_10_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC , "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    
-    assertTrue(
-        "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
-        bcs.isCompatible());
-  }
-  /**
-   * If a field was non-transient is changed to transient, then it will not break compatibility.
-   * 
-   */
-  @Test
-  public void test_jdk_chap13_4_10_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC , "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    
-    assertTrue(
-        "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Testing deleting a public/protected method when there is no such a method in the superclass breaks binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_11_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getFooLen", "(Ljava/lang/STring;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getFooLen(java.lang.STring) has been deleted or its return type or parameter list has changed.", bcs.getReason());
-    assertFalse(
-        "Deleting a public/protected method when there is no such a method in the superclass breaks binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Testing deleting a public/protected method when there is such a method in the superclass does not break binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_11_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getBarLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertTrue(
-        "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Testing deleting a public/protected method when there is less accessible method in the superclass does break binary compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_11_3()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getFooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getFooLen(java.lang.String) is less accessible.", bcs.getReason());
-    assertFalse(
-        "If a change to the direct superclass or the set of direct superinterfaces results in any class or interface no longer being a superclass or superinterface, respectively, it will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Adding a parameter is a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_12_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;I)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getCooLen(java.lang.String) has been deleted or its return type or parameter list has changed.", bcs.getReason());
-    assertFalse(
-        "Changing a parameter list will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-
-  /**
-   * Changing a method parameter type is a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_12_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/Object;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getCooLen(java.lang.String) has been deleted or its return type or parameter list has changed.", bcs.getReason());
-    assertFalse(
-        "Changing a method paramether type will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Changing a method formal type parameter is not a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_12_3()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/util/List;)I", "Ljava/lang/String;", null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/util/List;)I", "Lcome/ibm/blah;", null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertTrue(
-        "Changing a method formal type parameter should not break binary compatibility.",
-        bcs.isCompatible());
-  }
-  /**
-   * Changing a method return type is a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_13_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;)[I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getCooLen(java.lang.String) has been deleted or its return type or parameter list has changed.", bcs.getReason());
-    assertFalse(
-        "Changing a method return type will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Changing a method to be abstract is a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_14_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC + ACC_ABSTRACT, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getCooLen(java.lang.String) has changed from non abstract to abstract. ", bcs.getReason());
-    assertFalse(
-        "Changing a method to be abstract will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Changing a method to not to be abstract is not a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_14_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC + ACC_ABASTRACT, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC , "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Changing a method not to be abstract will not break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-
-  /**
-   * Changing an instance method that is not final to be final is a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_15_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC + ACC_FINAL , "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    BinaryCompatibilityStatus bcs = newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration()));
-    assertEquals(htmlOneLineBreak  +"The method int getCooLen(java.lang.String) was not final but has been changed to be final.", bcs.getReason());
-    assertFalse(
-        "Changing an instance method from non-final to final will break binary compatibility.",
-        bcs.isCompatible());
-  }
-
-  /**
-   * Changing an instance method that is final to be non-final is a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_15_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC + ACC_FINAL, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC , "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Changing an instance method from final to non-final will not break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * Changing a static method that is not final to final is not a binary compatibility change.
-   */
-  @Test
-  public void test_jdk_chap13_4_15_3()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC + ACC_STATIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC + ACC_STATIC+ ACC_FINAL , "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Changing a static method from non-final to final will not break binary compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * Adding a native modifier of a method does not break compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_16_1()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC + ACC_STATIC, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC + ACC_STATIC+ ACC_NATIVE , "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(
-        "Adding a native modifier of a method does not break compatibility.",
-        newCV.getClassDeclaration().getBinaryCompatibleStatus((oldCV.getClassDeclaration())).isCompatible());
-  }
-
-  /**
-   * Deleting a native modifier of a method does not break compatibility.
-   */
-  @Test
-  public void test_jdk_chap13_4_16_2()
-  {
-    ClassWriter cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitMethod(ACC_PUBLIC  + ACC_NATIVE, "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] oldBytes = cw.toByteArray();
-
-
-    cw = new ClassWriter(0);
-    cw.visit(V1_5, ACC_PUBLIC, "pkg/Test", null, "versioning/java/files/TestA", null);
-    cw.visitField(ACC_PUBLIC + ACC_TRANSIENT, "aa", "Ljava/lang/String;", null, new String("newBar")).visitEnd();
-    cw.visitMethod(ACC_PUBLIC  , "getCooLen", "(Ljava/lang/String;)I", null, null).visitEnd();
-    cw.visitEnd();
-    byte[] newBytes = cw.toByteArray();
-    SemanticVersioningClassVisitor oldCV = new SemanticVersioningClassVisitor(loader);
-    SemanticVersioningClassVisitor newCV = new SemanticVersioningClassVisitor(loader);
-    ClassReader newCR = new ClassReader(newBytes);
-    ClassReader oldCR = new ClassReader(oldBytes);
-
-    newCR.accept(newCV, 0);
-    oldCR.accept(oldCV, 0);
-    assertTrue(

[... 1483 lines stripped ...]


Mime
View raw message