Flipper: Discussion: Allow to mock response status code in request mocking feature

[gildor]

Existing request mocking feature doesn't support providing custom HTTP status code, it always returns 200, which is quite a big limitation, the testing HTTP statuses is one of the most important features of request mocking

[bizzguy]

I agree with the request to add return code. Also, I would also like to see DELETE added as a type. Mocking is really an AWESOME addition to the plugin!

[bizzguy]

One more thing. If the user has selected (highlighted) a network request when the Mock button is clicked, it would be useful if the current highlighted request was populated in the data section (and the URL also). Some of our requests are really long and this would save us time rather than having to copy and paste from the original request.

[bizzguy]

A few more things.

It appears that "export" does not export the mock definitions to the "FlipperExport.flipper" file.

Also, having a way to temporarily disable the mock without deleting the definition would be useful. Maybe put a checkbox next to the mock definition so it could be disabled.

[bizzguy]

Just submitted a PR to allow for specifying status code on response. Would be interested in feedback from anyone on this thread. Thanks.

https://github.com/facebook/flipper/pull/1441

[bizzguy]

The PR to fix this issue has been merged so this issue can now be closed.

[stale[bot]]

Hey there, it looks like there has been no activity on this issue recently. Has the issue been fixed, or does it still require the community's attention? This issue may be closed if no further activity occurs. You may also label this issue as a "Discussion" or add it to the "Backlog" and I will leave it open. Thank you for your contributions.

[bizzguy]

This issue can be closed. The functionality to add response status code to mocking was added in PR https://github.com/facebook/flipper/pull/1447