Fix #82 by commenting out black magic that test was doing to try to v…

…erify name of generated (de)serializer
FasterXML · Sep 29, 2019 · b96ccd2 · b96ccd2
1 parent 27030d4
commit b96ccd2
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 13 deletions.
diff --git a/...r/src/test/java/com/fasterxml/jackson/module/afterburner/deser/TestSimpleDeserialize.java b/...r/src/test/java/com/fasterxml/jackson/module/afterburner/deser/TestSimpleDeserialize.java
@@ -1,8 +1,6 @@
 package com.fasterxml.jackson.module.afterburner.deser;
 
-import java.lang.reflect.Field;
 import java.util.List;
-import java.util.Vector;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
@@ -145,10 +143,6 @@ static class Issue60Pojo {
         public List<Object> foos;
     }    
 
-    static class CheckGeneratedDeserializerName {
-        public String stringField;
-    }
-
     /*
     /**********************************************************************
     /* Test methods, method access
@@ -331,6 +325,15 @@ public void testProblemWithIndentation() throws Exception {
         assertEquals(0, pojo.foos.size());
     }
 
+    // 28-Sep-2019, tatu: I do not fully understand what this method tried to do;
+    //   it seemed to be a contribution from July 2016. But I noticed that it breaks
+    //   with JDK 12 and unlikely to be allowed so let's comment it out.
+    /*
+
+    static class CheckGeneratedDeserializerName {
+        public String stringField;
+    }
+    
     @SuppressWarnings("unchecked")
     public void testGeneratedDeserializerName() throws Exception {
         MAPPER.readValue("{\"stringField\":\"foo\"}", CheckGeneratedDeserializerName.class);
@@ -355,4 +358,5 @@ public void testGeneratedDeserializerName() throws Exception {
         }
         fail("Expected class not found:" + expectedClassName);
     }
+    */
 }
diff --git a/...urner/src/test/java/com/fasterxml/jackson/module/afterburner/ser/TestSimpleSerialize.java b/...urner/src/test/java/com/fasterxml/jackson/module/afterburner/ser/TestSimpleSerialize.java
@@ -1,8 +1,5 @@
 package com.fasterxml.jackson.module.afterburner.ser;
 
-import java.lang.reflect.Field;
-import java.util.Vector;
-
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.fasterxml.jackson.annotation.JsonPropertyOrder;
@@ -115,10 +112,6 @@ public static class BooleansBean {
         public boolean getB() { return false; }
     }
 
-    static class CheckGeneratedSerializerName {
-        public String stringField;
-    }
-
     /*
     /**********************************************************************
     /* Test methods, method access
@@ -211,6 +204,15 @@ public void testFiveMinuteDoc() throws Exception
         assertEquals(jsonPlain, jsonAb);
     }
 
+    // 28-Sep-2019, tatu: I do not fully understand what this method tried to do;
+    //   it seemed to be a contribution from July 2016. But I noticed that it breaks
+    //   with JDK 12 and unlikely to be allowed so let's comment it out.
+/*
+
+    static class CheckGeneratedSerializerName {
+        public String stringField;
+    }
+
     @SuppressWarnings("unchecked")
     public void testGeneratedSerializerName() throws Exception {
         CheckGeneratedSerializerName bean = new CheckGeneratedSerializerName();
@@ -238,4 +240,5 @@ public void testGeneratedSerializerName() throws Exception {
         }
         assertTrue("Expected class not found:" + expectedClassName, found);
     }
+    */
 }