From eb35a5ac1e7b6abe81947e22417f34508f89f091 Mon Sep 17 00:00:00 2001 From: Andrew Fitzgerald Date: Thu, 16 Nov 2023 17:46:14 -0800 Subject: [PATCH] Clean up: simplify asserts (#34111) --- runtime/src/bank/tests.rs | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/runtime/src/bank/tests.rs b/runtime/src/bank/tests.rs index 11cc567e8..23278f0da 100644 --- a/runtime/src/bank/tests.rs +++ b/runtime/src/bank/tests.rs @@ -9707,18 +9707,16 @@ fn test_verify_and_hash_transaction_sig_len() { { let tx = make_transaction(TestCase::RemoveSignature); assert_eq!( - bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification) - .err(), - Some(TransactionError::SanitizeFailure), + bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification), + Err(TransactionError::SanitizeFailure), ); } // Too many signatures: Sanitization failure { let tx = make_transaction(TestCase::AddSignature); assert_eq!( - bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification) - .err(), - Some(TransactionError::SanitizeFailure), + bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification), + Err(TransactionError::SanitizeFailure), ); } } @@ -9754,9 +9752,8 @@ fn test_verify_transactions_packet_data_size() { let tx = make_transaction(25); assert!(bincode::serialized_size(&tx).unwrap() > PACKET_DATA_SIZE as u64); assert_eq!( - bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification) - .err(), - Some(TransactionError::SanitizeFailure), + bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification), + Err(TransactionError::SanitizeFailure), ); } // Assert that verify fails as soon as serialized