Merge pull request #58 from rollup/Fix-#57

Fix #57

src/finalisers/umd.js

@@ -11,7 +11,7 @@ export default function umd ( bundle, magicString, { exportMode, indentString },
 	let amdDeps = bundle.externalModules.map( quoteId );
 	let cjsDeps = bundle.externalModules.map( req );
 	let globalDeps = bundle.externalModules.map( module => {
-		return globalNames[ module.id ] || module.name;
+		return 'global.' + (globalNames[ module.id ] || module.name);
 	});
 
 	let args = bundle.externalModules.map( getName );

test/form/external-imports/_config.js

@@ -0,0 +1,6 @@
+module.exports = {
+	description: 'prefixes global names with `global.` when creating UMD bundle (#57)',
+	options: {
+		external: [ 'factory' ]
+	}
+};

test/form/external-imports/_expected/amd.js

@@ -0,0 +1,5 @@
+define(['factory'], function (factory) { 'use strict';
+
+	factory( null );
+
+});

test/form/external-imports/_expected/cjs.js

@@ -0,0 +1,6 @@
+'use strict';
+
+var factory = require('factory');
+factory = 'default' in factory ? factory['default'] : factory;
+
+factory( null );

test/form/external-imports/_expected/es6.js

@@ -0,0 +1 @@
+factory( null );

test/form/external-imports/_expected/iife.js

@@ -0,0 +1,5 @@
+(function (factory) { 'use strict';
+
+	factory( null );
+
+})(factory);

test/form/external-imports/_expected/umd.js

@@ -0,0 +1,9 @@
+(function (global, factory) {
+	typeof exports === 'object' && typeof module !== 'undefined' ? factory(require('factory')) :
+	typeof define === 'function' && define.amd ? define(['factory'], factory) :
+	factory(global.factory);
+}(this, function (factory) { 'use strict';
+
+	factory( null );
+
+}));

test/form/external-imports/main.js

@@ -0,0 +1,2 @@
+import factory from 'factory';
+factory( null );