Consolidate test cases for discussion

src/utils/systemJsRendering.ts

@@ -5,12 +5,12 @@ export function getSystemExportStatement(exportedVariables: Variable[]): string
 		return `exports('${exportedVariables[0].safeExportName ||
 			exportedVariables[0].exportName[0]}', ${exportedVariables[0].getName()})`;
 	} else {
-		return `exports({ ${exportedVariables
+		return `exports({${exportedVariables
 			.map(variable =>
 				// TODO: do we always have export names at this point?
 				variable.exportName!.map(exportName => `${exportName}: ${variable.getName()}`)
 				.join(', ')
 			)
-			.join(', ')} })`;
+			.join(', ')}})`;
 	}
 }

test/form/samples/system-multiple-export-bindings/_config.js

@@ -0,0 +1,9 @@
+module.exports = {
+	description: 'supports multiple live bindings for the same symbol in systemJS',
+	options: {
+		output: {
+			format: 'system',
+			exports: 'named'
+		}
+	}
+};

test/form/samples/system-multiple-export-bindings/_expected.js

@@ -0,0 +1,37 @@
+System.register([], function (exports) {
+	'use strict';
+	return {
+		execute: function () {
+
+			const x = exports('x', 123);
+			const y = exports('y', 456);
+
+			var namespace = /*#__PURE__*/Object.freeze({
+				__proto__: null,
+				x: x,
+				y: y
+			});
+			exports({namespace: namespace, namespace2: namespace});
+
+			// Namespace variable
+
+			// Variable Declaration
+			let a = exports('default', exports('a2', exports('a', 1))), b = exports('b', 2), c = exports('c2', exports('c' = 3));
+
+			// Export default expression
+
+			// Assignment Expression
+			a = exports('default', exports('a2', exports('a', b = exports('b', c = exports('c2', exports('c', 0))))));
+
+			// Destructing Assignment Expression
+			(function (v) {exports({a: a, a2: a, default: a, b: b, c: c, c2: c}); return v;} ({a, b, c} = { c: 4, b: 5, a: 6 }));
+
+			// Update Expression
+			a = exports('default', exports('a2', exports('a', a + 1))), b = exports('b', b + 1), c = exports('c2', exports('c', c + 1));
+
+			// Class Declaration
+			export class A {} exports({ A: A, B: A });
+
+		}
+	};
+});

test/form/samples/system-multiple-export-bindings/main.js

@@ -0,0 +1,25 @@
+// Namespace variable
+import * as namespace from './namespace';
+export { namespace, namespace as namespace2 }
+export * from './namespace';
+
+// Variable Declaration
+export let a = 1, b = 2, c = 3;
+export { a as a2, c as c2 }
+
+// Export default expression
+export default a;
+
+// Assignment Expression
+a = b = c = 0;
+
+// Destructing Assignment Expression
+({ a, b, c } = { c: 4, b: 5, a: 6 });
+
+// Update Expression
+a++, b++, ++c;
+
+// Class Declaration
+export class A {} 
+export { A as B }
+

test/form/samples/system-multiple-export-bindings/namespace.js

@@ -0,0 +1,2 @@
+export const x = 123;
+export const y = 456;