fix(schematics): migrate spec to skipTest to be in line with Angular CLI

Closes: ngrx#2242

BREAKING CHANGES:

To be inline with the Angular CLI, we migrated the `--spec` to `--skipTest`.
By default skipTest is false, this way you will always be provided with `*.spec.ts files`

BEFORE:

```sh

ng generate action User --spec

```

AFTER:

```sh

ng generate action User

```

modules/effects/schematics/ng-add/index.spec.ts

@@ -21,7 +21,6 @@ describe('Effects ng-add Schematic', () => {
     name: 'foo',
     skipPackageJson: false,
     project: 'bar',
-    spec: true,
     module: 'app',
     flat: false,
     group: false,
@@ -124,8 +123,8 @@ describe('Effects ng-add Schematic', () => {
     expect(thrownError).toBeDefined();
   });
 
-  it('should respect the spec flag', () => {
-    const options = { ...defaultOptions, spec: false };
+  it('should respect the skipTest flag', () => {
+    const options = { ...defaultOptions, skipTest: true };
 
     const tree = schematicRunner.runSchematic('ng-add', options, appTree);
     const files = tree.files;
@@ -200,7 +199,12 @@ describe('Effects ng-add Schematic', () => {
   });
 
   it('should group within an "effects" folder if group is set', () => {
-    const options = { ...defaultOptions, flat: true, spec: false, group: true };
+    const options = {
+      ...defaultOptions,
+      flat: true,
+      skipTest: true,
+      group: true,
+    };
 
     const tree = schematicRunner.runSchematic('ng-add', options, appTree);
     const files = tree.files;
@@ -210,7 +214,7 @@ describe('Effects ng-add Schematic', () => {
   });
 
   it('should inject the effect service correctly', async () => {
-    const options = { ...defaultOptions, spec: true };
+    const options = { ...defaultOptions, skipTest: false };
     const tree = await schematicRunner
       .runSchematicAsync('ng-add', options, appTree)
       .toPromise();

modules/effects/schematics/ng-add/index.ts

@@ -129,9 +129,9 @@ export default function(options: EffectOptions): Rule {
     options.path = parsedPath.path;
 
     const templateSource = apply(url('./files'), [
-      options.spec
-        ? noop()
-        : filter(path => !path.endsWith('.spec.ts.template')),
+      options.skipTest
+        ? filter(path => !path.endsWith('.spec.ts.template'))
+        : noop(),
       options.minimal ? filter(_ => false) : noop(),
       applyTemplates({
         ...stringUtils,

modules/effects/schematics/ng-add/schema.json

@@ -26,10 +26,10 @@
       "default": true,
       "description": "Flag to indicate if a dir is created."
     },
-    "spec": {
+    "skipTest": {
       "type": "boolean",
-      "default": true,
-      "description": "Specifies if a spec file is generated."
+      "default": false,
+      "description": "When true, does not create test files."
     },
     "project": {
       "type": "string",

modules/effects/schematics/ng-add/schema.ts

@@ -3,7 +3,7 @@ export interface Schema {
   skipPackageJson?: boolean;
   path?: string;
   flat?: boolean;
-  spec?: boolean;
+  skipTest?: boolean;
   project?: string;
   module?: string;
   group?: boolean;

modules/schematics/src/action/index.spec.ts

@@ -19,7 +19,6 @@ describe('Action Schematic', () => {
   const defaultOptions: ActionOptions = {
     name: 'foo',
     project: 'bar',
-    spec: false,
     group: false,
     flat: true,
   };
@@ -61,10 +60,23 @@ describe('Action Schematic', () => {
     ).toBeGreaterThanOrEqual(0);
   });
 
-  it('should create two files if spec is true', () => {
+  it('should create two files test files by default', () => {
     const options = {
       ...defaultOptions,
-      spec: true,
+    };
+    const tree = schematicRunner.runSchematic('action', options, appTree);
+    expect(
+      tree.files.indexOf(`${projectPath}/src/app/foo.actions.spec.ts`)
+    ).toBeGreaterThanOrEqual(0);
+    expect(
+      tree.files.indexOf(`${projectPath}/src/app/foo.actions.ts`)
+    ).toBeGreaterThanOrEqual(0);
+  });
+
+  it('should not create two files test files when skipTests is set to true', () => {
+    const options = {
+      ...defaultOptions,
+      skipTests: false,
     };
     const tree = schematicRunner.runSchematic('action', options, appTree);
     expect(
@@ -116,7 +128,7 @@ describe('Action Schematic', () => {
     });
 
     it('should create spec class with right imports', () => {
-      const options = { ...defaultOptions, spec: true };
+      const options = { ...defaultOptions };
       const tree = schematicRunner.runSchematic('action', options, appTree);
       const fileContent = tree.readContent(
         `${projectPath}/src/app/foo.actions.spec.ts`

modules/schematics/src/action/index.ts

@@ -32,9 +32,9 @@ export default function(options: ActionOptions): Rule {
     const templateSource = apply(
       url(options.creators ? './creator-files' : './files'),
       [
-        options.spec
-          ? noop()
-          : filter(path => !path.endsWith('.spec.ts.template')),
+        options.skipTests
+          ? filter(path => !path.endsWith('.spec.ts.template'))
+          : noop(),
         applyTemplates({
           ...stringUtils,
           'if-flat': (s: string) =>

modules/schematics/src/action/schema.json

@@ -24,9 +24,9 @@
       "description": "The name of the project.",
       "aliases": ["p"]
     },
-    "spec": {
+    "skipTest": {
       "type": "boolean",
-      "description": "Specifies if a spec file is generated.",
+      "description": "When true, does not create test files.",
       "default": false
     },
     "flat": {

modules/schematics/src/action/schema.ts

@@ -15,9 +15,9 @@ export interface Schema {
   project?: string;
 
   /**
-   * Specifies if a spec file is generated.
+   * When true, does not create test files.
    */
-  spec?: boolean;
+  skipTests?: boolean;
 
   /**
    * Flag to indicate if a dir is created.

modules/schematics/src/container/index.spec.ts

@@ -22,7 +22,6 @@ describe('Container Schematic', () => {
     inlineTemplate: false,
     changeDetection: 'Default',
     styleext: 'css',
-    spec: true,
     module: undefined,
     export: false,
     prefix: 'app',
@@ -110,7 +109,7 @@ describe('Container Schematic', () => {
   });
 
   it('should update the component spec', async () => {
-    const options = { ...defaultOptions, spec: true, testDepth: 'unit' };
+    const options = { ...defaultOptions, testDepth: 'unit' };
     const tree = await schematicRunner
       .runSchematicAsync('container', options, appTree)
       .toPromise();
@@ -123,7 +122,7 @@ describe('Container Schematic', () => {
   });
 
   it('should use the provideMockStore helper if unit', async () => {
-    const options = { ...defaultOptions, spec: true, testDepth: 'unit' };
+    const options = { ...defaultOptions, testDepth: 'unit' };
     const tree = await schematicRunner
       .runSchematicAsync('container', options, appTree)
       .toPromise();
@@ -134,7 +133,7 @@ describe('Container Schematic', () => {
   });
 
   it('should inject Store correctly', async () => {
-    const options = { ...defaultOptions, spec: true };
+    const options = { ...defaultOptions };
     const tree = await schematicRunner
       .runSchematicAsync('container', options, appTree)
       .toPromise();
@@ -145,7 +144,7 @@ describe('Container Schematic', () => {
   });
 
   it('should use StoreModule if integration test', async () => {
-    const options = { ...defaultOptions, spec: true };
+    const options = { ...defaultOptions };
     const tree = await schematicRunner
       .runSchematicAsync('container', options, appTree)
       .toPromise();

modules/schematics/src/container/index.ts

@@ -139,9 +139,9 @@ export default function(options: ContainerOptions): Rule {
     const templateSource = apply(
       url(options.testDepth === 'unit' ? './files' : './integration-files'),
       [
-        options.spec
-          ? noop()
-          : filter(path => !path.endsWith('.spec.ts.template')),
+        options.skipTest
+          ? filter(path => !path.endsWith('.spec.ts.template'))
+          : noop(),
         applyTemplates({
           'if-flat': (s: string) => (options.flat ? '' : s),
           ...stringUtils,

modules/schematics/src/container/schema.json

@@ -60,10 +60,10 @@
       "type": "string",
       "default": "css"
     },
-    "spec": {
+    "skipTest": {
       "type": "boolean",
-      "description": "Specifies if a spec file is generated.",
-      "default": true
+      "description": "When true, does not create test files.",
+      "default": false
     },
     "flat": {
       "type": "boolean",

modules/schematics/src/container/schema.ts

@@ -36,9 +36,9 @@ export interface Schema {
    */
   styleext?: string;
   /**
-   * Specifies if a spec file is generated.
+   * When true, does not create test files.
    */
-  spec?: boolean;
+  skipTest?: boolean;
   /**
    * Flag to indicate if a dir is created.
    */

modules/schematics/src/data/index.spec.ts

@@ -19,7 +19,6 @@ describe('Data Schematic', () => {
   const defaultOptions: DataOptions = {
     name: 'foo',
     project: 'bar',
-    spec: false,
     group: false,
     flat: true,
   };
@@ -60,10 +59,9 @@ describe('Data Schematic', () => {
     ).toBeGreaterThanOrEqual(0);
   });
 
-  it('should create two files if spec is true', () => {
+  it('should create two files if skipTest is false(as it is by default)', () => {
     const options = {
       ...defaultOptions,
-      spec: true,
     };
     const tree = schematicRunner.runSchematic('data', options, appTree);
     expect(
@@ -76,7 +74,7 @@ describe('Data Schematic', () => {
 
   describe('service class', () => {
     it('should import the correct model', () => {
-      const options = { ...defaultOptions, spec: true };
+      const options = { ...defaultOptions };
       const tree = schematicRunner.runSchematic('data', options, appTree);
       const fileContent = tree.readContent(
         `${projectPath}/src/app/foo.service.ts`
@@ -86,7 +84,7 @@ describe('Data Schematic', () => {
     });
 
     it('should extending EntityCollectionServiceBase', () => {
-      const options = { ...defaultOptions, spec: true };
+      const options = { ...defaultOptions };
       const tree = schematicRunner.runSchematic('data', options, appTree);
       const fileContent = tree.readContent(
         `${projectPath}/src/app/foo.service.ts`
@@ -101,7 +99,7 @@ describe('Data Schematic', () => {
   });
 
   it('should create model', () => {
-    const options = { ...defaultOptions, spec: true };
+    const options = { ...defaultOptions };
     const tree = schematicRunner.runSchematic('data', options, appTree);
     const fileContent = tree.readContent(`${projectPath}/src/app/foo.ts`);
 
@@ -110,7 +108,7 @@ describe('Data Schematic', () => {
   });
 
   it('should create spec class with right imports', () => {
-    const options = { ...defaultOptions, spec: true };
+    const options = { ...defaultOptions };
     const tree = schematicRunner.runSchematic('data', options, appTree);
     const fileContent = tree.readContent(
       `${projectPath}/src/app/foo.service.spec.ts`

modules/schematics/src/data/index.ts

@@ -28,9 +28,9 @@ export default function(options: DataOptions): Rule {
     options.path = parsedPath.path;
 
     const templateSource = apply(url('./files'), [
-      options.spec
-        ? noop()
-        : filter(path => !path.endsWith('.spec.ts.template')),
+      options.skipTest
+        ? filter(path => !path.endsWith('.spec.ts.template'))
+        : noop(),
       applyTemplates({
         ...stringUtils,
         'if-flat': (s: string) =>

modules/schematics/src/data/schema.json

@@ -23,9 +23,9 @@
       "description": "The name of the project.",
       "aliases": ["p"]
     },
-    "spec": {
+    "skipTest": {
       "type": "boolean",
-      "description": "Specifies if a spec file is generated.",
+      "description": "When true, does not create test files.",
       "default": false
     },
     "flat": {

modules/schematics/src/data/schema.ts

@@ -15,9 +15,9 @@ export interface Schema {
   project?: string;
 
   /**
-   * Specifies if a spec file is generated.
+   * When true, does not create test files.
    */
-  spec?: boolean;
+  skipTest?: boolean;
 
   /**
    * Flag to indicate if a dir is created.

modules/schematics/src/effect/index.spec.ts

@@ -21,7 +21,6 @@ describe('Effect Schematic', () => {
   const defaultOptions: EffectOptions = {
     name: 'foo',
     project: 'bar',
-    spec: true,
     module: undefined,
     flat: false,
     feature: false,
@@ -104,8 +103,8 @@ describe('Effect Schematic', () => {
     expect(thrownError).toBeDefined();
   });
 
-  it('should respect the spec flag', () => {
-    const options = { ...defaultOptions, spec: false };
+  it('should respect the skipTest flag', () => {
+    const options = { ...defaultOptions, skipTest: true };
 
     const tree = schematicRunner.runSchematic('effect', options, appTree);
     const files = tree.files;
@@ -241,7 +240,12 @@ describe('Effect Schematic', () => {
   });
 
   it('should group within an "effects" folder if group is set', () => {
-    const options = { ...defaultOptions, flat: true, spec: false, group: true };
+    const options = {
+      ...defaultOptions,
+      flat: true,
+      skipTest: true,
+      group: true,
+    };
 
     const tree = schematicRunner.runSchematic('effect', options, appTree);
     const files = tree.files;
@@ -253,7 +257,7 @@ describe('Effect Schematic', () => {
   it('should group and nest the effect within a feature', () => {
     const options = {
       ...defaultOptions,
-      spec: false,
+      skipTest: true,
       group: true,
       flat: false,
       feature: true,
@@ -405,7 +409,7 @@ describe('Effect Schematic', () => {
   });
 
   it('should inject the effect service correctly', async () => {
-    const options = { ...defaultOptions, spec: true };
+    const options = { ...defaultOptions };
     const tree = await schematicRunner
       .runSchematicAsync('effect', options, appTree)
       .toPromise();

modules/schematics/src/effect/index.ts

@@ -130,9 +130,9 @@ export default function(options: EffectOptions): Rule {
     options.path = parsedPath.path;
 
     const templateSource = apply(url('./files'), [
-      options.spec
-        ? noop()
-        : filter(path => !path.endsWith('.spec.ts.template')),
+      options.skipTest
+        ? filter(path => !path.endsWith('.spec.ts.template'))
+        : noop(),
       options.root && options.minimal ? filter(_ => false) : noop(),
       applyTemplates({
         ...stringUtils,