Revert "Adding token factory specific denom endpoint" (#463)

Reverts #457 in favor of
sei-protocol/sei-chain#1444 as tokenfactory is
more appropriate module for the endpoint

go.mod

@@ -78,13 +78,16 @@ require (
 	github.com/cockroachdb/tokenbucket v0.0.0-20230807174530-cc333fc44b06 // indirect
 	github.com/cosmos/gorocksdb v1.2.0 // indirect
 	github.com/creachadair/taskgroup v0.3.2 // indirect
+	github.com/danieljoos/wincred v1.1.2 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/desertbit/timer v0.0.0-20180107155436-c41aec40b27f // indirect
 	github.com/dgraph-io/badger/v3 v3.2103.2 // indirect
 	github.com/dgraph-io/ristretto v0.1.1 // indirect
 	github.com/dustin/go-humanize v1.0.1 // indirect
+	github.com/dvsekhvalnov/jose2go v1.5.0 // indirect
 	github.com/felixge/httpsnoop v1.0.1 // indirect
 	github.com/fsnotify/fsnotify v1.6.0 // indirect
+	github.com/gin-gonic/gin v1.7.7 // indirect
 	github.com/go-kit/kit v0.12.0 // indirect
 	github.com/go-kit/log v0.2.1 // indirect
 	github.com/go-logfmt/logfmt v0.5.1 // indirect
@@ -94,8 +97,10 @@ require (
 	github.com/golang/glog v1.1.0 // indirect
 	github.com/golang/groupcache v0.0.0-20210331224755-41bb18bfe9da // indirect
 	github.com/golang/snappy v0.0.4 // indirect
+	github.com/google/btree v1.1.2 // indirect
 	github.com/google/flatbuffers v1.12.1 // indirect
 	github.com/google/go-cmp v0.5.9 // indirect
+	github.com/google/gofuzz v1.2.0 // indirect
 	github.com/google/orderedcode v0.0.1 // indirect
 	github.com/google/uuid v1.3.0 // indirect
 	github.com/gorilla/websocket v1.5.0 // indirect
@@ -121,6 +126,7 @@ require (
 	github.com/mtibben/percent v0.2.1 // indirect
 	github.com/oasisprotocol/curve25519-voi v0.0.0-20210609091139-0a56a4bca00b // indirect
 	github.com/pelletier/go-toml v1.9.5 // indirect
+	github.com/pelletier/go-toml/v2 v2.0.7 // indirect
 	github.com/petermattis/goid v0.0.0-20230317030725-371a4b8eda08 // indirect
 	github.com/pmezard/go-difflib v1.0.0 // indirect
 	github.com/prometheus/client_model v0.3.0 // indirect
@@ -144,11 +150,13 @@ require (
 	go.opencensus.io v0.23.0 // indirect
 	golang.org/x/mod v0.11.0 // indirect
 	golang.org/x/net v0.17.0 // indirect
+	golang.org/x/sync v0.4.0 // indirect
 	golang.org/x/sys v0.13.0 // indirect
 	golang.org/x/term v0.13.0 // indirect
 	golang.org/x/text v0.13.0 // indirect
 	golang.org/x/tools v0.6.0 // indirect
 	google.golang.org/genproto v0.0.0-20231002182017-d307bd883b97 // indirect
+	google.golang.org/genproto/googleapis/rpc v0.0.0-20231012201019-e917dd12ba7a // indirect
 	gopkg.in/ini.v1 v1.67.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 	lukechampine.com/uint128 v1.2.0 // indirect
@@ -164,17 +172,6 @@ require (
 	nhooyr.io/websocket v1.8.6 // indirect
 )
 
-require (
-	github.com/danieljoos/wincred v1.1.2 // indirect
-	github.com/dvsekhvalnov/jose2go v1.5.0 // indirect
-	github.com/gin-gonic/gin v1.7.7 // indirect
-	github.com/google/btree v1.1.2
-	github.com/google/gofuzz v1.2.0 // indirect
-	github.com/pelletier/go-toml/v2 v2.0.7 // indirect
-	golang.org/x/sync v0.4.0 // indirect
-	google.golang.org/genproto/googleapis/rpc v0.0.0-20231012201019-e917dd12ba7a // indirect
-)
-
 replace (
 	github.com/99designs/keyring => github.com/cosmos/keyring v1.1.7-0.20210622111912-ef00f8ac3d76
 	github.com/confio/ics23/go => github.com/cosmos/cosmos-sdk/ics23/go v0.8.0

proto/cosmos/bank/v1beta1/query.proto

@@ -47,11 +47,6 @@ service Query {
     option (google.api.http).get = "/cosmos/bank/v1beta1/denoms_metadata/{denom}";
   }
 
-  // TokenFactoryDenomMetadata queries the client metadata of a given token factory coin denomination.
-  rpc TokenFactoryDenomMetadata(QueryTokenFactoryDenomMetadataRequest) returns (QueryDenomMetadataResponse) {
-    option (google.api.http).get = "/cosmos/bank/v1beta1/token_factory/denoms_metadata";
-  }
-
   // DenomsMetadata queries the client metadata for all registered coin denominations.
   rpc DenomsMetadata(QueryDenomsMetadataRequest) returns (QueryDenomsMetadataResponse) {
     option (google.api.http).get = "/cosmos/bank/v1beta1/denoms_metadata";
@@ -190,12 +185,6 @@ message QueryDenomMetadataRequest {
   string denom = 1;
 }
 
-// QueryTokenFactoryDenomMetadataRequest is the request type for the Query/TokenFactoryDenomMetadata RPC method.
-message QueryTokenFactoryDenomMetadataRequest {
-  // denom is the (token factory) coin denom to query the metadata for.
-  string denom = 1;
-}
-
 // QueryDenomMetadataResponse is the response type for the Query/DenomMetadata RPC
 // method.
 message QueryDenomMetadataResponse {

x/bank/keeper/grpc_query.go

@@ -200,8 +200,3 @@ func (k BaseKeeper) DenomMetadata(c context.Context, req *types.QueryDenomMetada
 		Metadata: metadata,
 	}, nil
 }
-
-// TokenFactoryDenomMetadata implements Query/TokenFactoryDenomMetadata gRPC method.
-func (k BaseKeeper) TokenFactoryDenomMetadata(c context.Context, req *types.QueryTokenFactoryDenomMetadataRequest) (*types.QueryDenomMetadataResponse, error) {
-	return k.DenomMetadata(c, &types.QueryDenomMetadataRequest{Denom: req.Denom})
-}

x/bank/keeper/grpc_query_test.go

@@ -180,10 +180,10 @@ func (suite *IntegrationTestSuite) TestQueryParams() {
 	suite.Require().Equal(suite.app.BankKeeper.GetParams(suite.ctx), res.GetParams())
 }
 
-func (suite *IntegrationTestSuite) TestQueryDenomsMetadataRequest() {
+func (suite *IntegrationTestSuite) QueryDenomsMetadataRequest() {
 	var (
 		req         *types.QueryDenomsMetadataRequest
-		expMetadata []types.Metadata
+		expMetadata = []types.Metadata{}
 	)
 
 	testCases := []struct {
@@ -282,7 +282,7 @@ func (suite *IntegrationTestSuite) TestQueryDenomsMetadataRequest() {
 	}
 }
 
-func (suite *IntegrationTestSuite) TestQueryDenomMetadataRequest() {
+func (suite *IntegrationTestSuite) QueryDenomMetadataRequest() {
 	var (
 		req         *types.QueryDenomMetadataRequest
 		expMetadata = types.Metadata{}
@@ -312,7 +312,7 @@ func (suite *IntegrationTestSuite) TestQueryDenomMetadataRequest() {
 		{
 			"success",
 			func() {
-				expMetadata = types.Metadata{
+				expMetadata := types.Metadata{
 					Description: "The native staking token of the Cosmos Hub.",
 					DenomUnits: []*types.DenomUnit{
 						{
@@ -358,76 +358,3 @@ func (suite *IntegrationTestSuite) TestQueryDenomMetadataRequest() {
 		})
 	}
 }
-
-func (suite *IntegrationTestSuite) TestQueryTokenFactoryDenomMetadataRequest() {
-	var (
-		req         *types.QueryTokenFactoryDenomMetadataRequest
-		expMetadata = types.Metadata{}
-	)
-	tokenFactoryDenom := "factory/sei1gxskuzvhr4s8sdm2rpruaf7yx2dnmjn0zfdu9q/NEWCOIN"
-	testCases := []struct {
-		msg      string
-		malleate func()
-		expPass  bool
-	}{
-		{
-			"empty denom",
-			func() {
-				req = &types.QueryTokenFactoryDenomMetadataRequest{}
-			},
-			false,
-		},
-		{
-			"not found denom",
-			func() {
-				req = &types.QueryTokenFactoryDenomMetadataRequest{
-					Denom: tokenFactoryDenom,
-				}
-			},
-			false,
-		},
-		{
-			"success",
-			func() {
-
-				expMetadata = types.Metadata{
-					Description: "Token factory custom token",
-					DenomUnits: []*types.DenomUnit{
-						{
-							Denom:    tokenFactoryDenom,
-							Exponent: 0,
-							Aliases:  []string{tokenFactoryDenom},
-						},
-					},
-					Base:    tokenFactoryDenom,
-					Display: tokenFactoryDenom,
-				}
-
-				suite.app.BankKeeper.SetDenomMetaData(suite.ctx, expMetadata)
-				req = &types.QueryTokenFactoryDenomMetadataRequest{
-					Denom: expMetadata.Base,
-				}
-			},
-			true,
-		},
-	}
-
-	for _, tc := range testCases {
-		suite.Run(fmt.Sprintf("Case %s", tc.msg), func() {
-			suite.SetupTest() // reset
-
-			tc.malleate()
-			ctx := sdk.WrapSDKContext(suite.ctx)
-
-			res, err := suite.queryClient.TokenFactoryDenomMetadata(ctx, req)
-
-			if tc.expPass {
-				suite.Require().NoError(err)
-				suite.Require().NotNil(res)
-				suite.Require().Equal(expMetadata, res.Metadata)
-			} else {
-				suite.Require().Error(err)
-			}
-		})
-	}
-}