Problem: Some unnecessary TODOs

Solution: Removed 'em
This commit is contained in:
z-bowen 2018-08-29 18:41:43 +02:00
parent ba856c2044
commit 9896888e16
2 changed files with 0 additions and 8 deletions

View File

@ -216,7 +216,6 @@ def run_upsert_validator_show(args, bigchain):
logger.info(response) logger.info(response)
# TODO: Remove this once tests/commands/test_utils.test_configure_bigchaindb_logging.py is fixed
return response return response

View File

@ -155,13 +155,6 @@ def test_upsert_validator_show(caplog, ongoing_election, b):
show_args = Namespace(action='show', show_args = Namespace(action='show',
election_id=election_id) election_id=election_id)
# TODO: Turn this back on once tests/commands/test_utils.test_configure_bigchaindb_logging.py is fixed
# with caplog.at_level(logging.INFO):
# run_upsert_validator_show(show_args, b)
# msg = caplog.records.pop().msg
# assert msg == f'public_key={public_key}\npower={power}\nnode_id={node_id}\nstatus={status}'
# TODO: Remove everything after this once tests/commands/test_utils.test_configure_bigchaindb_logging.py is fixed
msg = run_upsert_validator_show(show_args, b) msg = run_upsert_validator_show(show_args, b)
assert msg == f'public_key={public_key}\npower={power}\nnode_id={node_id}\nstatus={status}' assert msg == f'public_key={public_key}\npower={power}\nnode_id={node_id}\nstatus={status}'