Skip to content

Commit e0e5de7

Browse files
committed
refac: ApiError is thrown in failed status
1 parent 3b284a1 commit e0e5de7

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

app/actions/v3/service_instance_create_managed.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,6 @@ def provision(instance, parameters: {}, accepts_incomplete: false)
7373
end
7474
rescue StandardError => e
7575
save_failed_state(instance, e)
76-
7776
raise e
7877
end
7978

@@ -100,7 +99,7 @@ def poll(instance)
10099
raise LastOperationFailedState.new(last_operation_result[:last_operation][:description])
101100
end
102101
rescue LastOperationFailedState => e
103-
raise e
102+
raise CloudController::Errors::ApiError.new_from_details('UnableToPerform', 'create managed', e.message)
104103
rescue StandardError => e
105104
save_failed_state(instance, e)
106105
raise e

spec/unit/actions/v3/service_instance_create_managed_spec.rb

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -681,8 +681,7 @@ module V3
681681
it 'raises and updates the last operation description' do
682682
expect do
683683
action.poll(service_instance)
684-
end.to raise_error(VCAP::CloudController::V3::ServiceInstanceCreateManaged::LastOperationFailedState)
685-
684+
end.to raise_error(CloudController::Errors::ApiError)
686685
expect(ServiceInstance.first.last_operation.type).to eq('create')
687686
expect(ServiceInstance.first.last_operation.state).to eq('failed')
688687
expect(ServiceInstance.first.last_operation.broker_provided_operation).to eq(operation_id)

0 commit comments

Comments
 (0)