From c9bf9ce094f1831546afedc8de199b54b3f58510 Mon Sep 17 00:00:00 2001 From: Rob Walker Date: Mon, 28 Jan 2019 15:21:08 -0800 Subject: [PATCH] eliminate re-use of a TX here, we're testing for empty account balance (#2576) --- src/bank.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/bank.rs b/src/bank.rs index f4fe70233f..3fc81c15a5 100644 --- a/src/bank.rs +++ b/src/bank.rs @@ -1510,15 +1510,17 @@ mod tests { let tx = Transaction::system_new(&keypair1, keypair4.pubkey(), 1, tick.id); let entry_2 = next_entry(&tick.id, 1, vec![tx]); assert_eq!( - bank.par_process_entries(&[entry_1.clone(), tick.clone(), entry_2]), + bank.par_process_entries(&[entry_1.clone(), tick.clone(), entry_2.clone()]), Ok(()) ); assert_eq!(bank.get_balance(&keypair3.pubkey()), 1); assert_eq!(bank.get_balance(&keypair4.pubkey()), 1); assert_eq!(bank.last_id(), tick.id); - // ensure that errors are returned + // ensure that an error is returned for an empty account (keypair2) + let tx = Transaction::system_new(&keypair2, keypair3.pubkey(), 1, tick.id); + let entry_3 = next_entry(&entry_2.id, 1, vec![tx]); assert_eq!( - bank.par_process_entries(&[entry_1]), + bank.par_process_entries(&[entry_3]), Err(BankError::AccountNotFound) ); }