Require more gas compute for create denom instead of a cost (#14)

* Charge extra gas for createdenom instead

* Adds DenomCreationGasConsume Param

* Bump to 2mil defai;t

* lint

* remove duplicate test

proto/buf.lock

@@ -8,7 +8,7 @@ deps:
   - remote: buf.build
     owner: cosmos
     repository: cosmos-sdk
-    commit: 4e1072d2e34943e79c5699c02aad7066
+    commit: 172f6ce4e9054eb4af8b96e8131da354
   - remote: buf.build
     owner: cosmos
     repository: gogo-proto

proto/osmosis/tokenfactory/v1beta1/params.proto

@@ -15,4 +15,11 @@ message Params {
     (gogoproto.moretags) = "yaml:\"denom_creation_fee\"",
     (gogoproto.nullable) = false
   ];
+  // if denom_creation_fee is an empty array, then this field is used to add more gas consumption
+  // to the base cost.
+  // https://github.com/CosmWasm/token-factory/issues/11
+  uint64 denom_creation_gas_consume = 2 [    
+    (gogoproto.moretags) = "yaml:\"denom_creation_gas_consume\"",
+    (gogoproto.nullable) = true
+  ];
 }

scripts/test_node.sh

@@ -67,7 +67,8 @@ from_scratch () {
   update_test_genesis '.app_state["gov"]["deposit_params"]["min_deposit"]=[{"denom": "ujuno","amount": "1000000"}]'
   update_test_genesis '.app_state["crisis"]["constant_fee"]={"denom": "ujuno","amount": "1000"}'  
 
-  update_test_genesis '.app_state["tokenfactory"]["params"]["denom_creation_fee"]=[{"denom":"ujuno","amount":"100"}]'
+  # removed fee, so it will be nil
+  update_test_genesis '.app_state["tokenfactory"]["params"]["denom_creation_fee"]=[]'  
 
   update_test_genesis '.app_state["feeshare"]["params"]["allowed_denoms"]=["ujuno"]'
 

x/tokenfactory/keeper/createdenom.go

@@ -81,6 +81,9 @@ func (k Keeper) chargeForCreateDenom(ctx sdk.Context, creatorAddr string, _ stri
 		if err := k.communityPoolKeeper.FundCommunityPool(ctx, creationFee, accAddr); err != nil {
 			return err
 		}
+	} else {
+		gasIncrease := k.GetParams(ctx).DenomCreationGasConsume
+		ctx.GasMeter().ConsumeGas(gasIncrease, "consume denom creation gas")
 	}
 	return nil
 }

x/tokenfactory/keeper/createdenom_test.go

@@ -152,7 +152,6 @@ func (suite *KeeperTestSuite) TestCreateDenom() {
 
 				suite.Require().NoError(err)
 				suite.Require().Equal(suite.TestAccs[0].String(), queryRes.AuthorityMetadata.Admin)
-
 			} else {
 				suite.Require().Error(err)
 				// Ensure we don't charge if we expect an error

x/tokenfactory/types/params.go

@@ -28,7 +28,8 @@ func DefaultParams() Params {
 	return Params{
 		// this was from osmosis
 		// DenomCreationFee: sdk.NewCoins(sdk.NewInt64Coin(appparams.BaseCoinUnit, 10_000_000)), // 10 OSMO
-		DenomCreationFee: sdk.NewCoins(sdk.NewInt64Coin(sdk.DefaultBondDenom, 10_000_000)), // 10 OSMO
+		DenomCreationFee:        sdk.NewCoins(sdk.NewInt64Coin(sdk.DefaultBondDenom, 10_000_000)), // 10 OSMO
+		DenomCreationGasConsume: 2_000_000,
 	}
 }