Permalink
Browse files

Use an ES5 compatible syntax for octal numbers

I run tests with Node v0.10 and v0.12 as well. I know that these versions aren't supported anymore, but many projects still use them.
This little change makes the tests run in these versions of Node.
  • Loading branch information...
1 parent c1eacd9 commit 6570faa3b23cbf889d085b79a702b1fe736673af @lukasbestle lukasbestle committed Nov 25, 2015
Showing with 2 additions and 2 deletions.
  1. +2 −2 test_lac/loadAndCrash.test.js
@@ -79,9 +79,9 @@ fs.writeFile = function(path, data, options, callback_) {
var callback = maybeCallback(arguments[arguments.length - 1]);
if (!options || typeof options === 'function') {
- options = { encoding: 'utf8', mode: 0o666, flag: 'w' };
+ options = { encoding: 'utf8', mode: 438, flag: 'w' }; // Mode 438 == 0o666 (compatibility with older Node releases)
} else if (typeof options === 'string') {
- options = { encoding: options, mode: 0o666, flag: 'w' };
+ options = { encoding: options, mode: 438, flag: 'w' }; // Mode 438 == 0o666 (compatibility with older Node releases)
} else if (typeof options !== 'object') {
throwOptionsError(options);
}

0 comments on commit 6570faa

Please sign in to comment.