summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatija Čupić <matteeyah@gmail.com>2017-12-04 01:59:29 +0100
committerMatija Čupić <matteeyah@gmail.com>2017-12-04 01:59:29 +0100
commit04c6d102616b48c95b09656efc720c7dfdc99d8a (patch)
treef1d00c50ada38750cdeee719ca22697406d7b240
parent50c8bd6350ebfd8d35deb2a1b41eb36e193d1a14 (diff)
downloadgitlab-ce-04c6d102616b48c95b09656efc720c7dfdc99d8a.tar.gz
Use RequestOptions in GCP Client user_agent_header
-rw-r--r--lib/google_api/cloud_platform/client.rb4
-rw-r--r--spec/lib/google_api/cloud_platform/client_spec.rb9
2 files changed, 10 insertions, 3 deletions
diff --git a/lib/google_api/cloud_platform/client.rb b/lib/google_api/cloud_platform/client.rb
index 615cd7dc60a..15401057903 100644
--- a/lib/google_api/cloud_platform/client.rb
+++ b/lib/google_api/cloud_platform/client.rb
@@ -84,7 +84,9 @@ module GoogleApi
end
def user_agent_header
- { 'User-Agent': "GitLab/#{Gitlab::VERSION.match('(\d+\.\d+)').captures.first} (GPN:GitLab;)" }
+ options = Google::Apis::RequestOptions.new
+ options.header = { 'User-Agent': "GitLab/#{Gitlab::VERSION.match('(\d+\.\d+)').captures.first} (GPN:GitLab;)" }
+ options
end
end
end
diff --git a/spec/lib/google_api/cloud_platform/client_spec.rb b/spec/lib/google_api/cloud_platform/client_spec.rb
index 41964034062..ee8f7afc285 100644
--- a/spec/lib/google_api/cloud_platform/client_spec.rb
+++ b/spec/lib/google_api/cloud_platform/client_spec.rb
@@ -129,9 +129,14 @@ describe GoogleApi::CloudPlatform::Client do
describe '#user_agent_header' do
subject { client.instance_eval { user_agent_header } }
- it 'returns the correct major and minor GitLab version ' do
+ it 'returns a RequestOptions object' do
+ expect(subject).to be_instance_of(Google::Apis::RequestOptions)
+ end
+
+ it 'has the correct GitLab version in User-Agent header' do
stub_const('Gitlab::VERSION', '10.3.0-pre')
- expect(subject).to eq({ 'User-Agent': 'GitLab/10.3 (GPN:GitLab;)' })
+
+ expect(subject.header).to eq({ 'User-Agent': 'GitLab/10.3 (GPN:GitLab;)' })
end
end
end