Skip to content

Commit 9e6db2d

Browse files
authored
test(node-resolve): add tests for mixing custom exportConditions with browser: true (#1043)
1 parent 8aa447a commit 9e6db2d

10 files changed

Lines changed: 46 additions & 0 deletions

File tree

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import sample from 'exports-only-worker-condition-with-browser-field';
2+
3+
export default sample;
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import sample from 'exports-worker-condition-with-browser-field';
2+
3+
export default sample;

packages/node-resolve/test/fixtures/node_modules/exports-only-worker-condition-with-browser-field/browser-field-entry.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/fixtures/node_modules/exports-only-worker-condition-with-browser-field/package.json

Lines changed: 7 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/fixtures/node_modules/exports-only-worker-condition-with-browser-field/webworker-condition-entry.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/fixtures/node_modules/exports-worker-condition-with-browser-field/browser-condition-entry.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/fixtures/node_modules/exports-worker-condition-with-browser-field/browser-field-entry.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/fixtures/node_modules/exports-worker-condition-with-browser-field/package.json

Lines changed: 8 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/fixtures/node_modules/exports-worker-condition-with-browser-field/webworker-condition-entry.js

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/node-resolve/test/package-entry-points.js

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -380,3 +380,23 @@ test('does not warn when resolving typescript imports with fallback', async (t)
380380
a: 'A'
381381
});
382382
});
383+
384+
test('custom condition takes precedence over browser field and condition with `browser: true`', async (t) => {
385+
const bundle = await rollup({
386+
input: 'exports-worker-condition-with-browser-field.js',
387+
plugins: [nodeResolve({ exportConditions: ['browser', 'webworker'], browser: true })]
388+
});
389+
const { module } = await testBundle(t, bundle);
390+
391+
t.deepEqual(module.exports, 'FROM WEBWORKER CONDITION');
392+
});
393+
394+
test('custom condition takes precedence over browser field with `browser: true`', async (t) => {
395+
const bundle = await rollup({
396+
input: 'exports-only-worker-condition-with-browser-field.js',
397+
plugins: [nodeResolve({ exportConditions: ['browser', 'webworker'], browser: true })]
398+
});
399+
const { module } = await testBundle(t, bundle);
400+
401+
t.deepEqual(module.exports, 'FROM WEBWORKER CONDITION');
402+
});

0 commit comments

Comments
 (0)