fix(schematics): do not modify tests when skipTests is set

Author: skydever

packages/schematics/src/collection/application/application.spec.ts

@@ -239,6 +239,17 @@ describe('app', () => {
  getFileContent(tree, 'apps/my-dir/my-app/src/app/app.component.spec.ts')
  ).toContain('imports: [RouterTestingModule]');
  });
+
+ it('should not modify tests when --skip-tests is set', () => {
+ const tree = schematicRunner.runSchematic(
+ 'app',
+ { name: 'myApp', directory: 'myDir', routing: true, skipTests: true },
+ appTree
+ );
+ expect(
+ tree.exists('apps/my-dir/my-app/src/app/app.component.spec.ts')
+ ).toBeFalsy();
+ });
  });
 
  describe('template generation mode', () => {

packages/schematics/src/collection/application/index.ts

@@ -73,29 +73,34 @@ function addRouterRootConfiguration(options: NormalizedSchema): Rule {
  )
  ]);
 
- const componentSpecPath = `${
- options.appProjectRoot
- }/src/app/app.component.spec.ts`;
- const componentSpecSource = host.read(componentSpecPath)!.toString('utf-8');
- const componentSpecSourceFile = ts.createSourceFile(
- componentSpecPath,
- componentSpecSource,
- ts.ScriptTarget.Latest,
- true
- );
- insert(host, componentSpecPath, [
- insertImport(
- componentSpecSourceFile,
- componentSpecPath,
- 'RouterTestingModule',
- '@angular/router/testing'
- ),
- ...addImportToTestBed(
- componentSpecSourceFile,
+ if (options.skipTests !== true) {
+ const componentSpecPath = `${
+ options.appProjectRoot
+ }/src/app/app.component.spec.ts`;
+ const componentSpecSource = host.read(componentSpecPath)!.toString(
+ 'utf-8'
+ );
+ const componentSpecSourceFile = ts.createSourceFile(
  componentSpecPath,
- `RouterTestingModule`
- )
- ]);
+ componentSpecSource,
+ ts.ScriptTarget.Latest,
+ true
+ );
+ insert(host, componentSpecPath, [
+ insertImport(
+ componentSpecSourceFile,
+ componentSpecPath,
+ 'RouterTestingModule',
+ '@angular/router/testing'
+ ),
+ ...addImportToTestBed(
+ componentSpecSourceFile,
+ componentSpecPath,
+ `RouterTestingModule`
+ )
+ ]);
+ }
+
  return host;
  };
 }