Test for multiple bindings

Author: jochenehret

command/v7/unbind_service_command_test.go

@@ -34,6 +34,7 @@ var _ = Describe("unbind-service Command", func() {
 		fakeSpaceName           = "fake-space-name"
 		fakeSpaceGUID           = "fake-space-guid"
 		fakeBindingGUID         = "fake-binding-guid"
+		fakeBindingGUID2        = "fake-binding-guid-2"
 	)
 
 	BeforeEach(func() {
@@ -60,7 +61,6 @@ var _ = Describe("unbind-service Command", func() {
 
 		fakeActor.GetCurrentUserReturns(configv3.User{Name: fakeUserName}, nil)
 
-		// Default: one binding found and deletion succeeds without stream
 		fakeActor.ListServiceAppBindingsReturns(
 			[]resources.ServiceCredentialBinding{{GUID: fakeBindingGUID}},
 			v7action.Warnings{"fake warning"},
@@ -88,18 +88,20 @@ var _ = Describe("unbind-service Command", func() {
 		Expect(actualSpace).To(BeTrue())
 	})
 
-	It("lists bindings then delegates deletion to the actor by GUID", func() {
-		Expect(fakeActor.ListServiceAppBindingsCallCount()).To(Equal(1))
-		Expect(fakeActor.ListServiceAppBindingsArgsForCall(0)).To(Equal(v7action.ListServiceAppBindingParams{
-			SpaceGUID:           fakeSpaceGUID,
-			ServiceInstanceName: fakeServiceInstanceName,
-			AppName:             fakeAppName,
-		}))
-
-		Expect(fakeActor.DeleteServiceAppBindingCallCount()).To(Equal(1))
-		Expect(fakeActor.DeleteServiceAppBindingArgsForCall(0)).To(Equal(v7action.DeleteServiceAppBindingParams{
-			ServiceBindingGUID: fakeBindingGUID,
-		}))
+	Context("one binding exists", func() {
+		It("lists binding then delegates deletion to the actor by GUID", func() {
+			Expect(fakeActor.ListServiceAppBindingsCallCount()).To(Equal(1))
+			Expect(fakeActor.ListServiceAppBindingsArgsForCall(0)).To(Equal(v7action.ListServiceAppBindingParams{
+				SpaceGUID:           fakeSpaceGUID,
+				ServiceInstanceName: fakeServiceInstanceName,
+				AppName:             fakeAppName,
+			}))
+
+			Expect(fakeActor.DeleteServiceAppBindingCallCount()).To(Equal(1))
+			Expect(fakeActor.DeleteServiceAppBindingArgsForCall(0)).To(Equal(v7action.DeleteServiceAppBindingParams{
+				ServiceBindingGUID: fakeBindingGUID,
+			}))
+		})
 	})
 
 	Describe("intro message", func() {
@@ -112,7 +114,7 @@ var _ = Describe("unbind-service Command", func() {
 			))
 
 			Expect(testUI.Out).To(Say(
-				`Unbinding app %s from service %s in org %s / space %s as %s\.{3}\n`,
+				`Unbinding app %s from service %s in org %s / space %s as %s\.\.\.\n`,
 				fakeAppName,
 				fakeServiceInstanceName,
 				fakeOrgName,
@@ -122,7 +124,7 @@ var _ = Describe("unbind-service Command", func() {
 		})
 	})
 
-	When("binding did not exist", func() {
+	When("binding does not exist", func() {
 		BeforeEach(func() {
 			fakeActor.ListServiceAppBindingsReturns(
 				nil,
@@ -362,4 +364,31 @@ var _ = Describe("unbind-service Command", func() {
 			Expect(executeErr).To(MatchError("bad thing"))
 		})
 	})
+
+	Context("multiple bindings exist", func() {
+		BeforeEach(func() {
+			fakeActor.ListServiceAppBindingsReturns(
+				[]resources.ServiceCredentialBinding{{GUID: fakeBindingGUID}, {GUID: fakeBindingGUID2}},
+				v7action.Warnings{"fake warning"},
+				nil,
+			)
+		})
+
+		It("deletes each binding by GUID and prints messages for both", func() {
+			Expect(fakeActor.DeleteServiceAppBindingCallCount()).To(Equal(2))
+
+			firstArgs := fakeActor.DeleteServiceAppBindingArgsForCall(0)
+			secondArgs := fakeActor.DeleteServiceAppBindingArgsForCall(1)
+			Expect([]string{firstArgs.ServiceBindingGUID, secondArgs.ServiceBindingGUID}).To(ConsistOf(fakeBindingGUID, fakeBindingGUID2))
+
+			Expect(testUI.Out).To(SatisfyAll(
+				Say(`Deleting service binding %s\.+\n`, fakeBindingGUID),
+				Say(`OK\n`),
+				Say(`Deleting service binding %s\.+\n`, fakeBindingGUID2),
+				Say(`OK\n`),
+			))
+
+			Expect(testUI.Err).To(Say("fake warning"))
+		})
+	})
 })