TESTING

Author: clydin

packages/angular/build/src/builders/application/execute-build.ts

@@ -87,6 +87,7 @@ export async function executeBuild(
     // all TypeScript related contexts.
     // TODO: Enable cached bundling for the typescript contexts
     const forceTypeScriptRebuild = codeBundleCache?.invalidate(allFileChanges);
+    console.log('TYPESCRIPT INVALID: ' + forceTypeScriptRebuild);
     const typescriptResults: BundleContextResult[] = [];
     for (const typescriptContext of bundlerContexts.typescriptContexts) {
       typescriptContext.invalidate(allFileChanges);

packages/angular/build/src/tools/esbuild/angular/source-file-cache.ts

@@ -32,10 +32,20 @@ export class SourceFileCache extends Map<string, ts.SourceFile> {
 
     const extraWatchFiles = new Set(this.referencedFiles?.map(path.normalize));
 
+    console.log(
+      'REFERENCED FILES: ' +
+        JSON.stringify(
+          [...extraWatchFiles].filter((file) => !file.endsWith('.d.ts')),
+          null,
+          2,
+        ),
+    );
+
     let invalid = false;
     for (let file of files) {
       file = path.normalize(file);
       invalid = this.loadResultCache.invalidate(file) || invalid;
+      invalid = extraWatchFiles.has(file) || invalid;
 
       // Normalize separators to allow matching TypeScript Host paths
       if (USING_WINDOWS) {
@@ -44,8 +54,6 @@ export class SourceFileCache extends Map<string, ts.SourceFile> {
 
       invalid = this.delete(file) || invalid;
       this.modifiedFiles.add(file);
-
-      invalid = extraWatchFiles.has(file) || invalid;
     }
 
     return invalid;

tests/legacy-cli/e2e/tests/basic/rebuild.ts

@@ -2,13 +2,18 @@ import { setTimeout } from 'node:timers/promises';
 import { getGlobalVariable } from '../../utils/env';
 import { appendToFile, replaceInFile, writeMultipleFiles } from '../../utils/fs';
 import { silentNg, waitForAnyProcessOutputToMatch } from '../../utils/process';
-import { ngServe } from '../../utils/project';
+import { ngServe, updateJsonFile } from '../../utils/project';
 
 export default async function () {
   const esbuild = getGlobalVariable('argv')['esbuild'];
   const validBundleRegEx = esbuild ? /sent to client/ : /Compiled successfully\./;
   const lazyBundleRegEx = esbuild ? /chunk-/ : /src_app_lazy_lazy_component_ts\.js/;
 
+  await updateJsonFile('angular.json', (configJson) => {
+    const appArchitect = configJson.projects['test-project'].architect;
+    appArchitect.build.options.verbose = true;
+  });
+
   // Disable component stylesheet HMR to support page reload based rebuild testing.
   // Ideally this environment variable would be passed directly to the new serve process
   // but this would require signficant test changes due to the existing `ngServe` signature.