fix test-watch

Author: Foxandxss

webpack.config.js

@@ -12,6 +12,8 @@ var CopyWebpackPlugin = require('copy-webpack-plugin');
  * Get npm lifecycle event to identify the environment
  */
 var ENV = process.env.npm_lifecycle_event;
+var isTest = ENV === 'test' || ENV === 'test-watch';
+var isProd = ENV === 'build';
 
 module.exports = function makeWebpackConfig () {
   /**
@@ -27,7 +29,7 @@ module.exports = function makeWebpackConfig () {
    * Should be an empty object if it's generating a test build
    * Karma will set this when it's a test build
    */
-  config.entry = ENV === 'test' ? {} : {
+  config.entry = isTest ? {} : {
     app: './src/app/app.js'
   };
 
@@ -37,31 +39,31 @@ module.exports = function makeWebpackConfig () {
    * Should be an empty object if it's generating a test build
    * Karma will handle setting it up for you when it's a test build
    */
-  config.output = ENV === 'test' ? {} : {
+  config.output = isTest ? {} : {
     // Absolute output directory
     path: __dirname + '/dist',
 
     // Output path from the view of the page
     // Uses webpack-dev-server in development
-    publicPath: ENV === 'build' ? '/' : 'http://localhost:8080/',
+    publicPath: isProd ? '/' : 'http://localhost:8080/',
 
     // Filename for entry points
     // Only adds hash in build mode
-    filename: ENV === 'build' ? '[name].[hash].js' : '[name].bundle.js',
+    filename: isProd ? '[name].[hash].js' : '[name].bundle.js',
 
     // Filename for non-entry points
     // Only adds hash in build mode
-    chunkFilename: ENV === 'build' ? '[name].[hash].js' : '[name].bundle.js'
+    chunkFilename: isProd ? '[name].[hash].js' : '[name].bundle.js'
   };
 
   /**
    * Devtool
    * Reference: http://webpack.github.io/docs/configuration.html#devtool
    * Type of sourcemap to use per build type
    */
-  if (ENV === 'test') {
+  if (isTest) {
     config.devtool = 'inline-source-map';
-  } else if (ENV === 'build') {
+  } else if (isProd) {
     config.devtool = 'source-map';
   } else {
     config.devtool = 'eval-source-map';
@@ -98,7 +100,7 @@ module.exports = function makeWebpackConfig () {
       //
       // Reference: https://github.com/webpack/style-loader
       // Use style-loader in development.
-      loader: ENV === 'test' ? 'null' : ExtractTextPlugin.extract('style', 'css?sourceMap!postcss')
+      loader: isTest ? 'null' : ExtractTextPlugin.extract('style', 'css?sourceMap!postcss')
     }, {
       // ASSET LOADER
       // Reference: https://github.com/webpack/file-loader
@@ -121,7 +123,7 @@ module.exports = function makeWebpackConfig () {
   // Reference: https://github.com/ColCh/isparta-instrumenter-loader
   // Instrument JS files with Isparta for subsequent code coverage reporting
   // Skips node_modules and files that end with .test.js
-  if (ENV === 'test') {
+  if (isTest) {
     config.module.preLoaders.push({
       test: /\.js$/,
       exclude: [
@@ -151,7 +153,7 @@ module.exports = function makeWebpackConfig () {
   config.plugins = [];
 
   // Skip rendering index.html in test mode
-  if (ENV !== 'test') {
+  if (!isTest) {
     // Reference: https://github.com/ampedandwired/html-webpack-plugin
     // Render index.html
     config.plugins.push(
@@ -163,12 +165,12 @@ module.exports = function makeWebpackConfig () {
       // Reference: https://github.com/webpack/extract-text-webpack-plugin
       // Extract css files
       // Disabled when in test mode or not in build mode
-      new ExtractTextPlugin('[name].[hash].css', {disable: ENV !== 'build'})
+      new ExtractTextPlugin('[name].[hash].css', {disable: !isProd})
     )
   }
 
   // Add build specific plugins
-  if (ENV === 'build') {
+  if (isProd) {
     config.plugins.push(
       // Reference: http://webpack.github.io/docs/list-of-plugins.html#noerrorsplugin
       // Only emit files when there are no errors