keptn / lifecycle-toolkit

Toolkit for cloud-native application lifecycle management
https://keptn.sh
Apache License 2.0
303 stars 121 forks source link

Increase Keptn Cert Manager coverage #3038

Closed RealAnna closed 2 months ago

RealAnna commented 7 months ago

Add unit tests to increase coverage of the cert-manager operator in Keptn. Specifically, improve coverage for the following files:

Ideally, code cov for the package should reach 70%.

Bharadwajshivam28 commented 7 months ago

Hello @RealAnna I have implemented unit test before can i work on this? as I am a new contributor also

Bharadwajshivam28 commented 7 months ago

Just I have few small doubts that are-

1) There are already test written for the files you mentioned so do i need to modify it to increase coverage and how and where can i see the coverage % 2) Do i need to fulfill prerequisites to check coverage % ?

mowies commented 7 months ago

please finish your other issue first :)

YashPimple commented 7 months ago

Hi @RealAnna, I'd like to take this up!

RealAnna commented 7 months ago

@YashPimple since you have no open PR I will assign you 🚀

odubajDT commented 7 months ago

Hi @YashPimple any updates on this issue?

YashPimple commented 7 months ago

Yes @odubajDT, I'm working on it and plan to submit a pull request within the next couple of days.