Some improvements

Signed-off-by: Pengyu Lv <pengyu.lv@arm.com>
This commit is contained in:
Pengyu Lv 2023-11-29 18:03:28 +08:00
parent 550cd6f9b2
commit 5dcfd0c613

View File

@ -87,19 +87,16 @@ def analyze_coverage(results: Results, outcomes: Outcomes,
"""Check that all available test cases are executed at least once.""" """Check that all available test cases are executed at least once."""
available = check_test_cases.collect_available_test_cases() available = check_test_cases.collect_available_test_cases()
for suite_case in available: for suite_case in available:
hit = False hit = any(suite_case in comp_outcomes.successes or
for comp_outcomes in outcomes.values(): suite_case in comp_outcomes.failures
if suite_case in comp_outcomes.successes or \ for comp_outcomes in outcomes.values())
suite_case in comp_outcomes.failures:
hit = True
break
if hit == 0 and suite_case not in allow_list: if not hit and suite_case not in allow_list:
if full_coverage: if full_coverage:
results.error('Test case not executed: {}', suite_case) results.error('Test case not executed: {}', suite_case)
else: else:
results.warning('Test case not executed: {}', suite_case) results.warning('Test case not executed: {}', suite_case)
elif hit != 0 and suite_case in allow_list: elif hit and suite_case in allow_list:
# Test Case should be removed from the allow list. # Test Case should be removed from the allow list.
if full_coverage: if full_coverage:
results.error('Allow listed test case was executed: {}', suite_case) results.error('Allow listed test case was executed: {}', suite_case)