Commit 7f2de443 authored by Xiyuan Xia's avatar Xiyuan Xia Committed by Commit Bot

Update test result format

- Update trace file key to 'trace/1.json'. Per spec, key name should
  start with 'trace/' and extension should be '.json' since gtest
  output trace json.
- Update script test data as well;

Bug: 945711
Change-Id: I9fc36b154e68bae869f5221e243854847aaaf307
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899309
Auto-Submit: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: default avatarScott Violet <sky@chromium.org>
Reviewed-by: default avatarMikhail Khokhlov <khokhlov@google.com>
Cr-Commit-Position: refs/heads/master@{#712771}
parent 84a97048
......@@ -64,7 +64,7 @@ perf_test::LuciTestResult CreateTestResult(
const std::vector<std::string>& tbm_metrics) {
perf_test::LuciTestResult result =
perf_test::LuciTestResult::CreateForGTest();
result.AddOutputArtifactFile("trace/1", trace_file, "application/json");
result.AddOutputArtifactFile("trace/1.json", trace_file, "application/json");
for (auto& metric : tbm_metrics)
result.AddTag("tbmv2", metric);
......
......@@ -79,7 +79,7 @@ perf_test::LuciTestResult CreateTestResult(
const std::vector<std::string>& tbm_metrics) {
perf_test::LuciTestResult result =
perf_test::LuciTestResult::CreateForGTest();
result.AddOutputArtifactFile("trace/1", trace_file, "application/json");
result.AddOutputArtifactFile("trace/1.json", trace_file, "application/json");
for (auto& metric : tbm_metrics)
result.AddTag("tbmv2", metric);
......
......@@ -4,7 +4,7 @@
"status": "PASS",
"expected": true,
"outputArtifacts":{
"trace": {
"trace/1.json": {
"contentType":"application/json",
"filePath":"FULL_TRACE_FILE_PATH"
}
......
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