refactor: code

Author: cap-Bernardito

src/utils.js

@@ -2,7 +2,7 @@ function resolveExposes(item) {
   let result;
 
   if (typeof item === 'string') {
-    const splittedItem = item.split(' ');
+    const splittedItem = item.split('|');
 
     if (splittedItem.length > 2) {
       throw new Error(`Invalid "${item}" for expose`);

test/__snapshots__/loader.test.js.snap

@@ -34,7 +34,7 @@ exports[`loader should be different modules id: warnings 1`] = `Array []`;
 exports[`loader should emit error because of many arguments: errors 1`] = `
 Array [
   "ModuleBuildError: Module build failed (from \`replaced original path\`):
-Error: Invalid \\"myGlobal_alias globalObject6 excessArgument\\" for expose",
+Error: Invalid \\"myGlobal_alias|globalObject6|excessArgument\\" for expose",
 ]
 `;
 

test/fixtures/inline-import-2.js

@@ -1,4 +1,4 @@
 import myExports from '../../src/cjs.js?exposes=myGlobal!./simple-commonjs2-single-export.js';
-import myExports2 from '../../src/cjs.js?exposes[]=myOtherGlobal.globalObject2%20globalObject2&exposes[]=myOtherGlobal.globalObject3%20globalObject3!./simple-commonjs2-multiple-export.js';
+import myExports2 from '../../src/cjs.js?exposes[]=myOtherGlobal.globalObject2|globalObject2&exposes[]=myOtherGlobal.globalObject3|globalObject3!./simple-commonjs2-multiple-export.js';
 
 export { myExports, myExports2 };

test/fixtures/inline-import.js

@@ -1,3 +1,3 @@
-import myExports from '../../src/cjs.js?exposes=myGlobal%20default!./custom.js?foo=bar';
+import myExports from '../../src/cjs.js?exposes=myGlobal|default!./custom.js?foo=bar';
 
 export default myExports;

test/loader.test.js

@@ -48,7 +48,7 @@ describe('loader', () => {
 
   it('should work multiple commonjs exports', async () => {
     const compiler = getCompiler('simple-commonjs2-multiple-export.js', {
-      exposes: ['myOtherGlobal', 'myGlobal.globalObject2 globalObject2'],
+      exposes: ['myOtherGlobal', 'myGlobal.globalObject2|globalObject2'],
     });
     const stats = await compile(compiler);
 
@@ -80,7 +80,7 @@ describe('loader', () => {
 
   it('should work for nested properties for a global object', async () => {
     const compiler = getCompiler('simple-commonjs2-single-export.js', {
-      exposes: ['myGlobal.nested', 'myOtherGlobal.nested foo'],
+      exposes: ['myGlobal.nested', 'myOtherGlobal.nested|foo'],
     });
     const stats = await compile(compiler);
 
@@ -113,9 +113,9 @@ describe('loader', () => {
   it('should work string config', async () => {
     const compiler = getCompiler('simple-module-named-export.js', {
       exposes: [
-        'myGlobal_alias.globalObject6 globalObject6',
-        'myGlobal_alias.globalObject7 globalObject7',
-        'myGlobal_alias.default default',
+        'myGlobal_alias.globalObject6|globalObject6',
+        'myGlobal_alias.globalObject7|globalObject7',
+        'myGlobal_alias.default|default',
         'myGlobal',
         'myOtherGlobal',
       ],
@@ -162,7 +162,7 @@ describe('loader', () => {
           globalName: ['myGlobal_alias', 'globalObject7'],
           localName: 'globalObject7',
         },
-        'myGlobal_alias.default default',
+        'myGlobal_alias.default|default',
       ],
     });
     const stats = await compile(compiler);
@@ -264,7 +264,7 @@ describe('loader', () => {
     const stats = await compile(compiler);
 
     const webpack4Filename = 'main-b0db002e.bundle.js';
-    const webpack5Filename = 'main-effaed92.bundle.js';
+    const webpack5Filename = 'main-22966fe7.bundle.js';
     const bundleName =
       webpack.version[0] === '5' ? webpack5Filename : webpack4Filename;
 
@@ -312,7 +312,7 @@ describe('loader', () => {
 
   it('should emit error because of many arguments', async () => {
     const compiler = getCompiler('simple-module-named-export.js', {
-      exposes: ['myGlobal_alias globalObject6 excessArgument'],
+      exposes: ['myGlobal_alias|globalObject6|excessArgument'],
     });
     const stats = await compile(compiler);