Fix test

Author: gleocadie

bin_tests/src/bin/crashing_test_app.rs

@@ -124,10 +124,12 @@ mod unix {
         let is_panic_mode = matches!(crash_type, CrashType::Panic);
 
         let called_panic_hook = Arc::new(AtomicBool::new(false));
+        let old_hook = std::panic::take_hook();
         if is_panic_mode {
             let called_panic_hook_clone = Arc::clone(&called_panic_hook);
-            std::panic::set_hook(Box::new(move |_| {
+            std::panic::set_hook(Box::new(move |panic_info| {
                 called_panic_hook_clone.store(true, Ordering::SeqCst);
+                old_hook(panic_info);
             }));
         }
 

bin_tests/tests/crashtracker_bin_test.rs

@@ -351,7 +351,6 @@ fn test_crash_tracking_app(crash_type: &str) {
                 .arg(&fixtures.output_dir)
                 .arg(crash_type);
         },
-        false, // expect crash (not success)
         validator,
     )
     .unwrap();
@@ -399,7 +398,6 @@ fn test_crash_tracking_bin_panic_hook_after_fork() {
                 .arg("panic_hook_after_fork") // mode
                 .arg("donothing"); // crash method (not used in this mode)
         },
-        false, // expect crash (not success)
         validator,
     )
     .unwrap();
@@ -445,9 +443,9 @@ fn test_crash_tracking_callstack() {
         |cmd, fixtures| {
             cmd.arg(format!("file://{}", fixtures.crash_profile_path.display()))
                 .arg(&artifacts_map[&crashtracker_receiver])
-                .arg(&fixtures.output_dir);
+                .arg(&fixtures.output_dir)
+                .arg("segfault");
         },
-        false, // expect crash (not success)
         |payload, _fixtures| {
             // Use the new callstack validator
             PayloadValidator::new(payload).validate_callstack_functions(&expected_functions)?;