🌱 Add negative profile lookup test case#2633
🌱 Add negative profile lookup test case#2633openshift-merge-bot[bot] merged 1 commit intooperator-framework:mainfrom
Conversation
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Pull request overview
This PR restores a negative test case for TLS profile lookup that was removed in PR #2632. The test verifies that looking up a non-existent profile returns an error, complementing the existing positive test cases in TestProfilesMapCompleteness.
Changes:
- Add a negative test case in
TestProfilesMapCompletenessto verify that looking up a non-existent profile returns an error
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2633 +/- ##
==========================================
+ Coverage 68.93% 68.95% +0.02%
==========================================
Files 139 139
Lines 9891 9891
==========================================
+ Hits 6818 6820 +2
+ Misses 2563 2562 -1
+ Partials 510 509 -1
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:
|
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Signed-off-by: Todd Short <tshort@redhat.com>
3c95471 to
e904867
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: grokspawn The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
42f101e
into
operator-framework:main
This restores the negative test that was removed in #2632
Description
Reviewer Checklist