New issue
Advanced search Search tips

Issue 775619 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 772208



Sign in to add a comment

Update documentation of JSON test results format to include artifacts

Project Member Reported by nedngu...@google.com, Oct 17 2017

Issue description

We have agreement from John/Dirk on extending JSON test results format to include artifacts (bit.ly/chromium-test-artifacts). We should update the documentation to reflect the new standard.

I think we should first start with porting https://www.chromium.org/developers/the-json-test-results-format to a markdown doc in https://chromium.googlesource.com/chromium/src/+/master/docs/testing/ first. This will allow updates to the doc to be code-review-able & enables history tracking.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 18 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/89f9ca6ed88f89bd0602fd61add1504b4a87b1db

commit 89f9ca6ed88f89bd0602fd61add1504b4a87b1db
Author: Stephen Martinis <martiniss@chromium.org>
Date: Wed Oct 18 19:12:35 2017

Convert test results json doc to markdown.

This is a conversion of 
https://sites.google.com/a/chromium.org/dev/developers/the-json-test-results-format
to markdown.

Bug:  775619 
Change-Id: Ibd9b6f95c8aedb21ba1fe9af129765ba84de80c9
Reviewed-on: https://chromium-review.googlesource.com/724219
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#509829}
[add] https://crrev.com/89f9ca6ed88f89bd0602fd61add1504b4a87b1db/docs/testing/json_test_results_format.md

Blocking: 772208
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 4 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e1e5bf9144cbdbeba1f5033a3c2303849d63cdad

commit e1e5bf9144cbdbeba1f5033a3c2303849d63cdad
Author: Stephen Martinis <martiniss@chromium.org>
Date: Mon Dec 04 23:44:33 2017

Clarify artifact documentation for test results

This clarifies the meaning of forward slashes in artifact file paths,
and also clarifies the directory these paths are relative to, when no
artifact permanent location is set.

Bug:  775619 
Change-Id: I1f41d86273455123823350a39b239cc04b6a0216
Reviewed-on: https://chromium-review.googlesource.com/802092
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521525}
[modify] https://crrev.com/e1e5bf9144cbdbeba1f5033a3c2303849d63cdad/docs/testing/json_test_results_format.md

Status: Fixed (was: Assigned)

Sign in to add a comment