ChangeType leads to botched imports for nested fields

[timtebeek]

After

• Recreate static imports for identifiers in ChangeType #4291

We continued to see issues in

• https://github.com/openrewrite/rewrite-jackson/pull/6

I've updated the test to better replicate the issue there. With this change the test fails with left imports.

Unexpected result in "A.java":
diff --git a/A.java b/A.java
index 98d3bcb..da17811 100644
--- a/A.java
+++ b/A.java
@@ -1,6 +1,8 @@ 
 import com.fasterxml.jackson.databind.ObjectMapper;
+import com.fasterxml.jackson.databind.SerializationFeature;
 
 import static com.fasterxml.jackson.databind.SerializationFeature.WRAP_ROOT_VALUE;
+import static org.codehaus.jackson.map.SerializationConfig.SerializationFeature.WRAP_ROOT_VALUE;
 
 class A {
     void test() {

[timtebeek]

@sambsnyd if you're up for a puzzle, here's that updated test. I didn't yet get into looking why this now happens

[timtebeek]

This comparison fails

rewrite/rewrite-java/src/main/java/org/openrewrite/java/ChangeType.java

Line 272 in f6473b6

if (maybeClass.toString().equals(oldType.getClassName())) {

For this test

rewrite/rewrite-java-test/src/test/java/org/openrewrite/java/ChangeTypeTest.java

Lines 164 to 166 in 1d8b3c3

	void changeInnerClassToOuterClass() {
	rewriteRun(
	spec -> spec.recipe(new ChangeType("java.util.Map$Entry", "java.util.List", true)),

Because everything up to java. is a J.FieldAccess, all of it ending up in maybeClass, whereas that should only contain the simple class name.