Skip to content

Commit

Permalink
fix spurious test failure with panic=abort
Browse files Browse the repository at this point in the history
  • Loading branch information
pietroalbini committed Jul 11, 2023
1 parent af9df2f commit d8fb568
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions tests/ui/panic-handler/weak-lang-item-2.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,10 +6,18 @@

extern crate weak_lang_items as other;

use std::thread;

fn main() {
let _ = thread::spawn(move|| {
let _ = std::thread::spawn(move || {
// The goal of the test is just to make sure other::foo() is called. Since the function
// panics, it's executed in its own thread. That way, the panic is isolated within the
// thread and wont't affect the overall exit code.
//
// That causes a spurious failures in panic=abort targets though: if the program exits
// before the thread is fully initialized the test will pass, but if the thread gets
// executed first the whole program will abort. Adding a 60 seconds sleep will (hopefully!)
// ensure the program always exits before the thread is executed.
std::thread::sleep(std::time::Duration::from_secs(60));

other::foo()
});
}

0 comments on commit d8fb568

Please sign in to comment.