Merge pull request #8596 from brave/fix-test-options

don't erase the rest of the array when splicing test command lines options
brave · Apr 21, 2021 · 3a710ed · 3a710ed
2 parents cfb3a19 + 9ac7023
commit 3a710ed
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/build/commands/lib/test.js b/build/commands/lib/test.js
@@ -79,7 +79,7 @@ const test = (passthroughArgs, suite, buildConfig = config.defaultBuildConfig, o
     // Run the tests
     getTestsToRun(config, suite).forEach((testSuite) => {
       if (options.output) {
-        braveArgs.splice(braveArgs.indexOf('--gtest_output=xml:' + options.output, 1))
+        braveArgs.splice(braveArgs.indexOf('--gtest_output=xml:' + options.output), 1)
         braveArgs.push(`--gtest_output=xml:${testSuite}.xml`)
       }
       if (config.targetOS === 'android') {