fix: check denom of SpendLimit (#292)

notional-labs · Jul 21, 2022 · 0aca1df · 0aca1df
1 parent b81ad24
commit 0aca1df
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/simapp/helpers/test_helpers.go b/simapp/helpers/test_helpers.go
@@ -13,8 +13,10 @@ import (
 )
 
 // SimAppChainID hardcoded chainID for simulation
+// TODO: Implement a tool to check for gas growth. For now,
+// we must increase this across the board for longer runs to work
 const (
-	DefaultGenTxGas = 1000000
+	DefaultGenTxGas = 2000000
 	SimAppChainID   = "simulation-app"
 )
 

diff --git a/x/authz/simulation/operations.go b/x/authz/simulation/operations.go
@@ -249,7 +249,7 @@ func SimulateMsgExec(ak authz.AccountKeeper, bk authz.BankKeeper, k keeper.Keepe
 			return simtypes.NoOpMsg(authz.ModuleName, TypeMsgExec, "not a send authorization"), nil, nil
 		}
 
-		if sendAuth.SpendLimit.IsAllLTE(coins) {
+		if !sendAuth.SpendLimit.IsAllGTE(coins) || !coins.DenomsSubsetOf(sendAuth.SpendLimit) {
 			return simtypes.NoOpMsg(authz.ModuleName, TypeMsgExec, "over spend limit"), nil, nil
 		}