Merge pull request #591 from sindresorhus/fix-010-watch-test

Fix watcher test with Node 0.10

Author: novemberborn

.travis.yml

@@ -8,6 +8,3 @@ before_install:
   - 'npm install -g npm@latest'
 after_success:
   - '[ -z "$COVERALLS_REPO_TOKEN" ] && tap --coverage-report=text-lcov | ./node_modules/.bin/coveralls'
-matrix:
-  allow_failures:
-    - node_js: '0.10'

package.json

@@ -134,7 +134,7 @@
     "cli-table2": "^0.1.9",
     "coveralls": "^2.11.4",
     "delay": "^1.3.0",
-    "get-stream": "^1.1.0",
+    "get-stream": "^2.0.0",
     "git-branch": "^0.3.0",
     "inquirer": "^0.11.1",
     "lolex": "^1.4.0",

test/cli.js

@@ -141,12 +141,9 @@ test('watcher works', function (t) {
 
 	var buffer = '';
 	var passedFirst = false;
-	// Pause the stream before attaching the 'data' listener. execCli() uses
-	// get-stream which read()s from the stream. The test just needs to piggyback
-	// on that without switching the stream to flowing mode.
-	child.stderr.pause().on('data', function (str) {
+	child.stderr.on('data', function (str) {
 		buffer += str;
-		if (/1 test passed/.test(str)) {
+		if (/1 test passed/.test(buffer)) {
 			if (!passedFirst) {
 				touch.sync(path.join(__dirname, 'fixture/watcher/test.js'));
 				buffer = '';