Index: test/runners/chromium_remote_process.js |
=================================================================== |
--- a/test/runners/chromium_remote_process.js |
+++ b/test/runners/chromium_remote_process.js |
@@ -54,17 +54,17 @@ |
catch (error) |
{ |
console.error(error); |
} |
} |
function startChromium(chromiumPath) |
{ |
- fs.chmodSync(chromiumPath, fs.constants.S_IRWXU); |
+ fs.chmodSync(chromiumPath, 0o700); |
Sebastian Noack
2018/09/25 22:12:46
fs.constants.S_IRWXU is 0o700. This change has no
hub
2018/09/26 01:03:25
on Windows? I expect it to be undefined there.
Sebastian Noack
2018/09/26 01:10:51
Geo, can you confirm?
geo
2018/09/26 13:34:09
Well, it get's rid of the `fs.chmodSync` error. Ho
|
let dataDir = fs.mkdtempSync(path.join(os.tmpdir(), "chromium-data")); |
let child = null; |
return { |
kill: () => child && child.kill(), |
done: new Promise((resolve, reject) => |
{ |
child = childProcess.execFile(chromiumPath, [ |