chore: Format recent changes

README.md

@@ -49,19 +49,27 @@ This plugin bundles two configurations to work with both `@playwright/test` or
 
 ## Global Settings
 
-The plugin reads global settings from your ESLint configuration's shared data under the `playwright` key. It supports the following settings:
-
-- `additionalAssertFunctionNames`: an array of function names to treat as assertion functions for the case of rules like `expect-expect`, which enforces the presence of at least one assertion per test case. This allows such rules to recognise custom assertion functions as valid assertions. The global setting applies to all modules. The [`expect-expect` rule accepts an option by the same name](./rules/expect-expect.md#additionalassertfunctionnames) to enable per-module configuration (.e.g, for module-specific custom assert functions).
+The plugin reads global settings from your ESLint configuration's shared data
+under the `playwright` key. It supports the following settings:
+
+- `additionalAssertFunctionNames`: an array of function names to treat as
+  assertion functions for the case of rules like `expect-expect`, which enforces
+  the presence of at least one assertion per test case. This allows such rules
+  to recognise custom assertion functions as valid assertions. The global
+  setting applies to all modules. The
+  [`expect-expect` rule accepts an option by the same name](./rules/expect-expect.md#additionalassertfunctionnames)
+  to enable per-module configuration (.e.g, for module-specific custom assert
+  functions).
 
 You can configure these settings like so:
 
 ```json
 {
-    "settings": {
-        "playwright": {
-            "additionalAssertFunctionNames": ["assertCustomCondition"]
-        }
+  "settings": {
+    "playwright": {
+      "additionalAssertFunctionNames": ["assertCustomCondition"]
     }
+  }
 }
 ```
 

docs/rules/expect-expect.md

@@ -43,7 +43,10 @@ test('should work with callbacks/async', async () => {
 
 ### `additionalAssertFunctionNames`
 
-An array of function names to treat as assertion functions. Only standalone functions are supported. Configure globally acceptable assert function names using [the global setting](../global-settings.md). You can also customize assert function names per-file. For example: 
+An array of function names to treat as assertion functions. Only standalone
+functions are supported. Configure globally acceptable assert function names
+using [the global setting](../global-settings.md). You can also customize assert
+function names per-file. For example:
 
 ```ts
 /* eslint playwright/expect-expect: ["error", { "additionalAssertFunctionNames": ["assertScrolledToBottom"] }] */
@@ -55,14 +58,15 @@ function assertScrolledToBottom(page) {
 describe('scrolling', () => {
   test('button click', async ({ page }) => {
     // ...
-    await assertScrolledToBottom(page)
-  })
+    await assertScrolledToBottom(page);
+  });
 
   test('another way to scroll', async ({ page }) => {
     // ...
-    await assertScrolledToBottom(page)
-  })
-})
+    await assertScrolledToBottom(page);
+  });
+});
 ```
 
-The rule option and the global setting are merged. On a file level, both are considered.
+The rule option and the global setting are merged. On a file level, both are
+considered.

src/rules/expect-expect.ts

@@ -18,7 +18,8 @@ function isAssertionCall(
 export default {
   create(context) {
     const unchecked: ESTree.CallExpression[] = [];
-    const additionalAssertFunctionNames = getAdditionalAssertFunctionNames(context)
+    const additionalAssertFunctionNames =
+      getAdditionalAssertFunctionNames(context);
 
     function checkExpressions(nodes: ESTree.Node[]) {
       for (const node of nodes) {

src/utils/misc.ts

@@ -9,11 +9,17 @@ export const getAmountData = (amount: number) => ({
 
 interface AssertFunctionNamesOptions {
   additionalAssertFunctionNames?: string[];
-};
+}
 
-export function getAdditionalAssertFunctionNames(context: Rule.RuleContext): string[] {
-  const globalSettings = (context.settings as Settings).playwright?.additionalAssertFunctionNames ?? [] as string[]
-  const ruleSettings = (context.options[0] as AssertFunctionNamesOptions | undefined)?.additionalAssertFunctionNames ?? [] as string[]
+export function getAdditionalAssertFunctionNames(
+  context: Rule.RuleContext
+): string[] {
+  const globalSettings =
+    (context.settings as Settings).playwright?.additionalAssertFunctionNames ??
+    ([] as string[]);
+  const ruleSettings =
+    (context.options[0] as AssertFunctionNamesOptions | undefined)
+      ?.additionalAssertFunctionNames ?? ([] as string[]);
 
-  return [...globalSettings, ...ruleSettings]
+  return [...globalSettings, ...ruleSettings];
 }

src/utils/types.ts

@@ -15,6 +15,6 @@ export type KnownCallExpression = ESTree.CallExpression & {
 
 export interface Settings {
   playwright?: {
-    additionalAssertFunctionNames?: readonly string[]
-  }
+    additionalAssertFunctionNames?: readonly string[];
+  };
 }

test/spec/expect-expect.spec.ts

@@ -29,7 +29,9 @@ runRuleTester('expect-expect', rule, {
       `,
       errors: [{ messageId: 'noAssertions' }],
       settings: {
-        playwright: { additionalAssertFunctionNames: ['wayComplexCustomCondition'] },
+        playwright: {
+          additionalAssertFunctionNames: ['wayComplexCustomCondition'],
+        },
       },
     },
     {
@@ -40,7 +42,9 @@ runRuleTester('expect-expect', rule, {
         })
       `,
       errors: [{ messageId: 'noAssertions' }],
-      options: [{ additionalAssertFunctionNames: ['wayComplexCustomCondition'] }],
+      options: [
+        { additionalAssertFunctionNames: ['wayComplexCustomCondition'] },
+      ],
     },
     {
       name: 'Global settings no false positives',
@@ -51,9 +55,13 @@ runRuleTester('expect-expect', rule, {
       `,
       errors: [{ messageId: 'noAssertions' }],
       settings: {
-        playwright: { additionalAssertFunctionNames: ['wayComplexGlobalCustomCondition'] },
+        playwright: {
+          additionalAssertFunctionNames: ['wayComplexGlobalCustomCondition'],
+        },
       },
-      options: [{ additionalAssertFunctionNames: ['wayComplexRuleCustomCondition'] }],
+      options: [
+        { additionalAssertFunctionNames: ['wayComplexRuleCustomCondition'] },
+      ],
     },
   ],
   valid: [
@@ -87,8 +95,10 @@ runRuleTester('expect-expect', rule, {
         })
       `,
       settings: {
-        playwright: { additionalAssertFunctionNames: ['assertCustomCondition'] }
-      }
+        playwright: {
+          additionalAssertFunctionNames: ['assertCustomCondition'],
+        },
+      },
     },
     {
       name: 'Rule settings only',
@@ -111,9 +121,13 @@ runRuleTester('expect-expect', rule, {
         })
       `,
       settings: {
-        playwright: { additionalAssertFunctionNames: ['assertCustomCondition'] },
+        playwright: {
+          additionalAssertFunctionNames: ['assertCustomCondition'],
+        },
       },
-      options: [{ additionalAssertFunctionNames: ['wayComplexCustomCondition'] }],
+      options: [
+        { additionalAssertFunctionNames: ['wayComplexCustomCondition'] },
+      ],
     },
   ],
 });