test: add the test case for invalid address (#88)

* test: add test case to `QueryInactiveContracts`

* chore: add this pr for changelog.md

* test: add the test case for invalid address

* test: add the test case for invalid address

* chore: add this pr to CHANGE.log

* fix: Fix address judgment and pagination key

* feat: add the function for byte array to use addresses order check

* test: add func for make test data for addresses order

* fix: Add a function to create an ordered address array

Fix to create an ordered address array and test it as an expected value

* fix: fix lint error

* fix: add the error handling for lint

* fix: fix `Error: unreachable: unreachable code (govet)` for `golangci-lint`

* fix: fix panic message

* refactor: Changed []byte array comparison to string comparison

* fix: revert commit for go.mod go.sum

* fix: fix lint

* fix: fix lint

* fix: delete unnecessary code

* fix: rename function name and add a contract for order

* fix: Changed to equal judgment as before with wrap error as expected value

* test: add the invalid address test case to `TestQueryContractInfo` and `TestQueryContractHistory`

CHANGELOG.md

@@ -22,6 +22,7 @@
 * [\#82](https://github.com/Finschia/wasmd/pull/82) add test case to QueryInactiveContracts
 * [\#87](https://github.com/Finschia/wasmd/pull/87) add an integration test for TestExecuteContract 
 * [\#79](https://github.com/Finschia/wasmd/pull/79) add an integration test for TestStoreAndInstantiateContract
+* [\#88](https://github.com/Finschia/wasmd/pull/88) add the test case for invalid address
 
 ### Bug Fixes
 * [\#62](https://github.com/Finschia/wasmd/pull/62) fill ContractHistory querier result's Updated field

x/wasm/keeper/querier_test.go

@@ -9,6 +9,7 @@ import (
 	"testing"
 	"time"
 
+	"github.com/cosmos/btcutil/bech32"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 	"google.golang.org/grpc/codes"
@@ -55,6 +56,10 @@ func TestQueryAllContractState(t *testing.T) {
 			srcQuery: &types.QueryAllContractStateRequest{Address: RandomBech32AccountAddress(t)},
 			expErr:   types.ErrNotFound,
 		},
+		"query all with invalid address": {
+			srcQuery: &types.QueryAllContractStateRequest{Address: "abcde"},
+			expErr:   fmt.Errorf("decoding bech32 failed: %w", bech32.ErrInvalidLength(5)),
+		},
 		"with pagination offset": {
 			srcQuery: &types.QueryAllContractStateRequest{
 				Address: contractAddr.String(),
@@ -163,6 +168,10 @@ func TestQuerySmartContractState(t *testing.T) {
 			srcQuery: &types.QuerySmartContractStateRequest{Address: RandomBech32AccountAddress(t), QueryData: []byte(`{"verifier":{}}`)},
 			expErr:   types.ErrNotFound,
 		},
+		"query smart with invalid address": {
+			srcQuery: &types.QuerySmartContractStateRequest{Address: "abcde", QueryData: []byte(`{"verifier":{}}`)},
+			expErr:   bech32.ErrInvalidLength(5),
+		},
 		"with empty request": {
 			srcQuery: nil,
 			expErr:   status.Error(codes.InvalidArgument, "empty request"),
@@ -267,6 +276,10 @@ func TestQueryRawContractState(t *testing.T) {
 			srcQuery: &types.QueryRawContractStateRequest{Address: RandomBech32AccountAddress(t), QueryData: []byte("foo")},
 			expErr:   types.ErrNotFound,
 		},
+		"query raw with invalid address": {
+			srcQuery: &types.QueryRawContractStateRequest{Address: "abcde", QueryData: []byte("foo")},
+			expErr:   bech32.ErrInvalidLength(5),
+		},
 		"with empty request": {
 			srcQuery: nil,
 			expErr:   status.Error(codes.InvalidArgument, "empty request"),
@@ -633,6 +646,16 @@ func TestQueryContractHistory(t *testing.T) {
 			}},
 			expErr: types.ErrEmpty,
 		},
+		"query with invalid address": {
+			req: &types.QueryContractHistoryRequest{Address: "abcde"},
+			srcHistory: []types.ContractCodeHistoryEntry{{
+				Operation: types.ContractCodeHistoryOperationTypeGenesis,
+				CodeID:    firstCodeID,
+				Updated:   types.NewAbsoluteTxPosition(ctx),
+				Msg:       []byte(`"init message"`),
+			}},
+			expErr: fmt.Errorf("decoding bech32 failed: %w", bech32.ErrInvalidLength(5)),
+		},
 		"with empty request": {
 			req:    nil,
 			expErr: status.Error(codes.InvalidArgument, "empty request"),
@@ -872,6 +895,10 @@ func TestQueryContractInfo(t *testing.T) {
 			stored: types.ContractInfoFixture(),
 			expErr: types.ErrNotFound,
 		},
+		"query with invalid address": {
+			src:    &types.QueryContractInfoRequest{Address: "abcde"},
+			expErr: bech32.ErrInvalidLength(5),
+		},
 		"with empty request": {
 			src:    nil,
 			expErr: status.Error(codes.InvalidArgument, "empty request"),