-
Notifications
You must be signed in to change notification settings - Fork 805
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
Test for Common/domain/replication_queue: GetMessagesfromDLQ & AckLevel #5734
Conversation
Codecov Report
Additional details and impacted filessee 9 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Pull Request Test Coverage Report for Build 018e156a-c04c-4289-8dac-a8b329c20f3bDetails
💛 - Coveralls |
Why?:
Strengthening the test coverage for key components of the replication queue, specifically for retrieving messages from the DLQ and querying acknowledgment levels.
What has changed?:
Refined the GetMessagesFromDLQ test cases to validate successful message retrievals from DLQ, along with handling of read failures, ensuring the function behaves as expected in various scenarios.
Expanded the testing for GetAckLevel to cover successful acknowledgment level retrievals and to check the function's robustness against potential data retrieval issues.
How did you test it?:
unit tests with mock objects
Potential risks:
No risk involved.