fix: raise an error if the document does not have a root

lddubeau · Jul 6, 2018 · f2de520 · f2de520
1 parent 7611a85
commit f2de520
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 15 deletions.
diff --git a/lib/saxes.js b/lib/saxes.js
@@ -175,6 +175,9 @@ class SAXParser {
   /* eslint-enable class-methods-use-this */
 
   end() {
+    if (!this.sawRoot) {
+      this.fail("Document must contain a root element");
+    }
     if (this.sawRoot && !this.closedRoot) {
       this.fail("Unclosed root tag");
     }

diff --git a/test/end_empty_stream.js b/test/end_empty_stream.js
@@ -1,9 +1,11 @@
 "use strict";
 
+const { expect } = require("chai");
 const saxes = require("../lib/saxes");
 
 it("end empty stream", () => {
   const saxesStream = saxes.createStream();
   // It musn't throw.
-  saxesStream.end();
+  expect(() => saxesStream.end()).to.throw(Error,
+                                           /Document must contain a root element/);
 });
diff --git a/test/stand-alone-comment.js b/test/stand-alone-comment.js