add a new unit test checking that JDK Map's inner classes are handled correctly (they appear to)

Author: cowtowncoder

src/test/java/com/fasterxml/classmate/OddJDKTypesTest.java

@@ -0,0 +1,32 @@
+package com.fasterxml.classmate;
+
+import java.util.*;
+
+public class OddJDKTypesTest extends BaseTest
+{
+    // Simple test, based on issues Jackson's type resolver had with inner
+    // classes of JDK (Hash)Map
+    public void testJDKMaps()
+    {
+        TypeResolver resolver = new TypeResolver();
+        Map<Object,Object> map = new HashMap<Object,Object>();
+        map.put("foo", "bar");
+
+        // map itself is trivial
+        ResolvedType type = resolver.resolve(map.getClass());
+        assertEquals(HashMap.class, type.getErasedType());
+
+        // but how about it's entry, key, value sets?
+        type = resolver.resolve(map.entrySet().getClass());
+        assertNotNull(type.getErasedType());
+        assertTrue(type.isInstanceOf(Set.class));
+        
+        type = resolver.resolve(map.keySet().getClass());
+        assertNotNull(type.getErasedType());
+        assertTrue(type.isInstanceOf(Set.class));
+
+        type = resolver.resolve(map.values().getClass());
+        assertNotNull(type.getErasedType());
+        assertTrue(type.isInstanceOf(Collection.class));
+    }
+}

src/test/java/com/fasterxml/classmate/ResolvedTypeTest.java

@@ -1,17 +1,16 @@
 package com.fasterxml.classmate;
 
-import com.fasterxml.classmate.types.ResolvedArrayType;
-import com.fasterxml.classmate.types.ResolvedInterfaceType;
-import com.fasterxml.classmate.types.ResolvedObjectType;
-import org.junit.Test;
-
 import java.io.Serializable;
 import java.nio.CharBuffer;
 import java.util.List;
 
-import static junit.framework.Assert.*;
+import org.junit.Test;
+
+import com.fasterxml.classmate.types.ResolvedArrayType;
+import com.fasterxml.classmate.types.ResolvedInterfaceType;
+import com.fasterxml.classmate.types.ResolvedObjectType;
 
-public class ResolvedTypeTest
+public class ResolvedTypeTest extends BaseTest
 {
     // For [Issue#16]
 
@@ -22,7 +21,7 @@ private static class Bar16 extends Zen16<Bar16, Foo16> { }
     private static class Zen16<A, B extends A>  { }
 
     @Test
-    public void canCreateSubtype() {
+    public void testCanCreateSubtype() {
         ResolvedObjectType stringType = ResolvedObjectType.create(String.class, null, null, null);
         assertTrue(stringType.canCreateSubtype(String.class));
         assertFalse(stringType.canCreateSubtype(CharBuffer.class));
@@ -40,13 +39,13 @@ public void canCreateSubtype() {
     }
 
     @Test
-    public void typeParametersFor() {
+    public void testtypeParametersFor() {
         ResolvedObjectType stringType = ResolvedObjectType.create(String.class, null, null, null);
         assertNull(stringType.typeParametersFor(CharBuffer.class));
     }
 
     @Test
-    public void findSupertype() {
+    public void testFindSupertype() {
         ResolvedInterfaceType comparableType = new ResolvedInterfaceType(Comparable.class, TypeBindings.create(String.class, ResolvedType.NO_TYPES), null);
         ResolvedObjectType stringType = new ResolvedObjectType(String.class, null, (ResolvedType) null, new ResolvedType[] { comparableType });
         assertNull(stringType.findSupertype(CharBuffer.class));
@@ -56,15 +55,15 @@ public void findSupertype() {
     }
 
     @Test
-    public void isConcrete() {
+    public void testIsConcrete() {
         ResolvedObjectType stringType = ResolvedObjectType.create(String.class, null, null, null);
         assertTrue(stringType.isConcrete());
         ResolvedObjectType charBufferType = ResolvedObjectType.create(CharBuffer.class, null, null, null);
         assertFalse(charBufferType.isConcrete());
     }
 
     @Test
-    public void accessors() {
+    public void testAccessors() {
         // the default accessor implementation is to return an empty-list; check for a new subtype
         ResolvedType type = new ResolvedType(String.class, null) {
             @Override public boolean canCreateSubtypes() {