Close #491 PR: Srcset regex should match any valid image candidate st…

…ring.

lib/fileprocessor.js

@@ -73,7 +73,7 @@ var _defaultPatterns = {
       'Update the HTML with the new image filenames for src links'
     ],
     [
-      /<(?:img|source)[^\>]*[^\>\S]+srcset=['"]([^"'\s]+)(?:\s\d[mx])["']/gm,
+      /<(?:img|source)[^\>]*[^\>\S]+srcset=['"]([^"'\s]+)\s*?(?:\s\d*?[w])?(?:\s\d*?[x])?\s*?["']/gm,
       'Update the HTML with the new image filenames for srcset links'
     ],
     [

test/test-fileprocessor.js

@@ -429,15 +429,53 @@ describe('FileProcessor', function () {
     });
 
     it('should replace img reference in srcset', function () {
+      var content = '<img srcset="image.png" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<img srcset="' + filemapping['app/image.png'] + '" />');
+    });
+
+    it('should replace img reference in srcset with pixel density descriptor', function () {
       var content = '<img srcset="[email protected] 2x" />';
       var replaced = fp.replaceWithRevved(content, ['app']);
       assert.equal(replaced, '<img srcset="' + filemapping['app/[email protected]'] + ' 2x" />');
+    });
 
-      content = '<source srcset="[email protected] 2x" />';
-      replaced = fp.replaceWithRevved(content, ['app']);
+    it('should replace img reference in srcset with width descriptor', function () {
+      var content = '<img srcset="image.png 200w" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<img srcset="' + filemapping['app/image.png'] + ' 200w" />');
+    });
+
+    it('should replace img reference in srcset with pixel density descriptor and with width descriptor', function () {
+      var content = '<img srcset="image.png 200w 2x" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<img srcset="' + filemapping['app/image.png'] + ' 200w 2x" />');
+    });
+
+    it('should replace source reference in srcset', function () {
+      var content = '<source srcset="image.png" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<source srcset="' + filemapping['app/image.png'] + '" />');
+    });
+
+    it('should replace source reference in srcset with pixel density descriptor', function () {
+      var content = '<source srcset="[email protected] 2x" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
       assert.equal(replaced, '<source srcset="' + filemapping['app/[email protected]'] + ' 2x" />');
     });
 
+    it('should replace source reference in srcset with width descriptor', function () {
+      var content = '<source srcset="image.png 200w" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<source srcset="' + filemapping['app/image.png'] + ' 200w" />');
+    });
+
+    it('should replace source reference in srcset with pixel density descriptor and with width descriptor', function () {
+      var content = '<source srcset="image.png 200w 2x" />';
+      var replaced = fp.replaceWithRevved(content, ['app']);
+      assert.equal(replaced, '<source srcset="' + filemapping['app/image.png'] + ' 200w 2x" />');
+    });
+
     it('should replace svg src reference with revved version for img tag', function () {
       var content = '<img src="image.svg#symbol">';
       var replaced = fp.replaceWithRevved(content, ['app']);