Commit 6e7dfb1
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 9abde3a commit 6e7dfb1
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1672,9 +1672,9 @@ def Main(): | |||
| 1672 | 1672 | ||
| 1673 | 1673 | tempdir = os.environ.get('NODE_TEST_DIR') or options.temp_dir | |
| 1674 | 1674 | if tempdir: | |
| 1675 | + os.environ['NODE_TEST_DIR'] = tempdir | ||
| 1675 | 1676 | try: | |
| 1676 | 1677 | os.makedirs(tempdir) | |
| 1677 | - os.environ['NODE_TEST_DIR'] = tempdir | ||
| 1678 | 1678 | except OSError as exception: | |
| 1679 | 1679 | if exception.errno != errno.EEXIST: | |
| 1680 | 1680 | print "Could not create the temporary directory", options.temp_dir | |
0 commit comments