Skip to content
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

Capture timing of requests #1153

Closed
nmck257 opened this issue Sep 27, 2024 · 6 comments · Fixed by #1154
Closed

Capture timing of requests #1153

nmck257 opened this issue Sep 27, 2024 · 6 comments · Fixed by #1154
Assignees

Comments

@nmck257
Copy link

nmck257 commented Sep 27, 2024

(I'm new to Hoverfly)

Is your feature request related to a problem? Please describe.
I think Hoverfly could be useful not just for unit testing, but performance testing.
In (some forms of) performance testing, I would like to stub out my application's connections to downstream webservices and return fixed responses, but with realistic (data-driven) delay.

Describe the solution you'd like
Could Capture mode be enhanced to grab not only request/response, but also the time taken for the roundtrip? This data could (minimally) be read by the developer to better-inform "hardcoded" delays, or a step further, could feed directly into Simulate mode.

Describe alternatives you've considered
Manually researching and configuring delays for requests.

@tommysitu
Copy link
Member

As a starting point, we can record the latency as fixed delay in capture mode. It seems easy to implement.

@kapishmalik
Copy link
Collaborator

@tommysitu can I pick this up?

@tommysitu
Copy link
Member

@tommysitu can I pick this up?

@kapishmalik of course if you have time. Thank you!

@kapishmalik
Copy link
Collaborator

Sure, I will manage and start working on the same.

@kapishmalik
Copy link
Collaborator

Raised PR - #1154

@nmck257
Copy link
Author

nmck257 commented Oct 10, 2024

Very cool to see the quick turnaround on this idea!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants