Add integration tests for stack reason display scenarios

- update-stack: test empty reason for non-active state, test reason with --reason flag
- stack: test no reason for active state, test empty reason for non-active state, test reason for non-active state with reason

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: simonjjones

integration/v7/isolated/stack_command_test.go

@@ -124,6 +124,15 @@ var _ = Describe("stack command", func() {
 				Eventually(session).Should(Say(`name:\s+%s`, stackName))
 				Eventually(session).Should(Say(`description:\s+%s`, stackDescription))
 				Eventually(session).Should(Say(`state:\s+ACTIVE`))
+				Consistently(session).ShouldNot(Say(`reason:`))
+				Eventually(session).Should(Exit(0))
+			})
+
+			It("does not show reason for an active stack", func() {
+				session := helpers.CF("stack", stackName)
+
+				Eventually(session).Should(Say(`state:\s+ACTIVE`))
+				Consistently(session).ShouldNot(Say(`reason:`))
 				Eventually(session).Should(Exit(0))
 			})
 
@@ -137,6 +146,42 @@ var _ = Describe("stack command", func() {
 				Eventually(session).Should(Say(`^[a-z0-9]{8}-[a-z0-9]{4}-[a-z0-9]{4}-[a-z0-9]{4}-[a-z0-9]{12}`))
 				Eventually(session).Should(Exit(0))
 			})
+
+			When("the stack is in a non-active state without a reason", func() {
+				BeforeEach(func() {
+					session := helpers.CF("update-stack", stackName, "--state", "deprecated")
+					Eventually(session).Should(Exit(0))
+				})
+
+				It("shows an empty reason field", func() {
+					session := helpers.CF("stack", stackName)
+
+					Eventually(session).Should(Say(`Getting info for stack %s as %s\.\.\.`, stackName, username))
+					Eventually(session).Should(Say(`name:\s+%s`, stackName))
+					Eventually(session).Should(Say(`description:\s+%s`, stackDescription))
+					Eventually(session).Should(Say(`state:\s+DEPRECATED`))
+					Eventually(session).Should(Say(`reason:\s*$`))
+					Eventually(session).Should(Exit(0))
+				})
+			})
+
+			When("the stack is in a non-active state with a reason", func() {
+				BeforeEach(func() {
+					session := helpers.CF("update-stack", stackName, "--state", "disabled", "--reason", "This stack is no longer supported.")
+					Eventually(session).Should(Exit(0))
+				})
+
+				It("shows the reason in the output", func() {
+					session := helpers.CF("stack", stackName)
+
+					Eventually(session).Should(Say(`Getting info for stack %s as %s\.\.\.`, stackName, username))
+					Eventually(session).Should(Say(`name:\s+%s`, stackName))
+					Eventually(session).Should(Say(`description:\s+%s`, stackDescription))
+					Eventually(session).Should(Say(`state:\s+DISABLED`))
+					Eventually(session).Should(Say(`reason:\s+This stack is no longer supported\.`))
+					Eventually(session).Should(Exit(0))
+				})
+			})
 		})
 	})
 })

integration/v7/isolated/update_stack_command_test.go

@@ -207,6 +207,32 @@ var _ = Describe("update-stack command", func() {
 				})
 			})
 
+			When("updating to a non-active state without a reason", func() {
+				It("shows an empty reason field in the output", func() {
+					session := helpers.CF("update-stack", stackName, "--state", "deprecated")
+
+					Eventually(session).Should(Say(`Updating stack %s as %s\.\.\.`, stackName, username))
+					Eventually(session).Should(Say("OK"))
+					Eventually(session).Should(Say(`name:\s+%s`, stackName))
+					Eventually(session).Should(Say(`state:\s+DEPRECATED`))
+					Eventually(session).Should(Say(`reason:\s*$`))
+					Eventually(session).Should(Exit(0))
+				})
+			})
+
+			When("updating with a reason", func() {
+				It("shows the reason in the update-stack output", func() {
+					session := helpers.CF("update-stack", stackName, "--state", "disabled", "--reason", "This stack is no longer supported.")
+
+					Eventually(session).Should(Say(`Updating stack %s as %s\.\.\.`, stackName, username))
+					Eventually(session).Should(Say("OK"))
+					Eventually(session).Should(Say(`name:\s+%s`, stackName))
+					Eventually(session).Should(Say(`state:\s+DISABLED`))
+					Eventually(session).Should(Say(`reason:\s+This stack is no longer supported\.`))
+					Eventually(session).Should(Exit(0))
+				})
+			})
+
 			When("state value is provided in different cases", func() {
 				It("accepts lowercase state value", func() {
 					session := helpers.CF("update-stack", stackName, "--state", "deprecated")