fix: emit CDATA on empty CDATA section too

lddubeau · Jul 10, 2018 · 95d192f · 95d192f
1 parent 285d859
commit 95d192f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/lib/saxes.js b/lib/saxes.js
@@ -986,9 +986,7 @@ class SaxesParser {
   sCDataEnding2(chunkState) {
     const c = this.getChar(chunkState);
     if (c === ">") {
-      if (this.cdata) {
-        this.emitNode("oncdata", this.cdata);
-      }
+      this.emitNode("oncdata", this.cdata);
       this.cdata = "";
       this.state = S_TEXT;
     }

diff --git a/test/cdata.js b/test/cdata.js
@@ -2,11 +2,12 @@
 
 require(".").test({
   name: "cdata",
-  xml: "<r><![CDATA[ this is character data  ]]></r>",
+  xml: "<r><![CDATA[ this is character data  ]]><![CDATA[]]></r>",
   expect: [
     ["opentagstart", { name: "r", attributes: {} }],
     ["opentag", { name: "r", attributes: {}, isSelfClosing: false }],
     ["cdata", " this is character data  "],
+    ["cdata", ""],
     ["closetag", "r"],
   ],
 });