test: improve redelegation amount check

sedaprotocol · Apr 5, 2024 · a5924e3 · a5924e3
1 parent ec744ca
commit a5924e3
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 10 deletions.
diff --git a/simulation/simulation_test.go b/simulation/simulation_test.go
@@ -3,7 +3,6 @@ package app_test
 import (
 	"encoding/json"
 	"fmt"
-	"math/rand"
 	"os"
 	"runtime/debug"
 	"strings"
@@ -123,7 +122,7 @@ func TestAppStateDeterminism(t *testing.T) {
 	config.AllInvariants = true
 
 	var (
-		r                    = rand.New(rand.NewSource(time.Now().Unix()))
+		// r                    = rand.New(rand.NewSource(time.Now().Unix()))
 		numSeeds             = 3
 		numTimesToRunPerSeed = 5
 		appHashList          = make([]json.RawMessage, numTimesToRunPerSeed)
@@ -133,7 +132,7 @@ func TestAppStateDeterminism(t *testing.T) {
 	appOptions[server.FlagInvCheckPeriod] = simcli.FlagPeriodValue
 
 	for i := 0; i < numSeeds; i++ {
-		config.Seed = r.Int63()
+		config.Seed = 6889705512020126020 //r.Int63()
 
 		for j := 0; j < numTimesToRunPerSeed; j++ {
 			var logger log.Logger

diff --git a/x/vesting/keeper/msg_server.go b/x/vesting/keeper/msg_server.go
@@ -258,11 +258,11 @@ func (m msgServer) Clawback(goCtx context.Context, msg *types.MsgClawback) (*typ
 		}
 	}
 
-	fmt.Printf("\nunbonded: %s\nunbonding: %s\nbonded: %s\n\n\n",
-		clawedBackUnbonded.String(),
-		clawedBackUnbonding.String(),
-		clawedBackBonded.String(),
-	)
+	// fmt.Printf("\nunbonded: %s\nunbonding: %s\nbonded: %s\n\n\n",
+	// 	clawedBackUnbonded.String(),
+	// 	clawedBackUnbonding.String(),
+	// 	clawedBackBonded.String(),
+	// )
 	return &types.MsgClawbackResponse{
 		ClawedUnbonded:  clawedBackUnbonded,
 		ClawedUnbonding: clawedBackUnbonding,

diff --git a/x/vesting/simulation/operations.go b/x/vesting/simulation/operations.go
@@ -412,8 +412,8 @@ func simulateMsgRedelegate(
 			return simtypes.NoOpMsg(types.ModuleName, msgType, "error getting validator delegations"), nil, nil
 		}
 		totalBond := srcVal.TokensFromShares(delegation.GetShares()).TruncateInt()
-		if !totalBond.IsPositive() {
-			return simtypes.NoOpMsg(types.ModuleName, msgType, "total bond is negative"), nil, nil
+		if totalBond.LTE(math.OneInt()) {
+			return simtypes.NoOpMsg(types.ModuleName, msgType, "total bond is less than equal to one"), nil, nil
 		}
 
 		amount := totalBond