fix(createBannedAttributeRule): check arguments length before accessing (#167)

Co-authored-by: Ben Monro <[email protected]>

Author: G-Rath

src/__tests__/__fixtures__/createBannedAttributeTestCases.js

@@ -3,6 +3,15 @@
 export default ({ preferred, negatedPreferred, attribute }) => {
   const doubleNegativeCases = negatedPreferred.startsWith("toBe")
     ? [
+        {
+          code: `expect().not.${negatedPreferred}`,
+          errors: [
+            {
+              message: `Use ${preferred} instead of not.${negatedPreferred}`,
+            },
+          ],
+          output: `expect().${preferred}`,
+        },
         {
           code: `const el = screen.getByText("foo"); expect(el).not.${negatedPreferred}`,
           errors: [
@@ -66,6 +75,7 @@ export default ({ preferred, negatedPreferred, attribute }) => {
 
   return {
     valid: [
+      `expect().not.toHaveProperty('value', 'foo')`,
       `const el = screen.getByText("foo"); expect(el).not.toHaveProperty('value', 'foo')`,
       `const el = screen.getByText("foo"); expect(el).${preferred}`,
       `const el = screen.getByText("foo"); expect(el).${negatedPreferred}`,

src/createBannedAttributeRule.js

@@ -13,6 +13,7 @@ export default ({ preferred, negatedPreferred, attributes }) => (context) => {
       : negatedPreferred;
 
   const isBannedArg = (node) =>
+    node.arguments.length &&
     attributes.some((attr) => attr === node.arguments[0].value);
 
   //expect(el).not.toBeEnabled() => expect(el).toBeDisabled()
@@ -42,6 +43,10 @@ export default ({ preferred, negatedPreferred, attributes }) => (context) => {
     "CallExpression[callee.property.name=/toBe(Truthy|Falsy)?|toEqual/][callee.object.callee.name='expect']"(
       node
     ) {
+      if (!node.callee.object.arguments.length) {
+        return;
+      }
+
       const {
         arguments: [{ object, property, property: { name } = {} }],
       } = node.callee.object;
@@ -77,11 +82,11 @@ export default ({ preferred, negatedPreferred, attributes }) => (context) => {
     "CallExpression[callee.property.name=/toHaveProperty|toHaveAttribute/][callee.object.property.name='not'][callee.object.object.callee.name='expect']"(
       node
     ) {
-      const arg = node.arguments[0].value;
       if (!isBannedArg(node)) {
         return;
       }
 
+      const arg = node.arguments[0].value;
       const correctFunction = getCorrectFunctionFor(node, true);
 
       const incorrectFunction = node.callee.property.name;