[Feature] Mock response status and headers #7
Conversation
| @@ -3,7 +3,19 @@ | |||
| * Partial implementation of RequestInit with the addition of "data" property which value will be returned from the mock. | |||
| */ | |||
| export type MockOptions = { | |||
| /** @deprecated use response.data */ | |||
There was a problem hiding this comment.
I don't feel confidend deprecating the root data property and using the response.data instead.
It's logically correct to have the data inside the response object, as it de facto mocks the response body, but it also gets rid of the simplicity of mock('..', { data: {...} }).
On the other hand, keeping both seems unnecessary.
There was a problem hiding this comment.
I don't think having two ways to do such a same thing is a good idea, I find it unnecessary and confusing. I would suggest deprecating it.
|
Can you please make a release with this PR merged? |
|
I am waiting for this to be released too |
|
@sadkebab I've added the dist folder to this fork temporarily as I wait for this release https://github.com/brianorwhatever/bun-bagel |
|
Sorry for the delay! Released 🚀 |
This PR will allow mocking the response status and headers.