Skip to content

Commit

Permalink
Merge pull request #1241 from brave/added-option-to-output-test-resul…
Browse files Browse the repository at this point in the history
…ts-to-file

added option to output test results to file
  • Loading branch information
w0ts0n authored Sep 22, 2018
2 parents d3d72c2 + 6ce9797 commit c92c037
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
4 changes: 4 additions & 0 deletions lib/test.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,10 @@ const test = (suite, buildConfig = config.defaultBuildConfig, options) => {
braveArgs.push('--gtest_filter=' + options.filter)
}

if (options.output) {
braveArgs.push('--gtest_output=xml:' + options.output)
}

if (options.disable_brave_extension) {
braveArgs.push('--disable-brave-extension')
}
Expand Down
1 change: 1 addition & 0 deletions scripts/commands.js
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,7 @@ program
.command('test <suite>')
.option('--v [log_level]', 'set log level to [log_level]', parseInt, '0')
.option('--filter <filter>', 'set test filter')
.option('--output <output>', 'set test output (results) file path')
.option('--disable_brave_extension', 'disable loading the Brave extension')
.option('--single_process', 'uses a single process to run tests to help with debugging')
.option('--test_launcher_jobs <test_launcher_jobs>', 'Number of jobs to launch')
Expand Down

0 comments on commit c92c037

Please sign in to comment.