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

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

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