test refactoring

Author: cowtowncoder

src/test/java/com/fasterxml/jackson/databind/filter/IgnorePropsTest.java renamed to src/test/java/com/fasterxml/jackson/databind/filter/IgnorePropsForSerTest.java

@@ -6,7 +6,7 @@
 
 import com.fasterxml.jackson.databind.*;
 
-public class IgnorePropsTest
+public class IgnorePropsForSerTest
     extends BaseMapTest
 {
     @JsonIgnoreProperties({"b", "c"})
@@ -23,7 +23,7 @@ static class IgnoreSome
     @JsonIgnoreProperties({"@class"})
     static class MyMap extends HashMap<String,String> { }
 
-    // [JACKSON-787]: allow use of @JsonIgnoreProperties for properties
+    //allow use of @JsonIgnoreProperties for properties
     static class WrapperWithPropIgnore
     {
         @JsonIgnoreProperties("y")

src/test/java/com/fasterxml/jackson/databind/filter/TestSimpleSerializationIgnore.java

@@ -70,21 +70,21 @@ static class NonIgnoredType
     /**********************************************************
      */
 
+    private final ObjectMapper MAPPER = new ObjectMapper();
+
     public void testSimpleIgnore() throws Exception
     {
-        ObjectMapper m = new ObjectMapper();
         // Should see "x", not "y"
-        Map<String,Object> result = writeAndMap(m, new SizeClassEnabledIgnore());
+        Map<String,Object> result = writeAndMap(MAPPER, new SizeClassEnabledIgnore());
         assertEquals(1, result.size());
         assertEquals(Integer.valueOf(1), result.get("x"));
         assertNull(result.get("y"));
     }
 
     public void testDisabledIgnore() throws Exception
     {
-        ObjectMapper m = new ObjectMapper();
         // Should see "x" and "y"
-        Map<String,Object> result = writeAndMap(m, new SizeClassDisabledIgnore());
+        Map<String,Object> result = writeAndMap(MAPPER, new SizeClassDisabledIgnore());
         assertEquals(2, result.size());
         assertEquals(Integer.valueOf(3), result.get("x"));
         assertEquals(Integer.valueOf(4), result.get("y"));
@@ -96,26 +96,20 @@ public void testDisabledIgnore() throws Exception
      */
     public void testIgnoreOver() throws Exception
     {
-        ObjectMapper m = new ObjectMapper();
-
         // should only see "y"
-        Map<String,Object> result = writeAndMap(m, new BaseClassIgnore());
+        Map<String,Object> result = writeAndMap(MAPPER, new BaseClassIgnore());
         assertEquals(1, result.size());
         assertEquals(Integer.valueOf(2), result.get("y"));
 
         // Should see "x" and "y"
-        result = writeAndMap(m, new SubClassNonIgnore());
+        result = writeAndMap(MAPPER, new SubClassNonIgnore());
         assertEquals(2, result.size());
         assertEquals(Integer.valueOf(3), result.get("x"));
         assertEquals(Integer.valueOf(2), result.get("y"));
     }
 
-    /**
-     * @since 1.7
-     */
     public void testIgnoreType() throws Exception
     {
-        ObjectMapper m = new ObjectMapper();
-        assertEquals("{\"value\":13}", m.writeValueAsString(new NonIgnoredType()));
+        assertEquals("{\"value\":13}", MAPPER.writeValueAsString(new NonIgnoredType()));
     }
 }