forked from celestiaorg/cosmos-sdk
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix!: update ABCI query to use request height (cosmos#9879)
- Loading branch information
1 parent
e656d5e
commit c04ab50
Showing
4 changed files
with
79 additions
and
4 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,63 @@ | ||
// +build norace | ||
|
||
package client_test | ||
|
||
import ( | ||
"fmt" | ||
|
||
abci "github.com/tendermint/tendermint/abci/types" | ||
|
||
banktypes "github.com/cosmos/cosmos-sdk/x/bank/types" | ||
) | ||
|
||
func (s *IntegrationTestSuite) TestQueryABCIHeight() { | ||
|
||
testCases := []struct { | ||
name string | ||
reqHeight int64 | ||
ctxHeight int64 | ||
expHeight int64 | ||
}{ | ||
{ | ||
name: "non zero request height", | ||
reqHeight: 3, | ||
ctxHeight: 1, // query at height 1 or 2 would cause an error | ||
expHeight: 3, | ||
}, | ||
{ | ||
name: "empty request height - use context height", | ||
reqHeight: 0, | ||
ctxHeight: 3, | ||
expHeight: 3, | ||
}, | ||
{ | ||
name: "empty request height and context height - use latest height", | ||
reqHeight: 0, | ||
ctxHeight: 0, | ||
expHeight: 4, | ||
}, | ||
} | ||
|
||
for _, tc := range testCases { | ||
s.Run(tc.name, func() { | ||
s.network.WaitForHeight(tc.expHeight) | ||
|
||
val := s.network.Validators[0] | ||
|
||
clientCtx := val.ClientCtx | ||
clientCtx = clientCtx.WithHeight(tc.ctxHeight) | ||
|
||
req := abci.RequestQuery{ | ||
Path: fmt.Sprintf("store/%s/key", banktypes.StoreKey), | ||
Height: tc.reqHeight, | ||
Data: banktypes.CreateAccountBalancesPrefix(val.Address), | ||
Prove: true, | ||
} | ||
|
||
res, err := clientCtx.QueryABCI(req) | ||
s.Require().NoError(err) | ||
|
||
s.Require().Equal(tc.expHeight, res.Height) | ||
}) | ||
} | ||
} |