fix(@angular-devkit/schematics): include the template path in errors thrown by content templates

Rethrow errors thrown while applying a content template, such as a
`ReferenceError` from a misspelled variable or a `SyntaxError` inside
`<% %>`, as `Failed to apply template "<path>": <error>` and keep the
original error as `cause`. The message did not say which template
failed, and the ng CLI printed only `foobat is not defined`.

Closes #13192

Author: Thanatos9404

packages/angular_devkit/schematics/src/rules/template.ts

@@ -71,10 +71,16 @@ export function applyContentTemplate<T>(options: T): FileOperator {
     }
     decodedContent = decodedContent.replace(/\r?\n/g, EOL);
 
-    return {
-      path,
-      content: Buffer.from(templateImpl(decodedContent, {})(options)),
-    };
+    try {
+      return {
+        path,
+        content: Buffer.from(templateImpl(decodedContent, {})(options)),
+      };
+    } catch (e) {
+      const message = e instanceof Error ? e.message : String(e);
+
+      throw new Error(`Failed to apply content template to "${path}": ${message}`, { cause: e });
+    }
   };
 }
 

packages/angular_devkit/schematics/src/rules/template_spec.ts

@@ -8,7 +8,7 @@
 
 /* eslint-disable import/no-extraneous-dependencies */
 import { normalize } from '@angular-devkit/core';
-import { of as observableOf } from 'rxjs';
+import { lastValueFrom, of as observableOf } from 'rxjs';
 import { UnitTestTree } from '../../testing';
 import { SchematicContext } from '../engine/interface';
 import { HostTree } from '../tree/host-tree';
@@ -183,6 +183,23 @@ describe('contentTemplate', () => {
 
     expect(applyContentTemplate({})(entry)).toBe(entry);
   });
+
+  it('names the template in errors thrown while applying it', () => {
+    const operator = applyContentTemplate({ foobar: 1 });
+
+    expect(() => operator(_entry('/a/b/file.ts', 'a<%= foobat %>b'))).toThrowError(
+      'Failed to apply content template to "/a/b/file.ts": foobat is not defined',
+    );
+    expect(() => operator(_entry('/a/b/file.ts', 'a<%= foobar unintended %>b'))).toThrowError(
+      /^Failed to apply content template to "\/a\/b\/file\.ts": Unexpected identifier/,
+    );
+  });
+
+  it('keeps the original template error as the cause', () => {
+    expect(() =>
+      applyContentTemplate({})(_entry('/a/b/file.ts', 'a<%= foobat %>b')),
+    ).toThrowMatching((e) => e.cause instanceof ReferenceError);
+  });
 });
 
 describe('applyTemplateFiles', () => {
@@ -212,4 +229,19 @@ describe('applyTemplateFiles', () => {
     ]);
     expect(tree.readContent('/a/b/file3')).toBe('hello 1 world');
   });
+
+  it('names the template file in errors', async () => {
+    const tree = new UnitTestTree(new HostTree());
+    tree.create('a/b/file__a__.template', 'hello <%= a.missing.length %> world');
+
+    const context: SchematicContext = {
+      strategy: MergeStrategy.Default,
+    } as SchematicContext;
+
+    await expectAsync(
+      lastValueFrom(callRule(applyTemplates({ a: 'foo' }), observableOf(tree), context)),
+    ).toBeRejectedWithError(
+      /^Failed to apply content template to "\/a\/b\/file__a__\.template": Cannot read /,
+    );
+  });
 });