sikkienl/dash
1
0
Fork 0
mirror of https://github.com/dashpay/dash.git synced 2024-12-26 12:32:48 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

[qa] Attach node index to test_node AssertionError and print messages

Browse Source 
to aid debugging. Feedback incorporated from @Empact.
This commit is contained in:
James O'Beirne 2018-04-18 15:17:22 -04:00
parent 615f7c2884
commit 80a5e59532
1 changed files with 24 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
test/functional/test_framework/test_node.py
View File

@ -19,7 +19,6 @@ import time
from .authproxy import JSONRPCException from .authproxy import JSONRPCException
from .util import ( from .util import (
append_config, append_config,
assert_equal,
delete_cookie_file, delete_cookie_file,
get_rpc_proxy, get_rpc_proxy,
rpc_url, rpc_url,
@ -103,6 +102,14 @@ class TestNode():
self.p2ps = [] self.p2ps = []
def _node_msg(self, msg: str) -> str:
"""Return a modified msg that identifies this node by its index as a debugging aid."""
return "[node %d] %s" % (self.index, msg)
def _raise_assertion_error(self, msg: str):
"""Raise an AssertionError with msg modified to identify this node."""
raise AssertionError(self._node_msg(msg))
def __del__(self): def __del__(self):
# Ensure that we don't leave any bitcoind processes lying around after # Ensure that we don't leave any bitcoind processes lying around after
# the test ends # the test ends
@ -110,7 +117,7 @@ class TestNode():
# Should only happen on test failure # Should only happen on test failure
# Avoid using logger, as that may have already been shutdown when # Avoid using logger, as that may have already been shutdown when
# this destructor is called. # this destructor is called.
print("Cleaning up leftover process") print(self._node_msg("Cleaning up leftover process"))
self.process.kill() self.process.kill()
def __getattr__(self, name): def __getattr__(self, name):
@ -118,7 +125,7 @@ class TestNode():
if self.use_cli: if self.use_cli:
return getattr(self.cli, name) return getattr(self.cli, name)
else: else:
assert self.rpc_connected and self.rpc is not None, "Error: no RPC connection" assert self.rpc_connected and self.rpc is not None, self._node_msg("Error: no RPC connection")
return getattr(self.rpc, name) return getattr(self.rpc, name)
def start(self, extra_args=None, stderr=None, *args, **kwargs): def start(self, extra_args=None, stderr=None, *args, **kwargs):
@ -141,7 +148,8 @@ class TestNode():
poll_per_s = 4 poll_per_s = 4
for _ in range(poll_per_s * self.rpc_timeout): for _ in range(poll_per_s * self.rpc_timeout):
if self.process.poll() is not None: if self.process.poll() is not None:
raise FailedToStartError('bitcoind exited with status {} during initialization'.format(self.process.returncode)) raise FailedToStartError(self._node_msg(
'bitcoind exited with status {} during initialization'.format(self.process.returncode)))
try: try:
self.rpc = get_rpc_proxy(rpc_url(self.datadir, self.index, self.rpchost), self.index, timeout=self.rpc_timeout, coveragedir=self.coverage_dir) self.rpc = get_rpc_proxy(rpc_url(self.datadir, self.index, self.rpchost), self.index, timeout=self.rpc_timeout, coveragedir=self.coverage_dir)
self.rpc.getblockcount() self.rpc.getblockcount()
@ -160,14 +168,13 @@ class TestNode():
if "No RPC credentials" not in str(e): if "No RPC credentials" not in str(e):
raise raise
time.sleep(1.0 / poll_per_s) time.sleep(1.0 / poll_per_s)
raise AssertionError("Unable to connect to bitcoind") self._raise_assertion_error("Unable to connect to bitcoind")
def get_wallet_rpc(self, wallet_name): def get_wallet_rpc(self, wallet_name):
if self.use_cli: if self.use_cli:
return self.cli("-rpcwallet={}".format(wallet_name)) return self.cli("-rpcwallet={}".format(wallet_name))
else: else:
assert self.rpc_connected assert self.rpc_connected and self.rpc, self._node_msg("RPC not connected")
assert self.rpc
wallet_path = "wallet/%s" % wallet_name wallet_path = "wallet/%s" % wallet_name
return self.rpc / wallet_path return self.rpc / wallet_path
@ -194,7 +201,8 @@ class TestNode():
return False return False
# process has stopped. Assert that it didn't return an error code. # process has stopped. Assert that it didn't return an error code.
assert_equal(return_code, 0) assert return_code == 0, self._node_msg(
"Node returned non-zero exit code (%d) when stopping" % return_code)
self.running = False self.running = False
self.process = None self.process = None
self.rpc_connected = False self.rpc_connected = False
@ -229,19 +237,22 @@ class TestNode():
stderr = log_stderr.read().decode('utf-8').strip() stderr = log_stderr.read().decode('utf-8').strip()
if match == ErrorMatch.PARTIAL_REGEX: if match == ErrorMatch.PARTIAL_REGEX:
if re.search(expected_msg, stderr, flags=re.MULTILINE) is None: if re.search(expected_msg, stderr, flags=re.MULTILINE) is None:
raise AssertionError('Expected message "{}" does not partially match stderr:\n"{}"'.format(expected_msg, stderr)) self._raise_assertion_error(
'Expected message "{}" does not partially match stderr:\n"{}"'.format(expected_msg, stderr))
elif match == ErrorMatch.FULL_REGEX: elif match == ErrorMatch.FULL_REGEX:
if re.fullmatch(expected_msg, stderr) is None: if re.fullmatch(expected_msg, stderr) is None:
raise AssertionError('Expected message "{}" does not fully match stderr:\n"{}"'.format(expected_msg, stderr)) self._raise_assertion_error(
'Expected message "{}" does not fully match stderr:\n"{}"'.format(expected_msg, stderr))
elif match == ErrorMatch.FULL_TEXT: elif match == ErrorMatch.FULL_TEXT:
if expected_msg != stderr: if expected_msg != stderr:
raise AssertionError('Expected message "{}" does not fully match stderr:\n"{}"'.format(expected_msg, stderr)) self._raise_assertion_error(
'Expected message "{}" does not fully match stderr:\n"{}"'.format(expected_msg, stderr))
else: else:
if expected_msg is None: if expected_msg is None:
assert_msg = "bitcoind should have exited with an error" assert_msg = "bitcoind should have exited with an error"
else: else:
assert_msg = "bitcoind should have exited with expected error " + expected_msg assert_msg = "bitcoind should have exited with expected error " + expected_msg
raise AssertionError(assert_msg) self._raise_assertion_error(assert_msg)
def node_encrypt_wallet(self, passphrase): def node_encrypt_wallet(self, passphrase):
""""Encrypts the wallet. """"Encrypts the wallet.
@ -272,7 +283,7 @@ class TestNode():
Convenience property - most tests only use a single p2p connection to each Convenience property - most tests only use a single p2p connection to each
node, so this saves having to write node.p2ps[0] many times.""" node, so this saves having to write node.p2ps[0] many times."""
assert self.p2ps, "No p2p connection" assert self.p2ps, self._node_msg("No p2p connection")
return self.p2ps[0] return self.p2ps[0]
def disconnect_p2ps(self): def disconnect_p2ps(self):