Skip to content

Commit

Permalink
test: set umask explicitly
Browse files Browse the repository at this point in the history
Some tests which create files and check file permissions assume the
umask is compatible with 022, and break when set to something like 007.
Explicitly set umask to 022

PR-URL: nodejs#25213
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
i8-pi authored and refack committed Jan 10, 2019
1 parent 8e9fd63 commit b5ff68e
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tools/test.py
Original file line number Diff line number Diff line change
@@ -70,6 +70,7 @@ def cmp(x, y): # Python 3

VERBOSE = False

os.umask(0o022)
os.environ['NODE_OPTIONS'] = ''

# ---------------------------------------------

0 comments on commit b5ff68e

Please sign in to comment.