Fix a barback bug where a closed stream could get an event.

pkg/barback/test/package_graph/add_remove_transform_test.dart

 // Copyright (c) 2013, 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.transform_test;
 
 import 'package:barback/src/utils.dart';
 import 'package:scheduled_test/scheduled_test.dart';
 
 import '../utils.dart';
@@ skipping 342 matching lines @@
     buildShouldSucceed();
 
     removeSources(["app|foo.txt"]);
     expectNoAsset("app|foo.out");
     buildShouldSucceed();
 
     updateSources(["app|foo.txt"]);
     expectAsset("app|foo.out", "foo.mid.out");
     buildShouldSucceed();
   });
+
+  // Regression test for issue 19540.
+  solo_test("a phase is removed and then one of its inputs is updated", () {

[Bob Nystrom, 2014/06/19 21:07:29]

Don't solo! :)

[nweiz, 2014/06/19 21:10:53]

I dunno, I think this is probably the only really important test in this file.

+    // Have an empty first phase because the first phase is never removed.
+    initGraph(["app|foo.txt"], {
+      "app": [[], [new RewriteTransformer("txt", "out")]]
+    });
+
+    updateSources(["app|foo.txt"]);
+    expectAsset("app|foo.out", "foo.out");
+    buildShouldSucceed();
+
+    // First empty both phases. This allows the second phase to be considered
+    // idle even when its transformer is no longer running.
+    updateTransformers("app", [[], []]);
+    buildShouldSucceed();
+
+    // Now remove the second phase. It should unsubscribe from its input's
+    // events.
+    updateTransformers("app", [[]]);
+    buildShouldSucceed();
+
+    // Update the input. With issue 19540, this would cause the removed phase to
+    // try to update its status, which would crash.
+    updateSources(["app|foo.txt"]);
+    buildShouldSucceed();
+  });
 }