Debug gov with inactive vals test

cosmos · Jul 19, 2024 · f0e651a · f0e651a
1 parent 42f86bb
commit f0e651a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/tests/e2e/config.go b/tests/e2e/config.go
@@ -579,7 +579,7 @@ func InactiveProviderValsTestConfig() TestConfig {
 	tr.chainConfigs[ChainID("provi")] = proviConfig
 	tr.chainConfigs[ChainID("consu")] = consuConfig
 
-	// make is to that carol does not use a consumer key
+	// make it so that carol does not use a consumer key
 	carolConfig := tr.validatorConfigs[ValidatorID("carol")]
 	carolConfig.UseConsumerKey = false
 	tr.validatorConfigs[ValidatorID("carol")] = carolConfig
@@ -606,7 +606,7 @@ func SmallMaxValidatorsTestConfig() TestConfig {
 func GovTestConfig() TestConfig {
 	cfg := DefaultTestConfig()
 
-	// set the MaxValidators to 2
+	// set the quorum to 50%
 	proviConfig := cfg.chainConfigs[ChainID("provi")]
 	proviConfig.GenesisChanges += "| .app_state.gov.params.quorum = \"0.5\""
 	cfg.chainConfigs[ChainID("provi")] = proviConfig
@@ -620,18 +620,13 @@ func GovTestConfig() TestConfig {
 }
 
 func InactiveValsGovTestConfig() TestConfig {
-	cfg := InactiveProviderValsTestConfig()
+	cfg := GovTestConfig()
 
-	// set the MaxValidators to 2
+	// set the MaxValidators to 1
 	proviConfig := cfg.chainConfigs[ChainID("provi")]
-	proviConfig.GenesisChanges += "| .app_state.gov.params.quorum = \"0.5\""
+	proviConfig.GenesisChanges += "| .app_state.staking.params.max_validators = 1"
 	cfg.chainConfigs[ChainID("provi")] = proviConfig
 
-	carolConfig := cfg.validatorConfigs["carol"]
-	// make carol use her own key
-	carolConfig.UseConsumerKey = false
-	cfg.validatorConfigs["carol"] = carolConfig
-
 	return cfg
 }
 

diff --git a/tests/e2e/steps_inactive_vals.go b/tests/e2e/steps_inactive_vals.go
@@ -656,7 +656,7 @@ func stepsInactiveProviderValidatorsGovernanceComparison() []Step {
 								InitialHeight: clienttypes.Height{RevisionNumber: 0, RevisionHeight: 1},
 								// the proposal should *not* have passed because only carol voted for it,
 								// and carol is not enough to pass the quorum
-								Status: strconv.Itoa(int(gov.ProposalStatus_PROPOSAL_STATUS_FAILED)),
+								Status: strconv.Itoa(int(gov.ProposalStatus_PROPOSAL_STATUS_REJECTED)),
 							},
 						},
 					},