Skip to content

test(license-generator): add missing unit & integration tests for CLI…#10

Merged
bsayli merged 1 commit intomainfrom
feature/next-steps
Sep 14, 2025
Merged

test(license-generator): add missing unit & integration tests for CLI…#10
bsayli merged 1 commit intomainfrom
feature/next-steps

Conversation

@bsayli
Copy link
Owner

@bsayli bsayli commented Sep 14, 2025

… and signature flows

  • Added integration tests for LicenseTokenCli, LicenseKeyGeneratorCli, KeygenCli, EncryptUserIdCli, and Ed25519KeygenCli
  • Added unit test for SignatureDemo to cover sample signing flows
  • Refactored CLIs to expose run() methods for testability
  • Improved validation and error handling coverage

Next steps:

  • Raise overall test coverage toward 70%+ target
  • Add Vault integration for secrets management
  • Extend Keycloak integration for license lifecycle operations

… and signature flows

- Added integration tests for LicenseTokenCli, LicenseKeyGeneratorCli, KeygenCli,
  EncryptUserIdCli, and Ed25519KeygenCli
- Added unit test for SignatureDemo to cover sample signing flows
- Refactored CLIs to expose run() methods for testability
- Improved validation and error handling coverage

Next steps:
- Raise overall test coverage toward 70%+ target
- Add Vault integration for secrets management
- Extend Keycloak integration for license lifecycle operations
@bsayli bsayli merged commit 219a589 into main Sep 14, 2025
1 check passed
@github-actions
Copy link

Coverage (lines): %

  • Covered:
  • Missed:
  • JaCoCo report path: target/site/jacoco-aggregate/jacoco.xml

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants