diff options
| author | Ben Gamari <ben@smart-cactus.org> | 2019-06-13 20:21:03 -0400 |
|---|---|---|
| committer | Ben Gamari <ben@smart-cactus.org> | 2019-06-14 12:57:31 -0400 |
| commit | 0336d838882bd07ceca8a2024a3f1f8009021890 (patch) | |
| tree | e9f0bb77b178b8676c57a9767f76fee94c95b3aa /testsuite/driver/testlib.py | |
| parent | b4ff741e3d6419695f369b722f9be890f1ff237c (diff) | |
| download | haskell-wip/check-testsuite-modifiers.tar.gz | |
testsuite: Add assertions that way lists are in fact listswip/check-testsuite-modifiers
Previously there were a few cases where operations like `omit_ways`
were incorrectly passed a single way (e.g. `omit_ways('threaded2')`).
This won't work as the author expected.
Diffstat (limited to 'testsuite/driver/testlib.py')
| -rw-r--r-- | testsuite/driver/testlib.py | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/testsuite/driver/testlib.py b/testsuite/driver/testlib.py index 9ec39267d6..e37bf5d49b 100644 --- a/testsuite/driver/testlib.py +++ b/testsuite/driver/testlib.py @@ -204,6 +204,7 @@ def use_specs( specs ): ['A B', '-O -prof -fno-prof-count-entries -v0']) """ + assert isinstance(specs, dict) return lambda name, opts, s=specs: _use_specs( name, opts, s ) def _use_specs( name, opts, specs ): @@ -213,6 +214,7 @@ def _use_specs( name, opts, specs ): # ----- def expect_fail_for( ways ): + assert isinstance(ways, list) return lambda name, opts, w=ways: _expect_fail_for( name, opts, w ) def _expect_fail_for( name, opts, ways ): @@ -228,6 +230,7 @@ def _expect_broken( name, opts, bug ): opts.expect = 'fail'; def expect_broken_for( bug, ways ): + assert isinstance(ways, list) return lambda name, opts, b=bug, w=ways: _expect_broken_for( name, opts, b, w ) def _expect_broken_for( name, opts, bug, ways ): @@ -271,6 +274,7 @@ def fragile_for( bug, ways ): # ----- def omit_ways( ways ): + assert isinstance(ways, list) return lambda name, opts, w=ways: _omit_ways( name, opts, w ) def _omit_ways( name, opts, ways ): @@ -280,6 +284,7 @@ def _omit_ways( name, opts, ways ): # ----- def only_ways( ways ): + assert isinstance(ways, list) return lambda name, opts, w=ways: _only_ways( name, opts, w ) def _only_ways( name, opts, ways ): @@ -288,6 +293,7 @@ def _only_ways( name, opts, ways ): # ----- def extra_ways( ways ): + assert isinstance(ways, list) return lambda name, opts, w=ways: _extra_ways( name, opts, w ) def _extra_ways( name, opts, ways ): |