Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Increase test coverage for generating kyma.status.modules #2356

Merged
merged 26 commits into from
Apr 3, 2025

Conversation

lindnerby
Copy link
Member

Description

Changes proposed in this pull request:

  • Extract to service
  • Add tests

Related issue(s)

@lindnerby lindnerby requested a review from a team as a code owner March 25, 2025 05:52
fmt
@lindnerby lindnerby enabled auto-merge (squash) March 28, 2025 07:31
@lindnerby lindnerby changed the title chore: Test update status modules chore: Increase test coverage for generating kyma.status.modules Mar 28, 2025
@lindnerby lindnerby linked an issue Mar 28, 2025 that may be closed by this pull request
1 task
lindnerby and others added 7 commits March 28, 2025 09:40

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fmt

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…o be collected.
@ruanxin
Copy link
Contributor

ruanxin commented Apr 1, 2025

To reviewer: pls check my updated commit 40c471f

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
ruanxin and others added 2 commits April 3, 2025 10:09

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Co-authored-by: Christoph Schwägerl <[email protected]>
@lindnerby lindnerby merged commit 6534394 into kyma-project:main Apr 3, 2025
73 of 75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unit test generateModuleStatus
4 participants