fix(getTypeDeclaration): getTypeDeclaration('?') should point to Object

INRIA · Nov 16, 2016 · d318222 · d318222
1 parent a98b96d
commit d318222
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 0 deletions.
diff --git a/src/main/java/spoon/support/reflect/reference/CtWildcardReferenceImpl.java b/src/main/java/spoon/support/reflect/reference/CtWildcardReferenceImpl.java
@@ -16,6 +16,7 @@
  */
 package spoon.support.reflect.reference;
 
+import spoon.reflect.declaration.CtType;
 import spoon.reflect.reference.CtReference;
 import spoon.reflect.reference.CtWildcardReference;
 import spoon.reflect.visitor.CtVisitor;
@@ -39,4 +40,10 @@ public <T extends CtReference> T setSimpleName(String simplename) {
  public CtWildcardReference clone() {
  return (CtWildcardReference) super.clone();
  }
+
+ @Override
+ public CtType<Object> getTypeDeclaration() {
+ return getFactory().Type().get(Object.class);
+ }
+
 }
diff --git a/src/test/java/spoon/test/reference/TypeReferenceTest.java b/src/test/java/spoon/test/reference/TypeReferenceTest.java
@@ -10,6 +10,7 @@
 import spoon.reflect.code.CtExpression;
 import spoon.reflect.code.CtFieldRead;
 import spoon.reflect.code.CtInvocation;
+import spoon.reflect.code.CtLocalVariable;
 import spoon.reflect.code.CtNewClass;
 import spoon.reflect.code.CtReturn;
 import spoon.reflect.code.CtStatement;
@@ -25,9 +26,11 @@
 import spoon.reflect.reference.CtFieldReference;
 import spoon.reflect.reference.CtTypeParameterReference;
 import spoon.reflect.reference.CtTypeReference;
+import spoon.reflect.reference.CtWildcardReference;
 import spoon.reflect.visitor.Query;
 import spoon.reflect.visitor.filter.ReferenceTypeFilter;
 import spoon.reflect.visitor.filter.TypeFilter;
+import spoon.support.compiler.SnippetCompilationHelper;
 import spoon.test.reference.testclasses.EnumValue;
 import spoon.test.reference.testclasses.Panini;
 import spoon.testing.utils.ModelUtils;
@@ -580,4 +583,20 @@ public void testGetTypeDeclaration() throws Exception {
  assertNotNull(iBar);
  assertEquals("compilation.IBar", iBar.getQualifiedName());
  }
+
+ @Test
+ public void testTypeDeclarationWildcard() throws Exception {
+ // contract1: getTypeDeclaration nevers returns null, even for wilddards
+ // contract2: getTypeDeclaration returns a CtTYpe representing Object as the compiler does
+ CtLocalVariable<?> s = new Launcher().getFactory().Code().createCodeSnippetStatement("java.util.List<?> l = null").compile();
+ assertEquals("?", s.getType().getActualTypeArguments().get(0).getSimpleName());
+ assertTrue(CtWildcardReference.class.isInstance(s.getType().getActualTypeArguments().get(0)));
+ assertEquals("Object", s.getType().getActualTypeArguments().get(0).getTypeDeclaration().getSimpleName());
+ assertEquals(Object.class, s.getType().getActualTypeArguments().get(0).getTypeDeclaration().getActualClass());
+
+ // some additional tests
+ CtLocalVariable<?> s2 = new Launcher().getFactory().Code().createCodeSnippetStatement("java.util.List<String> l = null").compile();
+ assertEquals("String", s2.getType().getActualTypeArguments().get(0).getSimpleName());
+ assertEquals(String.class, s2.getType().getActualTypeArguments().get(0).getTypeDeclaration().getActualClass());
+ }
 }