fix to work with pathnames for windows

Author: mandel59

lib/command-exists.js

@@ -3,6 +3,7 @@
 var exec = require('child_process').exec;
 var execSync = require('child_process').execSync;
 var fs = require('fs');
+var path = require('path');
 var access = fs.access;
 var accessSync = fs.accessSync;
 var constants = fs.constants || fs;
@@ -105,6 +106,18 @@ var cleanInput = function(s) {
   return s;
 }
 
+if (isUsingWindows) {
+  cleanInput = function(s) {
+    var isPathName = /[\\]/.test(s);
+    if (isPathName) {
+      var dirname = '"' + path.dirname(s) + '"';
+      var basename = '"' + path.basename(s) + '"';
+      return dirname + ':' + basename;
+    }
+    return '"' + s + '"';
+  }
+}
+
 module.exports = function commandExists(commandName, callback) {
   var cleanedCommandName = cleanInput(commandName);
   if (!callback && typeof Promise !== 'undefined') {

test/executable-script.cmd

@@ -110,5 +110,16 @@ describe('commandExists', function(){
                     });
             });
         }
+
+        if (isUsingWindows) {
+            it('it should report true if there is an executable file with that name', function(done) {
+                var commandToUse = 'test\\executable-script.cmd'
+                commandExists(commandToUse)
+                    .then(function(command){
+                        expect(command).to.be(commandToUse);
+                        done();
+                    });
+            });
+        }
     });
 });