diff --git a/common/static/common/js/karma.common.conf.js b/common/static/common/js/karma.common.conf.js index 09f6a21848..16501f47ad 100644 --- a/common/static/common/js/karma.common.conf.js +++ b/common/static/common/js/karma.common.conf.js @@ -49,17 +49,6 @@ var firefox = require('selenium-webdriver/firefox'); var webpackConfig = require(path.join(appRoot, 'webpack.dev.config.js')); -// The following crazy bit is to work around the webpack.optimize.CommonsChunkPlugin -// plugin. The problem is that it it factors out the code that defines webpackJsonp -// and puts in in the commons JS, which Karma doesn't know to load first. This is a -// workaround recommended in the karma-webpack bug report that basically just removes -// the plugin for the purposes of Karma testing (the plugin is meant to be an -// optimization only). -// https://github.com/webpack-contrib/karma-webpack/issues/24#issuecomment-257613167 -// -// This should be fixed in v3 of karma-webpack -var commonsChunkPluginIndex = webpackConfig[0].plugins.findIndex(function(plugin) { return plugin.chunkNames; }); - // Files which are needed by all lms/cms suites. var commonFiles = { libraryFiles: [ @@ -83,10 +72,6 @@ var commonFiles = { ] }; -webpackConfig[0].plugins.splice(commonsChunkPluginIndex, 1); - -delete webpackConfig[0].entry; - /** * Customize the name attribute in xml testcase element * @param {Object} browser