test: migrate app-shell-with-service-worker E2E test to Puppeteer

Replaces the Protractor-based ng e2e execution with the new Puppeteer executeBrowserTest utility in `build/app-shell/app-shell-with-service-worker.ts` E2E test.

Author: clydin

tests/e2e/tests/build/app-shell/app-shell-with-service-worker.ts

@@ -1,8 +1,10 @@
+import { setTimeout } from 'node:timers/promises';
 import { getGlobalVariable } from '../../../utils/env';
-import { appendToFile, expectFileToMatch, writeFile } from '../../../utils/fs';
+import { appendToFile, expectFileToMatch } from '../../../utils/fs';
 import { installPackage } from '../../../utils/packages';
 import { ng } from '../../../utils/process';
 import { updateJsonFile } from '../../../utils/project';
+import { executeBrowserTest } from '../../../utils/puppeteer';
 
 const snapshots = require('../../../ng-snapshot/package.json');
 
@@ -31,26 +33,26 @@ export default async function () {
     }
   }
 
-  await writeFile(
-    'e2e/app.e2e-spec.ts',
-    `
-    import { browser, by, element } from 'protractor';
+  await ng('build');
+  await expectFileToMatch('dist/test-project/browser/index.html', /app-shell works!/);
 
-    it('should have ngsw in normal state', () => {
-      browser.get('/');
+  await executeBrowserTest({
+    configuration: 'production',
+    checkFn: async (page) => {
       // Wait for service worker to load.
-      browser.sleep(2000);
-      browser.waitForAngularEnabled(false);
-      browser.get('/ngsw/state');
-      // Should have updated, and be in normal state.
-      expect(element(by.css('pre')).getText()).not.toContain('Last update check: never');
-      expect(element(by.css('pre')).getText()).toContain('Driver state: NORMAL');
-    });
-  `,
-  );
+      await setTimeout(2000);
 
-  await ng('build');
-  await expectFileToMatch('dist/test-project/browser/index.html', /app-shell works!/);
+      const baseUrl = page.url();
+      await page.goto(new URL('/ngsw/state', baseUrl).href);
 
-  await ng('e2e', '--configuration=production');
+      // Should have updated, and be in normal state.
+      const preText = await page.$eval('pre', (el) => el.textContent);
+      if (preText?.includes('Last update check: never')) {
+        throw new Error(`Expected service worker to have checked for updates, but got: ${preText}`);
+      }
+      if (!preText?.includes('Driver state: NORMAL')) {
+        throw new Error(`Expected service worker driver state to be NORMAL, but got: ${preText}`);
+      }
+    },
+  });
 }

tests/e2e/utils/puppeteer.ts

@@ -18,7 +18,7 @@ export async function executeBrowserTest(options: BrowserTestOptions = {}) {
     if (!url) {
       // Start serving and find address (1 - Webpack; 2 - Vite)
       const match = /(?:open your browser on|Local:)\s+(http:\/\/localhost:\d+\/)/;
-      const serveArgs = ['serve', '--port=0'];
+      const serveArgs = ['serve', '--port=0', '--no-watch', '--no-live-reload'];
       if (options.project) {
         serveArgs.push(options.project);
       }