Fix crash with 'compareAttributesAsJSON' option concerning the invalid input

Author: eGavr

lib/utils/utils.js

@@ -95,20 +95,24 @@ function sortAttrsValues(attrs, compareAttributesAsJSON) {
     }
 
     /**
-     * Parses the given JSON in HTML attribute
+     * Parses the given in HTML attribute JSON
      * @param {String} val
      * @param {Boolean} [isClick]
      * @returns {Object}
      */
     function _parseAttr(val, isClick) {
-        if (isClick) {
-            /*jshint evil: true */
-            var fn = Function(val);
+        try {
+            if (isClick) {
+                /*jshint evil: true */
+                var fn = Function(val);
 
-            return fn ? fn() : {};
-        }
+                return fn ? fn() : {};
+            }
 
-        return JSON.parse(val);
+            return JSON.parse(val);
+        } catch (err) {
+            return undefined;
+        }
     }
 
     _.forEach(compareAttributesAsJSON, function (attr) {
@@ -125,10 +129,10 @@ function sortAttrsValues(attrs, compareAttributesAsJSON) {
 
         _.forEach(attrIndexes, function (index) {
             attrValue = _parseAttr(attrs[index].value, isFunction);
-            attrValue = _sortObj(attrValue);
-            attrValue = JSON.stringify(attrValue);
 
-            attrs[index].value = (isFunction ? 'return ' : '') + attrValue;
+            attrValue && (attrValue = JSON.stringify(_sortObj(attrValue)));
+
+            attrs[index].value = (isFunction && attrValue ? 'return ' : '') + attrValue;
         });
     });
 

test/diff/fixtures/first/invalid-function.html

@@ -0,0 +1 @@
+<button onclick="{&quot;button&quot;:{}};"></button>

test/diff/fixtures/first/invalid-json.html

@@ -0,0 +1 @@
+<button data-bem="{&quot;button&quot;:{invalid}}"></button>

test/diff/fixtures/second/invalid-function.html

@@ -0,0 +1 @@
+<button onclick="undefined"></button>

test/diff/fixtures/second/invalid-json.html

@@ -0,0 +1 @@
+<button data-bem="undefined"></button>

test/diff/getDiffText.js

@@ -2,7 +2,7 @@ var diffLoger = require('../../lib/logger');
 
 require('colors');
 
-describe('\'diffHtml\'', function () {
+describe('\'getDiffText\'', function () {
     it('must return an empty string', function () {
         var inp = [ {
                 value: 'text',

test/diff/isEqual.js

@@ -47,12 +47,19 @@ describe('\'isEqual\'', function () {
     });
 
     it('must sort values of attributes as JSON when the content is not a function', function () {
-        var htmlDiffer = new HtmlDiffer({ compareAttributesAsJSON: [ 'a', { name: 'b', isFunction: false }] }),
+        var htmlDiffer = new HtmlDiffer({ compareAttributesAsJSON: ['a', { name: 'b', isFunction: false }] }),
             files = readFiles('sort-values-in-json-format');
 
         htmlDiffer.isEqual(files.html1, files.html2).must.be.true();
     });
 
+    it('must handle invalid JSON', function () {
+        var htmlDiffer = new HtmlDiffer({ compareAttributesAsJSON: ['data-bem'] }),
+            files = readFiles('invalid-json');
+
+        htmlDiffer.isEqual(files.html1, files.html2).must.be.true();
+    });
+
     it('must sort values of attributes as JSON when the content is a function', function () {
         var options = {
                 compareAttributesAsJSON: [
@@ -66,6 +73,14 @@ describe('\'isEqual\'', function () {
         htmlDiffer.isEqual(files.html1, files.html2).must.be.true();
     });
 
+    it('must handle invalid function', function () {
+        var options = { compareAttributesAsJSON: [{ name: 'onclick', isFunction: true }] },
+            htmlDiffer = new HtmlDiffer(options),
+            files = readFiles('invalid-function');
+
+        htmlDiffer.isEqual(files.html1, files.html2).must.be.true();
+    });
+
     it('must work option \'ignoreAttributes\'', function () {
         var htmlDiffer = new HtmlDiffer({ ignoreAttributes: ['id', 'for'] }),
             files = readFiles('ignore-attributes');