Fixed closing tag issues (#19)

chrisbottin · Jul 7, 2024 · 18cb361 · 18cb361
1 parent 4ee0ed2
commit 18cb361
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -210,7 +210,7 @@ function element(matchRoot: boolean): XmlParserNodeWrapper<XmlParserElementNode>
             throw new ParsingError('Failed to parse XML', `Closing tag not matching "${closingTag}"`);
         }
     } else {
-        match(/^<\/\s*[\w-:.\u00C0-\u00FF]+>/);
+        match(/^<\/[\w-:.\u00C0-\u00FF]+\s*>/);
     }
 
     return {

diff --git a/test/index.ts b/test/index.ts
@@ -152,7 +152,7 @@ describe('XML Parser', function() {
     });
 
     it('should support weird whitespace', function() {
-        const node = xmlParser('<foo \n\n\nbar\n\n=   \nbaz>\n\nhello world</\n\nfoo>');
+        const node = xmlParser('<foo \n\n\nbar\n\n=   \nbaz>\n\nhello world</foo>');
         assert.deepEqual(node.root, {
             type: 'Element',
             name: 'foo',
@@ -260,6 +260,16 @@ describe('XML Parser', function() {
         });
     });
 
+    it('should support closing tags with tailing whitespaces', function() {
+        const node = xmlParser('<a></a  \n>');
+        assert.deepEqual(node.root, {
+            type: 'Element',
+            name: 'a',
+            attributes: {},
+            children: []
+        });
+    });
+
     it('should support self-closing tags without attributes', function() {
         const node = xmlParser('<a><b>foo</b><b /> <b>bar</b></a>');
         assert.deepEqual(node.root, {