Skip to content

Commit

Permalink
Regression test println!() panic message on ErrorKind::BrokenPipe
Browse files Browse the repository at this point in the history
No existing test failed if the [`panic!()`][1] of the `println!()`
family of functions was removed, or if its message was changed.

So add such a test.

[1] https://github.com/rust-lang/rust/blob/104f4300cfddbd956e32820ef202a732f06ec848/library/std/src/io/stdio.rs#L1007-L1009
  • Loading branch information
Enselic committed Mar 10, 2023
1 parent 81be7b8 commit 4dc5e61
Show file tree
Hide file tree
Showing 3 changed files with 46 additions and 0 deletions.
43 changes: 43 additions & 0 deletions tests/ui/process/println-with-broken-pipe.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
// run-pass
// check-run-results
// ignore-windows
// ignore-emscripten
// ignore-fuchsia
// ignore-horizon
// ignore-android
// normalize-stderr-test ".rs:\d+:\d+" -> ".rs:LL:CC"

// Test what the error message looks like when `println!()` panics because of
// `std::io::ErrorKind::BrokenPipe`

#![feature(unix_sigpipe)]

use std::env;
use std::process::{Command, Stdio};

#[unix_sigpipe = "sig_ign"]
fn main() {
let mut args = env::args();
let me = args.next().unwrap();

if let Some(arg) = args.next() {
// More than enough iterations to fill any pipe buffer. Normally this
// loop will end with a panic more or less immediately.
for _ in 0..65536 * 64 {
println!("{arg}");
}
unreachable!("should have panicked because of BrokenPipe");
}

// Set up a pipeline with a short-lived consumer and wait for it to finish.
// This will produce the `println!()` panic message on stderr.
let mut producer = Command::new(&me)
.arg("this line shall appear exactly once on stdout")
.stdout(Stdio::piped())
.spawn()
.unwrap();
let mut consumer =
Command::new("head").arg("-n1").stdin(producer.stdout.take().unwrap()).spawn().unwrap();
consumer.wait().unwrap();
producer.wait().unwrap();
}
2 changes: 2 additions & 0 deletions tests/ui/process/println-with-broken-pipe.run.stderr
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
thread 'main' panicked at 'failed printing to stdout: Broken pipe (os error 32)', library/std/src/io/stdio.rs:LL:CC
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
1 change: 1 addition & 0 deletions tests/ui/process/println-with-broken-pipe.run.stdout
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
this line shall appear exactly once on stdout

0 comments on commit 4dc5e61

Please sign in to comment.