fix(@ngtools/webpack): dont error on non-identifier properties.

Author: hansl

packages/@ngtools/webpack/src/loader.spec.ts

@@ -0,0 +1,27 @@
+import * as ts from 'typescript';
+import {removeModuleIdOnlyForTesting} from './loader';
+import {WebpackCompilerHost} from './compiler_host';
+import {TypeScriptFileRefactor} from './refactor';
+
+describe('@ngtools/webpack', () => {
+  describe('loader', () => {
+    describe('removeModuleId', () => {
+      it('should work', () => {
+        const host = new WebpackCompilerHost({}, '');
+        host.writeFile('/file.ts', `
+          export const obj = { moduleId: 123 };
+          export const obj2 = { moduleId: 123, otherValue: 1 };
+          export const obj2 = { otherValue: 1, moduleId: 123 };
+        `);
+
+        const program = ts.createProgram(['/file.ts'], {}, host);
+
+        const refactor = new TypeScriptFileRefactor('/file.ts', host, program);
+        removeModuleIdOnlyForTesting(refactor);
+
+        expect(refactor.sourceText).toMatch(/obj = \{\s+};/);
+        expect(refactor.sourceText).toMatch(/obj2 = \{\s*otherValue: 1\s*};/);
+      });
+    });
+  });
+});

packages/@ngtools/webpack/src/loader.ts

@@ -97,10 +97,10 @@ function _removeModuleId(refactor: TypeScriptFileRefactor) {
   refactor.findAstNodes(sourceFile, ts.SyntaxKind.ObjectLiteralExpression, true)
     // Get all their property assignments.
     .filter((node: ts.ObjectLiteralExpression) =>
-      node.properties.some(prop => prop.name.getText() == 'moduleId'))
+      node.properties.some(prop => _getContentOfKeyLiteral(sourceFile, prop.name) == 'moduleId'))
     .forEach((node: ts.ObjectLiteralExpression) => {
       const moduleIdProp = node.properties.filter((prop: ts.ObjectLiteralElement, idx: number) => {
-        return prop.name.getText() == 'moduleId';
+        return _getContentOfKeyLiteral(sourceFile, prop.name) == 'moduleId';
       })[0];
       // get the trailing comma
       const moduleIdCommaProp = moduleIdProp.parent.getChildAt(1).getChildren()[1];

packages/@ngtools/webpack/src/refactor.ts

@@ -170,7 +170,7 @@ export class TypeScriptFileRefactor {
   }
 
   removeNodes(...nodes: ts.Node[]) {
-    nodes.forEach(node => this.removeNode(node));
+    nodes.forEach(node => node && this.removeNode(node));
   }
 
   replaceNode(node: ts.Node, replacement: string) {