From 49ca8eba491da920bca00103601ab51cc9fe5afc Mon Sep 17 00:00:00 2001
From: Ivan Sorokin <ivan@sorokin.io>
Date: Fri, 8 Feb 2019 12:31:36 +0100
Subject: [PATCH] fix: false positive wallet tests (#2539)

---
 wallet/tests/restore.rs     | 4 ++--
 wallet/tests/transaction.rs | 8 ++++----
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/wallet/tests/restore.rs b/wallet/tests/restore.rs
index 1c1b08373..05731bf7e 100644
--- 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
index e4b8b1358..3d0302646 100644
--- 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,7 +458,7 @@ 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());
 	}
 }
 
@@ -466,6 +466,6 @@ fn db_wallet_basic_transaction_api() {
 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());
 	}
 }