From b7caaf954b04a6c88795b953826b4e4c50bb3243 Mon Sep 17 00:00:00 2001 From: Jack Grigg Date: Mon, 19 Mar 2018 14:00:18 +0100 Subject: [PATCH] test: Move wait_and_assert_operationid_status debug output before asserts Otherwise we don't see them for failed tests :) --- qa/rpc-tests/test_framework/util.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/qa/rpc-tests/test_framework/util.py b/qa/rpc-tests/test_framework/util.py index f3ea481ee..ea6e8c832 100644 --- a/qa/rpc-tests/test_framework/util.py +++ b/qa/rpc-tests/test_framework/util.py @@ -393,12 +393,12 @@ def wait_and_assert_operationid_status(node, myopid, in_status='success', in_err elif status == "success": txid = results[0]['result']['txid'] break - assert_equal(in_status, status) - if errormsg is not None: - assert(in_errormsg is not None) - assert_equal(in_errormsg in errormsg, True) if os.getenv("PYTHON_DEBUG", ""): print('...returned status: {}'.format(status)) if errormsg is not None: print('...returned error: {}'.format(errormsg)) + assert_equal(in_status, status) + if errormsg is not None: + assert(in_errormsg is not None) + assert_equal(in_errormsg in errormsg, True) return txid