chore(test): add tests for mixing custom exportConditions with `bro…

…wser: true`
rollup · Nov 16, 2021 · 40c5f41 · 40c5f41
1 parent 02fb349
commit 40c5f41
Show file tree

Hide file tree

Showing 10 changed files with 48 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,25 @@ 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);
+
+  console.log({ module });
+
+  t.deepEqual(module.exports, 'FROM WEBWORKER CONDITION');
+});