Chromium Code Reviews| Index: sdk/lib/_internal/pub/test/hosted/version_negotiation_test.dart |
| diff --git a/sdk/lib/_internal/pub/test/hosted/version_negotiation_test.dart b/sdk/lib/_internal/pub/test/hosted/version_negotiation_test.dart |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..f2b497c64cbd303df23d6962873c1f037752d984 |
| --- /dev/null |
| +++ b/sdk/lib/_internal/pub/test/hosted/version_negotiation_test.dart |
| @@ -0,0 +1,72 @@ |
| +// 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. |
| + |
| +library pub_tests; |
| + |
| +import 'dart:io'; |
| + |
| +import 'package:scheduled_test/scheduled_server.dart'; |
| +import 'package:scheduled_test/scheduled_test.dart'; |
| + |
| +import '../descriptor.dart' as d; |
| +import '../test_pub.dart'; |
| + |
| +main() { |
| + initConfig(); |
| + |
| + forBothPubInstallAndUpdate((command) { |
| + integration('sends the correct Accept header', () { |
| + var server = new ScheduledServer(); |
| + |
| + d.appDir([{ |
| + "hosted": { |
| + "name": "foo", |
| + "url": server.url.then((url) => url.toString()) |
| + } |
| + }]).create(); |
| + |
| + var pub = startPub(args: [command.name]); |
| + |
| + server.handle('GET', '/api/packages/foo', (request) { |
| + expect(request.headers['Accept'], ['application/vnd.pub.v2+json']); |
| + }); |
| + |
| + pub.kill(); |
| + }); |
| + }); |
| + |
| + forBothPubInstallAndUpdate((command) { |
|
Bob Nystrom
2013/06/04 20:21:41
Other tests just wrap all of the integration() cal
nweiz
2013/06/04 21:04:17
Done.
|
| + integration('prints a user-friendly error if the version is out-of-date', |
|
Bob Nystrom
2013/06/04 20:21:41
"user-friendly" -> "friendly", just to get rid of
nweiz
2013/06/04 21:04:17
Done.
|
| + () { |
| + var server = new ScheduledServer(); |
| + |
| + d.appDir([{ |
| + "hosted": { |
| + "name": "foo", |
| + "url": server.url.then((url) => url.toString()) |
| + } |
| + }]).create(); |
| + |
| + var pub = startPub(args: [command.name]); |
| + |
| + server.handle('GET', '/api/packages/foo', (request) { |
| + request.response.statusCode = 406; |
| + request.response.close(); |
| + }); |
| + |
| + // TODO(nweiz): this shouldn't request the versions twice (issue 11077). |
| + server.handle('GET', '/api/packages/foo', (request) { |
| + request.response.statusCode = 406; |
| + request.response.close(); |
| + }); |
| + |
| + pub.shouldExit(1); |
| + |
| + expect(pub.remainingStderr(), completion(equals( |
| + "Pub 0.1.2+3 is incompatible with the current version of " |
| + "pub.dartlang.org.\n" |
|
Bob Nystrom
2013/06/04 20:21:41
Shouldn't it use the actual server URL here?
nweiz
2013/06/04 21:04:17
Done.
|
| + "Upgrade pub to the latest version and try again."))); |
| + }); |
| + }); |
| +} |