Commit 533ce48
tools: make --repeat work with -j in test.py
The repeat option in test.py did not work as expected if `-j` was set to
more than one. Repeated tests running at the same time could share temp
directories and cause test failures. This was observed with:
tools/test.py -J --repeat=10 parallel/test-fs-watch-recursive
By using copy.deepCopy(), the repeated tests are separate objects and
not references to the same objects. Setting `thread_id` on one of them
will now not change the `thread_id` on all of them. And `thread_id` is
how the temp directory (and common.PORT as well) are determined.
Refs: #9228
PR-URL: #9249
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>1 parent 9d5e7f5 commit 533ce48
1 file changed
Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -43,6 +43,7 @@ | |||
| 43 | 43 | import utils | |
| 44 | 44 | import multiprocessing | |
| 45 | 45 | import errno | |
| 46 | + import copy | ||
| 46 | 47 | ||
| 47 | 48 | from os.path import join, dirname, abspath, basename, isdir, exists | |
| 48 | 49 | from datetime import datetime | |
@@ -796,7 +797,9 @@ def AddTestsToList(self, result, current_path, path, context, arch, mode): | |||
| 796 | 797 | tests = self.GetConfiguration(context).ListTests(current_path, path, | |
| 797 | 798 | arch, mode) | |
| 798 | 799 | for t in tests: t.variant_flags = v | |
| 799 | - result += tests * context.repeat | ||
| 800 | + result += tests | ||
| 801 | + for i in range(1, context.repeat): | ||
| 802 | + result += copy.deepcopy(tests) | ||
| 800 | 803 | ||
| 801 | 804 | def GetTestStatus(self, context, sections, defs): | |
| 802 | 805 | self.GetConfiguration(context).GetTestStatus(sections, defs) | |
0 commit comments