change the method of instantiateClass from default to private and add test cases

Author: wei.zw

src/main/java/org/apache/ibatis/reflection/factory/DefaultObjectFactory.java

@@ -55,7 +55,7 @@ public void setProperties(Properties properties) {
     // no props for default
   }
 
-  <T> T instantiateClass(Class<T> type, List<Class<?>> constructorArgTypes, List<Object> constructorArgs) {
+  private  <T> T instantiateClass(Class<T> type, List<Class<?>> constructorArgTypes, List<Object> constructorArgs) {
     try {
       Constructor<T> constructor;
       if (constructorArgTypes == null || constructorArgs == null) {

src/test/java/org/apache/ibatis/reflection/factory/DefaultObjectFactoryTest.java

@@ -15,8 +15,18 @@
  */
 package org.apache.ibatis.reflection.factory;
 
+import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collection;
 import java.util.Collections;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.SortedSet;
+import java.util.TreeSet;
 
 import org.apache.ibatis.reflection.ReflectionException;
 import org.junit.Assert;
@@ -30,20 +40,20 @@
 public class DefaultObjectFactoryTest {
 
   @Test
-  public void instantiateClass() throws Exception {
+  public void createClass() throws Exception {
     DefaultObjectFactory defaultObjectFactory = new DefaultObjectFactory();
-    TestClass testClass = defaultObjectFactory.instantiateClass(TestClass.class,
+    TestClass testClass = defaultObjectFactory.create(TestClass.class,
         Arrays.<Class<?>>asList(String.class, Integer.class), Arrays.<Object>asList("foo", 0));
 
     Assert.assertEquals("myInteger didn't match expected", (Integer) 0, testClass.myInteger);
     Assert.assertEquals("myString didn't match expected", "foo", testClass.myString);
   }
 
   @Test
-  public void instantiateClassThrowsProperErrorMsg() {
+  public void createClassThrowsProperErrorMsg() {
     DefaultObjectFactory defaultObjectFactory = new DefaultObjectFactory();
     try {
-      defaultObjectFactory.instantiateClass(TestClass.class, Collections.<Class<?>>singletonList(String.class), Collections.<Object>singletonList("foo"));
+      defaultObjectFactory.create(TestClass.class, Collections.<Class<?>>singletonList(String.class), Collections.<Object>singletonList("foo"));
       Assert.fail("Should have thrown ReflectionException");
     } catch (Exception e) {
       Assert.assertTrue("Should be ReflectionException", e instanceof ReflectionException);
@@ -52,4 +62,39 @@ public void instantiateClassThrowsProperErrorMsg() {
     }
   }
 
+  @Test
+  public void creatHashMap() throws  Exception{
+     DefaultObjectFactory defaultObjectFactory=new DefaultObjectFactory();
+     Map  map= defaultObjectFactory.create(Map.class,null,null);
+     Assert.assertTrue("Should be HashMap",map instanceof HashMap);
+  }
+
+  @Test
+  public void createArrayList() throws Exception {
+    DefaultObjectFactory defaultObjectFactory = new DefaultObjectFactory();
+    List list = defaultObjectFactory.create(List.class);
+    Assert.assertTrue(" list should be ArrayList", list instanceof ArrayList);
+
+    Collection collection = defaultObjectFactory.create(Collection.class);
+    Assert.assertTrue(" collection should be ArrayList", collection instanceof ArrayList);
+
+    Iterable iterable = defaultObjectFactory.create(Iterable.class);
+    Assert.assertTrue(" iterable should be ArrayList", iterable instanceof ArrayList);
+  }
+
+
+  @Test
+  public void createTreeSet() throws Exception {
+    DefaultObjectFactory defaultObjectFactory = new DefaultObjectFactory();
+    SortedSet sortedSet = defaultObjectFactory.create(SortedSet.class);
+    Assert.assertTrue(" sortedSet should be TreeSet", sortedSet instanceof TreeSet);
+  }
+
+
+  @Test
+  public void createHashSet() throws Exception {
+    DefaultObjectFactory defaultObjectFactory = new DefaultObjectFactory();
+    Set set = defaultObjectFactory.create(Set.class);
+    Assert.assertTrue(" set should be HashSet", set instanceof HashSet);
+  }
 }