fix(cargo-fix): dont apply same suggestion twice

This assumes that if any of the machine applicable fixes (`solution`)
in a diagnostic is a duplicate, then `cargo fix` should only apply
the entire suggestion once.

src/cargo/ops/fix.rs

@@ -738,7 +738,18 @@ fn rustfix_and_fix(
         // As mentioned above in `rustfix_crate`, we don't immediately warn
         // about suggestions that fail to apply here, and instead we save them
         // off for later processing.
+        let mut already_applied = HashSet::new();
         for suggestion in suggestions.iter().rev() {
+            // Skip seemly duplicated suggestions.
+            if suggestion
+                .solutions
+                .iter()
+                .any(|sol| !already_applied.insert(sol))
+            {
+                // already applied
+                fixed_file.fixes_applied += 1;
+                break;
+            }
             match fixed.apply(suggestion) {
                 Ok(()) => fixed_file.fixes_applied += 1,
                 Err(e) => fixed_file.errors_applying_fixes.push(e.to_string()),

tests/testsuite/fix.rs

@@ -1921,14 +1921,14 @@ fn fix_only_once_for_duplicates() {
         .build();
 
     p.cargo("fix --allow-no-vcs")
-        .with_stderr_contains(
+        .with_stderr(
             "\
 [CHECKING] foo v0.0.1 ([CWD])
 [FIXED] src/lib.rs (2 fixes)
+[FINISHED] `dev` profile [..]
 ",
         )
-        .with_stderr_contains("[WARNING] unnecessary `unsafe` block[..]")
         .run();
 
-    assert!(p.read_file("src/lib.rs").contains("unsafe { unsafe"));
+    assert!(!p.read_file("src/lib.rs").contains("unsafe { unsafe"));
 }