Skip to content

Commit

Permalink
While the panic trace can be annoying..
Browse files Browse the repository at this point in the history
..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.
  • Loading branch information
jbesraa committed Jan 22, 2025
1 parent 0f6d89b commit 5015da3
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions roles/tests-integration/lib/sniffer.rs
Original file line number Diff line number Diff line change
Expand Up @@ -105,10 +105,6 @@ impl Sniffer {
check_on_drop: bool,
intercept_messages: Option<Vec<InterceptMessage>>,
) -> Self {
// Don't print backtrace on panic
std::panic::set_hook(Box::new(|_| {
println!();
}));
Self {
identifier,
listening_address,
Expand Down

0 comments on commit 5015da3

Please sign in to comment.