fix(karma): After adding 2nd webpack configuration

karma.conf.js

@@ -16,7 +16,6 @@ module.exports = function(config) {
         files: [
             './doc/example/libs/jquery-2.1.1.js',
             'node_modules/core-js/index.js',
-            './index.js',
             './modules/**/*.spec.js'
         ],
 
@@ -29,7 +28,6 @@ module.exports = function(config) {
         //  https://npmjs.org/browse/keyword/karma-preprocessor
         preprocessors: {
             'node_modules/core-js/**': [ 'webpack' ],
-            './index.js': [ 'webpack' ],
             './**/*.spec.js': [ 'webpack', 'sourcemap' ]
         },
 
@@ -62,6 +60,6 @@ module.exports = function(config) {
         // if true, Karma captures browsers, runs the tests and exits
         singleRun: true,
 
-        webpack: require('./webpack.config.js')
+        webpack: require('./webpack-shared-config')
     });
 };

webpack-shared-config.js

@@ -0,0 +1,93 @@
+/* global __dirname */
+
+const process = require('process');
+const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
+
+const analyzeBundle = process.argv.indexOf('--analyze-bundle') !== -1;
+
+const minimize
+    = process.argv.indexOf('-p') !== -1
+        || process.argv.indexOf('--optimize-minimize') !== -1;
+
+module.exports = {
+    // The inline-source-map is used to allow debugging the unit tests with Karma
+    devtool: minimize ? 'source-map' : 'inline-source-map',
+    mode: minimize ? 'production' : 'development',
+    module: {
+        rules: [ {
+            // Version this build of the lib-jitsi-meet library.
+
+            loader: 'string-replace-loader',
+            options: {
+                flags: 'g',
+                replace:
+                    process.env.LIB_JITSI_MEET_COMMIT_HASH || 'development',
+                search: '{#COMMIT_HASH#}'
+            },
+            test: `${__dirname}/JitsiMeetJS.js`
+        }, {
+            // Transpile ES2015 (aka ES6) to ES5.
+
+            exclude: [
+                new RegExp(`${__dirname}/node_modules/(?!@jitsi/js-utils)`)
+            ],
+            loader: 'babel-loader',
+            options: {
+                presets: [
+                    [
+                        '@babel/preset-env',
+
+                        // Tell babel to avoid compiling imports into CommonJS
+                        // so that webpack may do tree shaking.
+                        {
+                            modules: false,
+
+                            // Specify our target browsers so no transpiling is
+                            // done unnecessarily. For browsers not specified
+                            // here, the ES2015+ profile will be used.
+                            targets: {
+                                chrome: 58,
+                                electron: 2,
+                                firefox: 54,
+                                safari: 11
+                            }
+                        }
+                    ],
+                    '@babel/preset-flow'
+                ],
+                plugins: [
+                    '@babel/plugin-transform-flow-strip-types',
+                    '@babel/plugin-proposal-class-properties',
+                    '@babel/plugin-proposal-export-namespace-from'
+                ]
+            },
+            test: /\.js$/
+        } ]
+    },
+    node: {
+        // Allow the use of the real filename of the module being executed. By
+        // default Webpack does not leak path-related information and provides a
+        // value that is a mock (/index.js).
+        __filename: true
+    },
+    optimization: {
+        concatenateModules: minimize
+    },
+    output: {
+        filename: `[name]${minimize ? '.min' : ''}.js`,
+        path: process.cwd(),
+        sourceMapFilename: `[name].${minimize ? 'min' : 'js'}.map`
+    },
+    performance: {
+        hints: minimize ? 'error' : false,
+        maxAssetSize: 750 * 1024,
+        maxEntrypointSize: 750 * 1024
+    },
+    plugins: [
+        analyzeBundle
+            && new BundleAnalyzerPlugin({
+                analyzerMode: 'disabled',
+                generateStatsFile: true
+            })
+    ].filter(Boolean)
+};

webpack.config.js

@@ -1,96 +1,6 @@
-/* global __dirname */
-
 const process = require('process');
-const { BundleAnalyzerPlugin } = require('webpack-bundle-analyzer');
-
-const analyzeBundle = process.argv.indexOf('--analyze-bundle') !== -1;
-
-const minimize
-    = process.argv.indexOf('-p') !== -1
-        || process.argv.indexOf('--optimize-minimize') !== -1;
-
-const config = {
-    // The inline-source-map is used to allow debugging the unit tests with Karma
-    devtool: minimize ? 'source-map' : 'inline-source-map',
-    mode: minimize ? 'production' : 'development',
-    module: {
-        rules: [ {
-            // Version this build of the lib-jitsi-meet library.
-
-            loader: 'string-replace-loader',
-            options: {
-                flags: 'g',
-                replace:
-                    process.env.LIB_JITSI_MEET_COMMIT_HASH || 'development',
-                search: '{#COMMIT_HASH#}'
-            },
-            test: `${__dirname}/JitsiMeetJS.js`
-        }, {
-            // Transpile ES2015 (aka ES6) to ES5.
-
-            exclude: [
-                new RegExp(`${__dirname}/node_modules/(?!@jitsi/js-utils)`)
-            ],
-            loader: 'babel-loader',
-            options: {
-                presets: [
-                    [
-                        '@babel/preset-env',
-
-                        // Tell babel to avoid compiling imports into CommonJS
-                        // so that webpack may do tree shaking.
-                        {
-                            modules: false,
 
-                            // Specify our target browsers so no transpiling is
-                            // done unnecessarily. For browsers not specified
-                            // here, the ES2015+ profile will be used.
-                            targets: {
-                                chrome: 58,
-                                electron: 2,
-                                firefox: 54,
-                                safari: 11
-                            }
-                        }
-                    ],
-                    '@babel/preset-flow'
-                ],
-                plugins: [
-                    '@babel/plugin-transform-flow-strip-types',
-                    '@babel/plugin-proposal-class-properties',
-                    '@babel/plugin-proposal-export-namespace-from'
-                ]
-            },
-            test: /\.js$/
-        } ]
-    },
-    node: {
-        // Allow the use of the real filename of the module being executed. By
-        // default Webpack does not leak path-related information and provides a
-        // value that is a mock (/index.js).
-        __filename: true
-    },
-    optimization: {
-        concatenateModules: minimize
-    },
-    output: {
-        filename: `[name]${minimize ? '.min' : ''}.js`,
-        path: process.cwd(),
-        sourceMapFilename: `[name].${minimize ? 'min' : 'js'}.map`
-    },
-    performance: {
-        hints: minimize ? 'error' : false,
-        maxAssetSize: 750 * 1024,
-        maxEntrypointSize: 750 * 1024
-    },
-    plugins: [
-        analyzeBundle
-            && new BundleAnalyzerPlugin({
-                analyzerMode: 'disabled',
-                generateStatsFile: true
-            })
-    ].filter(Boolean)
-};
+const config = require('./webpack-shared-config');
 
 module.exports = [
     Object.assign({}, config, {