cf revisions no longer reports EXPERIMENTAL notice

cloudfoundry · Jul 29, 2024 · f3ba47c · f3ba47c
1 parent 04197bf
commit f3ba47c
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 10 deletions.
diff --git a/command/v7/revision_command.go b/command/v7/revision_command.go
@@ -1,7 +1,6 @@
 package v7
 
 import (
-	"code.cloudfoundry.org/cli/command"
 	"code.cloudfoundry.org/cli/command/flag"
 )
 
@@ -14,7 +13,6 @@ type RevisionCommand struct {
 }
 
 func (cmd RevisionCommand) Execute(_ []string) error {
-	cmd.UI.DisplayWarning(command.ExperimentalWarning)
 	cmd.UI.DisplayNewline()
 	return nil
 }
diff --git a/command/v7/revision_command_test.go b/command/v7/revision_command_test.go
@@ -49,7 +49,7 @@ var _ = Describe("revision Command", func() {
 		Expect(cmd.Execute(nil)).To(Succeed())
 	})
 
-	It("displays the experimental warning", func() {
-		Expect(testUI.Err).To(Say("This command is in EXPERIMENTAL stage and may change without notice"))
+	It("no longer displays the experimental warning", func() {
+		Expect(testUI.Err).NotTo(Say("This command is in EXPERIMENTAL stage and may change without notice"))
 	})
 })
diff --git a/command/v7/revisions_command.go b/command/v7/revisions_command.go
@@ -5,7 +5,6 @@ import (
 	"strconv"
 
 	"code.cloudfoundry.org/cli/actor/v7action"
-	"code.cloudfoundry.org/cli/command"
 	"code.cloudfoundry.org/cli/command/flag"
 	"code.cloudfoundry.org/cli/resources"
 	"code.cloudfoundry.org/cli/util/ui"
@@ -26,7 +25,6 @@ type RevisionsCommand struct {
 }
 
 func (cmd RevisionsCommand) Execute(_ []string) error {
-	cmd.UI.DisplayWarning(command.ExperimentalWarning)
 	cmd.UI.DisplayNewline()
 
 	err := cmd.SharedActor.CheckTarget(true, true)

diff --git a/command/v7/revisions_command_test.go b/command/v7/revisions_command_test.go
@@ -59,8 +59,8 @@ var _ = Describe("revisions Command", func() {
 		executeErr = cmd.Execute(nil)
 	})
 
-	It("displays the experimental warning", func() {
-		Expect(testUI.Err).To(Say("This command is in EXPERIMENTAL stage and may change without notice"))
+	It("no longer displays the experimental warning", func() {
+		Expect(testUI.Err).NotTo(Say("This command is in EXPERIMENTAL stage and may change without notice"))
 	})
 
 	When("checking target fails", func() {

diff --git a/integration/v7/isolated/revision_command_test.go b/integration/v7/isolated/revision_command_test.go
@@ -15,7 +15,7 @@ var _ = Describe("revision command", func() {
 			It("appears in cf help -a", func() {
 				session := helpers.CF("help", "-a")
 				Eventually(session).Should(Exit(0))
-				Expect(session).To(HaveCommandInCategoryWithDescription("revision", "EXPERIMENTAL COMMANDS", "Show details for a specific app revision"))
+				Expect(session).NotTo(HaveCommandInCategoryWithDescription("revision", "EXPERIMENTAL COMMANDS", "Show details for a specific app revision"))
 			})
 
 			It("Displays revision command usage to output", func() {

diff --git a/integration/v7/isolated/revisions_command_test.go b/integration/v7/isolated/revisions_command_test.go
@@ -33,7 +33,7 @@ var _ = Describe("revisions command", func() {
 			It("appears in cf help -a", func() {
 				session := helpers.CF("help", "-a")
 				Eventually(session).Should(Exit(0))
-				Expect(session).To(HaveCommandInCategoryWithDescription("revisions", "EXPERIMENTAL COMMANDS", "List revisions of an app"))
+				Expect(session).NotTo(HaveCommandInCategoryWithDescription("revisions", "EXPERIMENTAL COMMANDS", "List revisions of an app"))
 			})
 
 			It("Displays command usage to output", func() {