From f0e651a117080e0eb28c87b851e5544f6e94628f Mon Sep 17 00:00:00 2001 From: Philip Offtermatt Date: Fri, 19 Jul 2024 13:33:34 +0200 Subject: [PATCH] Debug gov with inactive vals test --- tests/e2e/config.go | 15 +++++---------- tests/e2e/steps_inactive_vals.go | 2 +- 2 files changed, 6 insertions(+), 11 deletions(-) diff --git a/tests/e2e/config.go b/tests/e2e/config.go index ff7227012e..e27f8a42cf 100644 --- 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 index ba21c46511..17bc419839 100644 --- 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)), }, }, },