Update: Improve tests for Windows

Author: heikki

index.js

@@ -29,7 +29,7 @@ var cli = new Liftoff({
 var failed = false;
 process.once('exit', function(code) {
   if (code === 0 && failed) {
-    process.exit(1);
+    exit(1);
   }
 });
 
@@ -78,7 +78,7 @@ function handleArguments(env) {
     if (env.modulePackage && typeof env.modulePackage.version !== 'undefined') {
       gutil.log('Local version', env.modulePackage.version);
     }
-    process.exit(0);
+    exit(0);
   }
 
   if (!env.modulePath) {
@@ -87,12 +87,12 @@ function handleArguments(env) {
       chalk.magenta(tildify(env.cwd))
     );
     gutil.log(chalk.red('Try running: npm install gulp'));
-    process.exit(1);
+    exit(1);
   }
 
   if (!env.configPath) {
     gutil.log(chalk.red('No gulpfile found'));
-    process.exit(1);
+    exit(1);
   }
 
   // check for semver difference between cli and local installation
@@ -207,6 +207,17 @@ function logEvents(gulpInst) {
       chalk.red('Task \'' + err.task + '\' is not in your gulpfile')
     );
     gutil.log('Please check the documentation for proper gulpfile formatting');
-    process.exit(1);
+    exit(1);
   });
 }
+
+// fix stdout truncation on windows
+function exit(code) {
+  if (process.platform === 'win32' && process.stdout.bufferSize) {
+    process.stdout.once('drain', function() {
+      process.exit(code);
+    });
+    return;
+  }
+  process.exit(code);
+}

test/flags-task-simple.js

@@ -6,12 +6,12 @@ var code = require('code');
 var fs = require('fs');
 var child = require('child_process');
 
-var output = fs.readFileSync(__dirname + '/expected/flags-tasks-simple.txt', 'utf8');
+var output = fs.readFileSync(__dirname + '/expected/flags-tasks-simple.txt', 'utf8').replace(/\r\n/g, '\n');
 
 lab.experiment('flag: --tasks-simple', function () {
 
   lab.test('prints the task list', function (done) {
-    child.exec(__dirname + '/../index.js --tasks-simple --cwd ./test', function(err, stdout) {
+    child.exec('node ' + __dirname + '/../index.js --tasks-simple --cwd ./test', function(err, stdout) {
       code.expect(stdout).to.equal(output);
       done(err);
     });

test/flags-tasks.js

@@ -8,7 +8,8 @@ var child = require('child_process');
 lab.experiment('flag: --tasks', function () {
 
   lab.test('prints the task list', function (done) {
-    child.exec(__dirname + '/../index.js --tasks --cwd ./test', function(err, stdout) {
+    child.exec('node ' + __dirname + '/../index.js --tasks --cwd ./test', function(err, stdout) {
+      stdout = stdout.replace(/\\/g, '/');
       code.expect(stdout).to.contain('/gulp-cli/test');
       code.expect(stdout).to.contain('├── test1');
       code.expect(stdout).to.contain('├── test2');

test/flags-version.js

@@ -11,7 +11,7 @@ var gulpVersion = require('gulp/package.json').version;
 lab.experiment('flag: --version', function () {
 
   lab.test('prints the task list', function (done) {
-    child.exec(__dirname + '/../index.js --version --cwd ./test', function(err, stdout) {
+    child.exec('node ' + __dirname + '/../index.js --version --cwd ./test', function(err, stdout) {
       code.expect(stdout).to.contain('CLI version ' + cliVersion);
       code.expect(stdout).to.contain('Local version ' + gulpVersion);
       done(err);