Add more testing for JDK serializability of exceptions

Author: cowtowncoder

release-notes/VERSION

@@ -18,7 +18,8 @@ JSON library.
 
 #209: Make use of `_allowMultipleMatches` in `FilteringParserDelegate`
  (contributed by Lokesh N)
-- Make `processor` transient in `JsonParseException` to keep it Serializable
+- Make `processor` transient in `JsonParseException`, `JsonGenerationException`
+  to keep both Serializable
 
 2.7.3 (16-Mar-2016)
 

src/main/java/com/fasterxml/jackson/core/JsonGenerationException.java

@@ -15,7 +15,8 @@ public class JsonGenerationException
 {
     private final static long serialVersionUID = 123; // Stupid eclipse...
 
-    protected JsonGenerator _processor;
+    // transient since 2.7.4
+    protected transient JsonGenerator _processor;
 
     @Deprecated // since 2.7
     public JsonGenerationException(Throwable rootCause) {

src/test/java/com/fasterxml/jackson/core/TestJDKSerializability.java

@@ -47,6 +47,58 @@ public void testPrettyPrinter() throws Exception
         // what should we test?
         assertNotNull(back);
     }
+
+    public void testLocation() throws Exception
+    {
+        JsonFactory jf = new JsonFactory();
+        JsonParser jp = jf.createParser("  { }");
+        assertToken(JsonToken.START_OBJECT, jp.nextToken());
+        JsonLocation loc = jp.getCurrentLocation();
+
+        byte[] stuff = jdkSerialize(loc);
+        JsonLocation loc2 = jdkDeserialize(stuff);
+        assertNotNull(loc2);
+        
+        assertEquals(loc.getLineNr(), loc2.getLineNr());
+        assertEquals(loc.getColumnNr(), loc2.getColumnNr());
+        jp.close();
+    }
+
+    public void testParseException() throws Exception
+    {
+        JsonFactory jf = new JsonFactory();
+        JsonParser p = jf.createParser("  { garbage! }");
+        JsonParseException exc = null;
+        try {
+            p.nextToken();
+            p.nextToken();
+            fail("Should not get here");
+        } catch (JsonParseException e) {
+            exc = e;
+        }
+        p.close();
+        byte[] stuff = jdkSerialize(exc);
+        JsonParseException result = jdkDeserialize(stuff);
+        assertNotNull(result);
+    }
+
+    public void testGenerationException() throws Exception
+    {
+        JsonFactory jf = new JsonFactory();
+        JsonGenerator g = jf.createGenerator(new ByteArrayOutputStream());
+        JsonGenerationException exc = null;
+        g.writeStartObject();
+        try {
+            g.writeNumber(4);
+            fail("Should not get here");
+        } catch (JsonGenerationException e) {
+            exc = e;
+        }
+        g.close();
+        byte[] stuff = jdkSerialize(exc);
+        JsonGenerationException result = jdkDeserialize(stuff);
+        assertNotNull(result);
+    }
 
     /*
     /**********************************************************