Tests for common/domain: tests TestGetDLQSize, TestRangeDeleteMessagesFromDLQ and TestDeleteMessageFromDLQ #5741
Conversation
abhishekj720
requested review from
Shaddoll,
neil-xie,
davidporter-id-au,
Groxx,
shijiesheng,
agautam478,
jakobht,
3vilhamster,
sankari165,
dkrotx,
taylanisikdemir and
demirkayaender
as code owners
| } | ||
| }) | ||
| } | ||
| } |
There was a problem hiding this comment.
will add extra line with next ongoing PR
There was a problem hiding this comment.
PR checks are failing. make fmt should take care
Codecov Report
Additional details and impacted filessee 5 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
| } | ||
| }) | ||
| } | ||
| } |
There was a problem hiding this comment.
PR checks are failing. make fmt should take care
Pull Request Test Coverage Report for Build 018e1658-d902-4b3d-b1c8-0b4f1bb3a4caDetails
💛 - Coveralls |
What changed?
Wrote tests for:
Why?
Increase code coverage and better coding standards
How did you test it?
Unit tests
Potential risks
No risks as it only contains test with mock calls.