fix: default to all targets when using --edition flags in cargo fix

Signed-off-by: Rustin170506 <[email protected]>

Author: 0xPoe

src/bin/cargo/commands/fix.rs

@@ -81,8 +81,11 @@ pub fn exec(gctx: &mut GlobalContext, args: &ArgMatches) -> CliResult {
 
     let mut opts = args.compile_options(gctx, mode, Some(&ws), ProfileChecking::LegacyTestOnly)?;
 
-    if !opts.filter.is_specific() {
-        // cargo fix with no target selection implies `--all-targets`.
+    let edition = args.flag("edition") || args.flag("edition-idioms");
+    if !opts.filter.is_specific() && edition {
+        // When `cargo fix` is run without specifying targets but with `--edition` or `--edition-idioms`,
+        // it should default to fixing all targets.
+        // See: https://github.com/rust-lang/cargo/issues/13527
         opts.filter = ops::CompileFilter::new_all_targets();
     }
 

tests/testsuite/fix.rs

@@ -198,7 +198,7 @@ fn prepare_for_2018() {
 }
 
 #[cargo_test]
-fn fix_tests() {
+fn do_not_fix_tests() {
     let p = project()
         .file(
             "src/lib.rs",
@@ -220,6 +220,36 @@ fn fix_tests() {
     p.cargo("fix --allow-no-vcs")
         .with_stderr_data(str![[r#"
 [CHECKING] foo v0.0.1 ([ROOT]/foo)
+[FINISHED] `dev` profile [unoptimized + debuginfo] target(s) in [ELAPSED]s
+
+"#]])
+        .with_stdout_data("")
+        .run();
+}
+
+#[cargo_test]
+fn fix_tests_with_edition() {
+    let p = project()
+        .file(
+            "src/lib.rs",
+            r#"
+                pub fn foo() {}
+
+                #[cfg(test)]
+                mod tests {
+                    #[test]
+                    fn it_works() {
+                        let mut x = 3;
+                        x;
+                    }
+                }
+            "#,
+        )
+        .build();
+
+    p.cargo("fix --edition-idioms --allow-no-vcs")
+        .with_stderr_data(str![[r#"
+[CHECKING] foo v0.0.1 ([ROOT]/foo)
 [FIXED] src/lib.rs (1 fix)
 [WARNING] path statement with no effect
  --> src/lib.rs:9:25
@@ -747,19 +777,6 @@ fn does_not_warn_about_dirty_ignored_files() {
     p.cargo("fix").run();
 }
 
-#[cargo_test]
-fn fix_all_targets_by_default() {
-    let p = project()
-        .file("src/lib.rs", "pub fn foo() { let mut x = 3; let _ = x; }")
-        .file("tests/foo.rs", "pub fn foo() { let mut x = 3; let _ = x; }")
-        .build();
-    p.cargo("fix --allow-no-vcs")
-        .env("__CARGO_FIX_YOLO", "1")
-        .run();
-    assert!(!p.read_file("src/lib.rs").contains("let mut x"));
-    assert!(!p.read_file("tests/foo.rs").contains("let mut x"));
-}
-
 #[cargo_test]
 fn prepare_for_unstable() {
     // During the period where a new edition is coming up, but not yet stable,