From 5015da37abda11841c38df8befe46892d38275f7 Mon Sep 17 00:00:00 2001 From: jbesraa Date: Wed, 22 Jan 2025 11:41:05 +0200 Subject: [PATCH] While the panic trace can be annoying.. ..if the error originates from unhandled message(s), it is quite useful if it is a different kind of error. Initially I thought removing the stack trace would be cleaner and would result in better logs, but in practice any error that occurs in testing is currently not printed, unless it is an unhandled message. This commit stops this behavior and will always print the stack trace in any case of failure in testing. --- roles/tests-integration/lib/sniffer.rs | 4 ---- 1 file changed, 4 deletions(-) diff --git a/roles/tests-integration/lib/sniffer.rs b/roles/tests-integration/lib/sniffer.rs index 6a6c634cb..6b680a4f5 100644 --- a/roles/tests-integration/lib/sniffer.rs +++ b/roles/tests-integration/lib/sniffer.rs @@ -105,10 +105,6 @@ impl Sniffer { check_on_drop: bool, intercept_messages: Option>, ) -> Self { - // Don't print backtrace on panic - std::panic::set_hook(Box::new(|_| { - println!(); - })); Self { identifier, listening_address,