yuzu/mbedtls
1
0
Fork 0
mirror of https://github.com/yuzu-emu/mbedtls.git synced 2024-12-24 05:35:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Consolidate tests for set with/without values

Browse source 
We currently test setting a symbol with a value even if it didn't
originally had one and vice versa. So there's no need to have separate
lists of symbols to test with. Just test everything we want to test
with each symbol.
This commit is contained in:
Gilles Peskine 2019-09-19 12:18:23 +02:00 committed by Darryl Green
parent 61a90bd32d
commit bc86f997ca
1 changed files with 10 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
tests/scripts/test_config_script.py
View file

@ -125,20 +125,17 @@ def run_one(options, args):
with open(status_filename, 'w') as status_file: with open(status_filename, 'w') as status_file:
status_file.write('{}\n'.format(status)) status_file.write('{}\n'.format(status))
### A list of symbols to test with set and unset. ### A list of symbols to test with.
### This script currently tests what happens when you change a symbol from
### having a value to not having a value or vice versa. This is not
### necessarily useful behavior, and we may not consider it a bug if
### config.py stops handling that case correctly.
TEST_SYMBOLS = [ TEST_SYMBOLS = [
'CUSTOM_OPTION', 'CUSTOM_SYMBOL', # does not exist
'MBEDTLS_AES_C', 'MBEDTLS_AES_C', # set, no value
'MBEDTLS_MPI_MAX_SIZE', 'MBEDTLS_MPI_MAX_SIZE', # unset, has a value
'MBEDTLS_NO_UDBL_DIVISION', 'MBEDTLS_NO_UDBL_DIVISION', # unset, in "System support"
'MBEDTLS_PLATFORM_ZEROIZE_ALT', 'MBEDTLS_PLATFORM_ZEROIZE_ALT', # unset, in "Customisation configuration options"
]
### A list of symbols to test with set with a value.
TEST_SYMBOLS_WITH_VALUE = [
'CUSTOM_VALUE',
'MBEDTLS_AES_C',
'MBEDTLS_MPI_MAX_SIZE',
] ]
def run_all(options): def run_all(options):
@ -150,9 +147,6 @@ def run_all(options):
run_one(options, ['get', symbol]) run_one(options, ['get', symbol])
run_one(options, ['set', symbol]) run_one(options, ['set', symbol])
run_one(options, ['--force', 'set', symbol]) run_one(options, ['--force', 'set', symbol])
run_one(options, ['unset', symbol])
for symbol in TEST_SYMBOLS_WITH_VALUE:
run_one(options, ['get', symbol])
run_one(options, ['set', symbol, 'value']) run_one(options, ['set', symbol, 'value'])
run_one(options, ['--force', 'set', symbol, 'value']) run_one(options, ['--force', 'set', symbol, 'value'])
run_one(options, ['unset', symbol]) run_one(options, ['unset', symbol])