-
Notifications
You must be signed in to change notification settings - Fork 63
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
Removed 400 AcceptedErrorCodes from MirrorNodeClient for GET_ACCOUNTS_BY_ID_ENDPOINT path #1651
Removed 400 AcceptedErrorCodes from MirrorNodeClient for GET_ACCOUNTS_BY_ID_ENDPOINT path #1651
Conversation
…ath, added UT to verify that errors are being thrown for both 400 and 500 error status codes Signed-off-by: Alfredo Gutierrez <alfredo@swirldslabs.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #1651 +/- ##
=======================================
Coverage 77.43% 77.43%
=======================================
Files 36 36
Lines 2761 2761
Branches 560 560
=======================================
Hits 2138 2138
Misses 447 447
Partials 176 176
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG
If we don't already have on there should be a UT to check that on 404 for those account methods that the method doesn't through an error but returns null
Yes there are already some UTs that cover those scenarios |
…_BY_ID_ENDPOINT path (hashgraph#1651) removed 400 from acceptedErrorCodes for GET_ACCOUNTS_BY_ID_ENDPOINT path, added UT to verify that errors are being thrown for both 400 and 500 error status codes Signed-off-by: Alfredo Gutierrez <alfredo@swirldslabs.com> Signed-off-by: Mo Shaikjee <shaikjeemohammed@gmail.com>
Description:
Removed 400 from acceptedErrorCodes for GET_ACCOUNTS_BY_ID_ENDPOINT path, added UT to verify that errors are being thrown for both 400 and 500 error status codes
Related issue(s): #1276
Fixes #
Notes for reviewer:
This replaces PR 1649
And PR 1366
Checklist