fix(schematics): use skipTests flag consistently, deprecate skipTest option (#2522)

Closes #2521

Author: aszechlicki

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

@@ -124,8 +124,8 @@ describe('Effects ng-add Schematic', () => {
     expect(thrownError).toBeDefined();
   });
 
-  it('should respect the skipTest flag', () => {
-    const options = { ...defaultOptions, skipTest: true };
+  it('should respect the skipTests flag', () => {
+    const options = { ...defaultOptions, skipTests: true };
 
     const tree = schematicRunner.runSchematic('ng-add', options, appTree);
     const files = tree.files;
@@ -203,7 +203,7 @@ describe('Effects ng-add Schematic', () => {
     const options = {
       ...defaultOptions,
       flat: true,
-      skipTest: true,
+      skipTests: true,
       group: true,
     };
 
@@ -215,7 +215,7 @@ describe('Effects ng-add Schematic', () => {
   });
 
   it('should inject the effect service correctly', async () => {
-    const options = { ...defaultOptions, skipTest: false };
+    const options = { ...defaultOptions, skipTests: false };
     const tree = await schematicRunner
       .runSchematicAsync('ng-add', options, appTree)
       .toPromise();

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

@@ -124,12 +124,16 @@ export default function(options: EffectOptions): Rule {
       options.module = findModuleFromOptions(host, options);
     }
 
+    if (!options.skipTests && options.skipTest) {
+      options.skipTests = options.skipTest;
+    }
+
     const parsedPath = parseName(options.path, options.name || '');
     options.name = parsedPath.name;
     options.path = parsedPath.path;
 
     const templateSource = apply(url('./files'), [
-      options.skipTest
+      options.skipTests
         ? filter(path => !path.endsWith('.spec.ts.template'))
         : noop(),
       options.minimal ? filter(_ => false) : noop(),

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

@@ -27,6 +27,12 @@
       "description": "Flag to indicate if a dir is created."
     },
     "skipTest": {
+      "type": "boolean",
+      "description": "When true, does not create test files.",
+      "x-deprecated": "Use skipTests instead.",
+      "default": false
+    },
+    "skipTests": {
       "type": "boolean",
       "default": false,
       "description": "When true, does not create test files."

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

@@ -3,7 +3,9 @@ export interface Schema {
   skipPackageJson?: boolean;
   path?: string;
   flat?: boolean;
+  /** @deprecated renamed to skipTests, use skipTests instead */
   skipTest?: boolean;
+  skipTests?: boolean;
   project?: string;
   module?: string;
   group?: boolean;

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

@@ -73,15 +73,15 @@ describe('Action Schematic', () => {
     ).toBeGreaterThanOrEqual(0);
   });
 
-  it('should not create two files test files when skipTests is set to true', () => {
+  it('should not create test files when skipTests is set to true', () => {
     const options = {
       ...defaultOptions,
-      skipTests: false,
+      skipTests: true,
     };
     const tree = schematicRunner.runSchematic('action', options, appTree);
     expect(
       tree.files.indexOf(`${projectPath}/src/app/foo.actions.spec.ts`)
-    ).toBeGreaterThanOrEqual(0);
+    ).toEqual(-1);
     expect(
       tree.files.indexOf(`${projectPath}/src/app/foo.actions.ts`)
     ).toBeGreaterThanOrEqual(0);

modules/schematics/src/action/index.ts

@@ -25,6 +25,10 @@ export default function(options: ActionOptions): Rule {
   return (host: Tree, context: SchematicContext) => {
     options.path = getProjectPath(host, options);
 
+    if (!options.skipTests && options.skipTest) {
+      options.skipTests = options.skipTest;
+    }
+
     const parsedPath = parseName(options.path, options.name);
     options.name = parsedPath.name;
     options.path = parsedPath.path;

modules/schematics/src/action/schema.json

@@ -25,6 +25,12 @@
       "aliases": ["p"]
     },
     "skipTest": {
+      "type": "boolean",
+      "description": "When true, does not create test files.",
+      "x-deprecated": "Use skipTests instead.",
+      "default": false
+    },
+    "skipTests": {
       "type": "boolean",
       "description": "When true, does not create test files.",
       "default": false

modules/schematics/src/action/schema.ts

@@ -14,6 +14,11 @@ export interface Schema {
    */
   project?: string;
 
+  /**
+   * When true, does not create test files.
+   * @deprecated Use skipTests instead
+   */
+  skipTest?: boolean;
   /**
    * When true, does not create test files.
    */

modules/schematics/src/container/index.ts

@@ -122,6 +122,10 @@ export default function(options: ContainerOptions): Rule {
   return (host: Tree, context: SchematicContext) => {
     options.path = getProjectPath(host, options);
 
+    if (!options.skipTests && options.skipTest) {
+      options.skipTests = options.skipTest;
+    }
+
     const parsedPath = parseName(options.path, options.name);
     options.name = parsedPath.name;
     options.path = parsedPath.path;
@@ -136,7 +140,7 @@ export default function(options: ContainerOptions): Rule {
     const templateSource = apply(
       url(options.testDepth === 'unit' ? './files' : './integration-files'),
       [
-        options.skipTest
+        options.skipTests
           ? filter(path => !path.endsWith('.spec.ts.template'))
           : noop(),
         applyTemplates({

modules/schematics/src/container/schema.json

@@ -58,6 +58,12 @@
       "type": "string"
     },
     "skipTest": {
+      "type": "boolean",
+      "description": "When true, does not create test files.",
+      "x-deprecated": "Use skipTests instead.",
+      "default": false
+    },
+    "skipTests": {
       "type": "boolean",
       "description": "When true, does not create test files."
     },