Refine late-entry tests

Author: lukastaegert

packages/commonjs/test/fixtures/function/module-side-effects-late-entry-strict/_config.js

@@ -0,0 +1,38 @@
+const path = require('path');
+const assert = require('assert');
+
+let referenceId;
+const testEntry = 'generated-foo.js';
+
+module.exports = {
+  description:
+    'use correct side-effects flags for files that become entry points after they are loaded (strictRequires: true)',
+  testEntry,
+  options: {
+    treeshake: { moduleSideEffects: false },
+    plugins: [
+      {
+        name: 'test',
+        load(id) {
+          if (id.endsWith('foo.js')) {
+            referenceId = this.emitFile({ type: 'chunk', id: path.join(__dirname, 'foo.js') });
+          }
+        },
+        generateBundle() {
+          assert.strictEqual(this.getFileName(referenceId), testEntry);
+        },
+        moduleParsed({ id, code }) {
+          console.log('\n===', id);
+          console.log(code);
+        }
+      }
+    ],
+    output: { chunkFileNames: 'generated-[name].js' }
+  },
+  pluginOptions: {
+    strictRequires: true
+  },
+  global: (global, t) => {
+    t.is(global.foo, 'foo');
+  }
+};

packages/commonjs/test/fixtures/function/module-side-effects-late-entry-strict/foo.js

@@ -0,0 +1,2 @@
+// This side-effect will only be respected if this is an entry point
+global.foo = 'foo';

packages/commonjs/test/fixtures/function/module-side-effects-late-entry-strict/main.js

@@ -0,0 +1,3 @@
+import './foo.js';
+
+export default 'main';

packages/commonjs/test/fixtures/function/module-side-effects-late-entry/_config.js

@@ -1,23 +1,32 @@
 const path = require('path');
+const assert = require('assert');
+
+let referenceId;
+const testEntry = 'generated-foo2.js';
 
 module.exports = {
   description:
-    'use correct side-effects flags for files that become entry points after they are loaded (strictRequires: "auto")',
+    'use correct side-effects flags for files that become entry points after they are loaded (strictRequires: false)',
+  testEntry,
   options: {
     treeshake: { moduleSideEffects: false },
     plugins: [
       {
+        name: 'test',
         load(id) {
           if (id.endsWith('foo.js')) {
-            this.emitFile({ type: 'chunk', id: path.join(__dirname, 'foo.js') });
+            referenceId = this.emitFile({ type: 'chunk', id: path.join(__dirname, 'foo.js') });
           }
+        },
+        generateBundle() {
+          assert.strictEqual(this.getFileName(referenceId), testEntry);
         }
       }
     ],
     output: { chunkFileNames: 'generated-[name].js' }
   },
   pluginOptions: {
-    strictRequires: 'auto'
+    strictRequires: false
   },
   global: (global, t) => {
     t.is(global.foo, 'foo');

packages/commonjs/test/snapshots/function.js.md

@@ -6575,6 +6575,44 @@ Generated by [AVA](https://avajs.dev).
       `,
     }
 
+## module-side-effects-late-entry-strict
+
+> Snapshot 1
+
+    {
+      'generated-foo.js': `'use strict';␊
+      ␊
+      var commonjsGlobal = typeof globalThis !== 'undefined' ? globalThis : typeof window !== 'undefined' ? window : typeof global !== 'undefined' ? global : typeof self !== 'undefined' ? self : {};␊
+      ␊
+      function getDefaultExportFromCjs (x) {␊
+      	return x && x.__esModule && Object.prototype.hasOwnProperty.call(x, 'default') ? x['default'] : x;␊
+      }␊
+      ␊
+      var foo$1 = {};␊
+      ␊
+      var hasRequiredFoo;␊
+      ␊
+      function requireFoo () {␊
+      	if (hasRequiredFoo) return foo$1;␊
+      	hasRequiredFoo = 1;␊
+      	// This side-effect will only be respected if this is an entry point␊
+      	commonjsGlobal.foo = 'foo';␊
+      	return foo$1;␊
+      }␊
+      ␊
+      var fooExports = requireFoo();␊
+      var foo = /*@__PURE__*/getDefaultExportFromCjs(fooExports);␊
+      ␊
+      module.exports = foo;␊
+      `,
+      'main.js': `'use strict';␊
+      ␊
+      var main = 'main';␊
+      ␊
+      module.exports = main;␊
+      `,
+    }
+
 ## module_require
 
 > Snapshot 1