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

adding additional test coverage #3587

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

miledxz
Copy link

@miledxz miledxz commented Oct 5, 2024

Description: Adding additional tests coverage for Client SSL multiple secrets and server SSL with empty secrets

Changes Proposed in PR:

This test Client SSL Multiple Secrets validates that the createSecretClientSSLProfile function can correctly handle multiple secrets, ensuring that it works with the most recent configuration provided, ensuring that systems where multiple TLS configurations might exist, function behaves correctly across various states.

This test Server SSL with empty secrets list ensures that the system can gracefully handle edge cases, specifically when no secrets are provided.

@vklohiya you can take a look if you have time and when available, thank you !!

goal of adding these tests is to add more cases,
thanks for your time

General Checklist

  • Updated Added functionality/ bug fix in release notes
  • Added examples for new feature
  • Updated the documentation
  • Smoke testing completed

CRD Checklist

  • Updated required CR schema

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.

1 participant