rabbitmq - chore: adding in test coverage#185
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #185 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 10 10
Lines 1266 1264 -2
Branches 231 231
=========================================
- Hits 1266 1264 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Code Review
This pull request introduces v8 ignore markers to an auto-reject block in the RabbitMQ task provider and adds several edge-case tests covering connection idempotency, timeout behaviors, and multiple handler rejections. Feedback suggests removing the coverage ignore markers in favor of explicit error-path testing, refactoring tests to use the public dequeue API instead of accessing private members, and adding assertions to ensure that side effects like dead-lettering occur only once.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
rabbitmq - chore: adding in test coverage