Fix: correctly emit 'css stream'

Previously, the event would only get emitted on the first run. This is a
problem for watchify-like environments.

Note: this is a breaking change.

Author: joeybaker

README.md

@@ -56,7 +56,7 @@ b.plugin(require('css-modulesify'), {
 });
 
 var bundle = b.bundle()
-bundle.on('css stream', function (css) {
+b.on('css stream', function (css) {
   css.pipe(fs.createWriteStream('mycss.css'));
 });
 ```
@@ -70,7 +70,7 @@ bundle.on('css stream', function (css) {
 - `generateScopedName`: (API only) a function to override the default behaviour of creating locally scoped classnames.
 
 ### Events
-- `b.bundle().on('css stream', callback)` The callback is called with a readable stream containing the compiled CSS. You can write this to a file.
+- `b.on('css stream', callback)` The callback is called with a readable stream containing the compiled CSS. You can write this to a file.
 
 ## Using CSS Modules on the backend
 

index.js

@@ -184,7 +184,7 @@ module.exports = function (browserify, options) {
     compiledCssStream = new ReadableStream();
     compiledCssStream._read = function () {};
 
-    bundle.emit('css stream', compiledCssStream);
+    browserify.emit('css stream', compiledCssStream);
 
     bundle.on('end', function () {
       // Combine the collected sources into a single CSS file

tests/stream-output.js

@@ -28,32 +28,42 @@ tape('stream output', function (t) {
   t.plan(cssFilesTotal * 2 + 1);
 
   var cssFilesCount = 0;
-  browserify(path.join(simpleCaseDir, 'main.js'))
+  var b = browserify(path.join(simpleCaseDir, 'main.js'));
+
+  b
     .plugin(cssModulesify, {
       rootDir: path.join(simpleCaseDir)
     })
     .on('error', t.error)
-    .bundle(function noop () {})
-      .on('css stream', function (stream) {
-        stream
-          .on('data', function onData (css) {
-            var cssString = css.toString();
-            // just get the first class name, use that as an id
-            var cssId = cssString.split('\n')[0].split(' ')[0];
-
-            t.ok(
-              ++cssFilesCount <= cssFilesTotal
-            , 'emits data for ' + cssId
-            );
-
-            t.ok(
-              cssString.indexOf('._styles') === 0
-            , 'emits compiled css for ' + cssId
-            );
-          })
-          .on('end', function onEnd () {
-            t.pass('ends the stream');
-          })
-          .on('error', t.error);
-      });
+    .bundle(function noop () {});
+
+  b
+    .once('css stream', function (stream) {
+      stream
+        .on('data', function onData (css) {
+          var cssString = css.toString();
+          // just get the first class name, use that as an id
+          var cssId = cssString.split('\n')[0].split(' ')[0];
+
+          t.ok(
+            ++cssFilesCount <= cssFilesTotal
+          , 'emits data for ' + cssId
+          );
+
+          t.ok(
+            cssString.indexOf('._styles') === 0
+          , 'emits compiled css for ' + cssId
+          );
+        })
+        .on('end', function onEnd () {
+          t.pass('ends the stream');
+
+          b.bundle(function noop () {});
+
+          b.once('css stream', function (stream2) {
+            t.ok(stream2, 'registers a second event for a CSS stream');
+          });
+        })
+        .on('error', t.error);
+    });
 });