Response handler for dynamic requests (POC) #160
Conversation
|
This PR is stale because it has been open for 30 days with no activity. Remove the Stale label or comment or this will be closed in 10 days. |
|
ping @AvdLee perhaps? |
|
This PR is stale because it has been open for 30 days with no activity. Remove the Stale label or comment or this will be closed in 10 days. |
|
ping @kairadiagne perhaps? |
|
This PR is stale because it has been open for 30 days with no activity. Remove the Stale label or comment or this will be closed in 10 days. |
|
.. |
|
This PR is stale because it has been open for 30 days with no activity. Remove the Stale label or comment or this will be closed in 10 days. |
|
. |
|
This PR is stale because it has been open for 30 days with no activity. Remove the Stale label or comment or this will be closed in 10 days. |
|
.. |
|
This PR is stale because it has been open for 30 days with no activity. Remove the Stale label or comment or this will be closed in 10 days. |
It still needs some polishing. Would you review and approve and merge the changes if I do this work? |
|
Hi @teameh, it looks good to me and I’ve just approved the PR, but I believe we need more approvals. I’m not a member of the WeTransfer team, but I use Mocker a lot to write unit tests. |
I'm using Mocker as mock backend for our app to be able to develop the app in "demo mode" as long as other developers are still working on the backend.
This sometimes requires a more dynamic response to be returned from the mocks based on the request body or previous requests. I considered creating something from scratch based off Mocker, but instead I've added a simple
responseHandleroption to my own fork to facilitate this.It works great and just using my own fork works fine, but I can imagine that this could be valuable for other developers as well. Here's a POC branch that demonstrates this. If you think this is something that would benefit Mocker, I could draft up a real PR with a nicer implementation if you like.