[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v6 10/11] iotests: rewrite check into python
From: |
Kevin Wolf |
Subject: |
Re: [PATCH v6 10/11] iotests: rewrite check into python |
Date: |
Tue, 12 Jan 2021 18:41:54 +0100 |
Am 09.01.2021 um 13:26 hat Vladimir Sementsov-Ogievskiy geschrieben:
> Just use classes introduced in previous three commits. Behavior
> difference is described in these three commits.
>
> Drop group file, as it becomes unused.
>
> Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
> ---
> tests/qemu-iotests/check | 994 ++-------------------------------------
> tests/qemu-iotests/group | 321 -------------
> 2 files changed, 28 insertions(+), 1287 deletions(-)
> delete mode 100644 tests/qemu-iotests/group
> +import sys
> +import os
> +from findtests import find_tests, TestFinder
> +from testenv import TestEnv
> +from testrunner import TestRunner
> +
> +if __name__ == '__main__':
> + if len(sys.argv) == 2 and sys.argv[1] in ['-h', '--help']:
> + print('Usage: ./check [options] [testlist]')
> + print()
> + TestFinder.get_argparser().print_help()
> + print()
> + TestEnv.get_argparser().print_help()
> + print()
> + TestRunner.get_argparser().print_help()
> + exit()
+check:34:8: R1722: Consider using sys.exit() (consider-using-sys-exit)
> +
> + env = TestEnv(sys.argv[1:])
> + tests, remaining_argv = find_tests(env.remaining_argv,
> + test_dir=env.source_iotests)
> +
> + with TestRunner(remaining_argv, env) as tr:
> + assert not tr.remaining_argv
> + tr.run_tests([os.path.join(env.source_iotests, t) for t in tests])
The assertion means that giving an unknown option results in an error
message like this:
$ build/check -T -raw
Traceback (most recent call last):
File "/home/kwolf/source/qemu/tests/qemu-iotests/build/check", line 41, in
<module>
assert not tr.remaining_argv
AssertionError
I think this could be a bit friendlier (especially since this doesn't
even tell you which of your options was wrong).
Kevin
- [PATCH v6 00/11] Rework iotests/check, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 01/11] iotests/277: use dot slash for nbd-fault-injector.py running, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 09/11] iotests: add testrunner.py, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 02/11] iotests/303: use dot slash for qcow2.py running, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 10/11] iotests: rewrite check into python, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 03/11] iotests: fix some whitespaces in test output files, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 04/11] iotests: make tests executable, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 05/11] iotests/294: add shebang line, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 11/11] iotests: rename and move 169 and 199 tests, Vladimir Sementsov-Ogievskiy, 2021/01/09
- [PATCH v6 07/11] iotests: add findtests.py, Vladimir Sementsov-Ogievskiy, 2021/01/09