villebro opened a new pull request, #111: URL: https://github.com/apache/superset-kubernetes-operator/pull/111
## Summary Adds focused unit tests for previously under-covered functions in `internal/controller` (and one in `internal/config`), targeting genuine gaps identified from the merged coverage profile rather than padding. Most additions use the fake client; no envtest required. `internal/controller` rises from 85.0% to 89.2% on the untagged suite alone (higher once the envtest specs are included), with no source changes and no `codecov.yml` edits. ## Details Notable gaps closed: `resetTaskStatusForRun` (0%→100%), `projectScheduleStatus` (41%→100%), `reconcileMaintenanceReturn` (33%→85%), `computeMaintenanceChecksum` (64%→93%); `applyExplicitOverrides`, `convertComponent`, `convertCloneComponent`, `warnEnvVarOverrides`, `preserveServiceAllocatedFields`, `mergeLabels`, `taskRequiresDrain`, `isTaskEnabled`, `defaultMigrateCommand` all to 100%. New test files: `checksum_test.go`, `component_descriptors_test.go`, `component_reconciler_test.go`, `websocket_config_test.go`; the rest extend existing test files. Remaining sub-85% functions are the deliberate residual: reconcile entrypoints covered by the envtest integration specs, and a few `client` error-return branches that need fault injection (out of scope). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
