fix: testutil/configurator: correctly create fresh copy of defaultCon…

…fig (backport #17089) (#17092) Co-authored-by: Emmanuel T Odeke <[email protected]> Co-authored-by: Julien Robert <[email protected]>
cosmos · Jul 23, 2023 · a811085 · a811085
1 parent 3b509c1
commit a811085
Showing 1 changed file with 66 additions and 64 deletions.
diff --git a/testutil/configurator/configurator.go b/testutil/configurator/configurator.go
@@ -33,69 +33,71 @@ type Config struct {
 	setInitGenesis     bool
 }
 
-var defaultConfig = &Config{
-	ModuleConfigs: make(map[string]*appv1alpha1.ModuleConfig),
-	BeginBlockersOrder: []string{
-		"upgrade",
-		"mint",
-		"distribution",
-		"slashing",
-		"evidence",
-		"staking",
-		"auth",
-		"bank",
-		"gov",
-		"crisis",
-		"genutil",
-		"authz",
-		"feegrant",
-		"nft",
-		"group",
-		"params",
-		"consensus",
-		"vesting",
-	},
-	EndBlockersOrder: []string{
-		"crisis",
-		"gov",
-		"staking",
-		"auth",
-		"bank",
-		"distribution",
-		"slashing",
-		"mint",
-		"genutil",
-		"evidence",
-		"authz",
-		"feegrant",
-		"nft",
-		"group",
-		"params",
-		"consensus",
-		"upgrade",
-		"vesting",
-	},
-	InitGenesisOrder: []string{
-		"auth",
-		"bank",
-		"distribution",
-		"staking",
-		"slashing",
-		"gov",
-		"mint",
-		"crisis",
-		"genutil",
-		"evidence",
-		"authz",
-		"feegrant",
-		"nft",
-		"group",
-		"params",
-		"consensus",
-		"upgrade",
-		"vesting",
-	},
-	setInitGenesis: true,
+func defaultConfig() *Config {
+	return &Config{
+		ModuleConfigs: make(map[string]*appv1alpha1.ModuleConfig),
+		BeginBlockersOrder: []string{
+			"upgrade",
+			"mint",
+			"distribution",
+			"slashing",
+			"evidence",
+			"staking",
+			"auth",
+			"bank",
+			"gov",
+			"crisis",
+			"genutil",
+			"authz",
+			"feegrant",
+			"nft",
+			"group",
+			"params",
+			"consensus",
+			"vesting",
+		},
+		EndBlockersOrder: []string{
+			"crisis",
+			"gov",
+			"staking",
+			"auth",
+			"bank",
+			"distribution",
+			"slashing",
+			"mint",
+			"genutil",
+			"evidence",
+			"authz",
+			"feegrant",
+			"nft",
+			"group",
+			"params",
+			"consensus",
+			"upgrade",
+			"vesting",
+		},
+		InitGenesisOrder: []string{
+			"auth",
+			"bank",
+			"distribution",
+			"staking",
+			"slashing",
+			"gov",
+			"mint",
+			"crisis",
+			"genutil",
+			"evidence",
+			"authz",
+			"feegrant",
+			"nft",
+			"group",
+			"params",
+			"consensus",
+			"upgrade",
+			"vesting",
+		},
+		setInitGenesis: true,
+	}
 }
 
 type ModuleOption func(config *Config)
@@ -295,7 +297,7 @@ func OmitInitGenesis() ModuleOption {
 }
 
 func NewAppConfig(opts ...ModuleOption) depinject.Config {
-	cfg := defaultConfig
+	cfg := defaultConfig()
 	for _, opt := range opts {
 		opt(cfg)
 	}