LBRYCommunity/lbrycrd
1
0
Fork 0
Code Issues 29 Pull requests 1 Projects Releases 32 Packages Wiki Activity

[tests] Remove EXPECTED_VIOLATION_COUNT

Browse source
This commit is contained in:
Anthony Towns 2018-01-25 09:44:30 +10:00
parent 3150b3fea7
commit 6f881cc880
1 changed files with 4 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
test/functional/test_runner.py
View file

@ -472,9 +472,8 @@ class TestResult():
def check_script_prefixes(): def check_script_prefixes():
"""Check that no more than `EXPECTED_VIOLATION_COUNT` of the """Check that at most a handful of the
test scripts don't start with one of the allowed name prefixes.""" test scripts don't start with one of the allowed name prefixes."""
EXPECTED_VIOLATION_COUNT = 0
# LEEWAY is provided as a transition measure, so that pull-requests # LEEWAY is provided as a transition measure, so that pull-requests
# that introduce new tests that don't conform with the naming # that introduce new tests that don't conform with the naming
@ -484,13 +483,10 @@ def check_script_prefixes():
good_prefixes_re = re.compile("(example|feature|interface|mempool|mining|p2p|rpc|wallet)_") good_prefixes_re = re.compile("(example|feature|interface|mempool|mining|p2p|rpc|wallet)_")
bad_script_names = [script for script in ALL_SCRIPTS if good_prefixes_re.match(script) is None] bad_script_names = [script for script in ALL_SCRIPTS if good_prefixes_re.match(script) is None]
if len(bad_script_names) < EXPECTED_VIOLATION_COUNT: if len(bad_script_names) > 0:
print("{}HURRAY!{} Number of functional tests violating naming convention reduced!".format(BOLD[1], BOLD[0])) print("INFO: %d tests not meeting naming conventions:" % (len(bad_script_names)))
print("Consider reducing EXPECTED_VIOLATION_COUNT from %d to %d" % (EXPECTED_VIOLATION_COUNT, len(bad_script_names)))
elif len(bad_script_names) > EXPECTED_VIOLATION_COUNT:
print("INFO: %d tests not meeting naming conventions (expected %d):" % (len(bad_script_names), EXPECTED_VIOLATION_COUNT))
print(" %s" % ("\n ".join(sorted(bad_script_names)))) print(" %s" % ("\n ".join(sorted(bad_script_names))))
assert len(bad_script_names) <= EXPECTED_VIOLATION_COUNT + LEEWAY, "Too many tests not following naming convention! (%d found, expected: <= %d)" % (len(bad_script_names), EXPECTED_VIOLATION_COUNT) assert len(bad_script_names) <= LEEWAY, "Too many tests not following naming convention! (%d found, maximum: %d)" % (len(bad_script_names), LEEWAY)
def check_script_list(src_dir): def check_script_list(src_dir):