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

Side by Side Diff: sdk/lib/_internal/pub_generated/test/dependency_computer/cycle_test.dart

Issue 599993004: Don't load transformers that aren't going to be used for an executable. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes Created 6 years, 2 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 library pub_tests; 1 library pub_tests;
2 import '../descriptor.dart' as d; 2 import '../descriptor.dart' as d;
3 import '../test_pub.dart'; 3 import '../test_pub.dart';
4 import 'utils.dart'; 4 import 'utils.dart';
5 void main() { 5 void main() {
6 initConfig(); 6 initConfig();
7 integration( 7 integration(
8 "allows a package dependency cycle that's unrelated to " "transformers", 8 "allows a package dependency cycle that's unrelated to " "transformers",
9 () { 9 () {
10 d.dir(appPath, [d.pubspec({ 10 d.dir(appPath, [d.pubspec({
(...skipping 211 matching lines...) Expand 10 before | Expand all | Expand 10 after
222 'lib', 222 'lib',
223 [ 223 [
224 d.file("myapp.dart", transformer(['foo.dart'])), 224 d.file("myapp.dart", transformer(['foo.dart'])),
225 d.file("foo.dart", "import 'bar.dart';"), 225 d.file("foo.dart", "import 'bar.dart';"),
226 d.file("bar.dart", "import 'myapp.dart';")])]).create(); 226 d.file("bar.dart", "import 'myapp.dart';")])]).create();
227 expectDependencies({ 227 expectDependencies({
228 'myapp': [] 228 'myapp': []
229 }); 229 });
230 }); 230 });
231 } 231 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698