atharvalade commented on PR #3135: URL: https://github.com/apache/iggy/pull/3135#issuecomment-4314335852
> can you make it a integration test instead of unit? code in core/server will be replaced when we introduce clustering, thus test will be lost. Good call, moved the tests from `core/server` unit tests to `core/integration/tests/storage/consumer_offsets.rs`. They follow the same pattern as `tests/state/` (direct `server::` imports, no full harness needed). This way they survive the `server` ti `server-ng` migration and can be re-run against the new binary. -- 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]
