flexysnap 0.4.4 → 0.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flexysnap",
-  "version": "0.4.4",
+  "version": "0.5.1",
   "description": "Flexible visual snapshot testing for Playwright, built for e-commerce.",
   "keywords": [
     "playwright",

package/src/index.js

@@ -10,7 +10,8 @@ export {
     scrollToTopOfElement,
     setClosePopups,
     logTimestamp,
-    setBaseUrl
+    setBaseUrl,
+    setCacheableUrlPattern
 } from './testUtils.js';
 
 export {

package/src/testUtils.js

@@ -6,15 +6,29 @@ async function callClosePopups(page) {
     await closePopups(page);
 }
 
+function logIfErrorResponse(response, url) {
+    if (response && response.status() >= 400) {
+        logTimestamp(`HTTP error ${response.status()} ${response.statusText()} while loading ${url}`);
+    }
+}
+
 async function gotoWithRetry(page, url) {
+    let response;
 	try {
 		logTimestamp('Loading ' + url);
-		await page.goto(url, { timeout: 30_000, waitUntil: "commit" });
+		response = await page.goto(url, { timeout: 5000, waitUntil: "commit" });
 	} catch (firstError) {
+        logTimestamp(firstError);
 		logTimestamp('Retry loading ' + url);
-		await page.goto(url, { timeout: 30_000, waitUntil: "commit" });
+		response = await page.goto(url, { timeout: 5000, waitUntil: "commit" });
 	}
-	await waitForCompleteLoad(page)
+    logIfErrorResponse(response, url);
+    try {
+        await waitForCompleteLoad(page);
+    } catch(e) {
+        logTimestamp(await page.pageErrors());
+        throw e;
+    }
 }
 
 
@@ -27,6 +41,12 @@ function setBaseUrl(url) {
     baseUrl = url;
 }
 
+let cacheableUrlPatterns = ['/wp-content/', '/wp-includes/']
+
+function setCacheableUrlPattern(pattern) {
+    cacheableUrlPatterns.push(pattern)
+}
+
 test.beforeEach(async ({ context }) => {
     startTimer();
     logTimestamp('Test execution started');
@@ -34,7 +54,13 @@ test.beforeEach(async ({ context }) => {
         const req = route.request();
         const url = new URL(req.url());
         const path = url.pathname.toLowerCase();
-        if (req.method() !== 'GET' || path.includes('/wp-content/') || !path.includes(baseUrl)) {
+
+        for (let pattern of cacheableUrlPatterns) {
+            if (path.includes(pattern))
+                return route.continue();
+        }
+
+        if (req.method() !== 'GET' || path.includes('/wp-content/') || !url.href.includes(baseUrl)) {
             return route.continue();
         }
         url.searchParams.set('_cb', Date.now().toString());
@@ -69,9 +95,20 @@ function logTimestamp(eventName) {
 }
 
 async function waitForCompleteLoad(page) {
-    await page.waitForLoadState('domcontentloaded');
-    await page.waitForLoadState('load');
-    await page.waitForTimeout(500);
+    try {
+        await page.waitForLoadState('domcontentloaded', { timeout: 5000 });
+    } catch (e) {
+        logTimestamp(e);
+        logTimestamp('Retry waiting for DOMContentLoaded event after 5 seconds');
+        await page.waitForLoadState('domcontentloaded', { timeout: 5000 });
+    }
+    try {
+        await page.waitForLoadState('load', { timeout: 5000 });
+    } catch (e) {
+        logTimestamp(e);
+        logTimestamp('Retry waiting for load event after 5 seconds');
+        await page.waitForLoadState('load', { timeout: 5000 });
+    }
 }
 
 
@@ -215,5 +252,6 @@ export {
     setClosePopups,
     callClosePopups,
     logTimestamp,
-    setBaseUrl
+    setBaseUrl,
+    setCacheableUrlPattern
 };