Skip to content

Commit f55309c

Browse files
authored
Deployment Updater: Print error class name in case of unexpected error (#4647)
* without we have really little information what went wrong
1 parent 003f49f commit f55309c

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

lib/cloud_controller/deployment_updater/updater.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def with_error_logging(error_message)
5959
if APPROVED_ERRORS.include?(e.class)
6060
deployment.update(error: e.message)
6161
else
62-
deployment.update(error: 'An unexpected error has occurred.')
62+
deployment.update(error: "An unexpected error has occurred: #{error_name}")
6363
end
6464
rescue StandardError => new_error
6565
logger.error(

spec/unit/lib/cloud_controller/deployment_updater/updater_spec.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ module VCAP::CloudController
184184
subject.scale
185185
end.not_to raise_error
186186

187-
expect(deployment.error).to eq 'An unexpected error has occurred.'
187+
expect(deployment.error).to eq 'An unexpected error has occurred: StandardError'
188188
end
189189
end
190190
end
@@ -396,7 +396,7 @@ module VCAP::CloudController
396396
subject.canary
397397
end.not_to raise_error
398398

399-
expect(deployment.error).to eq 'An unexpected error has occurred.'
399+
expect(deployment.error).to eq 'An unexpected error has occurred: StandardError'
400400
end
401401
end
402402
end
@@ -429,7 +429,7 @@ module VCAP::CloudController
429429
subject.cancel
430430
end.not_to raise_error
431431

432-
expect(deployment.error).to eq 'An unexpected error has occurred.'
432+
expect(deployment.error).to eq 'An unexpected error has occurred: StandardError'
433433
end
434434
end
435435
end
@@ -460,7 +460,7 @@ module VCAP::CloudController
460460

461461
it 'provides a helpful error on the deployment model' do
462462
subject.scale
463-
expect(deployment.error).to eq 'An unexpected error has occurred.'
463+
expect(deployment.error).to eq 'An unexpected error has occurred: StandardError'
464464
end
465465
end
466466

0 commit comments

Comments
 (0)