explise opened a new pull request, #3207:
URL: https://github.com/apache/iggy/pull/3207
## Which issue does this PR close?
Closes #3206
## Rationale
Closes the TODO at `core/integration/tests/server/cg.rs:27`. QUIC is already
supported by the harness and exercised by
`verify_consumer_group_partition_assignment.rs`,
so adding it to these scenarios is a free coverage gain on a production
transport.
## What changed?
Consumer group operations (join, multi-client polling, auto-commit
reconnection, message visibility after server restart, offset cleanup)
were only end-to-end verified against the TCP and WebSocket transports.
The QUIC transport had no coverage for these flows, even though QUIC is
a supported production transport and is already exercised elsewhere
(e.g. `verify_consumer_group_partition_assignment.rs`).
The harness now drives each of the six consumer-group scenarios through
a QUIC client in addition to TCP and WebSocket, closing the coverage
gap and catching any QUIC-specific framing, reconnect, or restart
regressions in consumer-group code paths.
## Local Execution
- Pre-commit hooks ran (`prek run` — all applicable hooks passed)
- Test count for `server::cg::*`: 12 → 18 (added 6 QUIC variants), all
passing
- `cargo test -p integration --test mod server::cg::`
## AI Usage
1. **Tool**: Claude (used as a research/explanation assistant via Claude
Code)
2. **Scope**: Codebase exploration and guidance only.
3. **Verification**: Ran the full test scope locally and confirmed 18/18
pass.
4. **Yes**, I can explain every line.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
