fix: add environment configuration replacement

Author: TheLarkInn

addon/ng2/blueprints/ng2/files/config/environment.dev.ts renamed to addon/ng2/blueprints/ng2/files/__path__/app/environment.dev.ts

@@ -1,4 +1,5 @@
 import { CliConfig } from './config';
+import { NgCliEnvironmentPlugin } from '../utilities/environment-plugin';
 import {
   getWebpackCommonConfig,
   getWebpackDevConfigPartial,
@@ -35,6 +36,7 @@ export class NgCliWebpackConfig {
     }
 
     this.generateConfig();
+    this.config.plugins.unshift(new NgCliEnvironmentPlugin({env: this.environment}));
   }
 
   generateConfig(): void {
@@ -59,4 +61,3 @@ export class NgCliWebpackConfig {
     }
   }
 }
-

addon/ng2/blueprints/ng2/files/config/environment.prod.ts renamed to addon/ng2/blueprints/ng2/files/__path__/app/environment.prod.ts

@@ -0,0 +1,87 @@
+import * as fs from 'fs';
+
+interface WebpackPlugin {
+  apply(compiler: any): void;
+}
+
+interface NgCliEnvrionmentConfig {
+  env?: string;
+  file?: string;
+  alias?: string;
+}
+
+export class NgCliEnvironmentPlugin implements WebpackPlugin {
+  _file: string;
+  _alias: string;
+  _env: string;
+
+  constructor(config: any) {
+    if (typeof config === 'string') {
+      config = {env: config};
+    }
+    if (typeof config.env !== 'string') {
+      throw new Error('must provide env')
+    }
+    const ALIAS = {
+      '"dev"':         'dev',
+      'development':   'dev',
+      '"development"': 'dev',
+      '"prod"':        'prod',
+      'production':    'prod',
+      '"production"':  'prod',
+      '"test"':        'test',
+      'testing':       'test',
+      '"testing"':     'test',
+    };
+    const ENV = config.env.toLowerCase();
+
+    this._file = config.file || 'environment';
+    this._alias = config.alias || ALIAS;
+    this._env =  this._alias[ENV] || ENV;
+  }
+
+  isEnvFile(file: string): boolean {
+    return file.indexOf(this._file + '.') !== -1;
+  }
+
+  replaceFile(file: string): any {
+    debugger;
+    return file
+      .replace(this._file, this._file + '.' + this._env);
+  }
+
+  updateResult(result: any): any {
+    ['request', 'userRequest', 'resource']
+      .filter((key) => { return result[key]; })
+      .forEach((key) => {
+        result[key] = this.replaceFile(result[key]);
+      });
+
+    return result;
+  }
+
+  apply(compiler: any): void {
+    compiler.plugin('normal-module-factory', (normalModuleFactory: any) => {
+        normalModuleFactory.plugin('after-resolve', (result, callback) => {
+          var _resource = result['resource'];
+          if (!this.isEnvFile(_resource)) {
+            return callback(null, result);
+          }
+          debugger;
+          var envFile = this.replaceFile(_resource);
+
+          fs.stat(envFile, (err, stats) => {
+            if (err || !stats.isFile()) {
+              var errorText = (!err && stats.isFile()) ? 'Is not a file.' : 'Does not exist.';
+              console.log('\nWARNING:\n' + envFile + '\n' + errorText + ' ' + 'Using file\n' + _resource + '\n');
+              return callback(null, result);
+            }
+            // mutate result
+            var newResult = this.updateResult(result);
+            return callback(null, newResult);
+          });
+
+        });
+    });
+  }
+}

addon/ng2/models/webpack-config.ts

@@ -82,11 +82,14 @@ describe('Basic end-to-end Workflow', function () {
     expect(sh.exec('git status --porcelain').output).to.be.equal(undefined);
   });
 
-  xit('Supports production builds config file replacement', function() {
-    var mainBundlePath = path.join(process.cwd(), 'dist', 'main.js');
+  it('Supports production builds config file replacement', function() {
+    this.timeout(420000);
+
+    sh.exec(`${ngBin} build --dev`);
+    var mainBundlePath = path.join(process.cwd(), 'dist', 'main.bundle.js');
     var mainBundleContent = fs.readFileSync(mainBundlePath, { encoding: 'utf8' });
-    // production: true minimized turns into production:!0
-    expect(mainBundleContent).to.include('production:!0');
+    // production: true minimized turns into production:!1
+    expect(mainBundleContent).to.include('production: false');
   });
 
   it_mobile('Enables mobile-specific production features in prod builds', () => {