Commit 7d917dc
test: fix temp-dir option in tools/test.py
If a temp-dir is specified and already exists, the NODE_TEST_DIR
environment variable will never be set. This fixes that.
PR-URL: #10723
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>1 parent 6b54024 commit 7d917dc
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1628,9 +1628,9 @@ def Main(): | |||
| 1628 | 1628 | ||
| 1629 | 1629 | tempdir = os.environ.get('NODE_TEST_DIR') or options.temp_dir | |
| 1630 | 1630 | if tempdir: | |
| 1631 | + os.environ['NODE_TEST_DIR'] = tempdir | ||
| 1631 | 1632 | try: | |
| 1632 | 1633 | os.makedirs(tempdir) | |
| 1633 | - os.environ['NODE_TEST_DIR'] = tempdir | ||
| 1634 | 1634 | except OSError as exception: | |
| 1635 | 1635 | if exception.errno != errno.EEXIST: | |
| 1636 | 1636 | print "Could not create the temporary directory", options.temp_dir | |
0 commit comments