Skip to content

Commit

Permalink
Uses Display for error in VerifyAccountsHashInBackground's panic mess…
Browse files Browse the repository at this point in the history
…age (#34219)
  • Loading branch information
brooksprumo authored Nov 27, 2023
1 parent 57ec207 commit deee5f8
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions accounts-db/src/verify_accounts_hash_in_background.rs
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ impl VerifyAccountsHashInBackground {
}
let result = lock.take().unwrap().join().unwrap();
if !result {
panic!("initial hash verification failed: {result:?}");
panic!("initial background accounts hash verification failed: {result}");
}
// we never have to check again
self.verification_complete();
Expand Down Expand Up @@ -139,7 +139,7 @@ pub mod tests {
}

#[test]
#[should_panic(expected = "initial hash verification failed")]
#[should_panic(expected = "initial background accounts hash verification failed")]
fn test_panic() {
let verify = Arc::new(VerifyAccountsHashInBackground::default());
start_thread_and_return(&verify, false, || {});
Expand Down

0 comments on commit deee5f8

Please sign in to comment.