2019-09-16 19:18:40 +02:00
|
|
|
#!/usr/bin/env python3
|
|
|
|
|
|
|
|
"""Sanity checks for test data.
|
2020-07-03 00:30:12 +02:00
|
|
|
|
|
|
|
This program contains a class for traversing test cases that can be used
|
|
|
|
independently of the checks.
|
2019-09-16 19:18:40 +02:00
|
|
|
"""
|
|
|
|
|
2020-08-07 13:07:28 +02:00
|
|
|
# Copyright The Mbed TLS Contributors
|
2023-11-02 20:47:20 +01:00
|
|
|
# SPDX-License-Identifier: Apache-2.0 OR GPL-2.0-or-later
|
2019-09-16 19:18:40 +02:00
|
|
|
|
2019-12-02 14:26:04 +01:00
|
|
|
import argparse
|
2019-09-16 19:18:40 +02:00
|
|
|
import glob
|
|
|
|
import os
|
|
|
|
import re
|
2023-02-24 07:53:29 +01:00
|
|
|
import subprocess
|
2019-09-16 19:18:40 +02:00
|
|
|
import sys
|
2023-08-17 16:11:10 +02:00
|
|
|
|
2019-09-16 19:18:40 +02:00
|
|
|
|
|
|
|
class Results:
|
2019-12-17 16:03:59 +01:00
|
|
|
"""Store file and line information about errors or warnings in test suites."""
|
2019-12-02 14:26:04 +01:00
|
|
|
|
|
|
|
def __init__(self, options):
|
2019-09-16 19:18:40 +02:00
|
|
|
self.errors = 0
|
|
|
|
self.warnings = 0
|
2019-12-02 14:26:04 +01:00
|
|
|
self.ignore_warnings = options.quiet
|
2019-09-16 19:18:40 +02:00
|
|
|
|
|
|
|
def error(self, file_name, line_number, fmt, *args):
|
|
|
|
sys.stderr.write(('{}:{}:ERROR:' + fmt + '\n').
|
|
|
|
format(file_name, line_number, *args))
|
|
|
|
self.errors += 1
|
|
|
|
|
|
|
|
def warning(self, file_name, line_number, fmt, *args):
|
2019-12-02 14:26:04 +01:00
|
|
|
if not self.ignore_warnings:
|
|
|
|
sys.stderr.write(('{}:{}:Warning:' + fmt + '\n')
|
|
|
|
.format(file_name, line_number, *args))
|
|
|
|
self.warnings += 1
|
2019-09-16 19:18:40 +02:00
|
|
|
|
2020-06-25 16:34:11 +02:00
|
|
|
class TestDescriptionExplorer:
|
|
|
|
"""An iterator over test cases with descriptions.
|
2020-06-25 16:16:25 +02:00
|
|
|
|
2020-06-25 16:34:11 +02:00
|
|
|
The test cases that have descriptions are:
|
|
|
|
* Individual unit tests (entries in a .data file) in test suites.
|
|
|
|
* Individual test cases in ssl-opt.sh.
|
|
|
|
|
|
|
|
This is an abstract class. To use it, derive a class that implements
|
|
|
|
the process_test_case method, and call walk_all().
|
2020-06-25 16:16:25 +02:00
|
|
|
"""
|
2020-06-25 16:34:11 +02:00
|
|
|
|
|
|
|
def process_test_case(self, per_file_state,
|
|
|
|
file_name, line_number, description):
|
|
|
|
"""Process a test case.
|
|
|
|
|
2020-07-03 00:30:12 +02:00
|
|
|
per_file_state: an object created by new_per_file_state() at the beginning
|
|
|
|
of each file.
|
2020-06-25 16:34:11 +02:00
|
|
|
file_name: a relative path to the file containing the test case.
|
|
|
|
line_number: the line number in the given file.
|
|
|
|
description: the test case description as a byte string.
|
2020-06-25 16:16:25 +02:00
|
|
|
"""
|
2020-06-25 16:34:11 +02:00
|
|
|
raise NotImplementedError
|
|
|
|
|
2020-07-03 00:30:12 +02:00
|
|
|
def new_per_file_state(self):
|
2020-06-25 16:34:11 +02:00
|
|
|
"""Return a new per-file state object.
|
|
|
|
|
|
|
|
The default per-file state object is None. Child classes that require per-file
|
|
|
|
state may override this method.
|
2020-06-25 16:16:25 +02:00
|
|
|
"""
|
2020-06-25 16:34:11 +02:00
|
|
|
#pylint: disable=no-self-use
|
|
|
|
return None
|
|
|
|
|
|
|
|
def walk_test_suite(self, data_file_name):
|
|
|
|
"""Iterate over the test cases in the given unit test data file."""
|
|
|
|
in_paragraph = False
|
2020-07-03 00:30:12 +02:00
|
|
|
descriptions = self.new_per_file_state() # pylint: disable=assignment-from-none
|
2020-06-25 16:34:11 +02:00
|
|
|
with open(data_file_name, 'rb') as data_file:
|
|
|
|
for line_number, line in enumerate(data_file, 1):
|
|
|
|
line = line.rstrip(b'\r\n')
|
|
|
|
if not line:
|
|
|
|
in_paragraph = False
|
|
|
|
continue
|
|
|
|
if line.startswith(b'#'):
|
|
|
|
continue
|
|
|
|
if not in_paragraph:
|
|
|
|
# This is a test case description line.
|
|
|
|
self.process_test_case(descriptions,
|
|
|
|
data_file_name, line_number, line)
|
|
|
|
in_paragraph = True
|
|
|
|
|
2023-09-01 18:41:16 +02:00
|
|
|
def collect_from_script(self, file_name):
|
|
|
|
"""Collect the test cases in a script by calling its listing test cases
|
|
|
|
option"""
|
2020-07-03 00:30:12 +02:00
|
|
|
descriptions = self.new_per_file_state() # pylint: disable=assignment-from-none
|
2023-09-01 18:41:16 +02:00
|
|
|
listed = subprocess.check_output(['sh', file_name, '--list-test-cases'])
|
|
|
|
# Assume test file is responsible for printing identical format of
|
|
|
|
# test case description between --list-test-cases and its OUTCOME.CSV
|
2023-09-04 11:23:04 +02:00
|
|
|
#
|
2023-08-30 12:31:35 +02:00
|
|
|
# idx indicates the number of test case since there is no line number
|
2023-10-27 12:45:26 +02:00
|
|
|
# in the script for each test case.
|
2023-09-04 11:23:04 +02:00
|
|
|
for idx, description in enumerate(listed.splitlines()):
|
|
|
|
self.process_test_case(descriptions,
|
|
|
|
file_name,
|
|
|
|
idx,
|
|
|
|
description.rstrip())
|
2023-02-24 07:53:29 +01:00
|
|
|
|
2020-06-25 16:40:10 +02:00
|
|
|
@staticmethod
|
|
|
|
def collect_test_directories():
|
|
|
|
"""Get the relative path for the TLS and Crypto test directories."""
|
|
|
|
if os.path.isdir('tests'):
|
|
|
|
tests_dir = 'tests'
|
|
|
|
elif os.path.isdir('suites'):
|
|
|
|
tests_dir = '.'
|
|
|
|
elif os.path.isdir('../suites'):
|
|
|
|
tests_dir = '..'
|
|
|
|
directories = [tests_dir]
|
|
|
|
return directories
|
|
|
|
|
2020-06-25 16:34:11 +02:00
|
|
|
def walk_all(self):
|
|
|
|
"""Iterate over all named test cases."""
|
2020-06-25 16:40:10 +02:00
|
|
|
test_directories = self.collect_test_directories()
|
2020-06-25 16:34:11 +02:00
|
|
|
for directory in test_directories:
|
|
|
|
for data_file_name in glob.glob(os.path.join(directory, 'suites',
|
|
|
|
'*.data')):
|
|
|
|
self.walk_test_suite(data_file_name)
|
2023-09-01 18:41:16 +02:00
|
|
|
|
|
|
|
for sh_file in ['ssl-opt.sh', 'compat.sh']:
|
|
|
|
sh_file = os.path.join(directory, sh_file)
|
|
|
|
if os.path.exists(sh_file):
|
|
|
|
self.collect_from_script(sh_file)
|
2020-06-25 16:34:11 +02:00
|
|
|
|
2022-01-07 15:58:38 +01:00
|
|
|
class TestDescriptions(TestDescriptionExplorer):
|
|
|
|
"""Collect the available test cases."""
|
|
|
|
|
|
|
|
def __init__(self):
|
|
|
|
super().__init__()
|
|
|
|
self.descriptions = set()
|
|
|
|
|
|
|
|
def process_test_case(self, _per_file_state,
|
|
|
|
file_name, _line_number, description):
|
|
|
|
"""Record an available test case."""
|
|
|
|
base_name = re.sub(r'\.[^.]*$', '', re.sub(r'.*/', '', file_name))
|
|
|
|
key = ';'.join([base_name, description.decode('utf-8')])
|
|
|
|
self.descriptions.add(key)
|
|
|
|
|
|
|
|
def collect_available_test_cases():
|
|
|
|
"""Collect the available test cases."""
|
|
|
|
explorer = TestDescriptions()
|
|
|
|
explorer.walk_all()
|
|
|
|
return sorted(explorer.descriptions)
|
|
|
|
|
2020-06-25 16:34:11 +02:00
|
|
|
class DescriptionChecker(TestDescriptionExplorer):
|
|
|
|
"""Check all test case descriptions.
|
|
|
|
|
|
|
|
* Check that each description is valid (length, allowed character set, etc.).
|
|
|
|
* Check that there is no duplicated description inside of one test suite.
|
|
|
|
"""
|
|
|
|
|
|
|
|
def __init__(self, results):
|
|
|
|
self.results = results
|
|
|
|
|
2020-07-03 00:30:12 +02:00
|
|
|
def new_per_file_state(self):
|
2020-06-25 16:40:10 +02:00
|
|
|
"""Dictionary mapping descriptions to their line number."""
|
2020-06-25 16:34:11 +02:00
|
|
|
return {}
|
|
|
|
|
|
|
|
def process_test_case(self, per_file_state,
|
|
|
|
file_name, line_number, description):
|
2020-06-25 16:40:10 +02:00
|
|
|
"""Check test case descriptions for errors."""
|
|
|
|
results = self.results
|
|
|
|
seen = per_file_state
|
|
|
|
if description in seen:
|
|
|
|
results.error(file_name, line_number,
|
|
|
|
'Duplicate description (also line {})',
|
|
|
|
seen[description])
|
|
|
|
return
|
|
|
|
if re.search(br'[\t;]', description):
|
|
|
|
results.error(file_name, line_number,
|
|
|
|
'Forbidden character \'{}\' in description',
|
|
|
|
re.search(br'[\t;]', description).group(0).decode('ascii'))
|
|
|
|
if re.search(br'[^ -~]', description):
|
|
|
|
results.error(file_name, line_number,
|
|
|
|
'Non-ASCII character in description')
|
|
|
|
if len(description) > 66:
|
|
|
|
results.warning(file_name, line_number,
|
|
|
|
'Test description too long ({} > 66)',
|
|
|
|
len(description))
|
|
|
|
seen[description] = line_number
|
2019-09-16 19:18:40 +02:00
|
|
|
|
|
|
|
def main():
|
2019-12-02 14:26:04 +01:00
|
|
|
parser = argparse.ArgumentParser(description=__doc__)
|
2022-01-07 15:58:55 +01:00
|
|
|
parser.add_argument('--list-all',
|
|
|
|
action='store_true',
|
|
|
|
help='List all test cases, without doing checks')
|
2019-12-02 14:26:04 +01:00
|
|
|
parser.add_argument('--quiet', '-q',
|
|
|
|
action='store_true',
|
|
|
|
help='Hide warnings')
|
|
|
|
parser.add_argument('--verbose', '-v',
|
|
|
|
action='store_false', dest='quiet',
|
|
|
|
help='Show warnings (default: on; undoes --quiet)')
|
|
|
|
options = parser.parse_args()
|
2022-01-07 15:58:55 +01:00
|
|
|
if options.list_all:
|
|
|
|
descriptions = collect_available_test_cases()
|
|
|
|
sys.stdout.write('\n'.join(descriptions + ['']))
|
|
|
|
return
|
2019-12-02 14:26:04 +01:00
|
|
|
results = Results(options)
|
2020-06-25 16:34:11 +02:00
|
|
|
checker = DescriptionChecker(results)
|
|
|
|
checker.walk_all()
|
2019-12-02 14:26:04 +01:00
|
|
|
if (results.warnings or results.errors) and not options.quiet:
|
2019-09-16 19:18:40 +02:00
|
|
|
sys.stderr.write('{}: {} errors, {} warnings\n'
|
|
|
|
.format(sys.argv[0], results.errors, results.warnings))
|
|
|
|
sys.exit(1 if results.errors else 0)
|
|
|
|
|
|
|
|
if __name__ == '__main__':
|
|
|
|
main()
|