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/pub_install_test.dart

Issue 10947050: Support dependencies in SDK packages. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Respond to review. Created 8 years, 3 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
« no previous file with comments | « utils/tests/pub/pub_install_sdk_test.dart ('k') | utils/tests/pub/pub_update_test.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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('dart:io'); 7 #import('dart:io');
8 8
9 #import('test_pub.dart'); 9 #import('test_pub.dart');
10 #import('../../../pkg/unittest/unittest.dart'); 10 #import('../../../pkg/unittest/unittest.dart');
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 ]).scheduleValidate(); 70 ]).scheduleValidate();
71 71
72 run(); 72 run();
73 }); 73 });
74 74
75 test('does not add a package if it does not have a "lib" directory', () { 75 test('does not add a package if it does not have a "lib" directory', () {
76 // Using an SDK source, but this should be true of all sources. 76 // Using an SDK source, but this should be true of all sources.
77 dir(sdkPath, [ 77 dir(sdkPath, [
78 file('revision', '1234'), 78 file('revision', '1234'),
79 dir('pkg', [ 79 dir('pkg', [
80 dir('foo', []) 80 dir('foo', [
81 libPubspec('foo', '0.0.0-not.used')
82 ])
81 ]) 83 ])
82 ]).scheduleCreate(); 84 ]).scheduleCreate();
83 85
84 dir(appPath, [ 86 dir(appPath, [
85 pubspec({"name": "myapp", "dependencies": {"foo": {"sdk": "foo"}}}) 87 pubspec({"name": "myapp", "dependencies": {"foo": {"sdk": "foo"}}})
86 ]).scheduleCreate(); 88 ]).scheduleCreate();
87 89
88 schedulePub(args: ['install'], 90 schedulePub(args: ['install'],
89 error: const RegExp(@'Warning: Package "foo" does not have a "lib" ' 91 error: const RegExp(@'Warning: Package "foo" does not have a "lib" '
90 'directory.'), 92 'directory.'),
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 schedulePub(args: ['install'], 243 schedulePub(args: ['install'],
242 error: ''' 244 error: '''
243 Warning: Package "myapp_name" is using a deprecated layout. 245 Warning: Package "myapp_name" is using a deprecated layout.
244 See http://www.dartlang.org/docs/pub-package-manager/package-layout.html for details. 246 See http://www.dartlang.org/docs/pub-package-manager/package-layout.html for details.
245 ''', 247 ''',
246 output: const RegExp(@"Dependencies installed!$")); 248 output: const RegExp(@"Dependencies installed!$"));
247 249
248 run(); 250 run();
249 }); 251 });
250 } 252 }
OLDNEW
« no previous file with comments | « utils/tests/pub/pub_install_sdk_test.dart ('k') | utils/tests/pub/pub_update_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698