Explore places to add more unit tests to library code #1750
Labels
Milestone
Comments
Racer159
added
tech-debt 💳
3 tasks
Racer159
pushed a commit
that referenced
this issue
Aug 10, 2023
## Description - Replaces use of the concrete `*kubernetes.Clientset` implementation with the `kubernetes.Interface` for client operations for easier mocking/testing - Adds unit test for `validatePackageArchitecture()` method ## Related Issue Relates to #1750 ## Type of change - [x] Other (security config, docs update, etc) ## Checklist before merging - [x] Test, docs, adr added or updated as needed - [x] [Contributor Guide Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow) followed
5 tasks
This was referenced Jan 25, 2024
|
Closing as a dupe |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe what should be investigated or refactored
We should look at targeted areas to add unit tests (specifically in
src/pkg) to provide better assurances over the interfaces in this code.Links to any relevant code
https://github.com/defenseunicorns/zarf/tree/main/src/pkg
Additional context
This is something we should look at as we move toward GA and get Zarf's interfaces (both schema and library code) to be more stable.
The text was updated successfully, but these errors were encountered: