Skip to content

Drop redundant GBS test tasks, tighten Kover, and add Codecov config - #52

Merged
btmonier merged 3 commits into
developfrom
fix/issues-25-33-34
Oct 23, 2026
Merged

btmonier merged 3 commits into
developfrom
fix/issues-25-33-34

Conversation

@btmonier

Copy link
Copy Markdown
Member

Description

This PR cleans up how tests and code coverage run in CI.

Since PR #16, all GBS v2 tests generate their own data with GBSSimData and run inside the main test task. None of them read -Dgbs.test.dataset, so gbsTest, gbsTestSmall, and gbsTestLarge were just re-running the same tests under different names. And because Kover instruments every Test task by default, the coverage job was running those three tasks plus statisticsTest on top of test.

This PR removes the three redundant GBS tasks, tells Kover to skip statisticsTest (which already runs in its own required CI job), and raises the Kover branch coverage minimum from 18% to 23% to sit one point below the measured 24.46%. It also adds a codecov.yml config with patch targets and per-area components.

Note:

Closes #25, closes #33, closes #34.

Type of change

What type of changes does your code introduce? Put an x in boxes that apply.

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Checklist:

  • The base branch of this PR is develop (use the hotfix template if this targets main)
  • I have updated the version variable in build.gradle.kts
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated relevant documentation

@btmonier btmonier self-assigned this Oct 21, 2026
@codecov-commenter

codecov-commenter commented Oct 21, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 25.85%. Comparing base (2b1f8cb) to head (d955a68).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop      #52      +/-   ##
=============================================
+ Coverage      25.39%   25.85%   +0.46%     
- Complexity      4918     5042     +124     
=============================================
  Files            610      610              
  Lines          73053    73056       +3     
  Branches       12590    12591       +1     
=============================================
+ Hits           18550    18892     +342     
+ Misses         52626    52274     -352     
- Partials        1877     1890      +13     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@btmonier
btmonier merged commit 198c06f into develop Oct 23, 2026
8 checks passed
@btmonier
btmonier deleted the fix/issues-25-33-34 branch October 23, 2026 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

5 participants