fix spurious test failure with panic=abort

1 files changed, 11 insertions, 3 deletions

diff --git a/tests/ui/panic-handler/weak-lang-item-2.rs b/tests/ui/panic-handler/weak-lang-item-2.rs
index a429d8fabc7..6d6e8e04b05 100644
--- a/tests/ui/panic-handler/weak-lang-item-2.rs
+++ b/tests/ui/panic-handler/weak-lang-item-2.rs
@@ -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()
     });
 }