fix: testutil/configurator: correctly create fresh copy of defaultConfig (#17089)

This commit is contained in:
Emmanuel T Odeke
2023-07-22 07:37:56 +00:00
committed by GitHub
parent e3482f2d41
commit 0fa85b7a42
+69 -67
View File
@@ -34,72 +34,74 @@ type Config struct {
setInitGenesis bool setInitGenesis bool
} }
var defaultConfig = &Config{ func defaultConfig() *Config {
ModuleConfigs: make(map[string]*appv1alpha1.ModuleConfig), return &Config{
BeginBlockersOrder: []string{ ModuleConfigs: make(map[string]*appv1alpha1.ModuleConfig),
"upgrade", BeginBlockersOrder: []string{
"mint", "upgrade",
"distribution", "mint",
"slashing", "distribution",
"evidence", "slashing",
"staking", "evidence",
"auth", "staking",
"bank", "auth",
"gov", "bank",
"crisis", "gov",
"genutil", "crisis",
"authz", "genutil",
"feegrant", "authz",
"nft", "feegrant",
"group", "nft",
"params", "group",
"consensus", "params",
"vesting", "consensus",
"circuit", "vesting",
}, "circuit",
EndBlockersOrder: []string{ },
"crisis", EndBlockersOrder: []string{
"gov", "crisis",
"staking", "gov",
"auth", "staking",
"bank", "auth",
"distribution", "bank",
"slashing", "distribution",
"mint", "slashing",
"genutil", "mint",
"evidence", "genutil",
"authz", "evidence",
"feegrant", "authz",
"nft", "feegrant",
"group", "nft",
"params", "group",
"consensus", "params",
"upgrade", "consensus",
"vesting", "upgrade",
"circuit", "vesting",
}, "circuit",
InitGenesisOrder: []string{ },
"auth", InitGenesisOrder: []string{
"bank", "auth",
"distribution", "bank",
"staking", "distribution",
"slashing", "staking",
"gov", "slashing",
"mint", "gov",
"crisis", "mint",
"genutil", "crisis",
"evidence", "genutil",
"authz", "evidence",
"feegrant", "authz",
"nft", "feegrant",
"group", "nft",
"params", "group",
"consensus", "params",
"upgrade", "consensus",
"vesting", "upgrade",
"circuit", "vesting",
}, "circuit",
setInitGenesis: true, },
setInitGenesis: true,
}
} }
type ModuleOption func(config *Config) type ModuleOption func(config *Config)
@@ -308,7 +310,7 @@ func OmitInitGenesis() ModuleOption {
} }
func NewAppConfig(opts ...ModuleOption) depinject.Config { func NewAppConfig(opts ...ModuleOption) depinject.Config {
cfg := defaultConfig cfg := defaultConfig()
for _, opt := range opts { for _, opt := range opts {
opt(cfg) opt(cfg)
} }