Fix unused import correction

CHANGELOG.md

@@ -6,7 +6,10 @@
 
 #### Experimental
 
-* None.
+* Fix `unused_import` correction deleting unrelated ranges when there are
+  multiple violations in a single file.  
+  [JP Simard](https://github.com/jpsim)
+  [#2561](https://github.com/realm/SwiftLint/issues/2561)
 
 #### Enhancements
 

Rules.md

@@ -21804,11 +21804,21 @@ class A {}
 ```swift
 ↓import Dispatch
 struct A {
-    static func dispatchMain() {}
+  static func dispatchMain() {}
 }
 A.dispatchMain()
 ```
 
+```swift
+↓import Foundation
+struct A {
+  static func dispatchMain() {}
+}
+A.dispatchMain()
+↓import Dispatch
+
+```
+
 ```swift
 ↓import Foundation
 dispatchMain()

Source/SwiftLintFramework/Rules/Lint/UnusedImportRule.swift

@@ -29,8 +29,19 @@ public struct UnusedImportRule: CorrectableRule, ConfigurationProviderRule, Anal
         triggeringExamples: [
             """
             ↓import Dispatch
-            struct A {\n    static func dispatchMain() {}\n}
+            struct A {
+              static func dispatchMain() {}
+            }
             A.dispatchMain()
+            """,
+            """
+            ↓import Foundation
+            struct A {
+              static func dispatchMain() {}
+            }
+            A.dispatchMain()
+            ↓import Dispatch
+
             """,
             """
             ↓import Foundation
@@ -45,12 +56,32 @@ public struct UnusedImportRule: CorrectableRule, ConfigurationProviderRule, Anal
         corrections: [
             """
             ↓import Dispatch
-            struct A {\n    static func dispatchMain() {}\n}
+            struct A {
+              static func dispatchMain() {}
+            }
+            A.dispatchMain()
+            """:
+            """
+            struct A {
+              static func dispatchMain() {}
+            }
             A.dispatchMain()
+            """,
+            """
+            ↓import Foundation
+            struct A {
+              static func dispatchMain() {}
+            }
+            A.dispatchMain()
+            ↓import Dispatch
+
             """:
             """
-            struct A {\n    static func dispatchMain() {}\n}
+            struct A {
+              static func dispatchMain() {}
+            }
             A.dispatchMain()
+
             """,
             """
             ↓import Foundation
@@ -166,14 +197,16 @@ private extension File {
         if unusedImports.contains("Foundation") && containsAttributesRequiringFoundation() {
             unusedImports.remove("Foundation")
         }
-        return unusedImports.map { module in
-            let testableImportRange = contentsNSString.range(of: "@testable import \(module)\n")
-            if testableImportRange.location != NSNotFound {
-                return (module, testableImportRange)
-            }
+        return unusedImports
+            .map { module in
+                let testableImportRange = contentsNSString.range(of: "@testable import \(module)\n")
+                if testableImportRange.location != NSNotFound {
+                    return (module, testableImportRange)
+                }
 
-            return (module, contentsNSString.range(of: "import \(module)\n"))
-        }
+                return (module, contentsNSString.range(of: "import \(module)\n"))
+            }
+            .sorted(by: { $0.1.location < $1.1.location })
     }
 
     private func containsAttributesRequiringFoundation() -> Bool {