Auto merge of rust-lang#124807 - GuillaumeGomez:migrate-rustdoc-io-er…

…ror, r=<try>

Migrate `run-make/rustdoc-io-error` to `rmake.rs`

Part of rust-lang#121876.

r? `@jieyouxu`

try-job: armhf-gnu

src/tools/tidy/src/allowed_run_make_makefiles.txt

@@ -239,7 +239,6 @@ run-make/rlib-format-packed-bundled-libs-3/Makefile
 run-make/rlib-format-packed-bundled-libs/Makefile
 run-make/rmeta-preferred/Makefile
 run-make/rustc-macro-dep-files/Makefile
-run-make/rustdoc-io-error/Makefile
 run-make/rustdoc-scrape-examples-invalid-expr/Makefile
 run-make/rustdoc-scrape-examples-macros/Makefile
 run-make/rustdoc-scrape-examples-multiple/Makefile

tests/run-make/rustdoc-io-error/rmake.rs

@@ -0,0 +1,41 @@
+// This test verifies that rustdoc doesn't ICE when it encounters an IO error
+// while generating files. Ideally this would be a rustdoc-ui test, so we could
+// verify the error message as well.
+//
+// It operates by creating a temporary directory and modifying its
+// permissions so that it is not writable. We have to take special care to set
+// the permissions back to normal so that it's able to be deleted later.
+
+use run_make_support::{rustdoc, tmp_dir};
+use std::fs;
+
+fn main() {
+    // FIXME: Remove this line once figured out what's wrong.
+    std::env::set_var("RUST_BACKTRACE", "1");
+    let out_dir = tmp_dir().join("rustdoc-io-error");
+    let output = fs::create_dir(&out_dir).unwrap();
+    let mut permissions = fs::metadata(&out_dir).unwrap().permissions();
+    permissions.set_readonly(true);
+    fs::set_permissions(&out_dir, permissions.clone()).unwrap();
+
+    let output = rustdoc().input("foo.rs").output(&out_dir).command_output();
+
+    // Changing back permissions.
+    permissions.set_readonly(false);
+    fs::set_permissions(&out_dir, permissions).unwrap();
+
+    let stderr = String::from_utf8(output.stderr).unwrap();
+
+    // Checks that rustdoc failed with the error code 1.
+    #[cfg(target_os = "linux")]
+    if output.status.code().unwrap() != 1 {
+        eprintln!("=== STDOUT ===");
+        eprintln!("{}", String::from_utf8(output.stdout).unwrap());
+        eprintln!("=== STDERR ===");
+        eprintln!("{stderr}");
+        panic!("Expected status code to be 1, found {}", output.status.code().unwrap());
+    }
+    assert!(!output.status.success());
+
+    assert!(stderr.contains("error: couldn't generate documentation: Permission denied"));
+}