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

Unified Diff: utils/tests/pub/install/pub_install_test.dart

Issue 11280018: Remove support for old package layouts. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Better error message. Created 8 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « utils/pub/io.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/tests/pub/install/pub_install_test.dart
diff --git a/utils/tests/pub/install/pub_install_test.dart b/utils/tests/pub/install/pub_install_test.dart
index 1ce3a6a0bf23091aaedf3ae823bada1bf95cd77d..4f9dc433974db0de3bdef1a208e57f141978db22 100644
--- a/utils/tests/pub/install/pub_install_test.dart
+++ b/utils/tests/pub/install/pub_install_test.dart
@@ -89,7 +89,8 @@ main() {
schedulePub(args: ['install'],
error: new RegExp(r'Warning: Package "foo" does not have a "lib" '
- 'directory.'),
+ 'directory so you will not be able to import any libraries from '
+ 'it.'),
output: new RegExp(r"Dependencies installed!$"));
run();
@@ -272,23 +273,4 @@ main() {
run();
});
});
-
- // TODO(rnystrom): Remove this when old layout support is removed. (#4964)
- test('shows a warning if the entrypoint uses the old layout', () {
- // The symlink should use the name in the pubspec, not the name of the
- // directory.
- dir(appPath, [
- pubspec({"name": "myapp_name"}),
- file("foo.dart", 'main() => "foo";'),
- ]).scheduleCreate();
-
- schedulePub(args: ['install'],
- error: '''
- Warning: Package "myapp_name" is using a deprecated layout.
- See http://www.dartlang.org/docs/pub-package-manager/package-layout.html for details.
- ''',
- output: new RegExp(r"Dependencies installed!$"));
-
- run();
- });
}
« no previous file with comments | « utils/pub/io.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698