Fix tests

dymensionxyz · Sep 25, 2024 · 3e91172 · 3e91172
1 parent ccbd284
commit 3e91172
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/ibctesting/utils_test.go b/ibctesting/utils_test.go
@@ -21,6 +21,7 @@ import (
 	channeltypes "github.com/cosmos/ibc-go/v7/modules/core/04-channel/types"
 	ibctesting "github.com/cosmos/ibc-go/v7/testing"
 	"github.com/cosmos/ibc-go/v7/testing/mock"
+	"github.com/dymensionxyz/sdk-utils/utils/uptr"
 	"github.com/stretchr/testify/require"
 	"github.com/stretchr/testify/suite"
 
@@ -144,7 +145,7 @@ func (s *utilSuite) createRollapp(transfersEnabled bool, channelID *string) {
 				Base:     "aden",
 				Exponent: 18,
 			},
-			InitialSupply: sdk.NewInt(1000),
+			InitialSupply: uptr.To(sdk.NewInt(1000)),
 		},
 	)
 

diff --git a/x/incentives/keeper/suite_test.go b/x/incentives/keeper/suite_test.go
@@ -8,6 +8,7 @@ import (
 
 	tmrand "github.com/cometbft/cometbft/libs/rand"
 	sdk "github.com/cosmos/cosmos-sdk/types"
+	"github.com/dymensionxyz/sdk-utils/utils/uptr"
 	"github.com/dymensionxyz/sdk-utils/utils/urand"
 
 	"github.com/dymensionxyz/dymension/v3/x/incentives/types"
@@ -222,7 +223,7 @@ func (suite *KeeperTestSuite) CreateDefaultRollapp(addr sdk.AccAddress) string {
 				Base:     "aden",
 				Exponent: 18,
 			},
-			InitialSupply: sdk.NewInt(1000),
+			InitialSupply: uptr.To(sdk.NewInt(1000)),
 		},
 	}
 

diff --git a/x/rollapp/keeper/rollapp.go b/x/rollapp/keeper/rollapp.go
@@ -52,7 +52,7 @@ func (k Keeper) CheckAndUpdateRollappFields(ctx sdk.Context, update *types.MsgUp
 		current.GenesisInfo.NativeDenom = update.GenesisInfo.NativeDenom
 	}
 
-	if update.GenesisInfo.InitialSupply != nil && !update.GenesisInfo.InitialSupply.IsPositive() {
+	if update.GenesisInfo.InitialSupply != nil && update.GenesisInfo.InitialSupply.IsPositive() {
 		current.GenesisInfo.InitialSupply = update.GenesisInfo.InitialSupply
 	}
 

diff --git a/x/rollapp/types/message_update_rollapp.go b/x/rollapp/types/message_update_rollapp.go
@@ -82,5 +82,5 @@ func (msg *MsgUpdateRollappInformation) UpdatingGenesisInfo() bool {
 	return msg.GenesisInfo.GenesisChecksum != "" ||
 		msg.GenesisInfo.Bech32Prefix != "" ||
 		msg.GenesisInfo.NativeDenom.Base != "" ||
-		(msg.GenesisInfo.InitialSupply != nil && !msg.GenesisInfo.InitialSupply.IsPositive())
+		(msg.GenesisInfo.InitialSupply != nil && msg.GenesisInfo.InitialSupply.IsPositive())
 }
diff --git a/x/rollapp/types/rollapp.go b/x/rollapp/types/rollapp.go
@@ -103,7 +103,7 @@ func (r Rollapp) GenesisInfoFieldsAreSet() bool {
 	return r.GenesisInfo.GenesisChecksum != "" &&
 		r.GenesisInfo.NativeDenom.Validate() == nil &&
 		r.GenesisInfo.Bech32Prefix != "" &&
-		(r.GenesisInfo.InitialSupply != nil && !r.GenesisInfo.InitialSupply.IsPositive())
+		(r.GenesisInfo.InitialSupply != nil && !r.GenesisInfo.InitialSupply.IsNil())
 }
 
 func (r Rollapp) IsVulnerable() bool {