Commit2a5a34b

committed

fix: display the correct response for coder list

1 parent3590102 commit2a5a34bCopy full SHA for 2a5a34b

File tree

+31

-1

lines changed

+31

-1

lines changed

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -83,6 +83,10 @@ func (f *OutputFormatter) Format(ctx context.Context, data any) (string, error)`
`83`	`83`	`return"",xerrors.Errorf("unknown output format %q",f.formatID)`
`84`	`84`	`}`
`85`	`85`
	`86`	`+func (f*OutputFormatter)IsMachineReadableFormat()bool {`
	`87`	`+returnf.formatID=="json"`
	`88`	`+}`
	`89`	`+`
`86`	`90`	`typetableFormatstruct {`
`87`	`91`	`defaultColumns []string`
`88`	`92`	`allColumns []string`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ func (r RootCmd) list() serpent.Command {`
`112`	`112`	`returnerr`
`113`	`113`	`}`
`114`	`114`
`115`		`-iflen(res)==0 {`
	`115`	`+iflen(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>"))`

Lines changed: 26 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -74,4 +74,30 @@ func TestList(t *testing.T) {`
`74`	`74`	`require.NoError(t,json.Unmarshal(out.Bytes(),&workspaces))`
`75`	`75`	`require.Len(t,workspaces,1)`
`76`	`76`	`})`
	`77`	`+`
	`78`	`+t.Run("NoWorkspacesJSON",func(t*testing.T) {`
	`79`	`+t.Parallel()`
	`80`	`+client:=coderdtest.New(t,nil)`
	`81`	`+owner:=coderdtest.CreateFirstUser(t,client)`
	`82`	`+member,_:=coderdtest.CreateAnotherUser(t,client,owner.OrganizationID)`
	`83`	`+`
	`84`	`+inv,root:=clitest.New(t,"list","--output=json")`
	`85`	`+clitest.SetupConfig(t,member,root)`
	`86`	`+`
	`87`	`+ctx,cancelFunc:=context.WithTimeout(context.Background(),testutil.WaitLong)`
	`88`	`+defercancelFunc()`
	`89`	`+`
	`90`	`+stdout:=bytes.NewBuffer(nil)`
	`91`	`+stderr:=bytes.NewBuffer(nil)`
	`92`	`+inv.Stdout=stdout`
	`93`	`+inv.Stderr=stderr`
	`94`	`+err:=inv.WithContext(ctx).Run()`
	`95`	`+require.NoError(t,err)`
	`96`	`+`
	`97`	`+varworkspaces []codersdk.Workspace`
	`98`	`+require.NoError(t,json.Unmarshal(stdout.Bytes(),&workspaces))`
	`99`	`+require.Len(t,workspaces,0)`
	`100`	`+`
	`101`	`+require.Len(t,stderr.Bytes(),0)`
	`102`	`+})`
`77`	`103`	`}`

Comments

(0)