Fix env config tests

License: MIT
Signed-off-by: Robert Ignat <robert.ignat91@gmail.com>

api/rest/config_test.go

@@ -110,14 +110,15 @@ func TestLoadJSON(t *testing.T) {
 	}
 }
 
-func TestLoadJSONEnvConfig(t *testing.T) {
+func TestApplyEnvVars(t *testing.T) {
 	username := "admin"
 	password := "thisaintmypassword"
 	user1 := "user1"
 	user1pass := "user1passwd"
 	os.Setenv("CLUSTER_RESTAPI_BASICAUTHCREDS", username+":"+password+","+user1+":"+user1pass)
 	cfg := &Config{}
-	err := cfg.LoadJSON(cfgJSON)
+	cfg.Default()
+	err := cfg.ApplyEnvVars()
 	if err != nil {
 		t.Fatal(err)
 	}

cluster_config_test.go

@@ -187,15 +187,6 @@ func TestLoadJSON(t *testing.T) {
 			t.Error("expected default replication factors")
 		}
 	})
-
-	t.Run("env var override", func(t *testing.T) {
-		os.Setenv("CLUSTER_PEERNAME", "envsetpeername")
-		cfg := &Config{}
-		cfg.LoadJSON(ccfgTestJSON)
-		if cfg.Peername != "envsetpeername" {
-			t.Fatal("failed to override peername with env var")
-		}
-	})
 }
 
 func TestToJSON(t *testing.T) {
@@ -220,6 +211,16 @@ func TestDefault(t *testing.T) {
 	}
 }
 
+func TestApplyEnvVars(t *testing.T) {
+	os.Setenv("CLUSTER_PEERNAME", "envsetpeername")
+	cfg := &Config{}
+	cfg.Default()
+	cfg.ApplyEnvVars()
+	if cfg.Peername != "envsetpeername" {
+		t.Fatal("failed to override peername with env var")
+	}
+}
+
 func TestValidate(t *testing.T) {
 	cfg := &Config{}
 	cfg.Default()

config/config_test.go

@@ -30,6 +30,10 @@ func (m *mockCfg) Default() error {
 	return nil
 }
 
+func (m *mockCfg) ApplyEnvVars() error {
+	return nil
+}
+
 func (m *mockCfg) Validate() error {
 	return nil
 }