Don't redeclare EnumSet as Set (#183)

* Don't redeclare EnumSet as Set That changes the type of the generic, potentially breaking iterators. Fixes #179 * Update src/test/java/org/openrewrite/staticanalysis/UseCollectionInterfacesTest.java --------- Co-authored-by: Tim te Beek <[email protected]>
openrewrite · Oct 2, 2023 · 9e9b5dc · 9e9b5dc
1 parent 2e3a961
commit 9e9b5dc
Show file tree

Hide file tree

Showing 2 changed files with 74 additions and 76 deletions.
diff --git a/src/main/java/org/openrewrite/staticanalysis/UseCollectionInterfaces.java b/src/main/java/org/openrewrite/staticanalysis/UseCollectionInterfaces.java
@@ -84,7 +84,6 @@ public Duration getEstimatedEffortPerOccurrence() {
         rspecRulesReplaceTypeMap.put("java.util.concurrent.ConcurrentLinkedQueue", "java.util.Queue");
         // Set
         rspecRulesReplaceTypeMap.put("java.util.AbstractSet", "java.util.Set");
-        rspecRulesReplaceTypeMap.put("java.util.EnumSet", "java.util.Set");
         rspecRulesReplaceTypeMap.put("java.util.HashSet", "java.util.Set");
         rspecRulesReplaceTypeMap.put("java.util.LinkedHashSet", "java.util.Set");
         rspecRulesReplaceTypeMap.put("java.util.concurrent.CopyOnWriteArraySet", "java.util.Set");