Clean up: simplify asserts (#34111)
This commit is contained in:
parent
3d3e7bb453
commit
eb35a5ac1e
|
@ -9707,18 +9707,16 @@ fn test_verify_and_hash_transaction_sig_len() {
|
||||||
{
|
{
|
||||||
let tx = make_transaction(TestCase::RemoveSignature);
|
let tx = make_transaction(TestCase::RemoveSignature);
|
||||||
assert_eq!(
|
assert_eq!(
|
||||||
bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification)
|
bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification),
|
||||||
.err(),
|
Err(TransactionError::SanitizeFailure),
|
||||||
Some(TransactionError::SanitizeFailure),
|
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
// Too many signatures: Sanitization failure
|
// Too many signatures: Sanitization failure
|
||||||
{
|
{
|
||||||
let tx = make_transaction(TestCase::AddSignature);
|
let tx = make_transaction(TestCase::AddSignature);
|
||||||
assert_eq!(
|
assert_eq!(
|
||||||
bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification)
|
bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification),
|
||||||
.err(),
|
Err(TransactionError::SanitizeFailure),
|
||||||
Some(TransactionError::SanitizeFailure),
|
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -9754,9 +9752,8 @@ fn test_verify_transactions_packet_data_size() {
|
||||||
let tx = make_transaction(25);
|
let tx = make_transaction(25);
|
||||||
assert!(bincode::serialized_size(&tx).unwrap() > PACKET_DATA_SIZE as u64);
|
assert!(bincode::serialized_size(&tx).unwrap() > PACKET_DATA_SIZE as u64);
|
||||||
assert_eq!(
|
assert_eq!(
|
||||||
bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification)
|
bank.verify_transaction(tx.into(), TransactionVerificationMode::FullVerification),
|
||||||
.err(),
|
Err(TransactionError::SanitizeFailure),
|
||||||
Some(TransactionError::SanitizeFailure),
|
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
// Assert that verify fails as soon as serialized
|
// Assert that verify fails as soon as serialized
|
||||||
|
|
Loading…
Reference in New Issue