fix(@ngtools/webpack): only add ctor params to decorated classes

Author: clydin

packages/ngtools/webpack/src/transformers/ctor-parameters.ts

@@ -287,7 +287,7 @@ export function decoratorDownlevelTransformer(
     }
 
     function visitor<T extends ts.Node>(node: T): ts.Node {
-      if (ts.isClassDeclaration(node)) {
+      if (ts.isClassDeclaration(node) && node.decorators && node.decorators.length > 0) {
         return ts.updateClassDeclaration(
           node,
           node.decorators,

packages/ngtools/webpack/src/transformers/ctor-parameters_spec.ts

@@ -22,14 +22,19 @@ describe('Constructor Parameter Transformer', () => {
     const input = `
       export class ClassInject {};
 
+      @Injectable()
       export class MyService {
         constructor(v: ClassInject) {}
       }
     `;
 
     const output = `
+      import * as tslib_1 from "tslib";
       export class ClassInject { } ;
-      export class MyService { constructor(v) { } } MyService.ctorParameters = () => [ { type: ClassInject } ];
+      let MyService = class MyService { constructor(v) { } };
+      MyService.ctorParameters = () => [ { type: ClassInject } ];
+      MyService = tslib_1.__decorate([ Injectable() ], MyService);
+      export { MyService };
     `;
 
     const result = transform(input);
@@ -47,19 +52,22 @@ describe('Constructor Parameter Transformer', () => {
         constructor() { }
       }
 
+      @Injectable()
       export class MyService {
         constructor(v: RootProvidedService) {}
       }
     `;
 
-    // tslint:disable:max-line-length
     const output = `
       import * as tslib_1 from "tslib";
       let RootProvidedService = class RootProvidedService { constructor() { } };
       RootProvidedService = tslib_1.__decorate([ Injectable({ providedIn: 'root' }) ], RootProvidedService);
-      export { RootProvidedService }; export class MyService { constructor(v) { } } MyService.ctorParameters = () => [ { type: RootProvidedService } ];
+      export { RootProvidedService };
+      let MyService = class MyService { constructor(v) { } };
+      MyService.ctorParameters = () => [ { type: RootProvidedService } ];
+      MyService = tslib_1.__decorate([ Injectable() ], MyService);
+      export { MyService };
     `;
-    // tslint:enable:max-line-length
 
     const result = transform(input);
 
@@ -84,6 +92,7 @@ describe('Constructor Parameter Transformer', () => {
     const input = `
       import { RootProvidedService } from './root-provided-service';
 
+      @Injectable()
       export class MyService {
         constructor(v: RootProvidedService) {}
       }
@@ -101,6 +110,7 @@ describe('Constructor Parameter Transformer', () => {
       export interface InterInject {}
       export const INTERFACE_INJECT = new InjectionToken<InterInject>('interface-inject');
 
+      @Injectable()
       export class MyService {
         constructor(@Inject(INTERFACE_INJECT) v: InterInject) {}
       }
@@ -111,7 +121,7 @@ describe('Constructor Parameter Transformer', () => {
       export const INTERFACE_INJECT = new InjectionToken('interface-inject');
       let MyService = class MyService { constructor(v) { } };
       MyService.ctorParameters = () => [ { type: undefined, decorators: [{ type: Inject, args: [INTERFACE_INJECT,] }] } ];
-      MyService = tslib_1.__decorate([ tslib_1.__param(0, Inject(INTERFACE_INJECT)) ], MyService);
+      MyService = tslib_1.__decorate([ Injectable(), tslib_1.__param(0, Inject(INTERFACE_INJECT)) ], MyService);
       export { MyService };
     `;
 
@@ -125,6 +135,7 @@ describe('Constructor Parameter Transformer', () => {
       interface InterInject {}
       export const INTERFACE_INJECT = new InjectionToken<InterInject>('interface-inject');
 
+      @Injectable()
       export class MyService {
         constructor(@Inject(INTERFACE_INJECT) v: InterInject) {}
       }
@@ -135,7 +146,7 @@ describe('Constructor Parameter Transformer', () => {
       export const INTERFACE_INJECT = new InjectionToken('interface-inject');
       let MyService = class MyService { constructor(v) { } };
       MyService.ctorParameters = () => [ { type: undefined, decorators: [{ type: Inject, args: [INTERFACE_INJECT,] }] } ];
-      MyService = tslib_1.__decorate([ tslib_1.__param(0, Inject(INTERFACE_INJECT)) ], MyService);
+      MyService = tslib_1.__decorate([ Injectable(), tslib_1.__param(0, Inject(INTERFACE_INJECT)) ], MyService);
       export { MyService };
     `;
 
@@ -155,6 +166,7 @@ describe('Constructor Parameter Transformer', () => {
     const input = `
       import { INTERFACE_INJECT, InterInject } from './module-inject';
 
+      @Injectable()
       export class MyService {
         constructor(@Inject(INTERFACE_INJECT) v: InterInject) {}
       }
@@ -165,7 +177,7 @@ describe('Constructor Parameter Transformer', () => {
       import { INTERFACE_INJECT } from './module-inject';
       let MyService = class MyService { constructor(v) { } };
       MyService.ctorParameters = () => [ { type: undefined, decorators: [{ type: Inject, args: [INTERFACE_INJECT,] }] } ];
-      MyService = tslib_1.__decorate([ tslib_1.__param(0, Inject(INTERFACE_INJECT)) ], MyService);
+      MyService = tslib_1.__decorate([ Injectable(), tslib_1.__param(0, Inject(INTERFACE_INJECT)) ], MyService);
       export { MyService };
     `;