lifecycle-toolkit icon indicating copy to clipboard operation
lifecycle-toolkit copied to clipboard

Increase Keptn Cert Manager coverage

Open RealAnna opened this issue 1 year ago • 7 comments

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%.

RealAnna avatar Feb 13 '24 09:02 RealAnna

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

Bharadwajshivam28 avatar Feb 15 '24 00:02 Bharadwajshivam28

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 % ?

Bharadwajshivam28 avatar Feb 15 '24 00:02 Bharadwajshivam28

please finish your other issue first :)

mowies avatar Feb 15 '24 08:02 mowies

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

yashpimple avatar Feb 17 '24 05:02 yashpimple

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

RealAnna avatar Feb 19 '24 07:02 RealAnna

Hi @YashPimple any updates on this issue?

odubajDT avatar Feb 26 '24 14:02 odubajDT

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

yashpimple avatar Feb 26 '24 15:02 yashpimple