fix: guard against null property identifier node

Author: Belco90

lib/detect-testing-library-utils.ts

@@ -160,6 +160,11 @@ export function detectTestingLibraryUtils<
 
       const referenceNode = getReferenceNode(node);
       const referenceNodeIdentifier = getPropertyIdentifierNode(referenceNode);
+
+      if (!referenceNodeIdentifier) {
+        return false;
+      }
+
       const importedUtilSpecifier = getImportedUtilSpecifier(
         referenceNodeIdentifier
       );
@@ -609,7 +614,11 @@ export function detectTestingLibraryUtils<
       node: TSESTree.MemberExpression | TSESTree.Identifier
     ): TSESTree.ImportClause | TSESTree.Identifier | undefined => {
       const identifierName: string | undefined = getPropertyIdentifierNode(node)
-        .name;
+        ?.name;
+
+      if (!identifierName) {
+        return undefined;
+      }
 
       return findImportedUtilSpecifier(identifierName);
     };
@@ -637,7 +646,11 @@ export function detectTestingLibraryUtils<
       }
 
       const identifierName: string | undefined = getPropertyIdentifierNode(node)
-        .name;
+        ?.name;
+
+      if (!identifierName) {
+        return false;
+      }
 
       return hasImportMatch(importNode, identifierName);
     };

lib/rules/no-await-sync-events.ts

@@ -81,7 +81,7 @@ export default createTestingLibraryRule<Options, MessageIds>({
           messageId: 'noAwaitSyncEvents',
           data: {
             name: `${
-              getPropertyIdentifierNode(node).name
+              getPropertyIdentifierNode(node)?.name
             }.${simulateEventFunctionName}`,
           },
         });

lib/rules/no-debug.ts

@@ -101,6 +101,10 @@ export default createTestingLibraryRule<Options, MessageIds>({
         const referenceNode = getReferenceNode(node);
         const referenceIdentifier = getPropertyIdentifierNode(referenceNode);
 
+        if (!referenceIdentifier) {
+          return;
+        }
+
         const isDebugUtil = helpers.isDebugUtil(callExpressionIdentifier);
         const isDeclaredDebugVariable = suspiciousDebugVariableNames.includes(
           callExpressionIdentifier.name

lib/rules/no-wait-for-empty-callback.ts

@@ -35,6 +35,10 @@ export default createTestingLibraryRule<Options, MessageIds>({
       const parentCallExpression = node.parent as TSESTree.CallExpression;
       const parentIdentifier = getPropertyIdentifierNode(parentCallExpression);
 
+      if (!parentIdentifier) {
+        return false;
+      }
+
       return helpers.isAsyncUtil(parentIdentifier, [
         'waitFor',
         'waitForElementToBeRemoved',

lib/rules/no-wait-for-multiple-assertions.ts

@@ -42,6 +42,10 @@ export default createTestingLibraryRule<Options, MessageIds>({
         callExpressionNode
       );
 
+      if (!callExpressionIdentifier) {
+        return;
+      }
+
       if (!helpers.isAsyncUtil(callExpressionIdentifier, ['waitFor'])) {
         return;
       }

lib/rules/no-wait-for-side-effects.ts

@@ -45,6 +45,10 @@ export default createTestingLibraryRule<Options, MessageIds>({
         callExpressionNode
       );
 
+      if (!callExpressionIdentifier) {
+        return;
+      }
+
       if (!helpers.isAsyncUtil(callExpressionIdentifier, ['waitFor'])) {
         return;
       }

tests/create-testing-library-rule.test.ts

@@ -307,6 +307,10 @@ ruleTester.run(RULE_NAME, rule, {
       const utils = somethingElse.render()
       `,
     },
+
+    // Weird edge cases
+    `(window as any).__THING = false;`,
+    `thing.method.lastCall.args[0]();`,
   ],
   invalid: [
     // Test Cases for Imports