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

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

Issue 12794007: Use scheduled_test for Pub tests. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes. 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 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 d.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 d.file.
4 4
5 import '../../../../../pkg/pathos/lib/path.dart' as path; 5 import '../../../../../pkg/pathos/lib/path.dart' as path;
6 import '../../../../../pkg/scheduled_test/lib/scheduled_test.dart';
6 7
7 import '../../../../pub/exit_codes.dart' as exit_codes; 8 import '../../../../pub/exit_codes.dart' as exit_codes;
9 import '../../descriptor.dart' as d;
8 import '../../test_pub.dart'; 10 import '../../test_pub.dart';
9 11
10 main() { 12 main() {
11 initConfig(); 13 initConfig();
12 integration('path dependency when path is a file', () { 14 integration('path dependency when path is a d.file', () {
13 dir('foo', [ 15 d.dir('foo', [
14 libDir('foo'), 16 d.libDir('foo'),
15 libPubspec('foo', '0.0.1') 17 d.libPubspec('foo', '0.0.1')
16 ]).scheduleCreate(); 18 ]).create();
17 19
18 file('dummy.txt', '').scheduleCreate(); 20 d.file('dummy.txt', '').create();
19 var dummyPath = path.join(sandboxDir, 'dummy.txt'); 21 var dummyPath = path.join(sandboxDir, 'dummy.txt');
20 22
21 dir(appPath, [ 23 d.dir(appPath, [
22 pubspec({ 24 d.pubspec({
23 "name": "myapp", 25 "name": "myapp",
24 "dependencies": { 26 "dependencies": {
25 "foo": {"path": dummyPath} 27 "foo": {"path": dummyPath}
26 } 28 }
27 }) 29 })
28 ]).scheduleCreate(); 30 ]).create();
29 31
30 // TODO(rnystrom): The "\" in a Windows path gets treated like a regex 32 // TODO(rnystrom): The "\" in a Windows path gets treated like a regex
31 // character, so hack escape. A better fix is to use a literal string 33 // character, so hack escape. A better fix is to use a literal string
32 // instead of a RegExp to validate, but that requires us to move the 34 // instead of a RegExp to validate, but that requires us to move the
33 // stack traces out of the stderr when we invoke pub. See also: #4706. 35 // stack traces out of the stderr when we invoke pub. See also: #4706.
34 var escapePath = dummyPath.replaceAll(r"\", r"\\"); 36 var escapePath = dummyPath.replaceAll(r"\", r"\\");
35 37
36 schedulePub(args: ['install'], 38 schedulePub(args: ['install'],
37 error: new RegExp("Path dependency for package 'foo' must refer to a " 39 error: new RegExp("Path dependency for package 'foo' must refer to a "
38 "directory, not a file. Was '$escapePath'."), 40 "directory, not a file. Was '$escapePath'."),
39 exitCode: exit_codes.DATA); 41 exitCode: exit_codes.DATA);
40 }); 42 });
41 } 43 }
OLDNEW
« no previous file with comments | « utils/tests/pub/install/path/nonexistent_dir_test.dart ('k') | utils/tests/pub/install/path/relative_path_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698