pkg/webhook: unit test Interpreter #5542
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.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
karmada-bot
added
the
size/XL
Signed-off-by: Mohamed Awnallah <[email protected]>
In this commit, we introduce unit tests for the `Interpreter` webhook across `Decoder`, `HTTP`, `Injection`, `Response`, and `Webhook` components. Signed-off-by: Mohamed Awnallah <[email protected]>
mohamedawnallah
force-pushed
the
unitTestInterpreter
branch
from
bbcf86e
to
f01637c
Compare
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #5542 +/- ##
==========================================
+ Coverage 34.14% 34.40% +0.25%
==========================================
Files 643 643
Lines 44509 44509
==========================================
+ Hits 15197 15312 +115
+ Misses 28156 28038 -118
- Partials 1156 1159 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Description
In this commit, we introduce unit tests for the
Interpreterwebhook acrossDecoder,HTTP,Injection,Response, andWebhookcomponents.What type of PR is this?
Which issue(s) this PR fixes:
Part of #5491.
Does this PR introduce a user-facing change?: