-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Another way to condition a response #55
Comments
Hi, @irsooti. Thanks, this is a great concern! The search parameter approach is mainly meant for triggering different responses in the browser, and we are working on the improved browser support for I think #57 is going to solve your problem though! You will be able to trim down the operations (and, hopefully, their responses) during the test setup. |
NICE! I'll wait it be documented and merged then. We ended up to create a middleware to add the search params as temporary solution |
Description
In my common use cases, I don't have the ability to add a search param to do a conditional response.
Consider this silly scenario:
The function
getUser
contains a fetch request that retrieves users.In the case of a happy path (200), the handlers created using
fromOpenApi
are fine. Now, let's repeat the same test using another type of response, for instance a 400 one.According to the documentation, I should add a search parameter response to retrieve the response coupled with the status code, but as shown in the scenario, I can't just add the parameter because it's a detail I don't have control over (it could be something from a package, or a prebuild sdk with all the http calls inside).
Alternatives
Maybe it could be easier to consider some utility to control the status code of the msw handlers, I do not have a good idea to propose but conceptually is something like:
wdyt?
The text was updated successfully, but these errors were encountered: