Tests pass!

ammario · ammario · commit f7fb54cc09c2 · 2022-09-02T22:42:30.000Z
diff --git a/cli/delete.go b/cli/delete.go
@@ -39,6 +39,7 @@ func deleteWorkspace() *cobra.Command {
 			}
 
 			var state []byte
+
 			if orphan {
 				cliui.Warn(cmd.ErrOrStderr(), "Orphaning workspace",
 					"Template edit permission is required to orphan workspaces.",
@@ -48,12 +49,13 @@ func deleteWorkspace() *cobra.Command {
 				if err != nil {
 					return err
 				}
-
-				state, err = codersdk.OrphanTerraformState(state)
-				if err != nil {
-					return err
+				// If there's alreay no state, orphanage makes no sense.
+				if len(state) > 0 {
+					state, err = codersdk.OrphanTerraformState(state)
+					if err != nil {
+						return err
+					}
 				}
-				fmt.Printf("new state: %s\n", state)
 			}
 
 			before := time.Now()

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,7 @@ func deleteWorkspace() *cobra.Command {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`var state []byte`
	`42`	`+`
`42`	`43`	`if orphan {`
`43`	`44`	`cliui.Warn(cmd.ErrOrStderr(), "Orphaning workspace",`
`44`	`45`	`"Template edit permission is required to orphan workspaces.",`
`@@ -48,12 +49,13 @@ func deleteWorkspace() *cobra.Command {`
`48`	`49`	`if err != nil {`
`49`	`50`	`return err`
`50`	`51`	`}`
`51`		`-`
`52`		`- state, err = codersdk.OrphanTerraformState(state)`
`53`		`- if err != nil {`
`54`		`- return err`
	`52`	`+ // If there's alreay no state, orphanage makes no sense.`
	`53`	`+ if len(state) > 0 {`
	`54`	`+ state, err = codersdk.OrphanTerraformState(state)`
	`55`	`+ if err != nil {`
	`56`	`+ return err`
	`57`	`+ }`
`55`	`58`	`}`
`56`		`- fmt.Printf("new state: %s\n", state)`
`57`	`59`	`}`
`58`	`60`
`59`	`61`	`before := time.Now()`