lighthouse
lighthouse copied to clipboard
Add test for ActiveSamplingRequest
Issue Addressed
I've implemented batching sampling requests in #6256, but there are no unit tests for it yet.
Proposed Changes
Add a test for ActiveSamplingRequest to ensure that the requests are batched, and check the status of the requests.
Please update to point at unstable by either:
- Rebasing on
unstable(if your branch has a small number of commits that are easy to tease out), or - Merging
origin/dasinto this PR:git fetch origin; git merge origin/das. This will result in the smallest number of conflict resolutions and is better for branches that already contain merge commits or have extensive history.
Thanks for the comment Michael! I've rebased this on unstable.
Thanks @dapplion! I've written tests in the tests.rs file.
@mergify queue
queue
✅ The pull request has been merged automatically
The pull request has been merged automatically at 8cf686f5c11cbad19727885d07df5abfddeddb0f