tests(fix): add console message grabber to testing script

davidsneighbour · Dec 24, 2024 · 73cb65f · 73cb65f
1 parent b133b00
commit 73cb65f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/tests/e2e/all-pages.spec.ts b/tests/e2e/all-pages.spec.ts
@@ -39,9 +39,22 @@ test('Verify each link in links.json is reachable', async ({ page }) => {
   }
 
   for (const link of links) {
+    const errors = [];
     await page.goto(link);
     expect(page.url()).toBe(link);
     //console.log(`Successfully navigated to ${link}`);
+
+    page.on('console', (message) => {
+      if (message.type() === 'error') {
+        errors.push(`Error message: ${message.text()}`);
+      }
+    });
+
+    page.on('pageerror', (err) => {
+      console.log(err.message);
+    });
+
+    expect(errors).toHaveLength(0);
   }
 });
 

diff --git a/tests/html-validator/config.json b/tests/html-validator/config.json
@@ -10,7 +10,7 @@
     "element-name": [
       "warn",
       {
-        "whitelist": ["menu", "date", "lite-youtube", "web-vitals"]
+        "whitelist": ["menu", "date", "lite-youtube"]
       }
     ],
     "element-required-content": "off",
@@ -59,7 +59,6 @@
           ":[a-z0-9-]+",
           "x-[a-z0-9-]+:[a-z0-9-]+",
           "x-[a-z0-9-]+.[a-z0-9-]+.[a-z0-9-]+"
-
         ]
       }
     ]