fix: fix build command by listing the argv (#40)

* fix: fix build commad by listing the argv

* fix: update argument list for image-create shell out

Author: aslamcodes

internal/config/image.go

@@ -1,7 +1,6 @@
 package config
 
 import (
-	"fmt"
 	"strings"
 )
 
@@ -16,37 +15,40 @@ type Image struct {
 }
 
 func (i *Image) Args() []string {
-	args := []string{}
+	var args []string
 
 	if i.Name != "" {
-		args = append(args, fmt.Sprintf(`--name "%s"`, i.Name))
+		args = append(args, "--name", i.Name)
 	}
 
 	if i.DisplayName != "" {
-		args = append(args, fmt.Sprintf(`--display-name "%s"`, i.DisplayName))
+		args = append(args, "--display-name", i.DisplayName)
 	}
+
 	if i.Description != "" {
-		args = append(args, fmt.Sprintf(`--description "%s"`, i.Description))
+		args = append(args, "--description", i.Description)
 	}
+
 	if i.UseLatestAgentVersion {
 		args = append(args, "--use-latest-agent-version")
 	}
+
 	if i.EnableDynamicAppCatalog {
 		args = append(args, "--enable-dynamic-app-catalog")
 	}
+
 	if i.DryRun {
 		args = append(args, "--dry-run")
 	}
 
 	if len(i.Tags) > 0 {
-		tagString := []string{"--tags"}
+		args = append(args, "--tags")
 		for _, tag := range i.Tags {
 			parts := strings.SplitSeq(tag, ":")
 			for part := range parts {
-				tagString = append(tagString, fmt.Sprintf(`"%s"`, part))
+				args = append(args, part)
 			}
 		}
-		args = append(args, strings.Join(tagString, " "))
 	}
 
 	return args

internal/service/image_build_service_test.go

@@ -26,13 +26,13 @@ func TestImageBuild(t *testing.T) {
 		Description:             "test3",
 		EnableDynamicAppCatalog: true,
 		UseLatestAgentVersion:   true,
-		Tags:                    []string{"k1", "v1"},
+		Tags:                    []string{"k1", "v1", "k2", "built with appstreamfile"},
 		DryRun:                  true,
 	}
 
 	i.BuildImage(image)
 
-	expectedCommand := strings.TrimSpace(`image-assistant create-image --name "test" --display-name "test2" --description "test3" --use-latest-agent-version --enable-dynamic-app-catalog --dry-run --tags "k1" "v1"`)
+	expectedCommand := strings.TrimSpace(`image-assistant create-image --name test --display-name test2 --description test3 --use-latest-agent-version --enable-dynamic-app-catalog --dry-run --tags k1 v1 k2 built with appstreamfile`)
 	actual := strings.TrimSpace(fmt.Sprintf("%s %s", fc.LastCommand, strings.Join(fc.LastArgs, " ")))
 
 	if expectedCommand != actual {