feat(@schematics/angular): add strict compiler option to workspace tsconfig (#14905)

Closes #14694

Author: alan-agius4

packages/schematics/angular/workspace/files/tsconfig.json.template

@@ -2,7 +2,12 @@
   "compileOnSave": false,
   "compilerOptions": {
     "baseUrl": "./",
-    "outDir": "./dist/out-tsc",
+    "outDir": "./dist/out-tsc",<% if (strict) { %>
+    "noImplicitAny": true,
+    "noImplicitReturns": true,
+    "noImplicitThis": true,
+    "noFallthroughCasesInSwitch": true,
+    "strictNullChecks": true,<% } %>
     "sourceMap": true,
     "declaration": false,
     "downlevelIteration": true,

packages/schematics/angular/workspace/index_spec.ts

@@ -71,4 +71,16 @@ describe('Workspace Schematic', () => {
     expect(files).not.toContain('/tslint.json');
     expect(files).not.toContain('/.editorconfig');
   });
+
+  it('should not add strict compiler options when false', async () => {
+    const tree = await schematicRunner.runSchematicAsync('workspace', { ...defaultOptions, strict: false }).toPromise();
+    const { compilerOptions } = JSON.parse(tree.readContent('/tsconfig.json'));
+    expect(compilerOptions.strictNullChecks).not.toBeDefined();
+  });
+
+  it('should not add strict compiler options when true', async () => {
+    const tree = await schematicRunner.runSchematicAsync('workspace', { ...defaultOptions, strict: true }).toPromise();
+    const { compilerOptions } = JSON.parse(tree.readContent('/tsconfig.json'));
+    expect(compilerOptions.strictNullChecks).toBe(true);
+  });
 });

packages/schematics/angular/workspace/schema.json

@@ -31,6 +31,11 @@
       "type": "boolean",
       "default": false,
       "x-user-analytics": 14
+    },
+    "strict": {
+      "description": "Creates a workspace with stricter TypeScript compiler options.",
+      "type": "boolean",
+      "default": false
     }
   },
   "required": [

tests/legacy-cli/e2e/tests/build/platform-server.ts

@@ -10,6 +10,8 @@ export default async function () {
   const veEnabled = argv['ve'];
 
   await ng('add', '@nguniversal/express-engine', '--client-project', 'test-project');
+  // todo remove once https://github.com/angular/universal/pull/1229 is landed
+  await silentNpm('install', '@types/express', '--save-dev');
 
   await updateJsonFile('package.json', packageJson => {
     const dependencies = packageJson['dependencies'];

tests/legacy-cli/e2e/tests/misc/browsers.ts

@@ -39,7 +39,7 @@ export default async function () {
   await replaceInFile(
     'e2e/src/app.e2e-spec.ts',
     'await browser.manage().logs().get(logging.Type.BROWSER)',
-    '[]',
+    '[] as any',
   );
 
   // Workaround defect in getText WebDriver implementation for Safari/Edge