refactor: code

Author: cap-Bernardito

test/__snapshots__/loader.test.js.snap

@@ -275,9 +275,9 @@ Object {
 
 exports[`loader should work multiple commonjs exports: warnings 1`] = `Array []`;
 
-exports[`loader should work multiple name from one library: errors 1`] = `Array []`;
+exports[`loader should work multiple name from one package: errors 1`] = `Array []`;
 
-exports[`loader should work multiple name from one library: module 1`] = `
+exports[`loader should work multiple name from one package: module 1`] = `
 "const globalObject4 = { foo: 'bar' };
 
 module.exports = globalObject4;
@@ -290,7 +290,7 @@ ___EXPOSE_LOADER_GLOBAL_THIS___[\\"myGlobal\\"] = ___EXPOSE_LOADER_IMPORT___;
 "
 `;
 
-exports[`loader should work multiple name from one library: result 1`] = `
+exports[`loader should work multiple name from one package: result 1`] = `
 Object {
   "ExposeLoader": Object {
     "foo": "bar",
@@ -304,7 +304,7 @@ Object {
 }
 `;
 
-exports[`loader should work multiple name from one library: warnings 1`] = `Array []`;
+exports[`loader should work multiple name from one package: warnings 1`] = `Array []`;
 
 exports[`loader should work multiple syntax to array: errors 1`] = `Array []`;
 

test/loader.test.js

@@ -62,22 +62,6 @@ describe('loader', () => {
     expect(getWarnings(stats)).toMatchSnapshot('warnings');
   });
 
-  it('should work multiple name from one library', async () => {
-    const compiler = getCompiler('simple-commonjs2-single-export.js', {
-      exposes: ['myOtherGlobal', 'myGlobal'],
-    });
-    const stats = await compile(compiler);
-
-    expect(
-      getModuleSource('./global-commonjs2-single-export.js-exposed', stats)
-    ).toMatchSnapshot('module');
-    expect(
-      execute(readAsset('main.bundle.js', compiler, stats))
-    ).toMatchSnapshot('result');
-    expect(getErrors(stats)).toMatchSnapshot('errors');
-    expect(getWarnings(stats)).toMatchSnapshot('warnings');
-  });
-
   it('should work for a nested property for a global object', async () => {
     const compiler = getCompiler('simple-commonjs2-single-export.js', {
       exposes: 'myGlobal.nested',
@@ -267,6 +251,40 @@ describe('loader', () => {
     expect(getWarnings(stats)).toMatchSnapshot('warnings');
   });
 
+  it('should work multiple name from one package', async () => {
+    const compiler = getCompiler(
+      'simple-commonjs2-single-export.js',
+      {
+        exposes: ['myOtherGlobal', 'myGlobal'],
+      },
+      {
+        output: {
+          path: path.resolve(__dirname, './outputs'),
+          filename: '[name]-[contenthash:8].bundle.js',
+          chunkFilename: '[name]-[contenthash:8].chunk.js',
+          library: 'ExposeLoader',
+          libraryTarget: 'var',
+        },
+      }
+    );
+    const stats = await compile(compiler);
+
+    const webpack4Filename = 'main-bd1693c3.bundle.js';
+    const webpack5Filename = 'main-b4700c39.bundle.js';
+    const bundleName =
+      webpack.version[0] === '5' ? webpack5Filename : webpack4Filename;
+
+    expect(
+      getModuleSource('./global-commonjs2-single-export.js-exposed', stats)
+    ).toMatchSnapshot('module');
+    expect(Object.keys(stats.compilation.assets)[0]).toBe(bundleName);
+    expect(execute(readAsset(bundleName, compiler, stats))).toMatchSnapshot(
+      'result'
+    );
+    expect(getErrors(stats)).toMatchSnapshot('errors');
+    expect(getWarnings(stats)).toMatchSnapshot('warnings');
+  });
+
   it('should match hashes on all operating systems', async () => {
     const compiler = getCompiler(
       'inline-import-2.js',