[version-4-5] chore: bump @playwright/test from 1.42.1 to 1.49.0 (#4836)

package.json

@@ -70,7 +70,7 @@
   "devDependencies": {
     "@argos-ci/cli": "^2.4.2",
     "@argos-ci/playwright": "^2.0.0",
-    "@playwright/test": "^1.42.1",
+    "@playwright/test": "^1.49.0",
     "@semantic-release/changelog": "^6.0.1",
     "@semantic-release/exec": "^6.0.3",
     "@semantic-release/git": "latest",

playwright.config.ts

@@ -1,4 +1,5 @@
-import { PlaywrightTestConfig } from "@playwright/test";
+import { devices } from "@playwright/test";
+import type { PlaywrightTestConfig } from "@playwright/test";
 
 const config: PlaywrightTestConfig = {
   snapshotDir: "screenshots/",
@@ -20,6 +21,14 @@ const config: PlaywrightTestConfig = {
     url: "http://127.0.0.1:3000",
     reuseExistingServer: !process.env.CI,
   },
+  projects: [
+    {
+      name: "chromium",
+      use: {
+        ...devices["Desktop Chrome"],
+      },
+    },
+  ],
 };
 
 export default config;

visuals/exclude.json

@@ -1 +1 @@
-["/integrations/"]
+["/integrations/*", "/security-bulletins/reports/*"]

visuals/screenshot.docs.spec.ts

@@ -11,23 +11,29 @@ const stylesheet = fs.readFileSync(stylesheetPath).toString();
 test.describe.configure({ mode: "parallel" });
 
 function isVersionedDocsPathname(pathname: string, excludeList: string[]): boolean {
-  if (excludeList.some((excludedPath) => pathname === excludedPath || pathname.startsWith(excludedPath))) {
-    console.log(`Excluding ${pathname} because it matches or starts with an exclude list pattern`);
+  if (
+    excludeList.some((excludedPath) => {
+      if (excludedPath.endsWith("/*")) {
+        // Let's remove the trailing "/*" to match sub-paths
+        const basePath = excludedPath.slice(0, -2);
+        // Exclude sub-paths only, not the index page
+        return pathname.startsWith(basePath) && pathname !== `${basePath}/`;
+      }
+      // This is an exact match
+      return pathname === excludedPath;
+    })
+  ) {
     return false;
   }
 
+  // Additional exclusion criteria
   if (pathname.startsWith("/api/") || pathname.match(/\/\d+\.\d+\.x\//)) {
     return false;
   }
 
   return true;
 }
 
-test.beforeAll(() => {
-  console.log("Excluded pages: ", excludeList);
-  console.log("Total pages: ", extractSitemapPathnames(sitemapPath).length);
-});
-
 function screenshotPathname(pathname: string) {
   test(`pathname ${pathname}`, async ({ page }) => {
     console.log(`Taking screenshot of ${pathname}`);