Skip to content

test: add integration tests and CI pipeline updates #483

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

Closed
wants to merge 16 commits into from

Conversation

egbakou
Copy link
Contributor

@egbakou egbakou commented Mar 20, 2025

This PR introduces integration tests, as discussed in this conversation. The tests are structured to be easily extensible for future additions.

✅ All tests passed except for Create_secret_With_DefaultReflectorAnnotations_Should_Replicated_To_All_Newly_Created_Namespaces, which reproduces the issue reported in #478.

Additionally, I have integrated the test execution step into the CI pipeline to ensure automated validation in future updates.
Let me know if you have any feedback @winromulus :)

Copy link

github-actions bot commented May 1, 2025

Automatically marked as stale due to no recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added stale and removed stale labels May 1, 2025
@winromulus
Copy link
Contributor

@egbakou I "merged" your contribution to the latest version (with a bit of restructuring) as the base for future tests.
Thank you very much for your continued contributions and especially for this one! It will save a lot of time!

@winromulus winromulus closed this May 3, 2025
@egbakou
Copy link
Contributor Author

egbakou commented May 3, 2025

@egbakou I "merged" your contribution to the latest version (with a bit of restructuring) as the base for future tests. Thank you very much for your continued contributions and especially for this one! It will save a lot of time!

You're welcome @winromulus :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants