Browse Source

test: fix test.py command line options processing

https://github.com/nodejs/node/pull/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: https://github.com/nodejs/node/pull/11086
PR-URL: https://github.com/nodejs/node/pull/11153
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
v6.x
Julien Gilli 8 years ago
committed by Myles Borins
parent
commit
8f00f70d19
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 2
      tools/test.py

2
tools/test.py

@ -1399,7 +1399,7 @@ def BuildOptions():
default=1, type="int")
result.add_option('--abort-on-timeout',
help='Send SIGABRT instead of SIGTERM to kill processes that time out',
default=False, dest="abort_on_timeout")
default=False, action="store_true", dest="abort_on_timeout")
return result

Loading…
Cancel
Save