Add tests for Cache RemoveOldest, Len and Clear - #359
magic-peach wants to merge 1 commit into
Conversation
Signed-off-by: Akanksha Trehun <akankshatrehun@gmail.com>
|
@magic-peach: The label(s) DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: magic-peach 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 |
|
@skitt: Closed this PR. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
What type of PR is this?
/kind test
What this PR does / why we need it:
Cache.RemoveOldest, Cache.Len, and Cache.Clear had zero test coverage. Adds direct tests for each, including RemoveOldest and Clear on an empty cache.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Verified with go test ./lru/...: all 10 tests pass, including the pre-existing race and eviction tests.
Release note: