nits

cosmos · Oct 4, 2023 · d35c983 · d35c983
1 parent e29cb92
commit d35c983
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 15 deletions.
diff --git a/tests/difference/core/driver/setup.go b/tests/difference/core/driver/setup.go
@@ -177,17 +177,17 @@ func (b *Builder) getAppBytesAndSenders(
 		require.NoError(b.suite.T(), err)
 
 		validator := stakingtypes.Validator{
-			OperatorAddress: sdk.ValAddress(val.Address).String(),
-			ConsensusPubkey: pkAny,
-			Jailed:          false,
-			Status:          status,
-			Tokens:          tokens,
-			DelegatorShares: sumShares,
-			Description:     stakingtypes.Description{},
-			UnbondingHeight: int64(0),
-			UnbondingTime:   time.Unix(0, 0).UTC(),
-			Commission:      stakingtypes.NewCommission(sdk.ZeroDec(), sdk.ZeroDec(), sdk.ZeroDec()),
-			// MinSelfDelegation: sdk.ZeroInt(),
+			OperatorAddress:   sdk.ValAddress(val.Address).String(),
+			ConsensusPubkey:   pkAny,
+			Jailed:            false,
+			Status:            status,
+			Tokens:            tokens,
+			DelegatorShares:   sumShares,
+			Description:       stakingtypes.Description{},
+			UnbondingHeight:   int64(0),
+			UnbondingTime:     time.Unix(0, 0).UTC(),
+			Commission:        stakingtypes.NewCommission(sdk.ZeroDec(), sdk.ZeroDec(), sdk.ZeroDec()),
+			MinSelfDelegation: sdk.ZeroInt(),
 		}
 
 		stakingValidators = append(stakingValidators, validator)
@@ -433,8 +433,7 @@ func (b *Builder) addValidatorToStakingModule(privVal mock.PV) {
 		coin,
 		stakingtypes.Description{},
 		stakingtypes.NewCommissionRates(sdk.ZeroDec(), sdk.ZeroDec(), sdk.ZeroDec()),
-		sdk.NewInt(0),
-	)
+		sdk.ZeroInt())
 	b.suite.Require().NoError(err)
 	providerStaking := b.providerStakingKeeper()
 	pskServer := stakingkeeper.NewMsgServerImpl(&providerStaking)

diff --git a/tests/e2e/steps.go b/tests/e2e/steps.go
@@ -24,7 +24,6 @@ var happyPathSteps = concatSteps(
 	stepsRedelegate("consu"),
 	stepsDowntime("consu"),
 	stepsDoubleSignOnProvider("consu"), // carol double signs on provider
-	// TODO: add double sign on consumer
 	stepsStartRelayer(),
 	stepsConsumerRemovalPropNotPassing("consu", 2), // submit removal prop but vote no on it - chain should stay
 	stepsStopChain("consu", 3),                     // stop chain
@@ -37,7 +36,6 @@ var shortHappyPathSteps = concatSteps(
 	stepsRedelegateShort("consu"),
 	stepsDowntime("consu"),
 	stepsDoubleSignOnProvider("consu"), // carol double signs on provider
-	// TODO: add double sign on consumer
 	stepsStartRelayer(),
 	stepsConsumerRemovalPropNotPassing("consu", 2), // submit removal prop but vote no on it - chain should stay
 	stepsStopChain("consu", 3),                     // stop chain