Don't change order of tests and benches for cleaner diff.

ehuss · ehuss · commit efc0dd1b070f · 2017-10-20T16:59:32.000-07:00
diff --git a/src/cargo/ops/cargo_compile.rs b/src/cargo/ops/cargo_compile.rs
@@ -652,14 +652,6 @@ fn generate_targets<'a>(pkg: &'a Package,
                 pkg, bins, "bin", Target::is_bin, profile)?);
             targets.append(&mut propose_indicated_targets(
                 pkg, examples, "example", Target::is_example, profile)?);
-            // If --benches was specified, add all targets that would be
-            // generated by `cargo bench`.
-            let bench_filter = match benches {
-                FilterRule::All => Target::benched,
-                FilterRule::Just(_) => Target::is_bench
-            };
-            targets.append(&mut propose_indicated_targets(
-                pkg, benches, "bench", bench_filter, bench_profile)?);
             // If --tests was specified, add all targets that would be
             // generated by `cargo test`.
             let test_filter = match tests {
@@ -668,6 +660,14 @@ fn generate_targets<'a>(pkg: &'a Package,
             };
             targets.append(&mut propose_indicated_targets(
                 pkg, tests, "test", test_filter, test_profile)?);
+            // If --benches was specified, add all targets that would be
+            // generated by `cargo bench`.
+            let bench_filter = match benches {
+                FilterRule::All => Target::benched,
+                FilterRule::Just(_) => Target::is_bench
+            };
+            targets.append(&mut propose_indicated_targets(
+                pkg, benches, "bench", bench_filter, bench_profile)?);
             targets
         }
     };
