diff options
author | Douwe Maan <douwe@gitlab.com> | 2017-06-27 17:30:43 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2017-06-27 17:30:43 +0000 |
commit | 585e6aa5b20b31b1de01f7c5aa9aea67d7a377f4 (patch) | |
tree | 27359d8d3d04cfb363c3ed1c27722649765ebd63 /spec/features | |
parent | 471a40ace1952d43266b8814c08af72f40874e43 (diff) | |
parent | a274359ecd485218ed7c42b750a43992ff3142ef (diff) | |
download | gitlab-ce-585e6aa5b20b31b1de01f7c5aa9aea67d7a377f4.tar.gz |
Merge branch 'bugfix/ruby2.4-compat' into 'master'
ruby 2.4 compat: update to webmock 2.3.2
See merge request !12471
Diffstat (limited to 'spec/features')
-rw-r--r-- | spec/features/projects/services/jira_service_spec.rb | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/spec/features/projects/services/jira_service_spec.rb b/spec/features/projects/services/jira_service_spec.rb index 2ea50e8f672..8cd216c8fdb 100644 --- a/spec/features/projects/services/jira_service_spec.rb +++ b/spec/features/projects/services/jira_service_spec.rb @@ -6,7 +6,11 @@ feature 'Setup Jira service', :feature, :js do let(:service) { project.create_jira_service } let(:url) { 'http://jira.example.com' } - let(:project_url) { 'http://username:password@jira.example.com/rest/api/2/project/GitLabProject' } + + def stub_project_url + WebMock.stub_request(:get, 'http://jira.example.com/rest/api/2/project/GitLabProject') + .with(basic_auth: %w(username password)) + end def fill_form(active = true) check 'Active' if active @@ -28,7 +32,7 @@ feature 'Setup Jira service', :feature, :js do describe 'user sets and activates Jira Service' do context 'when Jira connection test succeeds' do before do - WebMock.stub_request(:get, project_url) + stub_project_url end it 'activates the JIRA service' do @@ -44,7 +48,7 @@ feature 'Setup Jira service', :feature, :js do context 'when Jira connection test fails' do before do - WebMock.stub_request(:get, project_url).to_return(status: 401) + stub_project_url.to_return(status: 401) end it 'shows errors when some required fields are not filled in' do |