kirito632 commented on PR #3502: URL: https://github.com/apache/kvrocks/pull/3502#issuecomment-4592285345
@jihuayu , I investigated the coverage report after adding the new XDELEX integration tests. The Go tests now cover: - KEEPREF / DELREF / ACKED - multi-ID requests - mixed deleted/skipped results - multiple consumer groups - dangling PEL cleanup - ACKED semantics with groups created using `$` - non-existent stream handling - POLLUPDATES propagation However, cmd_stream.cc and batch_extractor.cc still appear as 0% covered in the Sonar report, even though the integration tests must pass through those layers in order to reach redis_stream.cc. This makes me wonder whether the issue is related to coverage collection from the integration-test process rather than missing functional test coverage. Do you think additional C++ unit tests are still expected here, or should we investigate the coverage reporting path first? -- 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]
