opticdev / optic

OpenAPI linting, diffing and testing. Optic helps prevent breaking changes, publish accurate documentation and improve the design of your APIs.
https://useoptic.com
MIT License
1.36k stars 83 forks source link

set request_concurrency 1 in capture tests #2713

Closed niclim closed 9 months ago

niclim commented 9 months ago

🍗 Description

What does this PR do? Anything folks should know?

Noticed that we still were occasionally getting flaky tests in capture. this is because i think there's still the issue that requests comes in different orders which results in sometimes getting different text

We could also sort out the capture return statements but it isn't super important - for now we can stop the flakiness by limiting request concurrency to 1 in the tests

📚 References

Links to relevant docs (Notion, Twist, GH issues, etc.), if applicable.

👹 QA

How can other humans verify that this PR is correct?