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

Side by Side Diff: utils/tests/pub/install/path/nonexistent_dir_test.dart

Issue 12263018: Fix path tests on windows. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Mention bug in TODO. Created 7 years, 10 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) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, 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 import 'dart:io'; 5 import 'dart:io';
6 6
7 import '../../../../../pkg/path/lib/path.dart' as path; 7 import '../../../../../pkg/path/lib/path.dart' as path;
8 8
9 import '../../../../pub/exit_codes.dart' as exit_codes; 9 import '../../../../pub/exit_codes.dart' as exit_codes;
10 import '../../test_pub.dart'; 10 import '../../test_pub.dart';
11 11
12 main() { 12 main() {
13 initConfig(); 13 initConfig();
14 integration('path dependency to non-existent directory', () { 14 integration('path dependency to non-existent directory', () {
15 var badPath = path.join(sandboxDir, "bad_path"); 15 var badPath = path.join(sandboxDir, "bad_path");
16 16
17 dir(appPath, [ 17 dir(appPath, [
18 pubspec({ 18 pubspec({
19 "name": "myapp", 19 "name": "myapp",
20 "dependencies": { 20 "dependencies": {
21 "foo": {"path": badPath} 21 "foo": {"path": badPath}
22 } 22 }
23 }) 23 })
24 ]).scheduleCreate(); 24 ]).scheduleCreate();
25 25
26 // TODO(rnystrom): The "\" in a Windows path gets treated like a regex
27 // character, so hack escape. A better fix is to use a literal string
28 // instead of a RegExp to validate, but that requires us to move the
29 // stack traces out of the stderr when we invoke pub. See also: #4706.
30 var escapePath = badPath.replaceAll(r"\", r"\\");
31
26 schedulePub(args: ['install'], 32 schedulePub(args: ['install'],
27 error: 33 error:
28 new RegExp("Could not find package 'foo' at '$badPath'."), 34 new RegExp("Could not find package 'foo' at '$escapePath'."),
29 exitCode: exit_codes.DATA); 35 exitCode: exit_codes.DATA);
30 }); 36 });
31 } 37 }
OLDNEW
« no previous file with comments | « utils/tests/pub/install/path/no_pubspec_test.dart ('k') | utils/tests/pub/install/path/path_is_file_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698