Skip to content

Commit 23cda7d

Browse files
Julien Gilliaddaleax
Julien Gilli
authored andcommitted
test: fix test.py command line options processing
#11086 had introduced a regression that broke command line options processing for tools/test.py. Basically, it made tools/test.py discard the command line argument that would be passed after `--abort-on-timeout`. For instance, when running: ``` $ python tools/test.py --abort-on-timeout path/to/some-test ``` all tests would be run because the last command line argument (`/path/to/some-test`) would be discarded. This change fixes this regression. Refs: #11086 PR-URL: #11153 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anna Henningsen <[email protected]>
1 parent 5e52a9a commit 23cda7d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tools/test.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -1399,7 +1399,7 @@ def BuildOptions():
13991399
default=1, type="int")
14001400
result.add_option('--abort-on-timeout',
14011401
help='Send SIGABRT instead of SIGTERM to kill processes that time out',
1402-
default=False, dest="abort_on_timeout")
1402+
default=False, action="store_true", dest="abort_on_timeout")
14031403
return result
14041404

14051405

0 commit comments

Comments
 (0)