Hash :
f071aaaf
Author :
Date :
2021-07-22T21:21:35
Use first test name instead of shard index for TestArtifacts
Shard index is not available on Android if run_gtest_angle_test.py
is not used, since host doesn't pass it to device by default.
We'd like to remove run_gtest_angle_test.py usage, since custom scripts
are not supported on iOS, so this CL is a pre-requisite to that.
We still need to distinguish between dEQP test artifacts when we merge
test results of different shards, so we'll be using the name of the
first test in a batch instead.
However, now, on Android,
instead of a single TestArtifactsFakeTest entry per shard,
we will have multiple entries,
an entry for each GTest batch of 256 tests.
On other platforms we will still have one entry per shard,
since batching is done by TestSuite and not GTests there,
just with a different name:
Old:
"tests": {
"TestArtifactsFakeTest-Shard00": {
"actual": "PASS",
"artifacts": {
"TestResults-Batch001.qpa": [
"TestResults-Batch001.qpa"
],
New:
"tests": {
"TestArtifactsFakeTest-GLES3/info_renderer": {
"actual": "PASS",
"artifacts": {
"TestResults-Batch001.qpa": [
"TestResults-Batch001.qpa"
],
Bug: angleproject:5417
Change-Id: I649036c151e078c862e380ccab347d6c59acc929
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/3046764
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>