added a unit test for checking if the DefaultValueSet is set or not

Author: bagulm123

pkg/devspace/config/loader/variable/testing/with_default_value/devspace.yaml

@@ -0,0 +1,10 @@
+version: v2beta1
+name: test
+
+vars:
+  MYSQL_VERSION:
+    question: Which mysql version do you want to use?
+    default: ""
+pipelines:
+  dev: |-
+    echo ${MYSQL_VERSION}

pkg/devspace/config/loader/variable/testing/without_default_value/devspace.yaml

@@ -0,0 +1,9 @@
+version: v2beta1
+name: test
+
+vars:
+  MYSQL_VERSION:
+    question: Which mysql version do you want to use?
+pipelines:
+  dev: |-
+    echo ${MYSQL_VERSION}

pkg/devspace/config/loader/variable/undefined_variable.go

@@ -69,6 +69,17 @@ func convertStringValue(value string) interface{} {
 }
 
 func askQuestion(variable *latest.Variable, log log.Logger) (string, error) {
+	params := getParams(variable)
+
+	answer, err := log.Question(params)
+	if err != nil {
+		return "", err
+	}
+
+	return answer, nil
+}
+
+func getParams(variable *latest.Variable) *survey.QuestionOptions {
 	params := &survey.QuestionOptions{}
 
 	if variable == nil {
@@ -107,11 +118,5 @@ func askQuestion(variable *latest.Variable, log log.Logger) (string, error) {
 			}
 		}
 	}
-
-	answer, err := log.Question(params)
-	if err != nil {
-		return "", err
-	}
-
-	return answer, nil
+	return params
 }

pkg/devspace/config/loader/variable/undefined_variable_test.go

@@ -0,0 +1,34 @@
+package variable
+
+import (
+	"fmt"
+	"github.com/loft-sh/devspace/pkg/devspace/config/versions/latest"
+	"gopkg.in/yaml.v3"
+	"os"
+	"testing"
+)
+
+func TestGetParams(t *testing.T) {
+	testCases := map[string]bool{"testing/with_default_value/devspace.yaml": true, "testing/without_default_value/devspace.yaml": false}
+	for input, expected := range testCases {
+		config := getConfig(input)
+		variable := config.Vars["MYSQL_VERSION"]
+		actual := getParams(variable)
+		if expected != actual.DefaultValueSet {
+			t.Errorf("TestCase %s\nactual:%t\nexpected:%t", input, actual.DefaultValueSet, true)
+		}
+	}
+}
+
+func getConfig(filename string) *latest.Config {
+	v := &latest.Config{}
+	yamlFile, err := os.ReadFile(filename)
+	if err != nil {
+		fmt.Printf("yamlFile.Get err   #%v ", err)
+	}
+	err = yaml.Unmarshal(yamlFile, v)
+	if err != nil {
+		fmt.Printf("Unmarshal: %v", err)
+	}
+	return v
+}