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

Side by Side Diff: pkg/oauth2/test/utils.dart

Issue 14070010: Refactor Future constructors. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Added co19 issue number. Created 7 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « pkg/oauth2/test/credentials_test.dart ('k') | pkg/scheduled_test/lib/scheduled_process.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) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 library utils; 5 library utils;
6 6
7 import 'dart:async'; 7 import 'dart:async';
8 import 'dart:collection' show Queue; 8 import 'dart:collection' show Queue;
9 9
10 import 'package:http/http.dart' as http; 10 import 'package:http/http.dart' as http;
(...skipping 20 matching lines...) Expand all
31 expect(completer.future, completes); 31 expect(completer.future, completes);
32 32
33 _handlers.add((request) { 33 _handlers.add((request) {
34 completer.complete(null); 34 completer.complete(null);
35 return fn(request); 35 return fn(request);
36 }); 36 });
37 } 37 }
38 38
39 Future<http.Response> _handleRequest(http.Request request) { 39 Future<http.Response> _handleRequest(http.Request request) {
40 if (_handlers.isEmpty) { 40 if (_handlers.isEmpty) {
41 return new Future.immediate(new http.Response('not found', 404)); 41 return new Future.value(new http.Response('not found', 404));
42 } else { 42 } else {
43 return _handlers.removeFirst()(request); 43 return _handlers.removeFirst()(request);
44 } 44 }
45 } 45 }
46 } 46 }
47 47
48 /// A matcher for AuthorizationExceptions. 48 /// A matcher for AuthorizationExceptions.
49 const isAuthorizationException = const _AuthorizationException(); 49 const isAuthorizationException = const _AuthorizationException();
50 50
51 /// A matcher for functions that throw AuthorizationException. 51 /// A matcher for functions that throw AuthorizationException.
(...skipping 11 matching lines...) Expand all
63 63
64 /// A matcher for functions that throw ExpirationException. 64 /// A matcher for functions that throw ExpirationException.
65 const Matcher throwsExpirationException = 65 const Matcher throwsExpirationException =
66 const Throws(isExpirationException); 66 const Throws(isExpirationException);
67 67
68 class _ExpirationException extends TypeMatcher { 68 class _ExpirationException extends TypeMatcher {
69 const _ExpirationException() : super("ExpirationException"); 69 const _ExpirationException() : super("ExpirationException");
70 bool matches(item, MatchState matchState) => 70 bool matches(item, MatchState matchState) =>
71 item is oauth2.ExpirationException; 71 item is oauth2.ExpirationException;
72 } 72 }
OLDNEW
« no previous file with comments | « pkg/oauth2/test/credentials_test.dart ('k') | pkg/scheduled_test/lib/scheduled_process.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698