test(@angular/build): prevent flakiness in incremental-watch e2e test

In watch mode, 'Application bundle generation complete.' can be logged before all files have finished being emitted to disk. Sampling 'files.length > 0' initially could capture a partial file count, causing the final count comparison to time out once all assets are fully written.

Update the test to only track JavaScript chunk files (.js), which prevents race conditions with unrelated asset emission and directly aligns with the test's intent of verifying chunk addition and removal for dynamic imports.

Author: alan-agius4

tests/e2e/tests/build/incremental-watch.ts

@@ -33,6 +33,20 @@ async function getOutputFiles(
   return files;
 }
 
+async function getOutputChunks(
+  dir: string,
+  predicate: (chunks: string[]) => boolean,
+  timeout = 10_000,
+): Promise<string[]> {
+  const files = await getOutputFiles(
+    dir,
+    (files) => predicate(files.filter((file) => file.endsWith('.js'))),
+    timeout,
+  );
+
+  return files.filter((file) => file.endsWith('.js'));
+}
+
 export default async function () {
   const usingApplicationBuilder = getGlobalVariable('argv')['esbuild'];
   assert(
@@ -46,9 +60,9 @@ export default async function () {
     ['build', '--watch', '--configuration=development'],
     buildReadyRegEx,
   );
-  const initialOutputFiles = await getOutputFiles(
+  const initialOutputChunks = await getOutputChunks(
     'dist/test-project/browser',
-    (files) => files.length > 0,
+    (chunks) => chunks.length > 0,
   );
 
   const originalMain = await readFile('src/main.ts');
@@ -66,12 +80,12 @@ export default async function () {
     ),
     appendToFile('src/main.ts', `\nimport('./a').then((m) => m.sayHi());`),
   ]);
-  const intermediateOutputFiles = await getOutputFiles(
+  const intermediateOutputChunks = await getOutputChunks(
     'dist/test-project/browser',
-    (files) => files.length > initialOutputFiles.length,
+    (chunks) => chunks.length > initialOutputChunks.length,
   );
   assert(
-    initialOutputFiles.length < intermediateOutputFiles.length,
+    initialOutputChunks.length < intermediateOutputChunks.length,
     'Additional chunks should be present',
   );
 
@@ -80,13 +94,13 @@ export default async function () {
     waitForAnyProcessOutputToMatch(buildReadyRegEx),
     writeFile('src/main.ts', originalMain),
   ]);
-  const finalOutputFiles = await getOutputFiles(
+  const finalOutputChunks = await getOutputChunks(
     'dist/test-project/browser',
-    (files) => files.length === initialOutputFiles.length,
+    (chunks) => chunks.length === initialOutputChunks.length,
   );
   assert.equal(
-    initialOutputFiles.length,
-    finalOutputFiles.length,
+    initialOutputChunks.length,
+    finalOutputChunks.length,
     'Final chunk count should be equal to initial chunk count.',
   );
 }