mirror/mbedtls
1
0
Fork 0
mirror of https://github.com/Mbed-TLS/mbedtls.git synced 2025-03-25 13:43:31 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Don't use the "allow list" terminology any longer

Browse Source 
What was formerly called an allow list is now an ignore table.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
This commit is contained in:
Gilles Peskine 2024-09-26 19:54:38 +02:00
parent 54cfe77951
commit 0930b331c0
1 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/scripts/analyze_outcomes.py
View File

@ -164,6 +164,9 @@ class Task:
class CoverageTask(Task): class CoverageTask(Task):
"""Analyze test coverage.""" """Analyze test coverage."""
# Test cases whose suite and description are matched by an entry in
# IGNORED_TESTS are expected to be never executed.
# All other test cases are expected to be executed at least once.
IGNORED_TESTS = { IGNORED_TESTS = {
'test_suite_psa_crypto_metadata': [ 'test_suite_psa_crypto_metadata': [
# Algorithm not supported yet # Algorithm not supported yet
@ -208,11 +211,14 @@ class CoverageTask(Task):
else: else:
results.warning('Test case not executed: {}', suite_case) results.warning('Test case not executed: {}', suite_case)
elif hit and ignored: elif hit and ignored:
# Test Case should be removed from the allow list. # If a test case is no longer always skipped, we should remove
# it from the ignore list.
if self.full_coverage: if self.full_coverage:
results.error('Allow listed test case was executed: {}', suite_case) results.error('Test case was executed but marked as ignored for coverage: {}',
suite_case)
else: else:
results.warning('Allow listed test case was executed: {}', suite_case) results.warning('Test case was executed but marked as ignored for coverage: {}',
suite_case)
class DriverVSReference(Task): class DriverVSReference(Task):