feat: Add no-all-method rule to disallow .all() method calls

This commit introduces a new rule that prevents the use of `.all()` as a method call in the codebase. The rule is documented in the README and includes corresponding tests to validate its functionality.

Author: gustavo-meilus

README.md

@@ -162,6 +162,7 @@ CLI option\
 | [require-hook](https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/require-hook.md)                               | Require setup and teardown code to be within a hook                |     |     |     |
 | [require-soft-assertions](https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/require-soft-assertions.md)         | Require assertions to use `expect.soft()`                          |     | 🔧  |     |
 | [require-to-pass-timeout](https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/require-to-pass-timeout.md)         | Require a timeout for `toPass()`                                   |     |     |     |
+| [no-all-method](https://github.com/playwright-community/eslint-plugin-playwright)                                                                   | Disallow any use of `.all()` as a method call                      |     |     |     |
 | [require-to-throw-message](https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/require-to-throw-message.md)       | Require a message for `toThrow()`                                  |     |     |     |
 | [require-top-level-describe](https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/require-top-level-describe.md)   | Require test cases and hooks to be inside a `test.describe` block  |     |     |     |
 | [valid-describe-callback](https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/valid-describe-callback.md)         | Enforce valid `describe()` callback                                | ✅  |     |     |

src/index.ts

@@ -3,6 +3,7 @@ import expectExpect from './rules/expect-expect.js'
 import maxExpects from './rules/max-expects.js'
 import maxNestedDescribe from './rules/max-nested-describe.js'
 import missingPlaywrightAwait from './rules/missing-playwright-await.js'
+import noAllMethod from './rules/no-all-method.js'
 import noCommentedOutTests from './rules/no-commented-out-tests.js'
 import noConditionalExpect from './rules/no-conditional-expect.js'
 import noConditionalInTest from './rules/no-conditional-in-test.js'
@@ -57,6 +58,7 @@ const index = {
     'max-expects': maxExpects,
     'max-nested-describe': maxNestedDescribe,
     'missing-playwright-await': missingPlaywrightAwait,
+    'no-all-method': noAllMethod,
     'no-commented-out-tests': noCommentedOutTests,
     'no-conditional-expect': noConditionalExpect,
     'no-conditional-in-test': noConditionalInTest,

src/rules/no-all-method.test.ts

@@ -0,0 +1,39 @@
+import { runRuleTester } from '../utils/rule-tester.js'
+import rule from './no-all-method.js'
+
+runRuleTester('no-all-method', rule, {
+  invalid: [
+    {
+      code: "await page.getByText('foo').all()",
+      errors: [
+        {
+          messageId: 'noAllMethod',
+        },
+      ],
+    },
+    {
+      code: "obj['all']()",
+      errors: [
+        {
+          messageId: 'noAllMethod',
+        },
+      ],
+    },
+    {
+      code: "foo.bar.all()",
+      errors: [
+        {
+          messageId: 'noAllMethod',
+        },
+      ],
+    },
+  ],
+  valid: [
+    { code: "const all = 1;" },
+    { code: "obj.all;" },
+    { code: "obj['all'];" },
+    { code: "const str = 'all';" },
+    { code: "function all() {}" },
+    { code: "let all = () => {};" },
+  ],
+}) 

src/rules/no-all-method.ts

@@ -0,0 +1,35 @@
+import { createRule } from '../utils/createRule.js'
+
+export default createRule({
+  create(context) {
+    return {
+      CallExpression(node) {
+        if (
+          node.callee &&
+          node.callee.type === 'MemberExpression' &&
+          node.callee.property &&
+          ((node.callee.property.type === 'Identifier' && node.callee.property.name === 'all') ||
+            (node.callee.property.type === 'Literal' && node.callee.property.value === 'all'))
+        ) {
+          context.report({
+            messageId: 'noAllMethod',
+            node: node.callee.property,
+          })
+        }
+      },
+    }
+  },
+  meta: {
+    docs: {
+      category: 'Best Practices',
+      description: 'Disallow any use of .all() as a method call',
+      recommended: false,
+      url: 'https://github.com/playwright-community/eslint-plugin-playwright',
+    },
+    messages: {
+      noAllMethod: 'Do not use .all() method.',
+    },
+    schema: [],
+    type: 'problem',
+  },
+})