test(node-resolve): add tests for mixing custom exportConditions wi…

…th `browser: true` (#1043)
rollup · Dec 13, 2021 · 9e6db2d · 9e6db2d
1 parent 8aa447a
commit 9e6db2d
Show file tree

Hide file tree

Showing 10 changed files with 46 additions and 0 deletions.
diff --git a/packages/node-resolve/test/fixtures/exports-only-worker-condition-with-browser-field.js b/packages/node-resolve/test/fixtures/exports-only-worker-condition-with-browser-field.js
@@ -0,0 +1,3 @@
+import sample from 'exports-only-worker-condition-with-browser-field';
+
+export default sample;
diff --git a/packages/node-resolve/test/fixtures/exports-worker-condition-with-browser-field.js b/packages/node-resolve/test/fixtures/exports-worker-condition-with-browser-field.js
@@ -0,0 +1,3 @@
+import sample from 'exports-worker-condition-with-browser-field';
+
+export default sample;
diff --git a/...ures/node_modules/exports-only-worker-condition-with-browser-field/browser-field-entry.js b/...ures/node_modules/exports-only-worker-condition-with-browser-field/browser-field-entry.js
diff --git a/.../test/fixtures/node_modules/exports-only-worker-condition-with-browser-field/package.json b/.../test/fixtures/node_modules/exports-only-worker-condition-with-browser-field/package.json
diff --git a/...ode_modules/exports-only-worker-condition-with-browser-field/webworker-condition-entry.js b/...ode_modules/exports-only-worker-condition-with-browser-field/webworker-condition-entry.js
diff --git a/...tures/node_modules/exports-worker-condition-with-browser-field/browser-condition-entry.js b/...tures/node_modules/exports-worker-condition-with-browser-field/browser-condition-entry.js
diff --git a/.../fixtures/node_modules/exports-worker-condition-with-browser-field/browser-field-entry.js b/.../fixtures/node_modules/exports-worker-condition-with-browser-field/browser-field-entry.js
diff --git a/...solve/test/fixtures/node_modules/exports-worker-condition-with-browser-field/package.json b/...solve/test/fixtures/node_modules/exports-worker-condition-with-browser-field/package.json
diff --git a/...res/node_modules/exports-worker-condition-with-browser-field/webworker-condition-entry.js b/...res/node_modules/exports-worker-condition-with-browser-field/webworker-condition-entry.js
diff --git a/packages/node-resolve/test/package-entry-points.js b/packages/node-resolve/test/package-entry-points.js
@@ -380,3 +380,23 @@ test('does not warn when resolving typescript imports with fallback', async (t)
     a: 'A'
   });
 });
+
+test('custom condition takes precedence over browser field and condition with `browser: true`', async (t) => {
+  const bundle = await rollup({
+    input: 'exports-worker-condition-with-browser-field.js',
+    plugins: [nodeResolve({ exportConditions: ['browser', 'webworker'], browser: true })]
+  });
+  const { module } = await testBundle(t, bundle);
+
+  t.deepEqual(module.exports, 'FROM WEBWORKER CONDITION');
+});
+
+test('custom condition takes precedence over browser field with `browser: true`', async (t) => {
+  const bundle = await rollup({
+    input: 'exports-only-worker-condition-with-browser-field.js',
+    plugins: [nodeResolve({ exportConditions: ['browser', 'webworker'], browser: true })]
+  });
+  const { module } = await testBundle(t, bundle);
+
+  t.deepEqual(module.exports, 'FROM WEBWORKER CONDITION');
+});
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,3 @@
		import sample from 'exports-only-worker-condition-with-browser-field';

		export default sample;
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,3 @@
		import sample from 'exports-worker-condition-with-browser-field';

		export default sample;