-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
fix: Staking delegations should return empty list instead of rpc error when no records found #9423
fix: Staking delegations should return empty list instead of rpc error when no records found #9423
Conversation
…khita/staking-delegations-returns-empty-list-instead-of-rpc-error-when-no-records-found
Codecov Report
@@ Coverage Diff @@
## master #9423 +/- ##
==========================================
- Coverage 60.41% 60.39% -0.02%
==========================================
Files 589 589
Lines 37107 37103 -4
==========================================
- Hits 22417 22409 -8
- Misses 12712 12715 +3
- Partials 1978 1979 +1
|
…khita/staking-delegations-returns-empty-list-instead-of-rpc-error-when-no-records-found
…khita/staking-delegations-returns-empty-list-instead-of-rpc-error-when-no-records-found
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.
lgtm
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.
lgtm, changelog is missing
…list-instead-of-rpc-error-when-no-records-found
…r when no records found (cosmos#9423) * return empty list instead of rpc error when no records found for staking delegations * fix grpc query DelegatorDelegations tests * remove response code tests for staking delegations * fix failing tests * change staking delegations response code to 200 in grpc test * add staking delegations response code tests to TestQueryDelegatorDelegationsGRPC * add address without delegations testcase * add changes to grpc query tests of delegatorDelegations * remove getRequest unused function from x/staking/client/rest/grpc_query_test.go * minor fixes * add testcases for request with no delegations * address review comments Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> (cherry picked from commit da87ab0)
closes: #9356