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

Side by Side Diff: tools/gardening/test/test_client.dart

Issue 2997043002: Include build revision in BuildResult (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/test/buildbot_data_test.dart ('k') | no next file » | 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 /// Compares the test log of a build step with previous builds. 5 /// Compares the test log of a build step with previous builds.
6 /// 6 ///
7 /// Use this to detect flakiness of failures, especially timeouts. 7 /// Use this to detect flakiness of failures, especially timeouts.
8 8
9 import 'dart:async'; 9 import 'dart:async';
10 import 'dart:io'; 10 import 'dart:io';
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
61 } 61 }
62 62
63 @override 63 @override
64 int get mostRecentBuildNumber => -1; 64 int get mostRecentBuildNumber => -1;
65 } 65 }
66 66
67 class DummyClient implements BuildbotClient { 67 class DummyClient implements BuildbotClient {
68 @override 68 @override
69 Future<BuildResult> readResult(BuildUri buildUri) { 69 Future<BuildResult> readResult(BuildUri buildUri) {
70 return new Future.value( 70 return new Future.value(
71 new BuildResult(buildUri, 1, const [], const [], const [])); 71 new BuildResult(buildUri, 1, '', const [], const [], const []));
72 } 72 }
73 73
74 @override 74 @override
75 void close() {} 75 void close() {}
76 76
77 @override 77 @override
78 int get mostRecentBuildNumber => -1; 78 int get mostRecentBuildNumber => -1;
79 } 79 }
OLDNEW
« no previous file with comments | « tools/gardening/test/buildbot_data_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698