Merge pull request #101 from brave/default-config-tests

Use default config so ninja is in PATH
brave · Mar 14, 2018 · 49e96ba · 49e96ba
2 parents 4cb2306 + 57e3693
commit 49e96ba
Showing 1 changed file with 3 additions and 8 deletions.
diff --git a/build/commands/lib/test.js b/build/commands/lib/test.js
@@ -12,13 +12,8 @@ const test = (suite, options) => {
     '--v=' + options.v,
   ]
 
-  let cmdOptions = {
-    stdio: 'inherit',
-    shell: true
-  }
-
   // Build the tests
-  util.run('ninja', ['-C', config.outputDir, suite], cmdOptions)
+  util.run('ninja', ['-C', config.outputDir, suite], config.defaultOptions)
 
   let testBinary;
   if (process.platform === 'win32') {
@@ -28,7 +23,7 @@ const test = (suite, options) => {
   }
 
   // Run the tests
-  util.run(path.join(config.outputDir, testBinary), braveArgs, cmdOptions)
+  util.run(path.join(config.outputDir, testBinary), braveArgs, config.defaultOptions)
 }
 
-module.exports = test
+module.exports = test