Skip to content

Commit e810462

Browse files
committed
[current results] Do not store skipped tests in current results database
When a test is skipped on a configuration for the first time, the change is reported as a changed result with new result "skipped". Subsequent builder runs remove this skipped record completely. Ignore these test results in the current results server. Bug: #107 Change-Id: I8587aab909fd367edc642f4a97513ca1a39e0592 Reviewed-on: https://dart-review.googlesource.com/c/dart_ci/+/339921 Reviewed-by: Alexander Thomas <[email protected]>
1 parent 6f343fb commit e810462

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

current_results/lib/src/slice.dart

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@ class Slice {
7474
final result = Result.fromApi(api.Result()
7575
..mergeFromProto3Json(json.decode(line),
7676
supportNamesWithUnderscores: true));
77+
if (result.result == 'skipped') continue;
7778
if (configuration == null) {
7879
configuration = result.configuration;
7980
} else if (result.configuration != configuration) {

0 commit comments

Comments
 (0)