Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

test: pkg/certificate/providers/vault/certificate_manager.go - ListCertificates() #3460

Closed
SanyaKochhar opened this issue May 26, 2021 · 1 comment
Closed

test: pkg/certificate/providers/vault/certificate_manager.go - ListCertificates() #3460

SanyaKochhar opened this issue May 26, 2021 · 1 comment
Labels
area/tests Related to tests size/XS 1 day
Milestone
v0.9.0

Comments

@SanyaKochhar
Copy link
Contributor

In pkg/certificate/providers/vault/certificate_manager.go, the unit test coverage can be improved for the ListCertificates() func. See highlighted lines below.

image

ref #1489
@SanyaKochhar SanyaKochhar added size/XS 1 day area/tests Related to tests labels May 26, 2021
@SanyaKochhar SanyaKochhar added this to the v0.10.0 milestone May 26, 2021
@SanyaKochhar
Copy link
Contributor Author

Closing in favor of consolidating in #3458

@draychev draychev modified the milestones: v0.10.0, v0.9.0 Jun 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
area/tests Related to tests size/XS 1 day
Projects
None yet
Milestone
v0.9.0
Development

No branches or pull requests
2 participants
@SanyaKochhar @draychev