atharvalade opened a new pull request, #2859: URL: https://github.com/apache/iggy/pull/2859
## Which issue does this PR close? Closes #2858 ## Rationale Go SDK coverage was only capturing unit tests from `foreign/go/`, missing all BDD integration tests (`bdd/go/`) and e2e tests that exercise the SDK through a running server. This artificially deflated the reported coverage. ## What changed? The post-merge `go-coverage` job and pre-merge `e2e` task only ran `cd foreign/go && go test ./...`, ignoring 49 BDD test files in `bdd/go/` that exercise streams, topics, messaging, consumer groups, offsets, users, and more through the SDK. Now both workflows also run `bdd/go` tests with `-coverpkg=github.com/apache/iggy/foreign/go/...` to capture SDK code coverage from integration tests, merge the profiles, and upload the combined result to Codecov. The `_test.yml` upload step now triggers on both `test` and `e2e` tasks, matching the Node SDK pattern. ## Local Execution - Passed - Pre-commit hooks ran — no Go/Rust/Java code changed, only CI workflow YAML files ## AI Usage 1. Opus 4.6 2. Used for codebase exploration, pattern analysis across SDK workflows, and generating the workflow YAML changes 3. Verified by cross-referencing with existing Node/C# coverage patterns in the same repo, validating YAML syntax, and checking file path consistency across all three modified files 4. Yes -- 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]
