Closed wacban closed 3 months ago
Attention: Patch coverage is 94.91525%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 71.73%. Comparing base (
14e143e
) to head (f5fb456
). Report is 3 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
...egration-tests/src/test_loop/utils/transactions.rs | 92.50% | 1 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This test caught the congestion control / state sync integration bug that was fixed in #11712. It currently doesn't make any checks or assertions but it does exercise the stateless validation / single shard tracking / state sync / congestion combo that we will run in production.
The first commit deletes the old test and the second commit adds it back refactored. I would recommend reviewing the full test in the second commit because the original was not fully reviewed previously. I have no clue how to make github treat those as two separate files hence the commit based workaround.