Store externals in an array. This fixes #471

Author: David Ellingsworth

lib/WebpackConfig.js

@@ -47,7 +47,7 @@ class WebpackConfig {
         this.providedVariables = {};
         this.configuredFilenames = {};
         this.aliases = {};
-        this.externals = {};
+        this.externals = [];
 
         // Features/Loaders flags
         this.useVersioning = false;
@@ -298,12 +298,12 @@ class WebpackConfig {
         Object.assign(this.aliases, aliases);
     }
 
-    addExternals(externals = {}) {
-        if (typeof externals !== 'object') {
-            throw new Error('Argument 1 to addExternals() must be an object.');
+    addExternals(externals = []) {
+        if (!Array.isArray(externals)) {
+            externals = [externals];
         }
 
-        Object.assign(this.externals, externals);
+        this.externals = this.externals.concat(externals);
     }
 
     enableVersioning(enabled = true) {

lib/config-generator.js

@@ -107,7 +107,7 @@ class ConfigGenerator {
             config.resolve.alias['react-dom'] = 'preact-compat';
         }
 
-        config.externals = Object.assign({}, this.webpackConfig.externals);
+        config.externals = this.webpackConfig.externals;
 
         return config;
     }

test/WebpackConfig.js

@@ -940,24 +940,15 @@ describe('WebpackConfig object', () => {
         it('Adds new externals', () => {
             const config = createConfig();
 
-            expect(config.externals).to.deep.equals({});
+            expect(config.externals).to.deep.equals([]);
 
             config.addExternals({ 'jquery': 'jQuery', 'react': 'react' });
             config.addExternals({ 'lodash': 'lodash' });
 
-            expect(config.externals).to.deep.equals({
-                'jquery': 'jQuery',
-                'react': 'react',
-                'lodash': 'lodash'
-            });
-        });
-
-        it('Calling it with an invalid argument', () => {
-            const config = createConfig();
-
-            expect(() => {
-                config.addExternals('foo');
-            }).to.throw('must be an object');
+            expect(config.externals).to.deep.equals([
+                { 'jquery': 'jQuery', 'react': 'react'},
+                { 'lodash': 'lodash' }
+            ]);
         });
     });
 

test/config-generator.js

@@ -471,7 +471,7 @@ describe('The config-generator function', () => {
 
             const actualConfig = configGenerator(config);
 
-            expect(actualConfig.externals).to.deep.equals({});
+            expect(actualConfig.externals).to.deep.equals([]);
         });
 
         it('with addExternals()', () => {
@@ -485,10 +485,10 @@ describe('The config-generator function', () => {
 
             const actualConfig = configGenerator(config);
 
-            expect(actualConfig.externals).to.deep.equals({
+            expect(actualConfig.externals).to.deep.equals([{
                 'jquery': 'jQuery',
                 'react': 'react'
-            });
+            }]);
         });
     });