Skip to content

Commit

Permalink
Merge pull request #1016 from cloudfoundry/azure-polishing
Browse files Browse the repository at this point in the history
Polishing for Azure App Insights testing & Rubocop
  • Loading branch information
pivotal-david-osullivan authored May 26, 2023
2 parents 8229200 + fd68ad3 commit b890725
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@ def compile

# (see JavaBuildpack::Component::BaseComponent#release)
def release
credentials = @application.services.find_service(FILTER, [CONNECTION_STRING, INSTRUMENTATION_KEY])['credentials']
credentials = @application.services.find_service(FILTER, [CONNECTION_STRING,
INSTRUMENTATION_KEY])['credentials']

if credentials.key?(CONNECTION_STRING)
@droplet.java_opts.add_system_property('applicationinsights.connection.string',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,7 @@

before do
allow(services).to receive(:one_service?).with(/azure-application-insights/,
'connection_string',
'instrumentation_key')
%w[connection_string instrumentation_key])
.and_return(true)
end

Expand Down

0 comments on commit b890725

Please sign in to comment.