[#76] [#77] Don't ignore tests. Expect throwable instead

Author: lukaseder

jOOR-java-6/src/test/java/org/joor/test/CompileTest.java

@@ -180,8 +180,6 @@
 
 
 
-
-
 
 
 

jOOR-java-6/src/test/java/org/joor/test/ReflectTest.java

@@ -55,7 +55,7 @@
  */
 public class ReflectTest {
 
-    static final boolean JDK9 = false;
+    static final boolean JDK9 = false ;
 
     @Rule
     public final ExpectedException expectedException = ExpectedException.none();

jOOR-java-8/src/test/java/org/joor/test/CompileTest.java

@@ -47,8 +47,7 @@ public void testCompileWithClasspathDependency() throws Exception {
         assertEquals(new ArrayList<Object>(), v.validateTestClass(null));
     }
 
-    @Test
-    @Ignore // [#77]
+    @Test(expected = Throwable.class) // [#77]
     public void testCompileLocalInterfaceHierarchy() throws Exception {
         I i = Reflect.compile("org.joor.test.CompileTest1", "package org.joor.test; public class CompileTest1 implements org.joor.test.I {}").create().get();
         assertEquals("I.m()", i.m());
@@ -87,8 +86,7 @@ public void testCompileDifferentPackage() {
         assertEquals("Hello World!", supplier.get());
     }
 
-    @Test
-    @Ignore // [#76]
+    @Test(expected = Throwable.class) // [#76]
     public void testRecompileSameClassName() {
 
         // The class loader will cache the class name by default, so a new content shouldn't affect the type

jOOR-java-8/src/test/java/org/joor/test/ReflectTest.java

@@ -55,7 +55,7 @@
  */
 public class ReflectTest {
 
-    static final boolean JDK9 = false;
+    static final boolean JDK9 = false ;
 
     @Rule
     public final ExpectedException expectedException = ExpectedException.none();

jOOR/src/test/java/org/joor/test/CompileTest.java

@@ -47,8 +47,7 @@ public void testCompileWithClasspathDependency() throws Exception {
         assertEquals(new ArrayList<Object>(), v.validateTestClass(null));
     }
 
-    @Test
-    @Ignore // [#77]
+    @Test(expected = Throwable.class) // [#77]
     public void testCompileLocalInterfaceHierarchy() throws Exception {
         I i = Reflect.compile("org.joor.test.CompileTest1", "package org.joor.test; public class CompileTest1 implements org.joor.test.I {}").create().get();
         assertEquals("I.m()", i.m());
@@ -87,8 +86,7 @@ public void testCompileDifferentPackage() {
         assertEquals("Hello World!", supplier.get());
     }
 
-    @Test
-    @Ignore // [#76]
+    @Test(expected = Throwable.class) // [#76]
     public void testRecompileSameClassName() {
 
         // The class loader will cache the class name by default, so a new content shouldn't affect the type