Introduce tests for lifecycle-operator bugs

[odubajDT]

Recently multiple small bugs were found and fixed as part of this PR.

The places are:

• https://github.com/keptn/lifecycle-toolkit/pull/3113/files#diff-7e3a3dd6a5940394c614f51b5abf528919d2593178aad9cfbbe963bdc261dc6bR212
• https://github.com/keptn/lifecycle-toolkit/pull/3113/files#diff-7e3a3dd6a5940394c614f51b5abf528919d2593178aad9cfbbe963bdc261dc6bR238
• https://github.com/keptn/lifecycle-toolkit/pull/3113/files#diff-9a3cc86b3f6c7660bf48ab0906ad69cc9500e27fc6b648f0d577f92f0078933dR162

Introduce unit tests to cover the affected functionality

[SohamRatnaparkhi]

Hi @odubajDT , I can try to solve this issue. Can you assign it to me?

Thank you!

[odubajDT]

Hi @SohamRatnaparkhi are you actively working on this issue? Since you have this issue assigned for aprox. 10 days, we usually unassign asignees from their issues if there is no activity to not block other potential contributors from starting to tackle them.

Thanks for the update!

[SohamRatnaparkhi]

Hey @odubajDT
Really sorry for holding on to the issue. I'm busy right now and can't solve it. Feel free to assign it to someone else.

Once again, I'm really sorry :)

[odubajDT]

No worries, thank you very much for your answer! :) If you have time in the future, we would be happy if you reach out to us for potentially another issue!

[Bharadwajshivam28]

Hey I can take on this one If no one is working

[odubajDT]

Hi @Bharadwajshivam28 are you actively working on this issue? Since you have this issue assigned for aprox. 3 weeks, we usually unassign asignees from their issues if there is no activity to not block other potential contributors from starting to tackle them.

Thanks for the update!

[odubajDT]

Due to inactivity, unassigning @Bharadwajshivam28 from this issue. Feel free to request a different issue!

[YashPimple]

Hi @odubajDT I would like to work on the above issue

[odubajDT]

@YashPimple I assigned it to you but please before you start, there is a small change required in the PR you have opened #3143. try to please finish this one first before you start with a new one

[Jay179-sudo]

Hi! @odubajDT, I would like to take up this issue.

Since the non-blocking mode does not introduce any errors in the controller, we can write a unit test with a mock handler that does not return any errors when the KeptnAppVersionStatus resembles the manifest in the docs

Am I correctly understanding the task here?

[mowies]

@Jay179-sudo sure thing, I assigned it to you

[github-actions]

This issue will be unassigned in 1 week if no further activity is seen.
If you are active please provide an update on the status of the issue and if you would like to continue
working on it.

[mowies]

un-assigning, due to inactivity

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be
closed if no further activity occurs. Thank you for your contributions.

[KapilSareen]

Hi @odubajDT , is this issue still open to contributions? If yes, I would love to work on it!