code.cloudfoundry.org/cli@v7.1.0+incompatible/actor/v2action/service_instance_test.go

package v2action_test

import (
	"errors"

	"code.cloudfoundry.org/cli/actor/actionerror"
	. "code.cloudfoundry.org/cli/actor/v2action"
	"code.cloudfoundry.org/cli/actor/v2action/v2actionfakes"
	"code.cloudfoundry.org/cli/api/cloudcontroller/ccerror"
	"code.cloudfoundry.org/cli/api/cloudcontroller/ccv2"
	"code.cloudfoundry.org/cli/api/cloudcontroller/ccv2/constant"

	. "github.com/onsi/ginkgo"
	. "github.com/onsi/ginkgo/extensions/table"
	. "github.com/onsi/gomega"
)

var _ = Describe("Service Instance Actions", func() {
	var (
		actor                     *Actor
		fakeCloudControllerClient *v2actionfakes.FakeCloudControllerClient
	)

	BeforeEach(func() {
		fakeCloudControllerClient = new(v2actionfakes.FakeCloudControllerClient)
		actor = NewActor(fakeCloudControllerClient, nil, nil)
	})

	Describe("CreateServiceInstance", func() {
		var (
			serviceInstance       ServiceInstance
			createServiceWarnings Warnings
			createServiceErr      error
		)

		When("no broker name is specified", func() {
			JustBeforeEach(func() {
				serviceInstance, createServiceWarnings, createServiceErr = actor.CreateServiceInstance("space-guid", "service", "service-plan", "service-instance", "", map[string]interface{}{"some": "params"}, []string{"tag-1", "tag-2"})
			})

			When("there are no errors creating a service instance", func() {
				var returnedInstance ccv2.ServiceInstance

				BeforeEach(func() {
					returnedInstance = ccv2.ServiceInstance{
						GUID:            "some-service-guid",
						Name:            "service",
						SpaceGUID:       "space-guid",
						ServiceGUID:     "service-guid",
						ServicePlanGUID: "service-plan-guid",
						Type:            constant.ManagedService,
						Tags:            []string{"some", "tags"},
						DashboardURL:    "http://dashboard.com",
						LastOperation:   ccv2.LastOperation{},
					}
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{{
							GUID:              "a-service-guid",
							ServiceBrokerName: "some-broker",
						}},
						nil,
						nil,
					)
					fakeCloudControllerClient.GetServicePlansReturns(
						[]ccv2.ServicePlan{
							{
								GUID: "a-random-service-plan-guid",
								Name: "service-plan-2",
							},
							{
								GUID: "the-service-plan-guid",
								Name: "service-plan",
							},
						},
						nil,
						nil)
					fakeCloudControllerClient.CreateServiceInstanceReturns(
						returnedInstance,
						ccv2.Warnings{"create-service-instance-warning"},
						nil)
				})

				It("returns the service instance and warnings", func() {
					By("getting the space services")
					Expect(createServiceErr).ToNot(HaveOccurred())
					Expect(serviceInstance).To(Equal(ServiceInstance(returnedInstance)))
					Expect(createServiceWarnings).To(ConsistOf("create-service-instance-warning"))

					Expect(fakeCloudControllerClient.GetSpaceServicesCallCount()).To(Equal(1))
					spaceGUID, serviceFilter := fakeCloudControllerClient.GetSpaceServicesArgsForCall(0)
					Expect(spaceGUID).Should(Equal("space-guid"))
					Expect(serviceFilter).Should(Equal([]ccv2.Filter{{
						Type:     constant.LabelFilter,
						Operator: constant.EqualOperator,
						Values:   []string{"service"},
					}},
					))

					By("getting the plans of the service")
					Expect(fakeCloudControllerClient.GetServicePlansCallCount()).To(Equal(1))
					planFilter := fakeCloudControllerClient.GetServicePlansArgsForCall(0)
					Expect(planFilter).Should(Equal([]ccv2.Filter{{
						Type:     constant.ServiceGUIDFilter,
						Operator: constant.EqualOperator,
						Values:   []string{"a-service-guid"},
					}},
					))

					By("creating the service instance")
					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(1))
					spaceGUID, planGUID, serviceInstanceName, parameters, tags := fakeCloudControllerClient.CreateServiceInstanceArgsForCall(0)
					Expect(serviceInstanceName).To(Equal("service-instance"))
					Expect(spaceGUID).To(Equal("space-guid"))
					Expect(planGUID).To(Equal("the-service-plan-guid"))
					Expect(parameters).To(Equal(map[string]interface{}{
						"some": "params",
					}))
					Expect(tags).To(ConsistOf("tag-1", "tag-2"))
				})
			})

			When("there are errors creating a service instance", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{{
							GUID: "a-service-guid",
						}},
						nil,
						nil,
					)
					fakeCloudControllerClient.GetServicePlansReturns(
						[]ccv2.ServicePlan{{
							GUID: "the-service-plan-guid",
							Name: "service-plan",
						}},
						nil,
						nil)

					fakeCloudControllerClient.CreateServiceInstanceReturns(
						ccv2.ServiceInstance{},
						ccv2.Warnings{"create-service-instance-warning"},
						errors.New("boom"))
				})

				It("returns the error and warnings", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError("boom"))
					Expect(createServiceWarnings).To(ConsistOf("create-service-instance-warning"))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(1))
				})
			})

			When("there are no matching services", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{},
						ccv2.Warnings{"service-warnings"},
						nil,
					)
				})

				It("returns the error and warnings", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError(actionerror.ServiceNotFoundError{Name: "service"}))
					Expect(createServiceWarnings).To(ConsistOf("service-warnings"))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})

			When("getting services returns an error", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{},
						ccv2.Warnings{"service-warnings"},
						errors.New("boom"),
					)
				})

				It("returns the error and warnings", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError("boom"))
					Expect(createServiceWarnings).To(ConsistOf("service-warnings"))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})

			When("getting service plan returns an error", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{{
							GUID: "a-service-guid",
						}},
						nil,
						nil,
					)
					fakeCloudControllerClient.GetServicePlansReturns(
						[]ccv2.ServicePlan{},
						ccv2.Warnings{"get-plan-warning"},
						errors.New("boom"),
					)
				})

				It("returns the error and warnings", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError("boom"))
					Expect(createServiceWarnings).To(ConsistOf("get-plan-warning"))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})

			When("no matching plan", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{{
							GUID: "a-service-guid",
						}},
						nil,
						nil,
					)
					fakeCloudControllerClient.GetServicePlansReturns(
						[]ccv2.ServicePlan{{
							GUID: "the-service-plan-guid",
							Name: "service-plan-2",
						}},
						ccv2.Warnings{"get-plan-warning"},
						nil)
				})

				It("returns the error and warnings", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError(actionerror.ServicePlanNotFoundError{PlanName: "service-plan", OfferingName: "service"}))
					Expect(createServiceWarnings).To(ConsistOf("get-plan-warning"))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})

			When("there are warnings", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{
							{Label: "service-1", GUID: "service-guid-1"},
						},
						[]string{"foo"},
						nil)

					fakeCloudControllerClient.GetServicePlansReturns(
						[]ccv2.ServicePlan{
							{Name: "service-plan", GUID: "service-plan-guid-1"},
						},
						[]string{"bar"},
						nil)
					fakeCloudControllerClient.CreateServiceInstanceReturns(
						ccv2.ServiceInstance{},
						[]string{"baz", "qux"},
						nil)
				})

				It("returns the warnings", func() {
					Expect(createServiceWarnings).To(ConsistOf([]string{"foo", "bar", "baz", "qux"}))
				})
			})

			When("there are multiple available services by the same name", func() {
				BeforeEach(func() {
					// two services get returned, indicating that there are two services with the
					// same name (i.e. they are provided by different service brokers)
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{
							{
								GUID:              "a-service-guid",
								ServiceBrokerName: "some-broker",
							},
							{
								GUID:              "another-service-guid",
								ServiceBrokerName: "some-other-broker",
							},
						},
						nil,
						nil,
					)
				})

				It("returns an error and warnings", func() {
					Expect(fakeCloudControllerClient.GetSpaceServicesCallCount()).To(Equal(1))
					spaceGUID, serviceFilter := fakeCloudControllerClient.GetSpaceServicesArgsForCall(0)
					Expect(spaceGUID).Should(Equal("space-guid"))
					Expect(serviceFilter).Should(Equal([]ccv2.Filter{{
						Type:     constant.LabelFilter,
						Operator: constant.EqualOperator,
						Values:   []string{"service"},
					}},
					))

					Expect(createServiceErr).To(MatchError(actionerror.DuplicateServiceError{Name: "service"}))
					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})
		})

		When("a broker name is specified", func() {
			var brokerName string

			BeforeEach(func() {
				brokerName = "some-broker"
			})

			JustBeforeEach(func() {
				serviceInstance, createServiceWarnings, createServiceErr = actor.CreateServiceInstance("space-guid", "service", "service-plan", "service-instance", brokerName, map[string]interface{}{"some": "params"}, []string{"tag-1", "tag-2"})
			})

			When("the specified service broker exists", func() {
				var returnedInstance ccv2.ServiceInstance

				BeforeEach(func() {
					returnedInstance = ccv2.ServiceInstance{
						GUID:            "some-service-guid",
						Name:            "service",
						SpaceGUID:       "space-guid",
						ServiceGUID:     "service-guid",
						ServicePlanGUID: "service-plan-guid",
						Type:            constant.ManagedService,
						Tags:            []string{"some", "tags"},
						DashboardURL:    "http://dashboard.com",
						LastOperation:   ccv2.LastOperation{},
					}
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{
							{
								GUID:              "a-service-guid",
								ServiceBrokerName: "some-broker",
							},
							{
								GUID:              "another-service-guid",
								ServiceBrokerName: "some-other-broker",
							},
						},
						nil,
						nil,
					)
					fakeCloudControllerClient.GetServicePlansReturns(
						[]ccv2.ServicePlan{
							{
								GUID: "a-random-service-plan-guid",
								Name: "service-plan-2",
							},
							{
								GUID: "the-service-plan-guid",
								Name: "service-plan",
							},
						},
						nil,
						nil)
					fakeCloudControllerClient.CreateServiceInstanceReturns(
						returnedInstance,
						ccv2.Warnings{"create-service-instance-warning"},
						nil)
				})

				It("returns the service instance and warnings", func() {
					By("getting the space services")
					Expect(createServiceErr).ToNot(HaveOccurred())

					Expect(fakeCloudControllerClient.GetSpaceServicesCallCount()).To(Equal(1))
					spaceGUID, serviceFilters := fakeCloudControllerClient.GetSpaceServicesArgsForCall(0)
					Expect(spaceGUID).To(Equal("space-guid"))
					Expect(serviceFilters).To(Equal(
						[]ccv2.Filter{
							{
								Type:     constant.LabelFilter,
								Operator: constant.EqualOperator,
								Values:   []string{"service"},
							},
						},
					))

					By("getting the plans of the service")
					Expect(fakeCloudControllerClient.GetServicePlansCallCount()).To(Equal(1))
					planFilter := fakeCloudControllerClient.GetServicePlansArgsForCall(0)
					Expect(planFilter).To(Equal([]ccv2.Filter{{
						Type:     constant.ServiceGUIDFilter,
						Operator: constant.EqualOperator,
						Values:   []string{"a-service-guid"},
					}},
					))

					By("creating the service instance")
					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(1))
					spaceGUID, planGUID, serviceInstanceName, parameters, tags := fakeCloudControllerClient.CreateServiceInstanceArgsForCall(0)
					Expect(serviceInstanceName).To(Equal("service-instance"))
					Expect(spaceGUID).To(Equal("space-guid"))
					Expect(planGUID).To(Equal("the-service-plan-guid"))
					Expect(parameters).To(Equal(map[string]interface{}{
						"some": "params",
					}))
					Expect(tags).To(ConsistOf("tag-1", "tag-2"))

					Expect(serviceInstance).To(Equal(ServiceInstance(returnedInstance)))
					Expect(createServiceWarnings).To(ConsistOf("create-service-instance-warning"))
				})
			})

			When("the specified service broker is not present in the list of services accessible to the user", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{
							{
								GUID:              "a-service-guid",
								ServiceBrokerName: "the-wrong-broker",
							},
							{
								GUID:              "another-service-guid",
								ServiceBrokerName: "not-this-broker-either",
							},
						},
						nil,
						nil,
					)
				})

				It("returns the error", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError(actionerror.ServiceAndBrokerCombinationNotFoundError{ServiceName: "service", BrokerName: brokerName}))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})

			When("the list of services does not include service broker names", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{
							{GUID: "a-service-guid"},
							{GUID: "another-service-guid"},
						},
						nil,
						nil,
					)
				})

				It("returns the appropriate error", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError(actionerror.ServiceAndBrokerCombinationNotFoundError{ServiceName: "service", BrokerName: brokerName}))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})

			When("there is an error getting the list of space services", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetSpaceServicesReturns(
						[]ccv2.Service{{}},
						ccv2.Warnings{"get-space-services-warning"},
						errors.New("oh no"),
					)
				})

				It("returns the error and warnings", func() {
					Expect(serviceInstance).To(Equal(ServiceInstance{}))
					Expect(createServiceErr).To(MatchError("oh no"))
					Expect(createServiceWarnings).To(ConsistOf("get-space-services-warning"))

					Expect(fakeCloudControllerClient.CreateServiceInstanceCallCount()).To(Equal(0))
				})
			})
		})
	})

	Describe("ServiceInstance", func() {
		DescribeTable("IsManaged",
			func(iType constant.ServiceInstanceType, expected bool) {
				Expect(ServiceInstance{Type: iType}.IsManaged()).To(Equal(expected))
			},

			Entry("return true for managed service", constant.ManagedService, true),
			Entry("return false for any other type of service", constant.UserProvidedService, false),
		)

		DescribeTable("IsUserProvided",
			func(iType constant.ServiceInstanceType, expected bool) {
				Expect(ServiceInstance{Type: iType}.IsUserProvided()).To(Equal(expected))
			},

			Entry("return true for UserProvidedService service", constant.UserProvidedService, true),
			Entry("return false for any other type of service", constant.ManagedService, false),
		)
	})

	Describe("GetServiceInstance", func() {
		var (
			serviceInstanceGUID string

			serviceInstance ServiceInstance
			warnings        Warnings
			executeErr      error
		)

		BeforeEach(func() {
			serviceInstanceGUID = "service-instance-guid"
		})

		JustBeforeEach(func() {
			serviceInstance, warnings, executeErr = actor.GetServiceInstance(serviceInstanceGUID)
		})

		When("the service instance exists", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.GetServiceInstanceReturns(ccv2.ServiceInstance{Name: "some-service-instance", GUID: "service-instance-guid"}, ccv2.Warnings{"service-instance-warnings"}, nil)
			})

			It("returns the service instance and warnings", func() {
				Expect(executeErr).ToNot(HaveOccurred())
				Expect(serviceInstance).To(Equal(ServiceInstance{
					GUID: "service-instance-guid",
					Name: "some-service-instance",
				}))
				Expect(warnings).To(ConsistOf("service-instance-warnings"))

				Expect(fakeCloudControllerClient.GetServiceInstanceCallCount()).To(Equal(1))
				Expect(fakeCloudControllerClient.GetServiceInstanceArgsForCall(0)).To(Equal(serviceInstanceGUID))
			})
		})

		When("the service instance does not exist", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.GetServiceInstanceReturns(ccv2.ServiceInstance{}, ccv2.Warnings{"service-instance-warnings-1"}, ccerror.ResourceNotFoundError{})
			})

			It("returns errors and warnings", func() {
				Expect(executeErr).To(MatchError(actionerror.ServiceInstanceNotFoundError{GUID: serviceInstanceGUID}))
				Expect(warnings).To(ConsistOf("service-instance-warnings-1"))
			})
		})

		When("retrieving the application's bound services returns an error", func() {
			var expectedErr error

			BeforeEach(func() {
				expectedErr = errors.New("this is indeed an error, kudos!")
				fakeCloudControllerClient.GetServiceInstanceReturns(ccv2.ServiceInstance{}, ccv2.Warnings{"service-instance-warnings-1"}, expectedErr)
			})

			It("returns errors and warnings", func() {
				Expect(executeErr).To(MatchError(expectedErr))
				Expect(warnings).To(ConsistOf("service-instance-warnings-1"))
			})
		})
	})

	Describe("GetServiceInstanceByNameAndSpace", func() {
		When("the service instance exists", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.GetSpaceServiceInstancesReturns(
					[]ccv2.ServiceInstance{
						{
							GUID: "some-service-instance-guid",
							Name: "some-service-instance",
						},
					},
					ccv2.Warnings{"foo"},
					nil,
				)
			})

			It("returns the service instance and warnings", func() {
				serviceInstance, warnings, err := actor.GetServiceInstanceByNameAndSpace("some-service-instance", "some-space-guid")
				Expect(err).ToNot(HaveOccurred())
				Expect(serviceInstance).To(Equal(ServiceInstance{
					GUID: "some-service-instance-guid",
					Name: "some-service-instance",
				}))
				Expect(warnings).To(ConsistOf("foo"))

				Expect(fakeCloudControllerClient.GetSpaceServiceInstancesCallCount()).To(Equal(1))

				spaceGUID, includeUserProvidedServices, queries := fakeCloudControllerClient.GetSpaceServiceInstancesArgsForCall(0)
				Expect(spaceGUID).To(Equal("some-space-guid"))
				Expect(includeUserProvidedServices).To(BeTrue())
				Expect(queries).To(ConsistOf([]ccv2.Filter{
					ccv2.Filter{
						Type:     constant.NameFilter,
						Operator: constant.EqualOperator,
						Values:   []string{"some-service-instance"},
					},
				}))
			})
		})

		When("the service instance does not exists", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.GetSpaceServiceInstancesReturns([]ccv2.ServiceInstance{}, nil, nil)
			})

			It("returns a ServiceInstanceNotFoundError", func() {
				_, _, err := actor.GetServiceInstanceByNameAndSpace("some-service-instance", "some-space-guid")
				Expect(err).To(MatchError(actionerror.ServiceInstanceNotFoundError{Name: "some-service-instance"}))
			})
		})

		When("the cloud controller client returns an error", func() {
			var expectedError error

			BeforeEach(func() {
				expectedError = errors.New("I am a CloudControllerClient Error")
				fakeCloudControllerClient.GetSpaceServiceInstancesReturns([]ccv2.ServiceInstance{}, nil, expectedError)
			})

			It("returns the error", func() {
				_, _, err := actor.GetServiceInstanceByNameAndSpace("some-service-instance", "some-space-guid")
				Expect(err).To(MatchError(expectedError))
			})
		})
	})

	Describe("GetServiceInstancesByApplication", func() {
		var (
			appGUID string

			serviceInstances []ServiceInstance
			warnings         Warnings
			executeErr       error
		)

		BeforeEach(func() {
			appGUID = "some-app-guid"
		})

		JustBeforeEach(func() {
			serviceInstances, warnings, executeErr = actor.GetServiceInstancesByApplication(appGUID)
		})

		When("the application has services bound", func() {
			var serviceBindings []ccv2.ServiceBinding

			BeforeEach(func() {
				serviceBindings = []ccv2.ServiceBinding{
					{ServiceInstanceGUID: "service-instance-guid-1"},
					{ServiceInstanceGUID: "service-instance-guid-2"},
					{ServiceInstanceGUID: "service-instance-guid-3"},
				}

				fakeCloudControllerClient.GetServiceBindingsReturns(serviceBindings, ccv2.Warnings{"service-bindings-warnings-1", "service-bindings-warnings-2"}, nil)
			})

			When("retrieving the service instances is successful", func() {
				BeforeEach(func() {
					fakeCloudControllerClient.GetServiceInstanceReturnsOnCall(0, ccv2.ServiceInstance{Name: "some-service-instance-1"}, ccv2.Warnings{"service-instance-warnings-1"}, nil)
					fakeCloudControllerClient.GetServiceInstanceReturnsOnCall(1, ccv2.ServiceInstance{Name: "some-service-instance-2"}, ccv2.Warnings{"service-instance-warnings-2"}, nil)
					fakeCloudControllerClient.GetServiceInstanceReturnsOnCall(2, ccv2.ServiceInstance{Name: "some-service-instance-3"}, ccv2.Warnings{"service-instance-warnings-3"}, nil)
				})

				It("returns the service instances and warnings", func() {
					Expect(executeErr).ToNot(HaveOccurred())
					Expect(warnings).To(ConsistOf("service-bindings-warnings-1", "service-bindings-warnings-2", "service-instance-warnings-1", "service-instance-warnings-2", "service-instance-warnings-3"))
					Expect(serviceInstances).To(ConsistOf(
						ServiceInstance{Name: "some-service-instance-1"},
						ServiceInstance{Name: "some-service-instance-2"},
						ServiceInstance{Name: "some-service-instance-3"},
					))

					Expect(fakeCloudControllerClient.GetServiceInstanceCallCount()).To(Equal(3))
					Expect(fakeCloudControllerClient.GetServiceInstanceArgsForCall(0)).To(Equal("service-instance-guid-1"))
					Expect(fakeCloudControllerClient.GetServiceInstanceArgsForCall(1)).To(Equal("service-instance-guid-2"))
					Expect(fakeCloudControllerClient.GetServiceInstanceArgsForCall(2)).To(Equal("service-instance-guid-3"))
				})
			})

			When("retrieving the service instances returns an error", func() {
				var expectedErr error

				BeforeEach(func() {
					expectedErr = errors.New("this is indeed an error, kudos!")
					fakeCloudControllerClient.GetServiceInstanceReturns(ccv2.ServiceInstance{}, ccv2.Warnings{"service-instance-warnings-1", "service-instance-warnings-2"}, expectedErr)
				})

				It("returns errors and warnings", func() {
					Expect(executeErr).To(MatchError(expectedErr))
					Expect(warnings).To(ConsistOf("service-bindings-warnings-1", "service-bindings-warnings-2", "service-instance-warnings-1", "service-instance-warnings-2"))
				})
			})
		})

		When("the application has no services bound", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.GetServiceBindingsReturns(nil, ccv2.Warnings{"service-bindings-warnings-1", "service-bindings-warnings-2"}, nil)
			})

			It("returns an empty list and warnings", func() {
				Expect(executeErr).ToNot(HaveOccurred())
				Expect(warnings).To(ConsistOf("service-bindings-warnings-1", "service-bindings-warnings-2"))
				Expect(serviceInstances).To(BeEmpty())
			})
		})

		When("retrieving the application's bound services returns an error", func() {
			var expectedErr error

			BeforeEach(func() {
				expectedErr = errors.New("this is indeed an error, kudos!")
				fakeCloudControllerClient.GetServiceBindingsReturns(nil, ccv2.Warnings{"service-bindings-warnings-1", "service-bindings-warnings-2"}, expectedErr)
			})

			It("returns errors and warnings", func() {
				Expect(executeErr).To(MatchError(expectedErr))
				Expect(warnings).To(ConsistOf("service-bindings-warnings-1", "service-bindings-warnings-2"))
			})
		})
	})

	Describe("GetServiceInstancesBySpace", func() {
		When("there are service instances", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.GetSpaceServiceInstancesReturns(
					[]ccv2.ServiceInstance{
						{
							GUID: "some-service-instance-guid-1",
							Name: "some-service-instance-1",
						},
						{
							GUID: "some-service-instance-guid-2",
							Name: "some-service-instance-2",
						},
					},
					ccv2.Warnings{"warning-1", "warning-2"},
					nil,
				)
			})

			It("returns the service instances and warnings", func() {
				serviceInstances, warnings, err := actor.GetServiceInstancesBySpace("some-space-guid")
				Expect(err).ToNot(HaveOccurred())
				Expect(serviceInstances).To(ConsistOf(
					ServiceInstance{
						GUID: "some-service-instance-guid-1",
						Name: "some-service-instance-1",
					},
					ServiceInstance{
						GUID: "some-service-instance-guid-2",
						Name: "some-service-instance-2",
					},
				))
				Expect(warnings).To(ConsistOf("warning-1", "warning-2"))

				Expect(fakeCloudControllerClient.GetSpaceServiceInstancesCallCount()).To(Equal(1))

				spaceGUID, includeUserProvidedServices, queries := fakeCloudControllerClient.GetSpaceServiceInstancesArgsForCall(0)
				Expect(spaceGUID).To(Equal("some-space-guid"))
				Expect(includeUserProvidedServices).To(BeTrue())
				Expect(queries).To(BeNil())
			})
		})

		When("the cloud controller client returns an error", func() {
			var expectedError error

			BeforeEach(func() {
				expectedError = errors.New("I am a CloudControllerClient Error")
				fakeCloudControllerClient.GetSpaceServiceInstancesReturns(
					[]ccv2.ServiceInstance{},
					ccv2.Warnings{"warning-1", "warning-2"},
					expectedError)
			})

			It("returns the error and warnings", func() {
				_, warnings, err := actor.GetServiceInstancesBySpace("some-space-guid")
				Expect(err).To(MatchError(expectedError))
				Expect(warnings).To(ConsistOf("warning-1", "warning-2"))
			})
		})
	})

	Describe("UpdateServiceInstanceMaintenanceInfo", func() {
		const serviceInstanceGUID = "service-instance-guid"
		var maintenanceInfo MaintenanceInfo

		BeforeEach(func() {
			maintenanceInfo = MaintenanceInfo{
				Version: "1.2.3",
			}
		})

		When("the update is successful", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.UpdateServiceInstanceMaintenanceInfoReturns(
					ccv2.Warnings{"warning-1", "warning-2"},
					nil,
				)
			})

			It("returns all the warnings", func() {
				warnings, err := actor.UpdateServiceInstanceMaintenanceInfo(serviceInstanceGUID, maintenanceInfo)
				Expect(err).NotTo(HaveOccurred())

				Expect(warnings).To(ConsistOf("warning-1", "warning-2"))
				Expect(fakeCloudControllerClient.UpdateServiceInstanceMaintenanceInfoCallCount()).To(Equal(1))
				guid, minfo := fakeCloudControllerClient.UpdateServiceInstanceMaintenanceInfoArgsForCall(0)
				Expect(guid).To(Equal(serviceInstanceGUID))
				Expect(minfo).To(Equal(ccv2.MaintenanceInfo(maintenanceInfo)))
			})
		})

		When("the update fails", func() {
			BeforeEach(func() {
				fakeCloudControllerClient.UpdateServiceInstanceMaintenanceInfoReturns(
					ccv2.Warnings{"warning-1", "warning-2"},
					errors.New("update failed horribly!!!"),
				)
			})

			It("returns the error and all the warnings", func() {
				warnings, err := actor.UpdateServiceInstanceMaintenanceInfo(serviceInstanceGUID, maintenanceInfo)
				Expect(err).To(MatchError("update failed horribly!!!"))
				Expect(warnings).To(ConsistOf("warning-1", "warning-2"))
			})
		})
	})
})