Skip to content

Commit 06a82b5

Browse files
committed
[Fix] display-name: fix false positive when using memo
1 parent e7fc22f commit 06a82b5

File tree

2 files changed

+14
-1
lines changed

2 files changed

+14
-1
lines changed

lib/util/Components.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -515,7 +515,7 @@ function componentRule(rule, context) {
515515

516516
// Case like `React.memo(() => <></>)` or `React.forwardRef(...)`
517517
const pragmaComponentWrapper = utils.getPragmaComponentWrapper(node);
518-
if (pragmaComponentWrapper) {
518+
if (pragmaComponentWrapper && utils.isReturningJSXOrNull(node)) {
519519
return pragmaComponentWrapper;
520520
}
521521

tests/lib/rules/display-name.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -588,6 +588,19 @@ ruleTester.run('display-name', rule, {
588588
}
589589
`,
590590
},
591+
{
592+
// issue #3303
593+
code: `
594+
function MyComponent(props) {
595+
return <b>{props.name}</b>;
596+
}
597+
598+
const MemoizedMyComponent = React.memo(
599+
MyComponent,
600+
(prevProps, nextProps) => prevProps.name === nextProps.name
601+
)
602+
`,
603+
},
591604
]),
592605

593606
invalid: parsers.all([

0 commit comments

Comments
 (0)