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

Unified Diff: utils/tests/pub/oauth2/with_a_pre_existing_credentials_does_not_authenticate_test.dart

Issue 13472016: Split apart several asynchronous tests to reduce timeouts. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: utils/tests/pub/oauth2/with_a_pre_existing_credentials_does_not_authenticate_test.dart
diff --git a/utils/tests/pub/lish/upload_form_provides_invalid_json_test.dart b/utils/tests/pub/oauth2/with_a_pre_existing_credentials_does_not_authenticate_test.dart
similarity index 63%
copy from utils/tests/pub/lish/upload_form_provides_invalid_json_test.dart
copy to utils/tests/pub/oauth2/with_a_pre_existing_credentials_does_not_authenticate_test.dart
index 3f2c4866871e9889eb4a0942bc7ffcf661f2da4f..976903c47db69fb926c732ee848f66f2810f4f32 100644
--- a/utils/tests/pub/lish/upload_form_provides_invalid_json_test.dart
+++ b/utils/tests/pub/oauth2/with_a_pre_existing_credentials_does_not_authenticate_test.dart
@@ -1,34 +1,35 @@
-// Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
+// Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
+import 'dart:io';
import 'dart:json' as json;
import 'package:scheduled_test/scheduled_test.dart';
import 'package:scheduled_test/scheduled_server.dart';
+import '../../../pub/io.dart';
+import '../../../pub/utils.dart';
import '../descriptor.dart' as d;
import '../test_pub.dart';
import 'utils.dart';
main() {
- initConfig();
- setUp(d.validPackage.create);
+ integration('with a pre-existing credentials.json does not authenticate', () {
+ d.validPackage.create();
- integration('upload form provides invalid JSON', () {
var server = new ScheduledServer();
d.credentialsFile(server, 'access token').create();
var pub = startPublish(server);
-
confirmPublish(pub);
server.handle('GET', '/packages/versions/new.json', (request) {
- request.response.write('{not json');
+ expect(request.headers.value('authorization'),
+ equals('Bearer access token'));
+
request.response.close();
});
- expect(pub.nextErrLine(), completion(equals('Invalid server response:')));
- expect(pub.nextErrLine(), completion(equals('{not json')));
- pub.shouldExit(1);
+ pub.kill();
});
-}
+}

Powered by Google App Engine
This is Rietveld 408576698