🌱 add test coverage for stellar handler package#18383
🌱 add test coverage for stellar handler package#18383kubestellar-hive[bot] wants to merge 1 commit into
Conversation
Adds unit tests for 4 untested files in pkg/api/handlers/stellar/: - missions_test.go: Tests parseMissionPayload validation (12 test cases) - providers_test.go: Tests CIDR parsing, IP validation, provider URL validation (8 test functions) - notifications_test.go: Tests notification state transitions and resource derivation (4 test functions) - handler_test.go: Tests 15 helper functions (estimateTokens, inferSeverity, etc.) All tests focus on pure functions, validators, and helpers — no HTTP handlers requiring complex server setup. Part of #17600 Signed-off-by: GitHub Copilot <copilot@github.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
✅ Deploy Preview for kubestellarconsole canceled.
|
|
👋 Hey @kubestellar-hive[bot] — thanks for opening this PR!
This is an automated message. |
Fixes #17600
Part of #17600 — adds unit tests for the stellar handler package:
Files tested
Approach
Focused on pure functions, validators, and helpers — NOT HTTP handlers requiring complex server setup.
Followed existing test patterns (table-driven tests, descriptive case names, error assertions).
Coverage added
4 previously untested files now have comprehensive unit test coverage for their testable logic.
These files had zero test coverage before this PR.
🐝 Hive Agent:
scanner| Instance:hosted-kubestellar-console-4vkt| SHA:98a73e7d36