x/tokenfactory/keeper: unexpected inconsistency mixing non-gRPC errors without a status+ code in an area with gRPC errors

[odeke-em]

This code is quite inconsistent when an error is encountered while parsing an address https://github.com/noble-assets/noble/blob/main/x/tokenfactory/keeper/grpc_query_blacklisted.go#L49-L52

and also inside Keeper.Mint

noble/x/tokenfactory/keeper/msg_server_mint.go

Lines 35 to 37 in d809e7e

	_, addressBz, err = bech32.DecodeAndConvert(msg.Address)
	if err != nil {
	return nil, err

in an area where errors are being wrapped and sent over with a status code

Remedy

Use status codes to wrap that error with a code of InvalidArgument and please add a test for invalid addresses because the presence of that error not being caught means that the other RPC is going to encounter an unexpected error or could even crash