add tests for individual validators

cmd/bundle/init.go

@@ -14,7 +14,6 @@ var initCmd = &cobra.Command{
 	Args:  cobra.ExactArgs(1),
 	RunE: func(cmd *cobra.Command, args []string) error {
 		// initialize default value for config file path
-		// TODO: add a test for this
 		if configFile == "" {
 			configFile = filepath.Join(targetDir, template.ConfigFileName)
 		}

libs/template/execute.go

@@ -26,8 +26,6 @@ func executeTemplate(config map[string]any, templateDefination string) (string,
 	return result.String(), nil
 }
 
-// TODO: allow skipping directories
-
 func generateFile(config map[string]any, pathTemplate, contentTemplate string) error {
 	// compute file content
 	fileContent, err := executeTemplate(config, contentTemplate)

libs/template/schema.go

@@ -74,7 +74,6 @@ func validateType(v any, fieldType FieldType) error {
 	return validateFunc(v)
 }
 
-// TODO: add validation check for regex for string types
 func (schema Schema) ValidateConfig(config map[string]any) error {
 	// validate types defined in config
 	for k, v := range config {

libs/template/validators.go

@@ -9,7 +9,6 @@ import (
 
 type Validator func(v any) error
 
-// TODO: refactor tests into individual tests for individual validators
 func validateString(v any) error {
 	if _, ok := v.(string); !ok {
 		return fmt.Errorf("expected type string, but value is %#v", v)

libs/template/validators_test.go

@@ -0,0 +1,75 @@
+package template
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestValidatorString(t *testing.T) {
+	err := validateString("abc")
+	assert.NoError(t, err)
+
+	err = validateString(1)
+	assert.ErrorContains(t, err, "expected type string, but value is 1")
+
+	err = validateString(true)
+	assert.ErrorContains(t, err, "expected type string, but value is true")
+
+	err = validateString("false")
+	assert.NoError(t, err)
+}
+
+func TestValidatorBoolean(t *testing.T) {
+	err := validateBoolean(true)
+	assert.NoError(t, err)
+
+	err = validateBoolean(1)
+	assert.ErrorContains(t, err, "expected type boolean, but value is 1")
+
+	err = validateBoolean("abc")
+	assert.ErrorContains(t, err, "expected type boolean, but value is \"abc\"")
+
+	err = validateBoolean("false")
+	assert.ErrorContains(t, err, "expected type boolean, but value is \"false\"")
+}
+
+func TestValidatorFloat(t *testing.T) {
+	err := validateFloat(true)
+	assert.ErrorContains(t, err, "expected type float, but value is true")
+
+	err = validateFloat(int32(1))
+	assert.ErrorContains(t, err, "expected type float, but value is 1")
+
+	err = validateFloat(int64(1))
+	assert.ErrorContains(t, err, "expected type float, but value is 1")
+
+	err = validateFloat(float32(1))
+	assert.NoError(t, err)
+
+	err = validateFloat(float64(1))
+	assert.NoError(t, err)
+
+	err = validateFloat("abc")
+	assert.ErrorContains(t, err, "expected type float, but value is \"abc\"")
+}
+
+func TestValidatorInt(t *testing.T) {
+	err := validateInteger(true)
+	assert.ErrorContains(t, err, "expected type integer, but value is true")
+
+	err = validateInteger(int32(1))
+	assert.NoError(t, err)
+
+	err = validateInteger(int64(1))
+	assert.NoError(t, err)
+
+	err = validateInteger(float32(1))
+	assert.ErrorContains(t, err, "expected type integer, but value is 1")
+
+	err = validateInteger(float64(1))
+	assert.ErrorContains(t, err, "expected type integer, but value is 1")
+
+	err = validateInteger("abc")
+	assert.ErrorContains(t, err, "expected type integer, but value is \"abc\"")
+}