image list show image type remove id (#24)

The header shows IMAGE TYPE (not ID)
The rows contain image_type values (ubuntu-22.04-cuda-12.0, pytorch-2.0) from the mock server — not UUID-style IDs

Author: hi-lei

internal/verda-cli/cmd/images/images.go

@@ -106,16 +106,16 @@ func runImages(cmd *cobra.Command, f cmdutil.Factory, ioStreams cmdutil.IOStream
 	}
 
 	_, _ = fmt.Fprintf(ioStreams.Out, "  %d image(s) found\n\n", len(images))
-	_, _ = fmt.Fprintf(ioStreams.Out, "  %-38s  %-45s  %-12s  %s\n", "ID", "NAME", "CATEGORY", "DETAILS")
-	_, _ = fmt.Fprintf(ioStreams.Out, "  %-38s  %-45s  %-12s  %s\n", "--", "----", "--------", "-------")
+	_, _ = fmt.Fprintf(ioStreams.Out, "  %-38s  %-45s  %-12s  %s\n", "IMAGE TYPE", "NAME", "CATEGORY", "DETAILS")
+	_, _ = fmt.Fprintf(ioStreams.Out, "  %-38s  %-45s  %-12s  %s\n", "----------", "----", "--------", "-------")
 	for i := range images {
 		details := strings.Join(images[i].Details, ", ")
 		def := ""
 		if images[i].IsDefault {
 			def = " *"
 		}
 		_, _ = fmt.Fprintf(ioStreams.Out, "  %-38s  %-45s  %-12s  %s%s\n",
-			images[i].ID, images[i].Name, images[i].Category, details, def)
+			images[i].ImageType, images[i].Name, images[i].Category, details, def)
 	}
 	return nil
 }

internal/verda-cli/cmd/images/images_test.go

@@ -1,9 +1,15 @@
 package images
 
 import (
+	"bytes"
+	"strings"
 	"testing"
 
+	"github.com/spf13/cobra"
 	"github.com/verda-cloud/verdacloud-sdk-go/pkg/verda"
+	"github.com/verda-cloud/verdacloud-sdk-go/pkg/verda/testutil"
+
+	cmdutil "github/verda-cloud/verda-cli/internal/verda-cli/cmd/util"
 )
 
 func TestFilterExcludesClusterImages(t *testing.T) {
@@ -67,3 +73,43 @@ func TestFilterEmptyList(t *testing.T) {
 		t.Fatalf("expected 0 images, got %d", len(filtered))
 	}
 }
+
+func TestOutputShowsImageTypeNotID(t *testing.T) {
+	t.Parallel()
+
+	mockServer := testutil.NewMockServer()
+	defer mockServer.Close()
+
+	client := verda.NewTestClient(mockServer)
+
+	var buf bytes.Buffer
+	ioStreams := cmdutil.IOStreams{Out: &buf, ErrOut: &buf}
+	f := cmdutil.NewTestFactory(nil)
+	f.ClientOverride = client
+
+	root := &cobra.Command{Use: "verda", SilenceUsage: true, SilenceErrors: true}
+	root.AddCommand(NewCmdImages(f, ioStreams))
+	root.SetArgs([]string{"images"})
+
+	if err := root.Execute(); err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+
+	out := buf.String()
+
+	// Header should say IMAGE TYPE, not ID.
+	if !strings.Contains(out, "IMAGE TYPE") {
+		t.Errorf("expected header to contain IMAGE TYPE, got:\n%s", out)
+	}
+	if strings.Contains(out, "  ID  ") {
+		t.Errorf("header should not contain ID column, got:\n%s", out)
+	}
+
+	// Rows should contain image_type values from mock server, not UUIDs.
+	if !strings.Contains(out, "ubuntu-22.04-cuda-12.0") {
+		t.Errorf("expected image_type 'ubuntu-22.04-cuda-12.0' in output, got:\n%s", out)
+	}
+	if !strings.Contains(out, "pytorch-2.0") {
+		t.Errorf("expected image_type 'pytorch-2.0' in output, got:\n%s", out)
+	}
+}