eslint-plugin-playwright 1.5.0 → 1.5.1

package/dist/index.js

@@ -637,7 +637,8 @@ var missing_playwright_await_default = {
     docs: {
       category: "Possible Errors",
       description: `Identify false positives when async Playwright APIs are not properly awaited.`,
-      recommended: true
+      recommended: true,
+      url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/missing-playwright-await.md"
     },
     fixable: "code",
     messages: {
@@ -1220,7 +1221,8 @@ var no_networkidle_default = {
     docs: {
       category: "Possible Errors",
       description: "Prevent usage of the networkidle option",
-      recommended: true
+      recommended: true,
+      url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/no-networkidle.md"
     },
     messages: {
       noNetworkidle: "Unexpected use of networkidle."
@@ -1280,7 +1282,8 @@ var no_page_pause_default = {
     docs: {
       category: "Possible Errors",
       description: "Prevent usage of page.pause()",
-      recommended: true
+      recommended: true,
+      url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/no-page-pause.md"
     },
     messages: {
       noPagePause: "Unexpected use of page.pause()."
@@ -1728,7 +1731,8 @@ var no_useless_await_default = {
     docs: {
       category: "Possible Errors",
       description: "Disallow unnecessary awaits for Playwright methods",
-      recommended: true
+      recommended: true,
+      url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/no-useless-await.md"
     },
     fixable: "code",
     messages: {
@@ -2005,7 +2009,8 @@ var prefer_comparison_matcher_default = {
     docs: {
       category: "Best Practices",
       description: "Suggest using the built-in comparison matchers",
-      recommended: false
+      recommended: false,
+      url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/prefer-comparision-matcher.md"
     },
     fixable: "code",
     messages: {
@@ -2781,7 +2786,7 @@ var require_hook_default = {
         if (!isTypeOfFnCall(context, node, ["describe"])) {
           return;
         }
-        const [, testFn] = node.arguments;
+        const testFn = node.arguments.at(-1);
         if (!isFunction(testFn) || testFn.body.type !== "BlockStatement") {
           return;
         }
@@ -2964,6 +2969,12 @@ var paramsLocation = (params) => {
     start: first.loc.start
   };
 };
+function parseArgs(node) {
+  const [name, b, c] = node.arguments;
+  const options = node.arguments.length === 2 ? b : void 0;
+  const callback = node.arguments.length === 3 ? c : b;
+  return [name, options, callback];
+}
 var valid_describe_callback_default = {
   create(context) {
     return {
@@ -2974,14 +2985,14 @@ var valid_describe_callback_default = {
         if (call.members.some((s) => getStringValue(s) === "configure")) {
           return;
         }
+        const [name, _, callback] = parseArgs(node);
         if (node.arguments.length < 1) {
           return context.report({
             loc: node.loc,
             messageId: "nameAndCallback"
           });
         }
-        const [, callback] = node.arguments;
-        if (!callback) {
+        if (!name || !callback) {
           context.report({
             loc: paramsLocation(node.arguments),
             messageId: "nameAndCallback"
@@ -2991,7 +3002,7 @@ var valid_describe_callback_default = {
         if (!isFunction(callback)) {
           context.report({
             loc: paramsLocation(node.arguments),
-            messageId: "secondArgumentMustBeFunction"
+            messageId: "invalidCallback"
           });
           return;
         }
@@ -3001,7 +3012,7 @@ var valid_describe_callback_default = {
             node: callback
           });
         }
-        if (call.members.every((s) => getStringValue(s) !== "each") && callback.params.length) {
+        if (callback.params.length) {
           context.report({
             loc: paramsLocation(callback.params),
             messageId: "unexpectedDescribeArgument"
@@ -3034,9 +3045,9 @@ var valid_describe_callback_default = {
       url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/valid-describe-callback.md"
     },
     messages: {
+      invalidCallback: "Callback argument must be a function",
       nameAndCallback: "Describe requires name and callback arguments",
       noAsyncDescribeCallback: "No async describe callback",
-      secondArgumentMustBeFunction: "Second argument must be function",
       unexpectedDescribeArgument: "Unexpected argument(s) in describe callback",
       unexpectedReturnInDescribe: "Unexpected return statement in describe callback"
     },

package/dist/index.mjs

@@ -661,7 +661,8 @@ var init_missing_playwright_await = __esm({
         docs: {
           category: "Possible Errors",
           description: `Identify false positives when async Playwright APIs are not properly awaited.`,
-          recommended: true
+          recommended: true,
+          url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/missing-playwright-await.md"
         },
         fixable: "code",
         messages: {
@@ -1331,7 +1332,8 @@ var init_no_networkidle = __esm({
         docs: {
           category: "Possible Errors",
           description: "Prevent usage of the networkidle option",
-          recommended: true
+          recommended: true,
+          url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/no-networkidle.md"
         },
         messages: {
           noNetworkidle: "Unexpected use of networkidle."
@@ -1405,7 +1407,8 @@ var init_no_page_pause = __esm({
         docs: {
           category: "Possible Errors",
           description: "Prevent usage of page.pause()",
-          recommended: true
+          recommended: true,
+          url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/no-page-pause.md"
         },
         messages: {
           noPagePause: "Unexpected use of page.pause()."
@@ -1905,7 +1908,8 @@ var init_no_useless_await = __esm({
         docs: {
           category: "Possible Errors",
           description: "Disallow unnecessary awaits for Playwright methods",
-          recommended: true
+          recommended: true,
+          url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/no-useless-await.md"
         },
         fixable: "code",
         messages: {
@@ -2221,7 +2225,8 @@ var init_prefer_comparison_matcher = __esm({
         docs: {
           category: "Best Practices",
           description: "Suggest using the built-in comparison matchers",
-          recommended: false
+          recommended: false,
+          url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/prefer-comparision-matcher.md"
         },
         fixable: "code",
         messages: {
@@ -3086,7 +3091,7 @@ var init_require_hook = __esm({
             if (!isTypeOfFnCall(context, node, ["describe"])) {
               return;
             }
-            const [, testFn] = node.arguments;
+            const testFn = node.arguments.at(-1);
             if (!isFunction(testFn) || testFn.body.type !== "BlockStatement") {
               return;
             }
@@ -3287,6 +3292,12 @@ var init_require_top_level_describe = __esm({
 });
 
 // src/rules/valid-describe-callback.ts
+function parseArgs(node) {
+  const [name, b, c] = node.arguments;
+  const options = node.arguments.length === 2 ? b : void 0;
+  const callback = node.arguments.length === 3 ? c : b;
+  return [name, options, callback];
+}
 var paramsLocation, valid_describe_callback_default;
 var init_valid_describe_callback = __esm({
   "src/rules/valid-describe-callback.ts"() {
@@ -3311,14 +3322,14 @@ var init_valid_describe_callback = __esm({
             if (call.members.some((s) => getStringValue(s) === "configure")) {
               return;
             }
+            const [name, _, callback] = parseArgs(node);
             if (node.arguments.length < 1) {
               return context.report({
                 loc: node.loc,
                 messageId: "nameAndCallback"
               });
             }
-            const [, callback] = node.arguments;
-            if (!callback) {
+            if (!name || !callback) {
               context.report({
                 loc: paramsLocation(node.arguments),
                 messageId: "nameAndCallback"
@@ -3328,7 +3339,7 @@ var init_valid_describe_callback = __esm({
             if (!isFunction(callback)) {
               context.report({
                 loc: paramsLocation(node.arguments),
-                messageId: "secondArgumentMustBeFunction"
+                messageId: "invalidCallback"
               });
               return;
             }
@@ -3338,7 +3349,7 @@ var init_valid_describe_callback = __esm({
                 node: callback
               });
             }
-            if (call.members.every((s) => getStringValue(s) !== "each") && callback.params.length) {
+            if (callback.params.length) {
               context.report({
                 loc: paramsLocation(callback.params),
                 messageId: "unexpectedDescribeArgument"
@@ -3371,9 +3382,9 @@ var init_valid_describe_callback = __esm({
           url: "https://github.com/playwright-community/eslint-plugin-playwright/tree/main/docs/rules/valid-describe-callback.md"
         },
         messages: {
+          invalidCallback: "Callback argument must be a function",
           nameAndCallback: "Describe requires name and callback arguments",
           noAsyncDescribeCallback: "No async describe callback",
-          secondArgumentMustBeFunction: "Second argument must be function",
           unexpectedDescribeArgument: "Unexpected argument(s) in describe callback",
           unexpectedReturnInDescribe: "Unexpected return statement in describe callback"
         },

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "eslint-plugin-playwright",
   "description": "ESLint plugin for Playwright testing.",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "repository": "https://github.com/playwright-community/eslint-plugin-playwright",
   "author": "Mark Skelton <mark@mskelton.dev>",
   "packageManager": "pnpm@8.12.0",