diff options
| author | Matthew Pickering <matthewtpickering@gmail.com> | 2022-04-28 17:13:07 +0100 |
|---|---|---|
| committer | Matthew Pickering <matthewtpickering@gmail.com> | 2022-04-29 08:59:42 +0000 |
| commit | 724d0dc04645b21865bcd1877b7cc77e9a3fe9b2 (patch) | |
| tree | eee85036ac4886c99d8bf8c99cab6d8ad57d666f | |
| parent | 0198841877f6f04269d6050892b98b5c3807ce4c (diff) | |
| download | haskell-724d0dc04645b21865bcd1877b7cc77e9a3fe9b2.tar.gz | |
testsuite: Deduplicate ways correctlywip/deduplicate-ways
This was leading to a bug where we would run a profasm test twice which
led to invalid junit.xml which meant the test results database was not
being populated for the fedora33-perf job.
| -rw-r--r-- | testsuite/driver/testlib.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/testsuite/driver/testlib.py b/testsuite/driver/testlib.py index 9c9a1760a2..813e26f832 100644 --- a/testsuite/driver/testlib.py +++ b/testsuite/driver/testlib.py @@ -34,6 +34,7 @@ extra_src_files = {'T4198': ['exitminus1.c']} # TODO: See #12223 from my_typing import * from threading import Timer +from collections import OrderedDict global pool_sema if config.use_threads: @@ -1073,7 +1074,7 @@ def test_common_work(watcher: testutil.Watcher, all_ways = [WayName('normal')] # A test itself can request extra ways by setting opts.extra_ways - all_ways = all_ways + [way for way in opts.extra_ways if way not in all_ways] + all_ways = list(OrderedDict.fromkeys(all_ways + [way for way in opts.extra_ways if way not in all_ways])) t.total_test_cases += len(all_ways) |