-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Open
Labels
area/testingIssues or PRs related to testingIssues or PRs related to testinghelp wantedDenotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.kind/cleanupCategorizes issue or PR as related to cleaning up code, process, or technical debt.Categorizes issue or PR as related to cleaning up code, process, or technical debt.kind/tepCategorizes issue or PR as related to a TEP (or needs a TEP).Categorizes issue or PR as related to a TEP (or needs a TEP).
Description
Unit Test Coverage
Ensure comprehensive unit test coverage for all resolver cache logic. Achieve near 100% test coverage on new cache-related code paths. Many tests are already implemented, so this one is more of a sanity check.
[Update 05.12]: Check if there are duplicated tests in any of the existing cache tests and remove those. And check if any of the existing tests are not testing actual behavior. Those can also be removed.
Areas to verify:
- All cache operation paths in
pkg/remoteresolution/resolver/framework/cache/ - Cache mode resolution logic (parameter → ConfigMap → default hierarchy)
- Cache key generation for different parameter types
- Edge cases (empty cache, cache misses, eviction scenarios)
- Concurrent access patterns
How to Contribute
Read the parent issue #9228 (comment).
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
area/testingIssues or PRs related to testingIssues or PRs related to testinghelp wantedDenotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines.kind/cleanupCategorizes issue or PR as related to cleaning up code, process, or technical debt.Categorizes issue or PR as related to cleaning up code, process, or technical debt.kind/tepCategorizes issue or PR as related to a TEP (or needs a TEP).Categorizes issue or PR as related to a TEP (or needs a TEP).
Type
Projects
Status
Todo