review: Fix: bug related with TypeReference generics in noclasspath

src/main/java/spoon/support/compiler/jdt/JDTTreeBuilder.java

@@ -1244,7 +1244,9 @@ private boolean createParameterizedType(TypeReference parameterizedTypeReference
 		if (skipTypeInAnnotation) {
 			return true;
 		}
-		context.enter(factory.Code().createTypeAccessWithoutCloningReference(references.buildTypeReference(parameterizedTypeReference, null)), parameterizedTypeReference);
+		CtTypeReference typeReference = references.buildTypeReference(parameterizedTypeReference, null);
+		CtTypeAccess typeAccess = factory.Code().createTypeAccessWithoutCloningReference(typeReference);
+		context.enter(typeAccess, parameterizedTypeReference);
 		return true;
 	}
 

src/main/java/spoon/support/compiler/jdt/ReferenceBuilder.java

@@ -130,7 +130,8 @@ <T> CtTypeReference<T> buildTypeReference(TypeReference type, Scope scope) {
 		if (type == null) {
 			return null;
 		}
-		return buildTypeReferenceInternal(this.<T>getTypeReference(type.resolvedType, type), type, scope);
+		CtTypeReference<T> typeReference = this.<T>getTypeReference(type.resolvedType, type);
+		return buildTypeReferenceInternal(typeReference, type, scope);
 	}
 
 	/**
@@ -457,7 +458,8 @@ <T> CtTypeReference<T> getTypeReference(TypeBinding binding, TypeReference ref)
 			insertGenericTypesInNoClasspathFromJDTInSpoon(ref, ctRef);
 			return ctRef;
 		}
-		return getTypeReference(ref);
+		CtTypeReference<T> result = getTypeReference(ref);
+		return result;
 	}
 
 	CtTypeReference<Object> getTypeParameterReference(TypeBinding binding, TypeReference ref) {
@@ -542,7 +544,9 @@ <T> CtTypeReference<T> getTypeReference(TypeReference ref) {
 		}
 		if (!res.toString().replace(", ?", ",?").endsWith(CharOperation.toString(ref.getParameterizedTypeName()))) {
 			// verify that we did not match a class that have the same name in a different package
-			return this.jdtTreeBuilder.getFactory().Type().createReference(CharOperation.toString(ref.getParameterizedTypeName()));
+			CtTypeReference result = this.jdtTreeBuilder.getFactory().Type().createReference(res.getQualifiedName());
+			result.setActualTypeArguments(res.getActualTypeArguments());
+			return result;
 		}
 		return res;
 	}

src/test/java/spoon/test/imports/ImportTest.java

@@ -1230,4 +1230,25 @@ public void testImportStarredPackageWithNonVisibleClass() throws IOException {
 
 		assertEquals(3, cu.getImports().size());
 	}
+
+	@Test
+	public void testImportWithGenerics() throws IOException {
+		final Launcher launcher = new Launcher();
+		launcher.addInputResource("./src/test/resources/import-with-generics/TestWithGenerics.java");
+		launcher.getEnvironment().setAutoImports(true);
+		launcher.getEnvironment().setShouldCompile(true);
+		launcher.getEnvironment().setNoClasspath(true);
+		launcher.setSourceOutputDirectory("./target/import-with-generics");
+		launcher.run();
+
+		PrettyPrinter prettyPrinter = launcher.createPrettyPrinter();
+		CtType element = launcher.getFactory().Class().get("spoon.test.imports.testclasses.TestWithGenerics");
+		List<CtType<?>> toPrint = new ArrayList<>();
+		toPrint.add(element);
+
+		prettyPrinter.calculate(element.getPosition().getCompilationUnit(), toPrint);
+		String output = prettyPrinter.getResult();
+
+		assertTrue(output.contains("import spoon.test.imports.testclasses.withgenerics.Target;"));
+	}
 }

src/test/resources/import-with-generics/TestWithGenerics.java

@@ -0,0 +1,7 @@
+package spoon.test.imports.testclasses;
+/**
+ * Created by urli on 16/10/2017.
+ */
+public class TestWithGenerics {
+    public spoon.test.imports.testclasses.withgenerics.Target<Object, Object> myfields;
+}