ROASTER-45: Added ignored tests to reproduce the issue

forge · Nov 19, 2014 · adbe1f6 · adbe1f6
1 parent 586076b
commit adbe1f6
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 3 deletions.
diff --git a/impl/src/test/java/org/jboss/forge/test/roaster/model/JavaClassPropertyTest.java b/impl/src/test/java/org/jboss/forge/test/roaster/model/JavaClassPropertyTest.java
@@ -7,6 +7,8 @@
 
 package org.jboss.forge.test.roaster.model;
 
+import static org.junit.Assert.assertTrue;
+
 import java.io.InputStream;
 import java.sql.Timestamp;
 import java.util.Date;
@@ -16,6 +18,7 @@
 import org.jboss.forge.roaster.model.source.JavaClassSource;
 import org.jboss.forge.roaster.model.source.PropertySource;
 import org.junit.Assert;
+import org.junit.Ignore;
 import org.junit.Test;
 
 /**
@@ -71,6 +74,19 @@ public void testChangePropertyType()
       Assert.assertEquals("Timestamp", source.getField("myDate").getType().getName());
       Assert.assertEquals("Timestamp", source.getMethod("getMyDate").getReturnType().getName());
       Assert.assertNotNull(source.getMethod("setMyDate", Timestamp.class));
+   }
 
+   @Test
+   @Ignore("ROASTER-45")
+   public void testFQNTypes() throws Exception
+   {
+      JavaClassSource source = Roaster.create(JavaClassSource.class).setName("MyClass");
+      source.addProperty("java.util.List<java.lang.String>", "list1");
+      source.addProperty("java.util.List<String>", "list2");
+      source.addProperty("java.util.List<java.util.List<String>>", "list3");
+      assertTrue(source.hasProperty("list1"));
+      assertTrue(source.hasProperty("list2"));
+      assertTrue(source.hasProperty("list3"));
    }
+
 }
diff --git a/impl/src/test/java/org/jboss/forge/test/roaster/model/util/TypesTest.java b/impl/src/test/java/org/jboss/forge/test/roaster/model/util/TypesTest.java
@@ -7,13 +7,12 @@
 
 package org.jboss.forge.test.roaster.model.util;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.*;
 
 import java.util.Vector;
 
 import org.jboss.forge.roaster.model.util.Types;
+import org.junit.Ignore;
 import org.junit.Test;
 
 /**
@@ -129,6 +128,11 @@ public void testGenerics()
       assertEquals("java.lang.Class[]", Types.stripGenerics("java.lang.Class<? extends Number>[]"));
       assertEquals("java.lang.Class[]", Types.stripGenerics("java.lang.Class<E extends Enum<E>>[]"));
       assertEquals("int[]", Types.stripGenerics(int[].class.getName()));
+      //@Ignore("ROASTER-45")
+//      assertEquals("java.util.List", Types.stripGenerics("java.util.List<String>"));
+//      assertEquals("java.util.List", Types.stripGenerics("java.util.List<java.lang.String>"));
+//      assertEquals("java.util.List", Types.stripGenerics("java.util.List<List<String>>"));
+//      assertEquals("java.util.List", Types.stripGenerics("java.util.List<List<java.lang.String>>"));
    }
 
    @Test
@@ -143,4 +147,13 @@ public void testAssertClassIsNotJavaLang()
       assertFalse(Types.isJavaLang("AssertClass"));
    }
 
+   @Ignore("ROASTER-45")
+   @Test
+   public void testAssertToSimpleName() throws Exception
+   {
+      assertEquals("List<String>", Types.toSimpleName("java.util.List<java.lang.String>"));
+      assertEquals("List<String>", Types.toSimpleName("java.util.List<String>"));
+      assertEquals("List<List<String>>", Types.toSimpleName("java.util.List<java.util.List<String>>"));
+   }
+
 }