Commit 93877c8
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 0f3677d commit 93877c8
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1644,9 +1644,9 @@ def Main(): | |||
| 1644 | 1644 | ||
| 1645 | 1645 | tempdir = os.environ.get('NODE_TEST_DIR') or options.temp_dir | |
| 1646 | 1646 | if tempdir: | |
| 1647 | + os.environ['NODE_TEST_DIR'] = tempdir | ||
| 1647 | 1648 | try: | |
| 1648 | 1649 | os.makedirs(tempdir) | |
| 1649 | - os.environ['NODE_TEST_DIR'] = tempdir | ||
| 1650 | 1650 | except OSError as exception: | |
| 1651 | 1651 | if exception.errno != errno.EEXIST: | |
| 1652 | 1652 | print "Could not create the temporary directory", options.temp_dir | |
0 commit comments