Add tests for diff test-only split helpers - #104
magic-peach wants to merge 1 commit into
Conversation
Signed-off-by: Akanksha Trehun <akankshatrehun@gmail.com>
|
Welcome @magic-peach! |
|
[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 this PR does:
Adds unit tests for the helper functions behind the
--split-test-onlyflag ondepstat diff:filterDepsByTestStatus,filterEdgesByTestStatus,filterVersionChangesByTestStatus,computeFilteredCounts,buildSplitSection,buildSplitResult,diffSlices, andgetEdges. These functions had no direct unit test coverage even though the flag itself is exercised indirectly elsewhere.Test only change, no production code touched.
How it was tested:
Ran
go test ./...on a clean test cache, full suite passes.No existing PR or issue covers this. Not linked to an open issue.