Add playwright browser caching

De-flake a test, hopefully

Author: prushforth

.github/workflows/ci-testing.yml

@@ -13,13 +13,21 @@ jobs:
       uses: actions/setup-node@v4
       with:
         node-version: 22
+        cache: npm
     - run: sudo apt-get install xvfb
-    - run: npm install
-    - run: npx playwright install  --with-deps
+    - run: npm ci
+    - name: Cache Playwright browsers
+      id: playwright-cache
+      uses: actions/cache@v4
+      with:
+        path: ~/.cache/ms-playwright
+        key: playwright-${{ runner.os }}-${{ hashFiles('package-lock.json') }}
+    - run: npx playwright install --with-deps
+      if: steps.playwright-cache.outputs.cache-hit != 'true'
+    - run: npx playwright install-deps
+      if: steps.playwright-cache.outputs.cache-hit == 'true'
     - run: npm install -g grunt-cli
     - run: grunt default
     - run: xvfb-run --auto-servernum -- npx playwright test --grep-invert="popupTabNavigation\.test\.js|layerContextMenuKeyboard\.test\.js"  --workers=1 --retries=3
-#    - run: xvfb-run --auto-servernum -- npx playwright test --grep="popupTabNavigation\.test\.js|layerContextMenuKeyboard\.test\.js" --workers=1 --retries=3
-#    - run: xvfb-run --auto-servernum -- npm run jest
       env:
         CI: true

test/e2e/api/events/map-change-event.test.js

@@ -11,7 +11,9 @@ test.describe('Map-change event are only fired when layers/extents are checked o
     page =
       context.pages().find((page) => page.url() === 'about:blank') ||
       (await context.newPage());
-    await page.goto('events/map-change-event.html');
+    await page.goto('events/map-change-event.html', {
+      waitUntil: 'networkidle'
+    });
   });
 
   test.afterAll(async function () {