Playwright: Refactor to use assertions instead (#211)

Author: lamATnginx

Makefile

@@ -16,7 +16,7 @@ list help::
 	@echo "<PRE-COMMIT>"
 	@echo "	     setup-pre-commit:		Sets up pre-commit (assuming it is installed)"
 	@echo "<PLAYWRIGHT TESTS>"
-	@echo "		 	 test: 		Runs playwright against the old theme."
+	@echo "		 	tests: 		Runs playwright against the old theme."
 	@echo "     tests-update-screenshots:		Runs playwright against the old theme."
 
 .PHONY: biome-format biome-lint biome-all setup-pre-commit tests build-example-site

layouts/_default/baseof.html

@@ -171,7 +171,7 @@
       {{ block "header" . }}{{end}}
     </header>
 
-    <section class="base-layout">
+    <section class="base-layout" data-testid="base-layout">
       <section id="breadcrumb-v1">
         {{ if not .IsHome }}
           {{ if not (in .Params.display_breadcrumb "false" ) }}

tests/src/visual-regression.spec.js

@@ -5,7 +5,11 @@ test.describe('Testing old theme', () => {
     page,
   }) => {
     const currentPlus = 'nginx/installing-nginx-open-source/';
-    await page.goto(`/${currentPlus}`, { waitUntil: 'networkidle' });
+    await page.goto(`/${currentPlus}`);
+    await expect(
+      await page.locator('data-testid=base-layout').count()
+    ).toBeTruthy();
+
     await page.waitForFunction(() => window.scrollY === 0);
     await expect(page).toHaveScreenshot('example-site-screenshot.png', {
       fullPage: true,