Rewrite and rename issue-26092 to rmake

Author: Oneirical

src/tools/run-make-support/src/command.rs

@@ -167,25 +167,35 @@ impl CompletedProcess {
         self
     }
 
+    /// Checks that trimmed `stdout` does not contain trimmed `content`.
     #[track_caller]
     pub fn assert_stdout_not_contains<S: AsRef<str>>(&self, needle: S) -> &Self {
         assert_not_contains(&self.stdout_utf8(), needle.as_ref());
         self
     }
 
+    /// Checks that trimmed `stdout` contains trimmed `content`.
+    #[track_caller]
+    pub fn assert_stdout_contains<S: AsRef<str>>(&self, needle: S) -> &Self {
+        assert!(self.stdout_utf8().contains(needle.as_ref()));
+        self
+    }
+
     /// Checks that trimmed `stderr` matches trimmed `content`.
     #[track_caller]
     pub fn assert_stderr_equals<S: AsRef<str>>(&self, content: S) -> &Self {
         assert_eq!(self.stderr_utf8().trim(), content.as_ref().trim());
         self
     }
 
+    /// Checks that trimmed `stderr` contains trimmed `content`.
     #[track_caller]
     pub fn assert_stderr_contains<S: AsRef<str>>(&self, needle: S) -> &Self {
         assert!(self.stderr_utf8().contains(needle.as_ref()));
         self
     }
 
+    /// Checks that trimmed `stderr` does not contain trimmed `content`.
     #[track_caller]
     pub fn assert_stderr_not_contains<S: AsRef<str>>(&self, needle: S) -> &Self {
         assert_not_contains(&self.stdout_utf8(), needle.as_ref());

src/tools/tidy/src/allowed_run_make_makefiles.txt

@@ -88,7 +88,6 @@ run-make/issue-20626/Makefile
 run-make/issue-22131/Makefile
 run-make/issue-25581/Makefile
 run-make/issue-26006/Makefile
-run-make/issue-26092/Makefile
 run-make/issue-28595/Makefile
 run-make/issue-33329/Makefile
 run-make/issue-35164/Makefile

tests/run-make/CURRENT_RUSTC_VERSION/rmake.rs

@@ -12,7 +12,6 @@ fn main() {
 
     let output =
         rustc().input("main.rs").emit("metadata").extern_("stable", "libstable.rmeta").run();
-
     let version = fs_wrapper::read_to_string(source_root().join("src/version"));
     let expected_string = format!("stable since {}", version.trim());
     output.assert_stderr_contains(expected_string);

tests/run-make/issue-26092/blank.rs …n-make/clear-error-blank-output/blank.rstests/run-make/issue-26092/blank.rs renamed to tests/run-make/clear-error-blank-output/blank.rs tests/run-make/issue-26092/blank.rs renamed to tests/run-make/clear-error-blank-output/blank.rs

@@ -0,0 +1,13 @@
+// When an empty output file is passed to rustc, the ensuing error message
+// should be clear. However, calling file_stem on an empty path returns None,
+// which, when unwrapped, causes a panic, stopping execution of rustc
+// and printing an obscure message instead of reaching the helpful
+// error message. This test checks that the panic does not occur.
+// See https://github.com/rust-lang/rust/pull/26199
+
+use run_make_support::rustc;
+
+fn main() {
+    let output = rustc().output("").input("blank.rs").run_fail();
+    output.assert_stderr_not_contains("panic");
+}

tests/run-make/clear-error-blank-output/rmake.rs

@@ -4,21 +4,16 @@
 // This ensures that the compiler successfully parses this flag.
 // See https://github.com/rust-lang/rust/pull/36574
 
-use run_make_support::rustc;
+use run_make_support::{is_msvc, rustc};
 
 fn main() {
-    let output = String::from_utf8(
-        rustc()
-            .input("empty.rs")
-            .link_arg("-lfoo")
-            .link_arg("-lbar")
-            .print("link-args")
-            .command_output()
-            .stdout,
-    )
-    .unwrap();
-    assert!(
-        output.contains("lfoo") || output.contains("lbar"),
-        "The output did not contain the expected \"lfoo\" or \"lbar\" strings."
-    );
+    //FIXME(Oneirical): These link-args do not fail when passed to the MSVC linker.
+    // The original Makefile test did not check if rustc succeeded or failed.
+    let out = if is_msvc() {
+        rustc().input("empty.rs").link_arg("-lfoo").link_arg("-lbar").print("link-args").run()
+    } else {
+        rustc().input("empty.rs").link_arg("-lfoo").link_arg("-lbar").print("link-args").run_fail()
+    };
+    out.assert_stdout_contains("lfoo");
+    out.assert_stdout_contains("lbar");
 }

tests/run-make/issue-26092/Makefile

@@ -7,26 +7,18 @@
 
 //@ ignore-msvc
 
+use run_make_support::rustc;
+
 fn main() {
     rustc().input("depa.rs").run();
     rustc().input("depb.rs").run();
     rustc().input("depc.rs").run();
-    let output =
-        String::from_utf8(rustc().input("empty.rs").cfg("bar").command_output().stderr).unwrap();
-    let pos_a1 =
-        output.find("-ltesta").expect("empty.rs, compiled with --cfg, should contain -ltesta");
-    let pos_b = output[pos_a1..]
-        .find("-ltestb")
-        .map(|pos| pos + pos_a1)
-        .expect("empty.rs, compiled with --cfg, should contain -ltestb");
-    let _ = output[pos_b..]
-        .find("-ltesta")
-        .map(|pos| pos + pos_b)
-        .expect("empty.rs, compiled with --cfg, should contain a second -ltesta");
-    let output = String::from_utf8(rustc().input("empty.rs").command_output().stderr).unwrap();
-    assert!(output.contains("-ltesta"));
-    let output = String::from_utf8(rustc().input("empty.rs").command_output().stderr).unwrap();
-    assert!(!output.contains("-ltestb"));
-    let output = String::from_utf8(rustc().input("empty.rs").command_output().stderr).unwrap();
-    assert_eq!(output.matches("-ltesta").count, 1);
+    let output = rustc().input("empty.rs").cfg("bar").run_fail();
+    output.assert_stderr_contains("\"-ltesta\" \"-ltestb\" \"-ltesta\"");
+    let output = rustc().input("empty.rs").run_fail();
+    output.assert_stderr_contains("\"-ltesta\"");
+    let output = rustc().input("empty.rs").run_fail();
+    output.assert_stderr_not_contains("\"-ltestb\"");
+    let output = rustc().input("empty.rs").run_fail();
+    output.assert_stderr_not_contains("\"-ltesta\" \"-ltesta\" \"-ltesta\"");
 }

tests/run-make/link-arg/rmake.rs

@@ -5,7 +5,6 @@ use run_make_support::rustdoc;
 
 fn main() {
     let output = rustdoc().input("input.rs").arg("--test").run_fail().stdout_utf8();
-
     let should_contain = &[
         "input.rs - foo (line 5)",
         "input.rs:7:15",