Merge pull request #81 from TheDancingCode/multiExt

Add multiExt option
hparra · Jun 11, 2018 · 1ad76a5 · 1ad76a5
2 parents 239c2b6 + eda8133
commit 1ad76a5
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -3,12 +3,14 @@
 var Stream = require('stream');
 var Path = require('path');
 
-function gulpRename(obj) {
+function gulpRename(obj, options) {
+
+  options = options || {};
 
   var stream = new Stream.Transform({objectMode: true});
 
   function parsePath(path) {
-    var extname = Path.extname(path);
+    var extname = options.multiExt ? Path.basename(path).slice(Path.basename(path).indexOf('.')) : Path.extname(path);
     return {
       dirname: Path.dirname(path),
       basename: Path.basename(path, extname),

diff --git a/test/path-parsing.spec.js b/test/path-parsing.spec.js
@@ -102,4 +102,15 @@ describe('gulp-rename path parsing', function () {
       helper(srcPattern, obj, null, done);
     });
   });
+
+  describe('multiExt option', function () {
+    it('includes multiple extensions in extname', function (done) {
+      var srcPattern = 'test/fixtures/hello.min.txt';
+      var obj = function (path) {
+        path.extname.should.equal('.min.txt');
+        path.basename.should.equal('hello');
+      };
+      helper(srcPattern, obj, null, done, { multiExt: true });
+    });
+  });
 });
diff --git a/test/spec-helper.js b/test/spec-helper.js
@@ -7,10 +7,10 @@ var Path = require('path'),
   gulp = require('gulp'),
   rename = require('../');
 
-global.helper = function (srcArgs, obj, expectedPath, done) {
+global.helper = function (srcArgs, obj, expectedPath, done, options) {
   var srcPattern = srcArgs.pattern || srcArgs;
   var srcOptions = srcArgs.options || {};
-  var stream = gulp.src(srcPattern, srcOptions).pipe(rename(obj));
+  var stream = gulp.src(srcPattern, srcOptions).pipe(rename(obj, options));
   var count = 0;
   stream.on('error', done);
   stream.on('data', function () {