Add unit tests for configuration utility#288
Add unit tests for configuration utility#288GautamBytes wants to merge 1 commit intoopenkruise:masterfrom
Conversation
|
[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. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@hantmac please review it whenever feasible! |
Signed-off-by: GautamBytes <manchandanigautam@gmail.com>
c0d75d0 to
d44d5c0
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #288 +/- ##
==========================================
+ Coverage 47.52% 47.59% +0.07%
==========================================
Files 63 64 +1
Lines 9799 9820 +21
==========================================
+ Hits 4657 4674 +17
- Misses 4585 4587 +2
- Partials 557 559 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@furykerry can u rerun the workflow once , it will pass then I just updated branch! |
This pull request introduces comprehensive unit tests for the configuration utility functions in
pkg/util/configuration/configuration.go.Changes:
GetTrafficRoutingIngressLuaScriptandgetRolloutConfigurationfunctions.controller-runtimeto mock Kubernetes API interactions, ensuring tests are isolated and reliable.ConfigMap.ConfigMap.