Browse Source

Handle invalid VCAP config

pull/336/head
baarkerlounger 3 years ago
parent
commit
2298ea6e12
  1. 1
      app/services/paas_configuration_service.rb
  2. 16
      spec/services/paas_configuration_service_spec.rb

1
app/services/paas_configuration_service.rb

@ -27,6 +27,7 @@ private
JSON.parse(ENV["VCAP_SERVICES"], { symbolize_names: true })
rescue StandardError
@logger.warn("Could not parse VCAP_SERVICES!")
return {}
end
end

16
spec/services/paas_configuration_service_spec.rb

@ -22,6 +22,22 @@ RSpec.describe PaasConfigurationService do
end
end
context "when configuration is present but invalid" do
let(:vcap_services) do
{"aws-s3-bucket": [{instance_name: "bucket_1"}, {instance_name: "bucket_2"}]}
end
before do
allow(ENV).to receive(:[]).with("VCAP_SERVICES").and_return(vcap_services)
allow(logger).to receive(:warn)
end
it "logs an error" do
expect(logger).to receive(:warn).with("Could not parse VCAP_SERVICES!")
config_service.s3_config_present?
end
end
context "when the paas configuration is present with S3 buckets" do
let(:vcap_services) do
<<-JSON

Loading…
Cancel
Save