Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(219)

Side by Side Diff: tools/gardening/bin/find_timeouts.dart

Issue 2987253002: Add test to compare-failures (Closed)
Patch Set: Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tools/gardening/bin/find_shard.dart ('k') | tools/gardening/lib/src/buildbot_loading.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 /// Scans past dart2js-windows test steps for timeouts and reports the 5 /// Scans past dart2js-windows test steps for timeouts and reports the
6 /// frequency of each test that has timed out. 6 /// frequency of each test that has timed out.
7 7
8 import 'dart:async'; 8 import 'dart:async';
9 import 'dart:io'; 9 import 'dart:io';
10 import 'package:args/args.dart'; 10 import 'package:args/args.dart';
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 buildNumberIndex = -buildNumberOffset - 1; 96 buildNumberIndex = -buildNumberOffset - 1;
97 } else { 97 } else {
98 buildNumberIndex = buildNumbers.firstWhere((n) => n <= buildNumberOffset, 98 buildNumberIndex = buildNumbers.firstWhere((n) => n <= buildNumberOffset,
99 orElse: () => buildNumbers.length); 99 orElse: () => buildNumbers.length);
100 } 100 }
101 for (int i = 0; i < buildNumberCount; i++) { 101 for (int i = 0; i < buildNumberCount; i++) {
102 if (buildNumberIndex + i < buildNumbers.length) { 102 if (buildNumberIndex + i < buildNumbers.length) {
103 int buildNumber = buildNumbers[buildNumberIndex + i]; 103 int buildNumber = buildNumbers[buildNumberIndex + i];
104 for (BuildUri buildUri 104 for (BuildUri buildUri
105 in subgroup.createShardUris(shardName, buildNumber)) { 105 in subgroup.createShardUris(shardName, buildNumber)) {
106 BuildResult result = await readLogDogResult(buildUri); 106 BuildResult result = await readBuildResultFromLogDog(buildUri);
107 for (TestFailure timeout in result.timeouts) { 107 for (TestFailure timeout in result.timeouts) {
108 timeouts.putIfAbsent(timeout.id.testName, () => <Timeout>[]).add( 108 timeouts.putIfAbsent(timeout.id.testName, () => <Timeout>[]).add(
109 new Timeout(subgroup, result.buildNumber, buildUri, timeout)); 109 new Timeout(subgroup, result.buildNumber, buildUri, timeout));
110 } 110 }
111 buildUri = buildUri.prev(); 111 buildUri = buildUri.prev();
112 } 112 }
113 } 113 }
114 } 114 }
115 } 115 }
116 } 116 }
(...skipping 16 matching lines...) Expand all
133 } 133 }
134 134
135 class Timeout { 135 class Timeout {
136 final BuildSubgroup subgroup; 136 final BuildSubgroup subgroup;
137 final int buildNumber; 137 final int buildNumber;
138 final BuildUri buildUri; 138 final BuildUri buildUri;
139 final TestFailure timeout; 139 final TestFailure timeout;
140 140
141 Timeout(this.subgroup, this.buildNumber, this.buildUri, this.timeout); 141 Timeout(this.subgroup, this.buildNumber, this.buildUri, this.timeout);
142 } 142 }
OLDNEW
« no previous file with comments | « tools/gardening/bin/find_shard.dart ('k') | tools/gardening/lib/src/buildbot_loading.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698