Merge remote-tracking branch 'origin/main' into v4

# Conflicts:
#	lib/rules/no-await-sync-events.ts
#	tests/lib/rules/no-await-sync-events.test.ts

Author: Belco90

.github/workflows/pipeline.yml

@@ -21,7 +21,7 @@ jobs:
 
     steps:
       - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.6.0
+        uses: styfle/cancel-workflow-action@0.8.0
         with:
           access_token: ${{ secrets.GITHUB_TOKEN }}
 
@@ -58,7 +58,7 @@ jobs:
 
     steps:
       - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.6.0
+        uses: styfle/cancel-workflow-action@0.8.0
         with:
           access_token: ${{ secrets.GITHUB_TOKEN }}
       - name: Checkout
@@ -91,7 +91,7 @@ jobs:
 
     steps:
       - name: Cancel Previous Runs
-        uses: styfle/cancel-workflow-action@0.6.0
+        uses: styfle/cancel-workflow-action@0.8.0
         with:
           access_token: ${{ secrets.GITHUB_TOKEN }}
       - name: Checkout

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

@@ -39,10 +39,11 @@ export default ESLintUtils.RuleCreator(getDocsUrl)<Options, MessageIds>({
         const memberExpression = node.parent as TSESTree.MemberExpression;
         const methodNode = memberExpression.property as TSESTree.Identifier;
         const callExpression = memberExpression.parent as TSESTree.CallExpression;
+        const lastArg =
+          callExpression.arguments[callExpression.arguments.length - 1];
         const withDelay =
-          callExpression.arguments.length >= 3 &&
-          isObjectExpression(callExpression.arguments[2]) &&
-          callExpression.arguments[2].properties.some(
+          isObjectExpression(lastArg) &&
+          lastArg.properties.some(
             (property) =>
               isProperty(property) &&
               ASTUtils.isIdentifier(property.key) &&
@@ -52,7 +53,7 @@ export default ESLintUtils.RuleCreator(getDocsUrl)<Options, MessageIds>({
         if (
           !(
             node.name === 'userEvent' &&
-            methodNode.name === 'type' &&
+            ['type', 'keyboard'].includes(methodNode.name) &&
             withDelay
           )
         ) {

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

@@ -142,6 +142,12 @@ ruleTester.run(RULE_NAME, rule, {
       }
       `,
     },
+    {
+      code: `() => {
+        await userEvent.keyboard('foo', {delay: 1234})
+      }
+      `,
+    },
   ],
 
   invalid: [
@@ -161,9 +167,13 @@ ruleTester.run(RULE_NAME, rule, {
         import userEvent from '@testing-library/user-event';
         test('should report sync event awaited', async() => {
           await userEvent.type('foo', 'bar', {hello: 1234});
+          await userEvent.keyboard('foo', {hello: 1234});
         });
       `,
-      errors: [{ line: 4, messageId: 'noAwaitSyncEvents' }],
+      errors: [
+        { line: 4, messageId: 'noAwaitSyncEvents' },
+        { line: 5, messageId: 'noAwaitSyncEvents' },
+      ],
     },
   ],
 });