add appveyor to test on windows & get tests to actually pass

Author: phated

appveyor.yml

@@ -0,0 +1,24 @@
+# http://www.appveyor.com/docs/appveyor-yml
+# http://www.appveyor.com/docs/lang/nodejs-iojs
+
+environment:
+  matrix:
+    # node.js
+    - nodejs_version: "0.10"
+    - nodejs_version: "0.12"
+    - nodejs_version: "4"
+    - nodejs_version: "5"
+
+install:
+  - ps: Install-Product node $env:nodejs_version
+  - npm install
+
+test_script:
+  - node --version
+  - npm --version
+  - cmd: npm test
+
+build: off
+
+# build version format
+version: "{build}"

lib/dest/writeContents/writeDir.js

@@ -6,7 +6,11 @@ var mkdirp = require('mkdirp');
 var fo = require('../../fileOperations');
 
 function writeDir(writePath, file, written) {
-  mkdirp(writePath, file.stat.mode, onMkdirp);
+  var mkdirpOpts = {
+    mode: file.stat.mode,
+    fs: fs,
+  };
+  mkdirp(writePath, mkdirpOpts, onMkdirp);
 
   function onMkdirp(mkdirpErr) {
     if (mkdirpErr) {

lib/dest/writeContents/writeStream.js

@@ -9,7 +9,6 @@ function writeStream(writePath, file, written) {
   var opt = {
     mode: file.stat.mode,
     flag: file.flag,
-    autoClose: false,
   };
 
   var outStream = fs.createWriteStream(writePath, opt);

lib/fileOperations.js

@@ -69,8 +69,18 @@ function getTimesDiff(fsStat, vinylStat) {
 }
 
 function isOwner(fsStat) {
+  var hasGetuid = (typeof process.getuid === 'function');
+  var hasGeteuid = (typeof process.geteuid === 'function');
+
+  // If we don't have either, assume we don't have permissions.
+  // This should only happen on Windows.
+  // Windows basically noops fchmod and errors on futimes called on directories.
+  if (!hasGeteuid && !hasGetuid) {
+    return false;
+  }
+
   var uid;
-  if (typeof process.geteuid === 'function') {
+  if (hasGeteuid) {
     uid = process.geteuid();
   } else {
     uid = process.getuid();

lib/prepareWrite.js

@@ -54,7 +54,11 @@ function prepareWrite(outFolder, file, opt, cb) {
   file.path = writePath;
 
   // Mkdirp the folder the file is going in
-  mkdirp(writeFolder, options.dirMode, function(err) {
+  var mkdirpOpts = {
+    mode: options.dirMode,
+    fs: fs,
+  };
+  mkdirp(writeFolder, mkdirpOpts, function(err) {
     if (err) {
       return cb(err);
     }

lib/src/index.js

@@ -1,5 +1,6 @@
 'use strict';
 
+var path = require('path');
 var assign = require('object-assign');
 var through2 = require('through2');
 var gs = require('glob-stream');
@@ -13,6 +14,21 @@ var isValidGlob = require('is-valid-glob');
 var getContents = require('./getContents');
 var resolveSymlinks = require('./resolveSymlinks');
 
+function normalizePath(options) {
+
+  function normalize(globFile, enc, cb) {
+    // TODO: probably move this somewhere
+    // Ref https://github.com/gulpjs/vinyl/issues/80
+    var normalizedFile = assign({}, globFile, {
+      path: path.normalize(globFile.path),
+    });
+
+    cb(null, normalizedFile);
+  }
+
+  return through2.obj(options, normalize);
+}
+
 function createFile(globFile, enc, cb) {
   cb(null, new File(globFile));
 }
@@ -36,6 +52,7 @@ function src(glob, opt) {
   var globStream = gs.create(glob, options);
 
   var outputStream = globStream
+    .pipe(normalizePath(options))
     .pipe(resolveSymlinks(options))
     .pipe(through2.obj(opt, createFile));