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 533ce48Copy full SHA for 533ce48
File tree
Expand file treeCollapse file tree
1 file changed
+4
-1
lines changedOpen diff view settings
Filter options
- tools
Expand file treeCollapse file tree
1 file changed
+4
-1
lines changedOpen diff view settings
Collapse file
+4-1Lines changed: 4 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
43 | 43 | |
44 | 44 | |
45 | 45 | |
| 46 | + |
46 | 47 | |
47 | 48 | |
48 | 49 | |
| ||
796 | 797 | |
797 | 798 | |
798 | 799 | |
799 | | - |
| 800 | + |
| 801 | + |
| 802 | + |
800 | 803 | |
801 | 804 | |
802 | 805 | |
|
0 commit comments