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

Unified Diff: utils/tests/pub/pub_lish_test.dart

Issue 11569046: Add a pub command for managing uploaders for packages. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes Created 8 years 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
« no previous file with comments | « utils/tests/pub/oauth2_test.dart ('k') | utils/tests/pub/pub_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/tests/pub/pub_lish_test.dart
diff --git a/utils/tests/pub/pub_lish_test.dart b/utils/tests/pub/pub_lish_test.dart
index b787ec97afdb67321443d3cea2aebf80c37066a5..695557848e952a4732633a5d5bcb77ec44289b3a 100644
--- a/utils/tests/pub/pub_lish_test.dart
+++ b/utils/tests/pub/pub_lish_test.dart
@@ -79,35 +79,6 @@ main() {
// TODO(nweiz): Once a multipart/form-data parser in Dart exists, we should
// test that "pub lish" chooses the correct files to publish.
- test('credentials are invalid', () {
- var server = new ScheduledServer();
- credentialsFile(server, 'access token').scheduleCreate();
- var pub = startPubLish(server);
-
- confirmPublish(pub);
-
- server.handle('GET', '/packages/versions/new.json', (request, response) {
- response.statusCode = 401;
- response.headers.set('www-authenticate', 'Bearer error="invalid_token",'
- ' error_description="your token sucks"');
- response.outputStream.writeString(JSON.stringify({
- 'error': {'message': 'your token sucks'}
- }));
- response.outputStream.close();
- });
-
- expectLater(pub.nextErrLine(), equals('OAuth2 authorization failed (your '
- 'token sucks).'));
- // TODO(rnystrom): The confirm line is run together with this one because
- // in normal usage, the user will have entered a newline on stdin which
- // gets echoed to the terminal. Do something better here?
- expectLater(pub.nextLine(), equals(
- 'Looks great! Are you ready to upload your package (y/n)? '
- 'Pub needs your authorization to upload packages on your behalf.'));
- pub.kill();
- run();
- });
-
test('package validation has an error', () {
var package = package("test_pkg", "1.0.0");
package.remove("homepage");
« no previous file with comments | « utils/tests/pub/oauth2_test.dart ('k') | utils/tests/pub/pub_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698