-
Notifications
You must be signed in to change notification settings - Fork 880
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
operator/pkg/certs: unit test Certificate Manager and Karmada Store #5559
base: master
Are you sure you want to change the base?
operator/pkg/certs: unit test Certificate Manager and Karmada Store #5559
Conversation
755c2c4
to
f193990
Compare
f193990
to
f41b894
Compare
f41b894
to
cf2cbd1
Compare
cf2cbd1
to
10faf0f
Compare
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #5559 +/- ##
==========================================
+ Coverage 39.73% 40.09% +0.35%
==========================================
Files 650 650
Lines 55171 55193 +22
==========================================
+ Hits 21922 22128 +206
+ Misses 31840 31643 -197
- Partials 1409 1422 +13
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
In this commit, we introduce unit tests for operator `certs` package. The test cases include but not limited to: - Test private key generation. - Test self-signed certificate authority (CA) generation. - Verify creation of self-signed certificate and key files within the CA. - Include tests for utility functions, such as appending to Subject Alternative Names in SSL/TLS certificates. - Add tests for Karmada Store operations related to certificates stored in Kubernetes Secrets: - Create - Read - Update - Load Signed-off-by: Mohamed Awnallah <[email protected]>
10faf0f
to
a83c26e
Compare
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
cc @XiShanYongYe-Chang All test cases passed after PR #5561 was merged and rebasing on master branch :) |
Description
In this commit, we introduce unit tests for operator
certs
package. The test cases include but not limited to:What type of PR is this?
Which issue(s) this PR fixes:
Part of #5491.
Dependency
This PR depends on #5560 and #5561.
Does this PR introduce a user-facing change?: