-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix: Use iife version of MSW #711
Conversation
How come this works under Vite but not web-dev-server? |
I am not sure what module resolution Vite runs all files through, but they are doing something magical. It is this bug mswjs/msw#1384 we are running into. Hopefully esm support will be possible one day. Until then it doesn't make a big difference. It is only code we use internally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The msw module version currently doesn't work in a browser. The means that we can not run mocks as part of the test runner. We will also run into the same problems when setting up import maps in web-dev-server as it is the same server as web-test-runner.