fix panic when overriding config settings of a test that has no config

ethpandaops · Mar 9, 2024 · f3d320d · f3d320d
1 parent 9a891e8
commit f3d320d
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/pkg/coordinator/test/descriptor.go b/pkg/coordinator/test/descriptor.go
@@ -122,17 +122,22 @@ func LoadExternalTestConfig(ctx context.Context, globalVars types.Variables, ext
 	}
 
 	decoder := yaml.NewDecoder(reader)
-	testConfig := &types.TestConfig{
-		Config:     map[string]interface{}{},
-		ConfigVars: map[string]string{},
-	}
+	testConfig := &types.TestConfig{}
 	testVars := vars.NewVariables(nil)
 
 	err := decoder.Decode(testConfig)
 	if err != nil {
 		return nil, nil, fmt.Errorf("error decoding external test config %v: %v", extTestCfg.File, err)
 	}
 
+	if testConfig.Config == nil {
+		testConfig.Config = map[string]interface{}{}
+	}
+
+	if testConfig.ConfigVars == nil {
+		testConfig.ConfigVars = map[string]string{}
+	}
+
 	for k, v := range testConfig.Config {
 		testVars.SetVar(k, v)
 	}