Polish. Update template to account for types in MigrateCollectionsUnmodifiableList and MigrateCollectionsUnmodifiableSet.

Author: traceyyoshima

src/main/java/org/openrewrite/java/migrate/lang/MigrateCollectionsUnmodifiableList.java

@@ -76,7 +76,7 @@ public J visitMethodInvocation(J.MethodInvocation method, ExecutionContext execu
 
                             StringJoiner setOf = new StringJoiner(", ", "List.of(", ")");
                             List<Expression> args = arraysInvocation.getArguments();
-                            args.forEach(o -> setOf.add("#{}"));
+                            args.forEach(o -> setOf.add("#{any()}"));
 
                             return autoFormat(m.withTemplate(
                                     JavaTemplate

src/main/java/org/openrewrite/java/migrate/lang/MigrateCollectionsUnmodifiableSet.java

@@ -77,7 +77,7 @@ public J visitMethodInvocation(J.MethodInvocation method, ExecutionContext execu
 
                                 StringJoiner setOf = new StringJoiner(", ", "Set.of(", ")");
                                 List<Expression> args = ((J.MethodInvocation) newSet.getArguments().get(0)).getArguments();
-                                args.forEach(o -> setOf.add("#{}"));
+                                args.forEach(o -> setOf.add("#{any()}"));
 
                                 return autoFormat(m.withTemplate(
                                         JavaTemplate

src/test/kotlin/org/openrewrite/java/migrate/lang/MigrateCollectionsUnmodifiableListTest.kt

@@ -42,4 +42,25 @@ class MigrateCollectionsUnmodifiableListTest : JavaRecipeTest {
             }
         """
     )
+
+    @Issue("https://github.com/openrewrite/rewrite-migrate-java/issues/67")
+    @Test
+    fun unmodifiableListTyped() = assertChanged(
+        before = """
+            import java.util.*;
+            import java.time.LocalDate;
+            
+            class Test {
+                List<LocalDate> s = Collections.unmodifiableList(Arrays.asList(LocalDate.of(2010,1,1),LocalDate.now()));
+            }
+        """,
+        after = """
+            import java.util.List;
+            import java.time.LocalDate;
+            
+            class Test {
+                List<LocalDate> s = List.of(LocalDate.of(2010, 1, 1), LocalDate.now());
+            }
+        """
+    )
 }

src/test/kotlin/org/openrewrite/java/migrate/lang/MigrateCollectionsUnmodifiableSetTest.kt

@@ -42,4 +42,25 @@ class MigrateCollectionsUnmodifiableSetTest : JavaRecipeTest {
             }
         """
     )
+
+    @Issue("https://github.com/openrewrite/rewrite-migrate-java/issues/67")
+    @Test
+    fun unmodifiableSetTyped() = assertChanged(
+        before = """
+            import java.util.*;
+            import java.time.LocalDate;
+            
+            class Test {
+                Set<LocalDate> s = Collections.unmodifiableSet(new HashSet<>(Arrays.asList(LocalDate.of(2010,1,1),LocalDate.now())));
+            }
+        """,
+        after = """
+            import java.util.Set;
+            import java.time.LocalDate;
+            
+            class Test {
+                Set<LocalDate> s = Set.of(LocalDate.of(2010, 1, 1), LocalDate.now());
+            }
+        """
+    )
 }