Skip to content
Permalink
Browse files

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>
  • Loading branch information...
gibfahn authored and jasnell committed Jan 10, 2017
1 parent dd2d3d3 commit 57f6a106fbc69a47ece42bab9f3f9c65c2c6150b
Showing with 1 addition and 1 deletion.
  1. +1 −1 tools/test.py
@@ -1628,9 +1628,9 @@ def Main():

tempdir = os.environ.get('NODE_TEST_DIR') or options.temp_dir
if tempdir:
os.environ['NODE_TEST_DIR'] = tempdir
try:
os.makedirs(tempdir)
os.environ['NODE_TEST_DIR'] = tempdir
except OSError as exception:
if exception.errno != errno.EEXIST:
print "Could not create the temporary directory", options.temp_dir

0 comments on commit 57f6a10

Please sign in to comment.
You can’t perform that action at this time.