Merge branch 'main' into test/lint

.eslintrc.json

@@ -21,6 +21,13 @@
         "allowImplicit": true
       }
     ],
+    "eqeqeq": [
+      "error",
+      "always",
+      {
+        "null": "ignore"
+      }
+    ],
     "no-lonely-if": 2,
     "no-proto": 2,
     "curly": [2, "multi-line"],

.gitattributes

@@ -1,2 +1,5 @@
 # Enforce Unix newlines
 * text=auto eol=lf
+
+benchmark/documents/* binary
+benchmark/jquery*.js binary

lib/api/attributes.js

@@ -383,10 +383,10 @@ exports.val = function (value) {
       var option = this.find('option:selected');
       if (!option) return;
       if (!querying) {
-        if (this.attr('multiple') == null && typeof value == 'object') {
+        if (this.attr('multiple') == null && typeof value === 'object') {
           return this;
         }
-        if (typeof value != 'object') {
+        if (typeof value !== 'object') {
           value = [value];
         }
         this.find('option').removeAttr('selected');

lib/api/css.js

@@ -38,7 +38,7 @@ exports.css = function (prop, val) {
  * @param {number} [idx] - Optional index within the selection.
  */
 function setCss(el, prop, val, idx) {
-  if (typeof prop == 'string') {
+  if (typeof prop === 'string') {
     var styles = getCss(el);
     if (typeof val === 'function') {
       val = val.call(el, idx, styles[prop]);
@@ -51,7 +51,7 @@ function setCss(el, prop, val, idx) {
     }
 
     el.attribs.style = stringify(styles);
-  } else if (typeof prop == 'object') {
+  } else if (typeof prop === 'object') {
     Object.keys(prop).forEach(function (k) {
       setCss(el, k, prop[k]);
     });

lib/api/manipulation.js

@@ -413,11 +413,7 @@ exports.wrapAll = function (wrapper) {
       wrapper = wrapper.call(this[0]);
     }
 
-    var wrap = this._make(wrapper);
-
-    if (this[0].parent) {
-      wrap = wrap.insertBefore(this[0]);
-    }
+    var wrap = this._make(wrapper).insertBefore(this[0]);
 
     // if html is given as wrapper, wrap may contain text elements
     var elInsertLocation = { children: wrap };

package.json

@@ -39,12 +39,12 @@
     "parse5-htmlparser2-tree-adapter": "^6.0.1"
   },
   "devDependencies": {
-    "@types/node": "^14.14.16",
+    "@types/node": "^14.14.19",
     "benchmark": "^2.1.4",
-    "eslint": "^7.16.0",
+    "eslint": "^7.17.0",
     "eslint-config-prettier": "^7.1.0",
     "eslint-plugin-jest": "^24.1.3",
-    "eslint-plugin-jsdoc": "^30.7.9",
+    "eslint-plugin-jsdoc": "^30.7.13",
     "eslint-plugin-node": "^11.1.0",
     "husky": "^4.3.6",
     "jest": "^26.6.3",

test/__fixtures__/fixtures.js

@@ -23,7 +23,7 @@ exports.divcontainers = [
   '<div class="inner">Fourth</div>',
   '</div>',
   '<div id="new"><div>',
-  '<div><p><em><b></b></em></p></div>',
+  '<div>\n\n<p><em><b></b></em></p>\n\n</div>',
   '</div>',
 ].join('');
 

test/api/manipulation.js

@@ -369,6 +369,12 @@ describe('$(...)', function () {
       expect($container[0].children[0]).toBe($wrap[0]);
     });
 
+    it('(html) : should wrap elements with it', function () {
+      var parent = doc('<p>').wrapAll('<div></div>').parent();
+      expect(parent).toHaveLength(1);
+      expect(parent.is('div')).toBe(true);
+    });
+
     it('(selector) : should find element from dom, wrap elements with it', function () {
       $inner.wrapAll('#new');
       var $container = doc('.container');
@@ -387,6 +393,36 @@ describe('$(...)', function () {
       expect($new[0].parent).toBe($container[0]);
       expect($container[0].children[0]).toBe($new[0]);
     });
+
+    it('(function) : check execution', function () {
+      var $container = doc('.container');
+      var p = $container[0].parent;
+
+      var result = $container.wrapAll(function () {
+        return "<div class='red'><div class='tmp'></div></div>";
+      });
+
+      expect(result.parent()).toHaveLength(1);
+      expect($container.eq(0).parent().parent().is('.red')).toBe(true);
+      expect($container.eq(1).parent().parent().is('.red')).toBe(true);
+      expect($container.eq(0).parent().parent().parent().is(p)).toBe(true);
+    });
+
+    it('(function) : check execution characteristics', function () {
+      var $new = doc('#new');
+      var i = 0;
+
+      doc('no-result').wrapAll(function () {
+        i++;
+        return '';
+      });
+      expect(i).toBeFalsy();
+
+      $new.wrapAll(function (index) {
+        expect(this).toBe($new[0]);
+        expect(index).toBe(undefined);
+      });
+    });
   });
 
   describe('.append', function () {
@@ -797,7 +833,7 @@ describe('$(...)', function () {
 
   describe('.after', function () {
     it('() : should do nothing', function () {
-      expect($('#fruits').after()[0].tagName).toBe('ul');
+      expect($fruits.after()[0].tagName).toBe('ul');
     });
 
     it('(html) : should add element as next sibling', function () {

test/api/traversing.js

@@ -578,6 +578,13 @@ describe('$(...)', function () {
       expect(result).toHaveLength(1);
       expect(result[0].attribs.id).toBe('fruits');
     });
+
+    it('(cheerio object) : should return all parents until body element', function () {
+      var body = $('body')[0];
+      var result = $('.carrot').parentsUntil(body);
+      expect(result).toHaveLength(2);
+      expect(result.eq(0).is('ul#vegetables')).toBe(true);
+    });
   });
 
   describe('.parent', function () {

test/cheerio.js

@@ -226,6 +226,10 @@ describe('cheerio', function () {
     expect($empty.each).toBe(cheerio.prototype.each);
   });
 
+  it('cheerio.html(null) should return a "" string', function () {
+    expect(cheerio.html(null)).toBe('');
+  });
+
   it('should set html(number) as a string', function () {
     var $elem = cheerio('<div>');
     $elem.html(123);

test/parse.js

@@ -210,6 +210,15 @@ describe('parse', function () {
       expect(root.childNodes[0].type).toBe('directive');
     });
 
+    it('should simply return root ', function () {
+      var oldroot = parse(basic, defaultOpts, true);
+      var root = parse(oldroot, defaultOpts, true);
+      expect(root).toBe(oldroot);
+      rootTest(root);
+      expect(root.childNodes).toHaveLength(1);
+      expect(root.childNodes[0].tagName).toBe('html');
+    });
+
     it('should expose the DOM level 1 API', function () {
       var root = parse(
         '<div><a></a><span></span><p></p></div>',