Don't make lazy transformers eager when an asset is requested.

pkg/barback/test/package_graph/lazy_transformer_test.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS 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 barback.test.package_graph.lazy_asset_test;
 
 import 'package:barback/barback.dart';
 import 'package:barback/src/utils.dart';
 import 'package:scheduled_test/scheduled_test.dart';
 
@@ skipping 118 matching lines @@
       [new LazyRewriteTransformer("one", "two")],
       [new DeclaringRewriteTransformer("two", "three")],
       [new DeclaringRewriteTransformer("three", "four")],
       [new DeclaringRewriteTransformer("four", "five")]
     ]});
     updateSources(["app|foo.one"]);
     expectAsset("app|foo.five", "foo.two.three.four.five");
     buildShouldSucceed();
   });
 
+  test("a lazy transformer followed by a non-lazy transformer is re-run "
+      "eagerly", () {
+    var rewrite = new LazyRewriteTransformer("one", "two");
+    initGraph(["app|foo.one"], {"app": [
+      [rewrite],
+      [new RewriteTransformer("two", "three")]
+    ]});
+
+    updateSources(["app|foo.one"]);
+    expectAsset("app|foo.three", "foo.two.three");
+    buildShouldSucceed();
+
+    updateSources(["app|foo.one"]);
+    buildShouldSucceed();
+
+    expect(rewrite.numRuns, completion(equals(2)));
+  });
+
+  test("a lazy transformer followed by a declaring transformer isn't re-run "
+      "eagerly", () {
+    var rewrite = new LazyRewriteTransformer("one", "two");
+    initGraph(["app|foo.one"], {"app": [
+      [rewrite],
+      [new DeclaringRewriteTransformer("two", "three")]
+    ]});
+
+    updateSources(["app|foo.one"]);
+    expectAsset("app|foo.three", "foo.two.three");
+    buildShouldSucceed();
+
+    updateSources(["app|foo.one"]);
+    buildShouldSucceed();
+
+    expect(rewrite.numRuns, completion(equals(1)));
+  });
+
+  test("a declaring transformer added after a materialized lazy transformer "
+      "is still deferred", () {
+    var lazy = new LazyRewriteTransformer("one", "two");
+    var declaring = new DeclaringRewriteTransformer("two", "three");
+    initGraph(["app|foo.one"], {"app": [[lazy]]});
+
+    updateSources(["app|foo.one"]);
+    expectAsset("app|foo.two", "foo.two");
+    buildShouldSucceed();
+
+    updateTransformers("app", [[lazy], [declaring]]);
+    expectAsset("app|foo.three", "foo.two.three");
+    buildShouldSucceed();
+
+    updateSources(["app|foo.one"]);
+    buildShouldSucceed();
+
+    expect(lazy.numRuns, completion(equals(1)));
+    expect(declaring.numRuns, completion(equals(1)));
+  });
+
   test("a lazy asset works as a cross-package input", () {
     initGraph({
       "pkg1|foo.blub": "foo",
       "pkg2|a.txt": "pkg1|foo.blab"
     }, {"pkg1": [
       [new LazyRewriteTransformer("blub", "blab")],
     ], "pkg2": [
       [new ManyToOneTransformer("txt")]
     ]});
 
     updateSources(["pkg1|foo.blub", "pkg2|a.txt"]);
     expectAsset("pkg2|a.out", "foo.blab");
     buildShouldSucceed();
   });
 
   test("a lazy transformer can consume secondary inputs lazily", () {
     initGraph({
       "app|a.inc": "a",
       "app|a.txt": "a.inc"
     }, {"app": [
       [new LazyManyToOneTransformer("txt")]
     ]});
 
     updateSources(["app|a.inc", "app|a.txt"]);
     expectAsset("app|a.out", "a");
     buildShouldSucceed();
   });
 
-  test("once a lazy transformer is materialized, it runs eagerly afterwards",
-      () {
+  test("after being materialized a lazy transformer is still lazy", () {
     var transformer = new LazyRewriteTransformer("blub", "blab");
     initGraph(["app|foo.blub"], {"app": [[transformer]]});
 
     updateSources(["app|foo.blub"]);
     buildShouldSucceed();
 
     // Request the asset once to force it to be materialized.
     expectAsset("app|foo.blab", "foo.blab");
     buildShouldSucceed();
 
     updateSources(["app|foo.blub"]);
     buildShouldSucceed();
 
-    expect(transformer.numRuns, completion(equals(2)));
+    expect(transformer.numRuns, completion(equals(1)));
+  });
+
+  test("after being materialized a lazy transformer can be materialized again",
+      () {
+    var transformer = new LazyRewriteTransformer("blub", "blab");
+    initGraph(["app|foo.blub"], {"app": [[transformer]]});
+
+    updateSources(["app|foo.blub"]);
+    buildShouldSucceed();
+
+    // Request the asset once to force it to be materialized.
+    expectAsset("app|foo.blab", "foo.blab");
+    buildShouldSucceed();
+
+    modifyAsset("app|foo.blub", "bar");
+    updateSources(["app|foo.blub"]);
+    expectAsset("app|foo.blab", "bar.blab");
+    buildShouldSucceed();
   });
 
   test("an error emitted in a lazy transformer's declareOutputs method is "
       "caught and reported", () {
     initGraph(["app|foo.txt"], {"app": [
       [new LazyBadTransformer("app|foo.out")]
     ]});
 
     updateSources(["app|foo.txt"]);
     buildShouldFail([isTransformerException(equals(LazyBadTransformer.ERROR))]);
@@ skipping 80 matching lines @@
     resumeProvider();
     expectAsset("app|foo.txt", "foo");
     buildShouldSucceed();
 
     modifyAsset("app|foo.txt", "bar");
     updateSources(["app|foo.txt"]);
     expectAsset("app|foo.txt", "bar");
     buildShouldSucceed();
   });
 }