feat(@schematics/angular): add tslint no-any and typedef rules when i…

…n strict mode
angular · Jun 1, 2020 · cbc755c · cbc755c
1 parent ca42487
commit cbc755c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/packages/schematics/angular/workspace/files/tslint.json.template b/packages/schematics/angular/workspace/files/tslint.json.template
@@ -42,7 +42,8 @@
           "instance-method"
         ]
       }
-    ],
+    ],<% if (strict) { %>
+    "no-any": true,<% } %>
     "no-console": [
       true,
       "debug",
@@ -82,6 +83,7 @@
         "named": "never"
       }
     },
+    "typedef": [true, "call-signature"],
     "typedef-whitespace": {
       "options": [
         {

diff --git a/packages/schematics/angular/workspace/index_spec.ts b/packages/schematics/angular/workspace/index_spec.ts
@@ -81,10 +81,22 @@ describe('Workspace Schematic', () => {
     expect(angularCompilerOptions).toBeUndefined();
   });
 
-  it('should not add strict compiler options when true', async () => {
+  it('should add strict compiler options when true', async () => {
     const tree = await schematicRunner.runSchematicAsync('workspace', { ...defaultOptions, strict: true }).toPromise();
     const { compilerOptions, angularCompilerOptions } = JSON.parse(tree.readContent('/tsconfig.base.json'));
     expect(compilerOptions.strict).toBe(true);
     expect(angularCompilerOptions.strictTemplates).toBe(true);
   });
+
+  it('should not add strict lint options when false', async () => {
+    const tree = await schematicRunner.runSchematicAsync('workspace', { ...defaultOptions, strict: false }).toPromise();
+    const { rules } = JSON.parse(tree.readContent('/tslint.json'));
+    expect(rules['no-any']).toBeUndefined();
+  });
+
+  it('should add strict lint options when true', async () => {
+    const tree = await schematicRunner.runSchematicAsync('workspace', { ...defaultOptions, strict: true }).toPromise();
+    const { rules } = JSON.parse(tree.readContent('/tslint.json'));
+    expect(rules['no-any']).toBe(true);
+  });
 });