Merge pull request #101 from bem/ignoreEndTags

Rename option 'ignoreClosingTags' to 'ignoreEndTags'

Author: eGavr

lib/HtmlDiff.js

@@ -10,7 +10,7 @@ var diff = require('diff'),
      * @param {String[]} [options.compareAttributesAsJSON]
      * @param {Boolean} [options.ignoreWhitespaces=true]
      * @param {Boolean} [options.ignoreComments]
-     * @param {Boolean} [options.ignoreClosingTags=false]
+     * @param {Boolean} [options.ignoreEndTags=false]
      * @param {Boolean} [options.ignoreDuplicateAttributes=false]
      */
     HtmlDiff = function (options) {

lib/index.js

@@ -22,7 +22,7 @@ HtmlDiff.prototype.tokenize = function (html) {
  * @param {String[]} [options.compareAttributesAsJSON]
  * @param {Boolean} [options.ignoreWhitespaces=true]
  * @param {Boolean} [options.ignoreComments=true]
- * @param {Boolean} [options.ignoreClosingTags=false]
+ * @param {Boolean} [options.ignoreEndTags=false]
  * @param {Boolean} [options.ignoreDuplicateAttributes=false]
  */
 var HtmlDiffer = function (options) {

lib/utils/defaults.js

@@ -31,7 +31,7 @@ var _ = require('lodash');
         ignoreWhitespaces: true,
         ignoreComments: true,
 
-        ignoreClosingTags: false,
+        ignoreEndTags: false,
         ignoreDuplicateAttributes: false
     });
 };

lib/utils/modify.js

@@ -10,7 +10,7 @@ var SimpleApiParser = require('parse5').SimpleApiParser,
  * @param {String[]} [options.compareAttributesAsJSON]
  * @param {Boolean} [options.ignoreWhitespaces=true]
  * @param {Boolean} [options.ignoreComments=true]
- * @param {Boolean} [options.ignoreClosingTags=false]
+ * @param {Boolean} [options.ignoreEndTags=false]
  * @param {Boolean} [options.ignoreDuplicateAttributes=false]
  * returns {String}
  */
@@ -48,7 +48,7 @@ module.exports = function (value, options) {
          * @param {String} tagName
          */
         endTag: function (tagName) {
-            !options.ignoreClosingTags && (modifiedValue += serialize.endTag(tagName));
+            !options.ignoreEndTags && (modifiedValue += serialize.endTag(tagName));
         },
         /**
          * @param {String} text

…/fixtures/first/ignore-closing-tags.html …diff/fixtures/first/ignore-end-tags.htmltest/diff/fixtures/first/ignore-closing-tags.html renamed to test/diff/fixtures/first/ignore-end-tags.html test/diff/fixtures/first/ignore-closing-tags.html renamed to test/diff/fixtures/first/ignore-end-tags.html

@@ -81,9 +81,9 @@ describe('\'isEqual\'', function () {
         htmlDiffer.isEqual(files.html1, files.html2).must.be.true();
     });
 
-    it('must work option \'ignoreClosingTags\'', function () {
-        var htmlDiffer = new HtmlDiffer({ ignoreClosingTags: true }),
-            files = readFiles('ignore-closing-tags');
+    it('must work option \'ignoreEndTags\'', function () {
+        var htmlDiffer = new HtmlDiffer({ ignoreEndTags: true }),
+            files = readFiles('ignore-end-tags');
 
         htmlDiffer.isEqual(files.html1, files.html2).must.be.true();
     });