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

jOOQ · Mar 2, 2020 · ff36d22 · ff36d22
1 parent 52ec31d
commit ff36d22
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 12 deletions.
diff --git a/jOOR-java-6/src/test/java/org/joor/test/CompileTest.java b/jOOR-java-6/src/test/java/org/joor/test/CompileTest.java
@@ -180,8 +180,6 @@
 
 
 
-
-
 
 
 
diff --git a/jOOR-java-6/src/test/java/org/joor/test/ReflectTest.java b/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();

diff --git a/jOOR-java-8/src/test/java/org/joor/test/CompileTest.java b/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

diff --git a/jOOR-java-8/src/test/java/org/joor/test/ReflectTest.java b/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();

diff --git a/jOOR/src/test/java/org/joor/test/CompileTest.java b/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