Test2

.github/workflows/docker-image.yml

@@ -142,6 +142,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-buildx-
 
+      # We could perform the build in a separate job, but why for? let's save some bandwidth on caches
       - name: Build and push
         id: docker_build
         uses: docker/build-push-action@v5
@@ -165,12 +166,13 @@ jobs:
 
       - name: Give containers a moment to init
         shell: bash
-        run: sleep 30s
+        run: sleep 10s
 
       - name: Wait for wiki
         shell: bash
         run: curl --head -X GET --retry 10 --retry-connrefused --retry-delay 10 http://localhost:8000
 
+      # todo, migrate to docker image? https://playwright.dev/docs/docker
       - uses: actions/setup-node@v4
         with:
           node-version: 20

e2e/playwright.config.ts

@@ -12,7 +12,7 @@ import { defineConfig, devices } from '@playwright/test';
 export default defineConfig({
   testDir: './tests',
   /* Run tests in files in parallel */
-  fullyParallel: true,
+  fullyParallel: false,
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */
@@ -24,8 +24,7 @@ export default defineConfig({
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
     /* Base URL to use in actions like `await page.goto('/')`. */
-    // baseURL: 'http://127.0.0.1:3000',
-
+    baseURL: 'http://localhost:8000',
     /* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
     trace: 'on-first-retry',
   },

e2e/tests/example.spec.ts

@@ -1,8 +1,7 @@
 import { test, expect } from '@playwright/test';
 
 test('has title', async ({ page }) => {
-  await page.goto('http://localhost:8000/');
-
+  await page.goto('/');
   // Expect a title "to contain" a substring.
   await expect(page).toHaveTitle(/MediaWiki/);
 });