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

Unified Diff: pkg/barback/test/package_graph/lazy_transformer_test.dart

Issue 255483002: Expand barback's notion of dirtiness to understand declaredness. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 8 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: pkg/barback/test/package_graph/lazy_transformer_test.dart
diff --git a/pkg/barback/test/package_graph/lazy_transformer_test.dart b/pkg/barback/test/package_graph/lazy_transformer_test.dart
index 24ae2e5cfcb99b6825cc495e028a5b57bec6143b..5668545ed8352a61a4ae2cb2f79060972d469fd9 100644
--- a/pkg/barback/test/package_graph/lazy_transformer_test.dart
+++ b/pkg/barback/test/package_graph/lazy_transformer_test.dart
@@ -354,6 +354,23 @@ main() {
buildShouldSucceed();
});
+ test("a lazy transformer that generates fewer outputs than it declares is "
+ "forced when a declared but ungenerated output is requested", () {
+ initGraph({"app|foo.txt": "no"}, {"app": [
+ [new LazyCheckContentAndRenameTransformer(
+ "txt", "yes", "out", "done")]
+ ]});
+
+ updateSources(["app|foo.txt"]);
+ expectNoAsset("app|foo.out");
+ buildShouldSucceed();
+
+ modifyAsset("app|foo.txt", "yes");
+ updateSources(["app|foo.txt"]);
+ expectAsset("app|foo.out", "done");
+ buildShouldSucceed();
+ });
+
// Regression tests.
test("a lazy transformer that doesn't apply updates its passed-through asset",

Powered by Google App Engine
This is Rietveld 408576698