fixing missing prop(‘outerHTML’) implementation. Added an ‘outerHTML’…

… case to the switch in the prop function, which wraps a clone of in a container element, and sets to that container's HTML (#945)
cheeriojs · Mar 11, 2018 · 2c7a9e1 · 2c7a9e1
1 parent f7e05b5
commit 2c7a9e1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/lib/api/attributes.js b/lib/api/attributes.js
@@ -125,6 +125,9 @@ exports.prop = function (name, value) {
       case 'nodeName':
         property = this[0].name.toUpperCase();
         break;
+      case 'outerHTML':
+        property = this.clone().wrap('<container />').parent().html();
+        break;
       default:
         property = getProp(this[0], name);
     }

diff --git a/test/cheerio.js b/test/cheerio.js
@@ -447,5 +447,14 @@ describe('cheerio', function() {
         expect($b('div').foo).to.be(undefined);
       });
     });
+
+    describe('.prop', function () {
+      describe('("outerHTML")', function () {
+        var outerHtml = '<div><a></a></div>';
+        var $a = $(outerHtml);
+
+        expect($a.prop('outerHTML')).to.be(outerHtml);
+      });
+    });
   });
 });