Skip to content

Commit 5608ad0

Browse files
committed
Move the assert on new X in the test coverage up before the percentage
to try get the list of URLs out before the percentage - Legacy-Id: 13144
1 parent 5b8ac15 commit 5608ad0

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

ietf/utils/test_runner.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,11 +257,11 @@ def report_test_result(self, test):
257257
if self.runner.run_full_test_suite:
258258
# Permit 0.02% variation in results -- otherwise small code changes become a pain
259259
fudge_factor = 0.00005 # 0.005% -- a small change, less than the last digit we show
260+
self.assertLessEqual(len(test_missing), len(master_missing),
261+
msg = "New %s without test coverage since %s: %s" % (test, latest_coverage_version, list(set(test_missing) - set(master_missing))))
260262
self.assertGreaterEqual(test_coverage, master_coverage - fudge_factor,
261263
msg = "The %s coverage percentage is now lower (%.2f%%) than for version %s (%.2f%%)" %
262264
( test, test_coverage*100, latest_coverage_version, master_coverage*100, ))
263-
self.assertLessEqual(len(test_missing), len(master_missing),
264-
msg = "New %s without test coverage since %s: %s" % (test, latest_coverage_version, list(set(test_missing) - set(master_missing))))
265265

266266
def template_coverage_test(self):
267267
global loaded_templates

0 commit comments

Comments
 (0)