fix: self closing tag should not be combined - close #158 (#196)

* fix: self closing tag should not be combined - close #158

* fix: open tag index could be -1

Author: JounQin

packages/eslint-mdx/src/traverse.ts

@@ -76,7 +76,18 @@ export class Traverse {
           }
 
           if (!offset) {
-            acc.push(this.combineLeftJsxNodes(jsxNodes))
+            // fix #158
+            const firstOpenTagIndex = jsxNodes.findIndex(node =>
+              isOpenTag(node.value as string),
+            )
+            if (firstOpenTagIndex === -1) {
+              acc.push(...jsxNodes)
+            } else {
+              acc.push(...jsxNodes.slice(0, firstOpenTagIndex))
+              acc.push(
+                this.combineLeftJsxNodes(jsxNodes.slice(firstOpenTagIndex)),
+              )
+            }
             jsxNodes.length = 0
           }
         }

test/no-unescaped-entities.test.ts

@@ -19,6 +19,15 @@ ruleTester.run('no-unescaped-entities', noUnescapedEntities, {
       parserOptions,
       filename,
     },
+    {
+      // #158
+      code: `<YouTube youTubeId="dQw4w9WgXcQ" />
+      <Aside>I chose this video to test my theme. I did this to myself</Aside>
+      `,
+      parser,
+      parserOptions,
+      filename,
+    },
   ],
   invalid: [
     {