Auto merge of #13890 - weihanglo:rustfix, r=ehuss

test(rustfix): run some tests only on nightly

Author: bors

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "rustfix"
-version = "0.8.3"
+version = "0.8.4"
 authors = [
     "Pascal Hertleif <killercup@gmail.com>",
     "Oliver Schneider <oli-obk@users.noreply.github.com>",

crates/rustfix/Cargo.toml

@@ -1,4 +1,4 @@
-use std::collections::{HashSet};
+use std::collections::HashSet;
 
 fn main() {
     let _: HashSet<()>;

crates/rustfix/tests/everything/multiple-solutions.fixed.rs renamed to crates/rustfix/tests/everything/multiple-solutions.nightly.fixed.rs

@@ -1,4 +1,5 @@
 {
+  "$message_type": "diagnostic",
   "message": "unused imports: `HashMap`, `VecDeque`",
   "code": {
     "code": "unused_imports",
@@ -7,7 +8,7 @@
   "level": "warning",
   "spans": [
     {
-      "file_name": "src/main.rs",
+      "file_name": "./tests/everything/multiple-solutions.nightly.rs",
       "byte_start": 23,
       "byte_end": 30,
       "line_start": 1,
@@ -28,7 +29,7 @@
       "expansion": null
     },
     {
-      "file_name": "src/main.rs",
+      "file_name": "./tests/everything/multiple-solutions.nightly.rs",
       "byte_start": 41,
       "byte_end": 49,
       "line_start": 1,
@@ -51,7 +52,7 @@
   ],
   "children": [
     {
-      "message": "#[warn(unused_imports)] on by default",
+      "message": "`#[warn(unused_imports)]` on by default",
       "code": null,
       "level": "note",
       "spans": [],
@@ -64,7 +65,28 @@
       "level": "help",
       "spans": [
         {
-          "file_name": "src/main.rs",
+          "file_name": "./tests/everything/multiple-solutions.nightly.rs",
+          "byte_start": 22,
+          "byte_end": 23,
+          "line_start": 1,
+          "line_end": 1,
+          "column_start": 23,
+          "column_end": 24,
+          "is_primary": true,
+          "text": [
+            {
+              "text": "use std::collections::{HashMap, HashSet, VecDeque};",
+              "highlight_start": 23,
+              "highlight_end": 24
+            }
+          ],
+          "label": null,
+          "suggested_replacement": "",
+          "suggestion_applicability": "MachineApplicable",
+          "expansion": null
+        },
+        {
+          "file_name": "./tests/everything/multiple-solutions.nightly.rs",
           "byte_start": 23,
           "byte_end": 32,
           "line_start": 1,
@@ -85,7 +107,7 @@
           "expansion": null
         },
         {
-          "file_name": "src/main.rs",
+          "file_name": "./tests/everything/multiple-solutions.nightly.rs",
           "byte_start": 39,
           "byte_end": 49,
           "line_start": 1,
@@ -104,11 +126,41 @@
           "suggested_replacement": "",
           "suggestion_applicability": "MachineApplicable",
           "expansion": null
+        },
+        {
+          "file_name": "./tests/everything/multiple-solutions.nightly.rs",
+          "byte_start": 49,
+          "byte_end": 50,
+          "line_start": 1,
+          "line_end": 1,
+          "column_start": 50,
+          "column_end": 51,
+          "is_primary": true,
+          "text": [
+            {
+              "text": "use std::collections::{HashMap, HashSet, VecDeque};",
+              "highlight_start": 50,
+              "highlight_end": 51
+            }
+          ],
+          "label": null,
+          "suggested_replacement": "",
+          "suggestion_applicability": "MachineApplicable",
+          "expansion": null
         }
       ],
       "children": [],
       "rendered": null
     }
   ],
-  "rendered": "warning: unused imports: `HashMap`, `VecDeque`\n --> src/main.rs:1:24\n  |\n1 | use std::collections::{HashMap, HashSet, VecDeque};\n  |                        ^^^^^^^           ^^^^^^^^\n  |\n  = note: #[warn(unused_imports)] on by default\nhelp: remove the unused imports\n  |\n1 | use std::collections::{HashSet};\n  |                       --     --\n\n"
+  "rendered": "warning: unused imports: `HashMap`, `VecDeque`\n --> ./tests/everything/multiple-solutions.nightly.rs:1:24\n  |\n1 | use std::collections::{HashMap, HashSet, VecDeque};\n  |                        ^^^^^^^           ^^^^^^^^\n  |\n  = note: `#[warn(unused_imports)]` on by default\n\n"
+}
+{
+  "$message_type": "diagnostic",
+  "message": "1 warning emitted",
+  "code": null,
+  "level": "warning",
+  "spans": [],
+  "children": [],
+  "rendered": "warning: 1 warning emitted\n\n"
 }

crates/rustfix/tests/everything/multiple-solutions.json renamed to crates/rustfix/tests/everything/multiple-solutions.nightly.json

@@ -45,6 +45,26 @@ mod settings {
     pub const BLESS: &str = "RUSTFIX_TEST_BLESS";
 }
 
+static mut VERSION: (u32, bool) = (0, false);
+
+// Temporarily copy from `cargo_test_macro::version`.
+fn version() -> (u32, bool) {
+    static INIT: std::sync::Once = std::sync::Once::new();
+    INIT.call_once(|| {
+        let output = Command::new("rustc")
+            .arg("-V")
+            .output()
+            .expect("cargo should run");
+        let stdout = std::str::from_utf8(&output.stdout).expect("utf8");
+        let vers = stdout.split_whitespace().skip(1).next().unwrap();
+        let is_nightly = option_env!("CARGO_TEST_DISABLE_NIGHTLY").is_none()
+            && (vers.contains("-nightly") || vers.contains("-dev"));
+        let minor = vers.split('.').skip(1).next().unwrap().parse().unwrap();
+        unsafe { VERSION = (minor, is_nightly) }
+    });
+    unsafe { VERSION }
+}
+
 fn compile(file: &Path) -> Result<Output, Error> {
     let tmp = tempdir()?;
 
@@ -220,7 +240,20 @@ fn assert_fixtures(dir: &str, mode: &str) {
         .unwrap();
     let mut failures = 0;
 
+    let is_not_nightly = !version().1;
+
     for file in &files {
+        if file
+            .file_stem()
+            .unwrap()
+            .to_str()
+            .unwrap()
+            .ends_with(".nightly")
+            && is_not_nightly
+        {
+            info!("skipped: {file:?}");
+            continue;
+        }
         if let Err(err) = test_rustfix_with_file(file, mode) {
             println!("failed: {}", file.display());
             warn!("{:?}", err);