Add error when value is misconfigured

vars/vars.go

@@ -49,11 +49,7 @@ func (ev EnvVar) GetBooleanValue() (bool, error) {
 	if isPresent {
 		boolValue, err := strconv.ParseBool(oEnvValue)
 		if err != nil {
-			defaultBoolValue, err := strconv.ParseBool(ev.defaultValue)
-			if err != nil {
-				return false, fmt.Errorf("error parsing default value: %v", err)
-			}
-			return defaultBoolValue, nil
+			return false, fmt.Errorf("error parsing the boolean value: %v", err)
 		}
 		return boolValue, nil
 	}

vars/vars_test.go

@@ -90,7 +90,7 @@ func TestGetBooleanValue(t *testing.T) {
 			name:    "default value when env value is misconigured",
 			in:      EnvVar{envVarName: "TEST_ENV_VAR", defaultValue: "false", isRequired: false},
 			want:    false,
-			wantErr: nil,
+			wantErr: fmt.Errorf("error parsing the boolean value: strconv.ParseBool: parsing \"not_a_bool\": invalid syntax"),
 			envVars: map[string]string{"TEST_ENV_VAR": "not_a_bool"},
 		},
 		{
@@ -112,12 +112,6 @@ func TestGetBooleanValue(t *testing.T) {
 			wantErr: fmt.Errorf("error parsing default value: strconv.ParseBool: parsing \"not_a_bool\": invalid syntax"),
 			envVars: map[string]string{},
 		},
-		{
-			name:    "default value is misconfigured and env var is present",
-			in:      EnvVar{envVarName: "TEST_ENV_VAR", defaultValue: "not_a_bool", isRequired: false},
-			wantErr: fmt.Errorf("error parsing default value: strconv.ParseBool: parsing \"not_a_bool\": invalid syntax"),
-			envVars: map[string]string{"TEST_ENV_VAR": "again_not_a_bool"},
-		},
 	}
 	for _, tc := range tests {
 		t.Run(tc.name, func(t *testing.T) {