fix: false positive wallet tests (#2539)

mimblewimble · Feb 8, 2019 · 49ca8eb · 49ca8eb
1 parent a63cfa7
commit 49ca8eb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/wallet/tests/restore.rs b/wallet/tests/restore.rs
@@ -366,10 +366,10 @@ fn perform_restore(test_dir: &str) -> Result<(), libwallet::Error> {
 fn wallet_restore() {
 	let test_dir = "test_output/wallet_restore";
 	if let Err(e) = setup_restore(test_dir) {
-		println!("Set up restore: Libwallet Error: {}", e);
+		panic!("Libwallet Error: {} - {}", e, e.backtrace().unwrap());
 	}
 	if let Err(e) = perform_restore(test_dir) {
-		println!("Perform restore: Libwallet Error: {}", e);
+		panic!("Libwallet Error: {} - {}", e, e.backtrace().unwrap());
 	}
 	// let logging finish
 	thread::sleep(Duration::from_millis(200));

diff --git a/wallet/tests/transaction.rs b/wallet/tests/transaction.rs
@@ -240,8 +240,8 @@ fn basic_transaction_api(test_dir: &str) -> Result<(), libwallet::Error> {
 			None,
 		)?;
 		slate = client1.send_tx_slate_direct("wallet2", &slate_i)?;
-		sender_api.finalize_tx(&mut slate)?;
 		sender_api.tx_lock_outputs(&slate, lock_fn)?;
+		sender_api.finalize_tx(&mut slate)?;
 		Ok(())
 	})?;
 
@@ -337,8 +337,8 @@ fn tx_rollback(test_dir: &str) -> Result<(), libwallet::Error> {
 			None,
 		)?;
 		slate = client1.send_tx_slate_direct("wallet2", &slate_i)?;
-		sender_api.finalize_tx(&mut slate)?;
 		sender_api.tx_lock_outputs(&slate, lock_fn)?;
+		sender_api.finalize_tx(&mut slate)?;
 		Ok(())
 	})?;
 
@@ -458,14 +458,14 @@ fn tx_rollback(test_dir: &str) -> Result<(), libwallet::Error> {
 fn db_wallet_basic_transaction_api() {
 	let test_dir = "test_output/basic_transaction_api";
 	if let Err(e) = basic_transaction_api(test_dir) {
-		println!("Libwallet Error: {}", e);
+		panic!("Libwallet Error: {} - {}", e, e.backtrace().unwrap());
 	}
 }
 
 #[test]
 fn db_wallet_tx_rollback() {
 	let test_dir = "test_output/tx_rollback";
 	if let Err(e) = tx_rollback(test_dir) {
-		println!("Libwallet Error: {}", e);
+		panic!("Libwallet Error: {} - {}", e, e.backtrace().unwrap());
 	}
 }