fix: add checks for undefined variables for valid-graphql-wire-adapter-callback-parameter rule (#149)

Author: emanchan

lib/rules/valid-graphql-wire-adapter-callback-parameters.js

@@ -25,11 +25,16 @@ module.exports = {
                 const { decorators } = node;
 
                 // Check that the @wire decorator is using graphql
-                const graphQlDecorator = decorators.find((decorator) => {
-                    return decorator.expression.arguments.find((argument) => {
-                        return argument.name === 'graphql';
+                const graphQlDecorator =
+                    decorators !== undefined &&
+                    decorators.find((decorator) => {
+                        if (decorator.expression && decorator.expression.arguments) {
+                            return decorator.expression.arguments.some((argument) => {
+                                return argument.name === 'graphql';
+                            });
+                        }
+                        return false;
                     });
-                });
 
                 // Verify that the method definition is using 'errors' not 'error
                 if (graphQlDecorator !== undefined) {
@@ -38,12 +43,12 @@ module.exports = {
                             (param) => param.type === 'ObjectPattern',
                         );
                         if (objectPatternNode !== undefined) {
-                            const incorrectErrorParameter = objectPatternNode.properties.find(
+                            const incorrectErrorParameter = objectPatternNode.properties.some(
                                 (property) => {
                                     return property.value.name === 'error';
                                 },
                             );
-                            if (incorrectErrorParameter !== undefined) {
+                            if (incorrectErrorParameter) {
                                 context.report({
                                     node,
                                     message:

test/lib/rules/valid-graphql-wire-adapter-callback-parameters.js

@@ -15,6 +15,12 @@ const ruleTester = new RuleTester(ESLINT_TEST_CONFIG);
 
 ruleTester.run('valid-graphql-wire-adapter-callback-parameters', rule, {
     valid: [
+        {
+            code: `import { LightningElement, api } from 'lwc';
+            export default class Foo extends LightningElement {
+                @api bar_Foo() {}
+            }`,
+        },
         {
             code: `import { wire } from 'lwc';
             import getFoo from 'adapter';