refactor test vendor logic

Author: filipesilva

packages/@angular/cli/models/webpack-configs/test.ts

@@ -70,18 +70,6 @@ export function getTestConfig(testConfig: WebpackTestOptions) {
     });
   }
 
-  if (true) {
-    extraPlugins.push(new webpack.optimize.CommonsChunkPlugin({
-      minChunks: Infinity,
-      name: 'inline'
-    }));
-    extraPlugins.push(new webpack.optimize.CommonsChunkPlugin({
-      name: 'vendor',
-      chunks: ['main'],
-      minChunks: (module: any) => module.resource && module.resource.startsWith(nodeModules)
-    }));
-  }
-
   return {
     devtool: testConfig.sourcemaps ? 'inline-source-map' : 'eval',
     entry: {
@@ -91,6 +79,15 @@ export function getTestConfig(testConfig: WebpackTestOptions) {
       rules: [].concat(extraRules)
     },
     plugins: [
+      new webpack.optimize.CommonsChunkPlugin({
+        minChunks: Infinity,
+        name: 'inline'
+      }),
+      new webpack.optimize.CommonsChunkPlugin({
+        name: 'vendor',
+        chunks: ['main'],
+        minChunks: (module: any) => module.resource && module.resource.startsWith(nodeModules)
+      }),
       // new webpackBuildLogger({ logEnabled: true }),
       // new (<any>FileListPlugin)(),
       new KarmaWebpackEmitlessError()