fix: display the correct response for coder list

evgeniy-scherbina · evgeniy-scherbina · commit 43ac0f02d6c8 · 2025-02-12T16:53:10.000-05:00
diff --git a/cli/cliui/output.go b/cli/cliui/output.go
@@ -83,6 +83,10 @@ func (f *OutputFormatter) Format(ctx context.Context, data any) (string, error)
 	return "", xerrors.Errorf("unknown output format %q", f.formatID)
 }
 
+func (f *OutputFormatter) IsMachineReadableFormat() bool {
+	return f.formatID == "json"
+}
+
 type tableFormat struct {
 	defaultColumns []string
 	allColumns     []string
diff --git a/cli/list.go b/cli/list.go
@@ -112,7 +112,7 @@ func (r *RootCmd) list() *serpent.Command {
 				return err
 			}
 
-			if len(res) == 0 {
+			if len(res) == 0 && !formatter.IsMachineReadableFormat() {
 				pretty.Fprintf(inv.Stderr, cliui.DefaultStyles.Prompt, "No workspaces found! Create one:\n")
 				_, _ = fmt.Fprintln(inv.Stderr)
 				_, _ = fmt.Fprintln(inv.Stderr, "  "+pretty.Sprint(cliui.DefaultStyles.Code, "coder create <name>"))
diff --git a/cli/list_test.go b/cli/list_test.go
@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
+	"fmt"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -74,4 +75,33 @@ func TestList(t *testing.T) {
 		require.NoError(t, json.Unmarshal(out.Bytes(), &workspaces))
 		require.Len(t, workspaces, 1)
 	})
+
+	t.Run("NoWorkspacesJSON", func(t *testing.T) {
+		t.Parallel()
+		client := coderdtest.New(t, nil)
+		owner := coderdtest.CreateFirstUser(t, client)
+		member, _ := coderdtest.CreateAnotherUser(t, client, owner.OrganizationID)
+
+		inv, root := clitest.New(t, "list", "--output=json")
+		clitest.SetupConfig(t, member, root)
+
+		ctx, cancelFunc := context.WithTimeout(context.Background(), testutil.WaitLong)
+		defer cancelFunc()
+
+		stdout := bytes.NewBuffer(nil)
+		stderr := bytes.NewBuffer(nil)
+		inv.Stdout = stdout
+		inv.Stderr = stderr
+		err := inv.WithContext(ctx).Run()
+		require.NoError(t, err)
+
+		fmt.Printf("stdout.Bytes: %s\n", stdout.Bytes())
+		fmt.Printf("stderr.Bytes: %s\n", stderr.Bytes())
+
+		var workspaces []codersdk.Workspace
+		require.NoError(t, json.Unmarshal(stdout.Bytes(), &workspaces))
+		require.Len(t, workspaces, 0)
+
+		require.Len(t, stderr.Bytes(), 0)
+	})
 }

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ func (r RootCmd) list() serpent.Command {`
`112`	`112`	`return err`
`113`	`113`	`}`
`114`	`114`
`115`		`- if len(res) == 0 {`
	`115`	`+ if len(res) == 0 && !formatter.IsMachineReadableFormat() {`
`116`	`116`	`pretty.Fprintf(inv.Stderr, cliui.DefaultStyles.Prompt, "No workspaces found! Create one:\n")`
`117`	`117`	`_, _ = fmt.Fprintln(inv.Stderr)`
`118`	`118`	`_, _ = fmt.Fprintln(inv.Stderr, " "+pretty.Sprint(cliui.DefaultStyles.Code, "coder create <name>"))`