tast: results.json should include details about tests |
|||
Issue descriptionIt'd be nice if the tast executable included serialized testing.Test structs in results.json files: - People looking at results can read tests' descriptions. - cbuildbot test stages can avoid failing for errors in tests that have a "flaky" attribute set. - Probably more stuff that I haven't thought of yet. To do this, the local_tests and remote_tests executables will need to pass the structs to the tast executable; they should probably be included in control.TestStart messages.
,
Oct 20 2017
,
Jan 22 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform/tast/+/8f05ad10016e65edb014284ac7528d24d376df8d commit 8f05ad10016e65edb014284ac7528d24d376df8d Author: Daniel Erat <derat@chromium.org> Date: Mon Jan 22 18:13:39 2018 tast: Drop control.TestStart.Name field. Drop the Name field from TestStart control messages. The test name is now available via the Test field's Name field (and has been for months). BUG= chromium:773819 TEST=unit tests pass; also ran a local test and checked that results.json contains test info Change-Id: Id59ee6898b85bcc47316202bd9a2c8546c37046a Reviewed-on: https://chromium-review.googlesource.com/877799 Commit-Ready: Dan Erat <derat@chromium.org> Tested-by: Dan Erat <derat@chromium.org> Reviewed-by: Jason Clinton <jclinton@chromium.org> [modify] https://crrev.com/8f05ad10016e65edb014284ac7528d24d376df8d/src/chromiumos/cmd/tast/run/results_test.go [modify] https://crrev.com/8f05ad10016e65edb014284ac7528d24d376df8d/src/chromiumos/tast/control/control_test.go [modify] https://crrev.com/8f05ad10016e65edb014284ac7528d24d376df8d/src/chromiumos/tast/bundle/bundle.go [modify] https://crrev.com/8f05ad10016e65edb014284ac7528d24d376df8d/src/chromiumos/tast/control/control.go [modify] https://crrev.com/8f05ad10016e65edb014284ac7528d24d376df8d/src/chromiumos/cmd/tast/run/results.go [modify] https://crrev.com/8f05ad10016e65edb014284ac7528d24d376df8d/src/chromiumos/tast/bundle/bundle_test.go
,
Jan 23 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Oct 15 2017