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

Unified Diff: sdk/lib/_internal/pub/test/get/cache_transformed_dependency_test.dart

Issue 576333002: Precompile immutable dependencies on "pub get". (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Code review changes Created 6 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 side-by-side diff with in-line comments
Download patch
Index: sdk/lib/_internal/pub/test/get/cache_transformed_dependency_test.dart
diff --git a/sdk/lib/_internal/pub/test/get/cache_transformed_dependency_test.dart b/sdk/lib/_internal/pub/test/get/cache_transformed_dependency_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..86c3baa4b94f2e06d8a37d2922a9e0ba8168f62e
--- /dev/null
+++ b/sdk/lib/_internal/pub/test/get/cache_transformed_dependency_test.dart
@@ -0,0 +1,300 @@
+// Copyright (c) 2014, the Dart project authors. Please see the AUTHORS d.file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library pub_tests;
+
+import 'package:scheduled_test/scheduled_test.dart';
+
+import '../descriptor.dart' as d;
+import '../test_pub.dart';
+import '../serve/utils.dart';
+
+const MODE_TRANSFORMER = """
+import 'dart:async';
+
+import 'package:barback/barback.dart';
+
+class ModeTransformer extends Transformer {
+ final BarbackSettings _settings;
+
+ ModeTransformer.asPlugin(this._settings);
+
+ String get allowedExtensions => '.dart';
+
+ void apply(Transform transform) {
+ return transform.primaryInput.readAsString().then((contents) {
+ transform.addOutput(new Asset.fromString(
+ transform.primaryInput.id,
+ contents.replaceAll("MODE", _settings.mode.name)));
+ });
+ }
+}
+""";
+
+main() {
+ initConfig();
+
+ integration("caches a transformed dependency", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ deps: {'barback': 'any'},
+ pubspec: {'transformers': ['foo']},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", replaceTransformer("Hello", "Goodbye")),
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+
+ pubGet(output: contains("Precompiled foo."));
+
+ d.dir(appPath, [
+ d.dir(".pub/deps/debug/foo/lib", [
+ d.file("foo.dart", "final message = 'Goodbye!';")
+ ])
+ ]).validate();
+ });
+
+ integration("caches a dependency transformed by its dependency", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ deps: {'bar': '1.2.3'},
+ pubspec: {'transformers': ['bar']},
+ contents: [
+ d.dir("lib", [
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+
+ builder.serve("bar", "1.2.3",
+ deps: {'barback': 'any'},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", replaceTransformer("Hello", "Goodbye"))
+ ])
+ ]);
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+
+ pubGet(output: contains("Precompiled foo."));
+
+ d.dir(appPath, [
+ d.dir(".pub/deps/debug/foo/lib", [
+ d.file("foo.dart", "final message = 'Goodbye!';")
+ ])
+ ]).validate();
+ });
+
+ integration("doesn't cache an untransformed dependency", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ contents: [
+ d.dir("lib", [
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+
+ pubGet(output: isNot(contains("Precompiled foo.")));
+
+ d.dir(appPath, [d.nothing(".pub/deps")]).validate();
+ });
+
+ integration("recaches when the dependency is updated", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ deps: {'barback': 'any'},
+ pubspec: {'transformers': ['foo']},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", replaceTransformer("Hello", "Goodbye")),
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+
+ builder.serve("foo", "1.2.4",
+ deps: {'barback': 'any'},
+ pubspec: {'transformers': ['foo']},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", replaceTransformer("Hello", "See ya")),
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+
+ pubGet(output: contains("Precompiled foo."));
+
+ d.dir(appPath, [
+ d.dir(".pub/deps/debug/foo/lib", [
+ d.file("foo.dart", "final message = 'Goodbye!';")
+ ])
+ ]).validate();
+
+ // Upgrade to the new version of foo.
+ d.appDir({"foo": "1.2.4"}).create();
+
+ pubGet(output: contains("Precompiled foo."));
+
+ d.dir(appPath, [
+ d.dir(".pub/deps/debug/foo/lib", [
+ d.file("foo.dart", "final message = 'See ya!';")
+ ])
+ ]).validate();
+ });
+
+ integration("recaches when a transitive dependency is updated", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ deps: {
+ 'barback': 'any',
+ 'bar': 'any'
+ },
+ pubspec: {'transformers': ['foo']},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", replaceTransformer("Hello", "Goodbye")),
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+
+ builder.serve("bar", "5.6.7");
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+ pubGet(output: contains("Precompiled foo."));
+
+ servePackages((builder) => builder.serve("bar", "6.0.0"));
+ pubUpgrade(output: contains("Precompiled foo."));
+ });
+
+ integration("doesn't recache when an unrelated dependency is updated", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ deps: {'barback': 'any'},
+ pubspec: {'transformers': ['foo']},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", replaceTransformer("Hello", "Goodbye")),
+ d.file("foo.dart", "final message = 'Hello!';")
+ ])
+ ]);
+
+ builder.serve("bar", "5.6.7");
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+ pubGet(output: contains("Precompiled foo."));
+
+ servePackages((builder) => builder.serve("bar", "6.0.0"));
+ pubUpgrade(output: isNot(contains("Precompiled foo.")));
+ });
+
+ integration("caches the dependency in debug mode", () {
+ servePackages((builder) {
+ builder.serveRepoPackage('barback');
+
+ builder.serve("foo", "1.2.3",
+ deps: {'barback': 'any'},
+ pubspec: {'transformers': ['foo']},
+ contents: [
+ d.dir("lib", [
+ d.file("transformer.dart", MODE_TRANSFORMER),
+ d.file("foo.dart", "final mode = 'MODE';")
+ ])
+ ]);
+ });
+
+ d.appDir({"foo": "1.2.3"}).create();
+
+ pubGet(output: contains("Precompiled foo."));
+
+ d.dir(appPath, [
+ d.dir(".pub/deps/debug/foo/lib", [
+ d.file("foo.dart", "final mode = 'debug';")
+ ])
+ ]).validate();
+ });
+
+ // TODO(nweiz): Implement this.
+ // integration("loads code from the cache", () {
+ // servePackages((builder) {
+ // builder.serveRepoPackage('barback');
+ //
+ // builder.serve("foo", "1.2.3",
+ // deps: {'barback': 'any'},
+ // pubspec: {'transformers': ['foo']},
+ // contents: [
+ // d.dir("lib", [
+ // d.file("transformer.dart", replaceTransformer("Hello", "Goodbye")),
+ // d.file("foo.dart", "final message = 'Hello!';")
+ // ])
+ // ]);
+ // });
+ //
+ // d.dir(appPath, [
+ // d.appPubspec({"foo": "1.2.3"}),
+ // d.dir('bin', [
+ // d.file('script.dart', """
+ // import 'package:foo/foo.dart';
+ //
+ // void main() => print(message);""")
+ // ])
+ // ]).create();
+ //
+ // pubGet(output: contains("Precompiled foo."));
+ //
+ // d.dir(appPath, [
+ // d.dir(".pub/deps/debug/foo/lib", [
+ // d.file("foo.dart", "final message = 'Modified!';")
+ // ])
+ // ]).create();
+ //
+ // var pub = pubRun(args: ["script"]);
+ // pub.stdout.expect("Modified!");
+ // pub.shouldExit();
+ // });
+}
+
+String replaceTransformer(String input, String output) {
+ return """
+import 'dart:async';
+
+import 'package:barback/barback.dart';
+
+class ReplaceTransformer extends Transformer {
+ ReplaceTransformer.asPlugin();
+
+ String get allowedExtensions => '.dart';
+
+ Future apply(Transform transform) {
+ return transform.primaryInput.readAsString().then((contents) {
+ transform.addOutput(new Asset.fromString(
+ transform.primaryInput.id,
+ contents.replaceAll("$input", "$output")));
+ });
+ }
+}
+""";
+}
« no previous file with comments | « sdk/lib/_internal/pub/lib/src/utils.dart ('k') | sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698