Skip to content

Commit

Permalink
Don't test for trace messages from interrupt handler
Browse files Browse the repository at this point in the history
  • Loading branch information
sourcefrog committed Jun 11, 2024
1 parent fd149e6 commit 93fa91f
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions tests/main.rs
Original file line number Diff line number Diff line change
Expand Up @@ -709,9 +709,10 @@ fn interrupt_caught_and_kills_children() {
println!("stderr:\n{stderr}");

assert!(stderr.contains("interrupted"));
// Also because of `--level=trace` we see some debug details.
assert!(stderr.contains("terminating child process"));
assert!(stderr.contains("terminated child exit status"));
// We used to look here for some other trace messages about how it's interrupted, but
// that seems to be racy: sometimes the parent sees the child interrupted before it
// emits these messages? Anyhow, it's not essential.

// This shouldn't cause a panic though (#333)
assert!(!stderr.contains("panic"));
// And we don't want duplicate messages about workers failing.
Expand Down

0 comments on commit 93fa91f

Please sign in to comment.