You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iggy.apache.org by "atharvalade (via GitHub)" <gi...@apache.org> on 2026/03/03 18:37:05 UTC
[PR] ci(go): merge BDD and e2e test coverage into Go SDK Codecov reports [iggy]
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: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] ci(go): merge BDD and e2e test coverage into Go SDK Codecov reports [iggy]
Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #2859:
URL: https://github.com/apache/iggy/pull/2859#issuecomment-3993013439
## [Codecov](https://app.codecov.io/gh/apache/iggy/pull/2859?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 65.91%. Comparing base ([`5ec0d73`](https://app.codecov.io/gh/apache/iggy/commit/5ec0d73c827827fc8c775253489cee5e9937cb12?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`253d528`](https://app.codecov.io/gh/apache/iggy/commit/253d5283ee31378fab5a44427526304b7a6f94f6?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
<details><summary>Additional details and impacted files</summary>
```diff
@@ Coverage Diff @@
## master #2859 +/- ##
============================================
- Coverage 67.96% 65.91% -2.05%
Complexity 739 739
============================================
Files 1049 1049
Lines 84373 84373
Branches 60952 60940 -12
============================================
- Hits 57341 55614 -1727
- Misses 24674 26497 +1823
+ Partials 2358 2262 -96
```
| [Flag](https://app.codecov.io/gh/apache/iggy/pull/2859/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
|---|---|---|
| [go](https://app.codecov.io/gh/apache/iggy/pull/2859/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `36.55% <ø> (+30.22%)` | :arrow_up: |
| [node](https://app.codecov.io/gh/apache/iggy/pull/2859/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `91.11% <ø> (-1.30%)` | :arrow_down: |
Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
[see 153 files with indirect coverage changes](https://app.codecov.io/gh/apache/iggy/pull/2859/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
</details>
<details><summary> :rocket: New features to boost your workflow: </summary>
- :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
- :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
</details>
--
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: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] ci(go): merge BDD and e2e test coverage into Go SDK Codecov reports [iggy]
Posted by "hubcio (via GitHub)" <gi...@apache.org>.
hubcio merged PR #2859:
URL: https://github.com/apache/iggy/pull/2859
--
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: commits-unsubscribe@iggy.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org