Commit dc69e70b authored by Ben Reich's avatar Ben Reich Committed by Commit Bot

[js-code-coverage] Fix v8 coverage output directory

Had the params.task_output_dir as the source of the v8 coverage files,
this is incorrect it should be the tests folder in the
--javascript-coverage-dir flag path.

Bug: 1113941
Change-Id: I8491ce86f6c615be20e7660becedc3c7887cfd13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2526805Reviewed-by: default avatarYuke Liao <liaoyuke@chromium.org>
Commit-Queue: Ben Reich <benreich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#826186}
parent cd752950
...@@ -103,8 +103,9 @@ def main(): ...@@ -103,8 +103,9 @@ def main():
output_path = os.path.join(params.javascript_coverage_dir, output_path = os.path.join(params.javascript_coverage_dir,
'%s_javascript.json' % params.merged_js_cov_filename) '%s_javascript.json' % params.merged_js_cov_filename)
raw_coverage_folder = os.path.join(params.javascript_coverage_dir, 'tests')
logging.info('Merging v8 coverage output to %s', output_path) logging.info('Merging v8 coverage output to %s', output_path)
javascript_merger.merge_coverage_files(params.task_output_dir, output_path) javascript_merger.merge_coverage_files(raw_coverage_folder, output_path)
# Name the output profdata file name as {test_target}.profdata or # Name the output profdata file name as {test_target}.profdata or
# default.profdata. # default.profdata.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment