Merge pull request #160 from TrySound/sourcemaps-string

Allow specify dest sourcemaps option as string

Author: contra

README.md

@@ -199,7 +199,8 @@ Default: `true` (always overwrite existing files)
 
 ##### `options.sourcemaps`
 
-Enables sourcemap support on files passed through the stream.  Will write inline soucemaps if specified as `true`. Uses [gulp-sourcemaps] under the hood.
+Enables sourcemap support on files passed through the stream.  Will write inline soucemaps if specified as `true`.
+Specifying a `string` is shorthand for the path option. Uses [gulp-sourcemaps] under the hood.
 
 Examples:
 
@@ -211,9 +212,7 @@ vfs.dest('./', {
 
 // Write as files in the same folder
 vfs.dest('./', {
-  sourcemaps: {
-    path: '.'
-  }
+  sourcemaps: '.'
 });
 
 // Any other options are passed through to [gulp-sourcemaps]
@@ -226,7 +225,7 @@ vfs.dest('./', {
 });
 ```
 
-Type: `Boolean` or `Object`
+Type: `Boolean`, `String` or `Object`
 
 Default: `undefined` (do not write sourcemaps)
 

lib/dest/index.js

@@ -34,6 +34,11 @@ function dest(outFolder, opt) {
   if (typeof sourcemapOpt === 'boolean') {
     sourcemapOpt = {};
   }
+  if (typeof sourcemapOpt === 'string') {
+    sourcemapOpt = {
+      path: sourcemapOpt,
+    };
+  }
 
   var mapStream = sourcemaps.write(sourcemapOpt.path, sourcemapOpt);
   var outputStream = duplexify.obj(mapStream, saveStream);

test/dest.js

@@ -103,6 +103,32 @@ describe('dest stream', function() {
     stream.end();
   });
 
+  it('should not explode if the sourcemap option is string', function(done) {
+    var inputPath = path.join(__dirname, './fixtures/test.coffee');
+
+    var expectedFile = new File({
+      base: __dirname,
+      cwd: __dirname,
+      path: inputPath,
+      contents: null,
+    });
+
+    var buffered = [];
+
+    var onEnd = function() {
+      buffered.length.should.equal(1);
+      buffered[0].should.equal(expectedFile);
+      done();
+    };
+
+    var bufferStream = through.obj(dataWrap(buffered.push.bind(buffered)), onEnd);
+
+    var stream = vfs.dest(path.join(__dirname, './out-fixtures/'), { sourcemaps: '.' });
+    stream.pipe(bufferStream);
+    stream.write(expectedFile);
+    stream.end();
+  });
+
   it('should not explode if sourcemap option is an object', function(done) {
     var inputPath = path.join(__dirname, './fixtures/test.coffee');
 

test/not-owned/not-owned.txt

@@ -1 +1 @@
-Something new
+Something new