blockworks-foundation
/
solana
mirror of https://github.com/blockworks-foundation/solana.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Remove assert!()s that hide error codes, making failure debug harder

This commit is contained in:
Michael Vines 2019-01-21 10:04:46 -08:00
parent b0149a54d8
commit c7e5006bcf
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
wallet/tests/pay.rs
View File

@ -99,7 +99,6 @@ fn test_wallet_timestamp_tx() {
None, None,
); );
let sig_response = process_command(&config_payer); let sig_response = process_command(&config_payer);
assert!(sig_response.is_ok());
let object: Value = serde_json::from_str(&sig_response.unwrap()).unwrap(); let object: Value = serde_json::from_str(&sig_response.unwrap()).unwrap();
let process_id_str = object.get("processId").unwrap().as_str().unwrap(); let process_id_str = object.get("processId").unwrap().as_str().unwrap();
@ -117,8 +116,7 @@ fn test_wallet_timestamp_tx() {
// Sign transaction by config_witness // Sign transaction by config_witness
config_witness.command = WalletCommand::TimeElapsed(bob_pubkey, process_id, dt); config_witness.command = WalletCommand::TimeElapsed(bob_pubkey, process_id, dt);
let sig_response = process_command(&config_witness); process_command(&config_witness).unwrap();
assert!(sig_response.is_ok());
let params = json!([format!("{}", config_payer.id.pubkey())]); let params = json!([format!("{}", config_payer.id.pubkey())]);
check_balance(39, &rpc_client, params); // config_payer balance check_balance(39, &rpc_client, params); // config_payer balance
@ -195,7 +193,6 @@ fn test_wallet_witness_tx() {
None, None,
); );
let sig_response = process_command(&config_payer); let sig_response = process_command(&config_payer);
assert!(sig_response.is_ok());
let object: Value = serde_json::from_str(&sig_response.unwrap()).unwrap(); let object: Value = serde_json::from_str(&sig_response.unwrap()).unwrap();
let process_id_str = object.get("processId").unwrap().as_str().unwrap(); let process_id_str = object.get("processId").unwrap().as_str().unwrap();
@ -213,8 +210,7 @@ fn test_wallet_witness_tx() {
// Sign transaction by config_witness // Sign transaction by config_witness
config_witness.command = WalletCommand::Witness(bob_pubkey, process_id); config_witness.command = WalletCommand::Witness(bob_pubkey, process_id);
let sig_response = process_command(&config_witness); process_command(&config_witness).unwrap();
assert!(sig_response.is_ok());
let params = json!([format!("{}", config_payer.id.pubkey())]); let params = json!([format!("{}", config_payer.id.pubkey())]);
check_balance(39, &rpc_client, params); // config_payer balance check_balance(39, &rpc_client, params); // config_payer balance
@ -291,7 +287,6 @@ fn test_wallet_cancel_tx() {
Some(config_payer.id.pubkey()), Some(config_payer.id.pubkey()),
); );
let sig_response = process_command(&config_payer); let sig_response = process_command(&config_payer);
assert!(sig_response.is_ok());
let object: Value = serde_json::from_str(&sig_response.unwrap()).unwrap(); let object: Value = serde_json::from_str(&sig_response.unwrap()).unwrap();
let process_id_str = object.get("processId").unwrap().as_str().unwrap(); let process_id_str = object.get("processId").unwrap().as_str().unwrap();
@ -309,8 +304,7 @@ fn test_wallet_cancel_tx() {
// Sign transaction by config_witness // Sign transaction by config_witness
config_payer.command = WalletCommand::Cancel(process_id); config_payer.command = WalletCommand::Cancel(process_id);
let sig_response = process_command(&config_payer); process_command(&config_payer).unwrap();
assert!(sig_response.is_ok());
let params = json!([format!("{}", config_payer.id.pubkey())]); let params = json!([format!("{}", config_payer.id.pubkey())]);
check_balance(49, &rpc_client, params); // config_payer balance check_balance(49, &rpc_client, params); // config_payer balance