Merge pull request #17 from dannyfallon/df/add-gzipped-files-to-dist

Add the gzipped file to the context's distFiles when keep is true

Author: Aaron Chambers

index.js

@@ -53,6 +53,13 @@ module.exports = {
         return this._gzipFiles(distDir, distFiles, filePattern, keep)
           .then(function(gzippedFiles) {
             self.log('gzipped ' + gzippedFiles.length + ' files ok', { verbose: true });
+            if (keep) {
+              self.log('keep is enabled, added gzipped files to `context.distFiles`', { verbose: true });
+              return {
+                distFiles: gzippedFiles,
+                gzippedFiles: gzippedFiles
+              }
+            }
             return { gzippedFiles: gzippedFiles };
           })
           .catch(this._errorMessage.bind(this));

tests/unit/index-nodetest.js

@@ -155,15 +155,30 @@ describe('gzip plugin', function() {
         });
     });
 
-    it('gzips the matching files with .gz suffix when keep is enabled', function(done) {
-      context.config.gzip.keep = true;
-      return assert.isFulfilled(plugin.willUpload(context))
-        .then(function(result) {
-          assert.deepEqual(result, { gzippedFiles: ['assets/foo.js.gz'] });
-          done();
-        }).catch(function(reason){
-          done(reason);
-        });
+    describe('when keep is enabled', function() {
+      beforeEach(function() {
+        context.config.gzip.keep = true;
+      });
+
+      it('gzips the matching files with .gz suffix', function(done) {
+        return assert.isFulfilled(plugin.willUpload(context))
+          .then(function(result) {
+            assert.deepEqual(result.gzippedFiles, ['assets/foo.js.gz']);
+            done();
+          }).catch(function(reason){
+            done(reason);
+          });
+      });
+
+      it('adds the gzipped files to the distFiles', function(done) {
+        return assert.isFulfilled(plugin.willUpload(context))
+          .then(function(result) {
+            assert.include(result.distFiles, 'assets/foo.js.gz');
+            done();
+          }).catch(function(reason){
+            done(reason);
+          });
+      });
     });
   });
 });