feat(@schematics/angular): remove environment files in new applications

This commit removes the usage of environment files and `fileReplacements` in new application projects. Previously, the environment files was used to distinguish between a prod build to invoke `enableProdMode`. The `enableProdMode` however needed only for the case of JIT mode in production mode, which is a rare case as JIT mode is recommanded to be used in production.

In the CLI, calling `enableProdMode` is not needed as `ngDevMode` it's set using the minifier.

Author: alan-agius4

packages/angular_devkit/build_angular/src/builders/app-shell/app-shell_spec.ts

@@ -51,7 +51,6 @@ describe('AppShell Builder', () => {
 
       import { AppRoutingModule } from './app-routing.module';
       import { AppComponent } from './app.component';
-      import { environment } from '../environments/environment';
       import { RouterModule } from '@angular/router';
 
       @NgModule({
@@ -91,15 +90,8 @@ describe('AppShell Builder', () => {
       export class AppServerModule {}
     `,
     'src/main.ts': `
-      import { enableProdMode } from '@angular/core';
       import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
-
       import { AppModule } from './app/app.module';
-      import { environment } from './environments/environment';
-
-      if (environment.production) {
-        enableProdMode();
-      }
 
       document.addEventListener('DOMContentLoaded', () => {
         platformBrowserDynamic().bootstrapModule(AppModule)

packages/angular_devkit/build_angular/src/builders/browser/specs/rebuild_spec.ts

@@ -55,15 +55,8 @@ describe('Browser Builder rebuilds', () => {
         export let X = '$$_E2E_GOLDEN_VALUE_2';
       `,
       'src/main.ts': `
-        import { enableProdMode } from '@angular/core';
         import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
-
         import { AppModule } from './app/app.module';
-        import { environment } from './environments/environment';
-
-        if (environment.production) {
-          enableProdMode();
-        }
 
         platformBrowserDynamic().bootstrapModule(AppModule);
 

packages/angular_devkit/build_angular/src/builders/browser/specs/unused-files-warning_spec.ts

@@ -38,11 +38,11 @@ describe('Browser Builder unused files warnings', () => {
   });
 
   it('should show warning when some files are unused', async () => {
-    host.replaceInFile(
-      'src/tsconfig.app.json',
-      '"main.ts"',
-      '"main.ts", "environments/environment.prod.ts"',
-    );
+    host.writeMultipleFiles({
+      'src/unused-file.ts': `export const unused = '1';`,
+    });
+
+    host.replaceInFile('src/tsconfig.app.json', '"main.ts"', '"main.ts", "unused-file.ts"');
 
     const logger = new logging.Logger('');
     const logs: string[] = [];
@@ -51,7 +51,7 @@ describe('Browser Builder unused files warnings', () => {
     const run = await architect.scheduleTarget(targetSpec, undefined, { logger });
     const output = (await run.result) as BrowserBuilderOutput;
     expect(output.success).toBe(true);
-    expect(logs.join().includes(`environment.prod.ts ${warningMessageSuffix}`)).toBe(true);
+    expect(logs.join().includes(`unused-file.ts ${warningMessageSuffix}`)).toBe(true);
 
     await run.stop();
   });

packages/angular_devkit/build_angular/src/builders/browser/specs/vendor-source-map_spec.ts

@@ -107,7 +107,6 @@ describe('Identifying third-party code in source maps', () => {
     expect(mainMap.sources.filter((_, i) => !mainMap[IGNORE_LIST].includes(i))).toEqual([
       './src/app/app.component.ts',
       './src/app/app.module.ts',
-      './src/environments/environment.ts',
       './src/main.ts',
     ]);
 

packages/angular_devkit/build_angular/src/builders/browser/specs/web-worker_spec.ts

@@ -37,11 +37,8 @@ describe('Browser Builder Web Worker support', () => {
       });
     `,
     'src/main.ts': `
-      import { enableProdMode } from '@angular/core';
       import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
       import { AppModule } from './app/app.module';
-      import { environment } from './environments/environment';
-      if (environment.production) { enableProdMode(); }
       platformBrowserDynamic().bootstrapModule(AppModule).catch(err => console.error(err));
 
       const worker = new Worker(new URL('./app/app.worker', import.meta.url), { type: 'module' });

packages/angular_devkit/build_angular/test/hello-world-app/angular.json

@@ -43,12 +43,6 @@
           },
           "configurations": {
             "production": {
-              "fileReplacements": [
-                {
-                  "replace": "src/environments/environment.ts",
-                  "with": "src/environments/environment.prod.ts"
-                }
-              ],
               "optimization": true,
               "outputHashing": "all",
               "sourceMap": false,

packages/angular_devkit/build_angular/test/hello-world-app/src/environments/environment.prod.ts

@@ -7,12 +7,5 @@
  */
 
 import '@angular/localize/init';
-import { enableProdMode } from '@angular/core';
-
-import { environment } from './environments/environment';
-
-if (environment.production) {
-  enableProdMode();
-}
 
 export { AppServerModule } from './app/app.server.module';

packages/angular_devkit/build_angular/test/hello-world-app/src/environments/environment.ts

@@ -6,15 +6,9 @@
  * found in the LICENSE file at https://angular.io/license
  */
 
-import { enableProdMode } from '@angular/core';
 import { platformBrowserDynamic } from '@angular/platform-browser-dynamic';
 
 import { AppModule } from './app/app.module';
-import { environment } from './environments/environment';
-
-if (environment.production) {
-  enableProdMode();
-}
 
 platformBrowserDynamic()
   .bootstrapModule(AppModule)