Skip to content

Commit 0983664

Browse files
refactor: simplify isRenderUtil function (#330)
1 parent a8d2017 commit 0983664

File tree

1 file changed

+11
-24
lines changed

1 file changed

+11
-24
lines changed

lib/detect-testing-library-utils.ts

Lines changed: 11 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -554,31 +554,18 @@ export function detectTestingLibraryUtils<
554554
* Testing Library. Otherwise, it means `custom-module` has been set up, so
555555
* only those nodes coming from Testing Library will be considered as valid.
556556
*/
557-
const isRenderUtil: IsRenderUtilFn = (node) => {
558-
return isTestingLibraryUtil(
559-
node,
560-
(identifierNodeName, originalNodeName) => {
561-
if (isAggressiveRenderReportingEnabled()) {
562-
return identifierNodeName.toLowerCase().includes(RENDER_NAME);
563-
}
564-
565-
return [RENDER_NAME, ...getCustomRenders()].some(
566-
(validRenderName) => {
567-
let isMatch = false;
568-
569-
if (validRenderName === identifierNodeName) {
570-
isMatch = true;
571-
}
572-
573-
if (!!originalNodeName && validRenderName === originalNodeName) {
574-
isMatch = true;
575-
}
576-
return isMatch;
577-
}
578-
);
557+
const isRenderUtil: IsRenderUtilFn = (node) =>
558+
isTestingLibraryUtil(node, (identifierNodeName, originalNodeName) => {
559+
if (isAggressiveRenderReportingEnabled()) {
560+
return identifierNodeName.toLowerCase().includes(RENDER_NAME);
579561
}
580-
);
581-
};
562+
563+
return [RENDER_NAME, ...getCustomRenders()].some(
564+
(validRenderName) =>
565+
validRenderName === identifierNodeName ||
566+
(Boolean(originalNodeName) && validRenderName === originalNodeName)
567+
);
568+
});
582569

583570
const isRenderVariableDeclarator: IsRenderVariableDeclaratorFn = (node) => {
584571
if (!node.init) {

0 commit comments

Comments
 (0)