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

Side by Side Diff: sdk/lib/_internal/pub/test/cache/repair/handles_failure_test.dart

Issue 488323002: Make servePackages's contents argument saner. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes Created 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, 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 pub_tests; 5 library pub_tests;
6 6
7 import 'package:scheduled_test/scheduled_test.dart'; 7 import 'package:scheduled_test/scheduled_test.dart';
8 8
9 import '../../../lib/src/exit_codes.dart' as exit_codes; 9 import '../../../lib/src/exit_codes.dart' as exit_codes;
10 import '../../descriptor.dart' as d; 10 import '../../descriptor.dart' as d;
11 import '../../test_pub.dart'; 11 import '../../test_pub.dart';
12 12
13 main() { 13 main() {
14 initConfig(); 14 initConfig();
15 integration('handles failure to reinstall some packages', () { 15 integration('handles failure to reinstall some packages', () {
16 // Only serve two packages so repairing will have a failure. 16 // Only serve two packages so repairing will have a failure.
17 servePackages([ 17 servePackages((builder) {
18 packageMap("foo", "1.2.3"), 18 builder.serve("foo", "1.2.3");
19 packageMap("foo", "1.2.5") 19 builder.serve("foo", "1.2.5");
20 ]); 20 });
21 21
22 // Set up a cache with some packages. 22 // Set up a cache with some packages.
23 d.dir(cachePath, [ 23 d.dir(cachePath, [
24 d.dir('hosted', [ 24 d.dir('hosted', [
25 d.async(port.then((p) => d.dir('localhost%58$p', [ 25 d.async(port.then((p) => d.dir('localhost%58$p', [
26 d.dir("foo-1.2.3", [ 26 d.dir("foo-1.2.3", [
27 d.libPubspec("foo", "1.2.3"), 27 d.libPubspec("foo", "1.2.3"),
28 d.file("broken.txt") 28 d.file("broken.txt")
29 ]), 29 ]),
30 d.dir("foo-1.2.4", [ 30 d.dir("foo-1.2.4", [
(...skipping 17 matching lines...) Expand all
48 48
49 pub.stderr.expect(startsWith("Failed to repair foo 1.2.4. Error:")); 49 pub.stderr.expect(startsWith("Failed to repair foo 1.2.4. Error:"));
50 pub.stderr.expect("HTTP error 404: Not Found"); 50 pub.stderr.expect("HTTP error 404: Not Found");
51 51
52 pub.stdout.expect("Reinstalled 2 packages."); 52 pub.stdout.expect("Reinstalled 2 packages.");
53 pub.stdout.expect("Failed to reinstall 1 package."); 53 pub.stdout.expect("Failed to reinstall 1 package.");
54 54
55 pub.shouldExit(exit_codes.UNAVAILABLE); 55 pub.shouldExit(exit_codes.UNAVAILABLE);
56 }); 56 });
57 } 57 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698