Skip to content

Add automated code coverage threshold enforcement via Jest - #201

Merged
chizzy192 merged 1 commit into
StellarState:devfrom
KingFRANKHOOD:feature/coverage-thresholds
Jul 30, 2026
Merged

Add automated code coverage threshold enforcement via Jest#201
chizzy192 merged 1 commit into
StellarState:devfrom
KingFRANKHOOD:feature/coverage-thresholds

Conversation

@KingFRANKHOOD

@KingFRANKHOOD KingFRANKHOOD commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

Configured global code coverage thresholds in Jest and added coverage report generation to CI pipeline. Also resolved pre-existing type errors and test failures.
Closes #128
Closes #129
Closes #130
Closes #131

Changes

DevOps

  • jest.config.js: Added \coverageThreshold\ enforcing 60% branches, 80% lines/functions/statements
  • api-cy.yml: Updated test step to run with --coverage\ flag; added coverage artifact upload step

Bug fixes (pre-existing errors resolved)

  • stellar-address.utils.ts: Fixed \StrKey.isValidContract\ TypeScript type issue with type assertion
  • invoice-state-machine.test.ts: Removed invalid second argument from \expect()\ call
  • validate-invoice-for-publish.test.ts: Corrected expected error code from \FACE_VALUE_NOT_POSITIVE\ to \FACE_VALUE_TOO_LOW\
  • stellar-address.test.ts: Replaced invalid test addresses with real valid Stellar keys

Verification

  • Lint: ✅ passes
  • Type check: ✅ passes
  • Tests: 63 suites, 444 tests all passing
  • Coverage: all thresholds met

Closes #

@drips-wave

drips-wave Bot commented Jul 30, 2026

Copy link
Copy Markdown

@KingFRANKHOOD Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@chizzy192
chizzy192 merged commit 2e9656f into StellarState:dev Jul 30, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment