fix: handle classIgnore for img with no class attribute

dmnsgn · dmnsgn · commit 3233f0f5ee1e · 2020-05-29T10:39:14.000+01:00
- default classes to [] even when checking for attributes
- update test
diff --git a/lib/index.js b/lib/index.js
@@ -20,7 +20,7 @@ module.exports = function (options) {
   return function posthtmlWebp (tree) {
     tree.match([{ tag: 'img' }, { tag: 'amp-img' }], function (imgNode) {
       if (imgNode.skip) return imgNode
-      var classes = imgNode.attrs && imgNode.attrs.class && (imgNode.attrs.class.split(' ') || [])
+      var classes = (imgNode.attrs && imgNode.attrs.class && imgNode.attrs.class.split(' ')) || []
       var extension = imgNode.attrs.src.split('.').pop()
       var isIgnoredByClass = options.classIgnore.filter(className => classes.includes(className)).length > 0
       var isIgnoredByExtension = options.extensionIgnore.filter(fileExtension => fileExtension === extension).length > 0
diff --git a/test/fixtures/ignore.expected.html b/test/fixtures/ignore.expected.html
@@ -1,6 +1,7 @@
 <!doctype html>
 <html>
-    <body>
-        <img class="ignore-class" srcset="photo-lg.jpg 1000w, photo-md.jpg 500w, photo-sm.jpg 250w" src="photo.png">
-    </body>
+	<body>
+		<img class="ignore-class" srcset="photo-lg.jpg 1000w, photo-md.jpg 500w, photo-sm.jpg 250w" src="photo.png">
+		<picture><source type="image/webp" srcset="photo.png.webp"><img src="photo.png"></picture>
+	</body>
 </html>
diff --git a/test/fixtures/ignore.html b/test/fixtures/ignore.html
@@ -1,6 +1,7 @@
 <!doctype html>
 <html>
-    <body>
-        <img class="ignore-class" srcset="photo-lg.jpg 1000w, photo-md.jpg 500w, photo-sm.jpg 250w" src="photo.png">
-    </body>
+	<body>
+		<img class="ignore-class" srcset="photo-lg.jpg 1000w, photo-md.jpg 500w, photo-sm.jpg 250w" src="photo.png">
+		<img src="photo.png">
+	</body>
 </html>
