add unit test

humyna · Aug 14, 2017 · ed5fc5e · ed5fc5e
1 parent b792a75
commit ed5fc5e
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/hutool-core/src/main/java/com/xiaoleilu/hutool/util/BeanUtil.java b/hutool-core/src/main/java/com/xiaoleilu/hutool/util/BeanUtil.java
@@ -412,7 +412,7 @@ public static <T> T fillBean(T bean, ValueProvider<String> valueProvider, CopyOp
 		Class<?> actualEditable = bean.getClass();
 		if (copyOptions.editable != null) {
 			//检查限制类是否为target的父类或接口
-			if (!copyOptions.editable.isInstance(bean)) {
+			if (false == copyOptions.editable.isInstance(bean)) {
 				throw new IllegalArgumentException(StrUtil.format("Target class [{}] not assignable to Editable class [{}]", bean.getClass().getName(), copyOptions.editable.getName()));
 			}
 			actualEditable = copyOptions.editable;

diff --git a/hutool-core/src/test/java/com/xiaoleilu/hutool/core/util/BeanUtilTest.java b/hutool-core/src/test/java/com/xiaoleilu/hutool/core/util/BeanUtilTest.java
@@ -2,6 +2,7 @@
 
 import java.lang.reflect.Type;
 import java.util.HashMap;
+import java.util.Map;
 
 import org.junit.Assert;
 import org.junit.Test;
@@ -70,15 +71,36 @@ public void mapToBeanIgnoreCaseTest(){
 		HashMap<String,Object> map = CollectionUtil.newHashMap();
 		map.put("Name", "Joe");
 		map.put("aGe", 12);
+
 		Person person = BeanUtil.mapToBeanIgnoreCase(map, Person.class, false);
 		Assert.assertEquals(person.getName(), "Joe");
 		Assert.assertEquals(person.getAge(), 12);
 	}
 
+	@Test
+	public void beanToMapTest() {
+		SubPerson person = new SubPerson();
+		person.setAge(14);
+		person.setOpenid("11213232");
+		person.setName("测试A11");
+		person.setSubName("sub名字");
+
+		Map<String, Object> map = BeanUtil.beanToMap(person);
+		Assert.assertEquals(map.get("name"), "测试A11");
+		Assert.assertEquals(map.get("age"), 14);
+	}
+
 
 	//-----------------------------------------------------------------------------------------------------------------
 	public static class SubPerson extends Person{
-
+		private String subName;
+
+		public String getSubName() {
+			return subName;
+		}
+		public void setSubName(String subName) {
+			this.subName = subName;
+		}
 	}
 
 	public static class Person{