Fixed code style, added more tests

jsx-eslint · Jan 11, 2024 · 592484c · 592484c
1 parent 128e0bf
commit 592484c
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 2 deletions.
diff --git a/lib/rules/jsx-one-expression-per-line.js b/lib/rules/jsx-one-expression-per-line.js
@@ -65,8 +65,10 @@ module.exports = {
         return;
       }
 
-      if (options.allow === 'non-jsx'
-        && !children.find((child) => (child.type === 'JSXFragment' || child.type === 'JSXElement'))) {
+      if (
+        options.allow === 'non-jsx'
+        && !children.find((child) => (child.type === 'JSXFragment' || child.type === 'JSXElement'))
+      ) {
         return;
       }
 

diff --git a/tests/lib/rules/jsx-one-expression-per-line.js b/tests/lib/rules/jsx-one-expression-per-line.js
@@ -211,6 +211,27 @@ ruleTester.run('jsx-one-expression-per-line', rule, {
         </App>`,
       options: [{ allow: 'non-jsx' }],
     },
+    {
+      code: `
+        <App>
+          Hello {<Bar />} there!
+        </App>`,
+      options: [{ allow: 'non-jsx' }],
+    },
+    {
+      code: `
+        <App>
+          Hello {(<Bar />)} there!
+        </App>`,
+      options: [{ allow: 'non-jsx' }],
+    },
+    {
+      code: `
+        <App>
+          Hello {(() => <Bar />)()} there!
+        </App>`,
+      options: [{ allow: 'non-jsx' }],
+    },
   ]),
 
   invalid: parsers.all([