diff options
author | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-05-31 11:15:25 +0200 |
---|---|---|
committer | Grzegorz Bizon <grzesiek.bizon@gmail.com> | 2017-05-31 11:15:25 +0200 |
commit | c970702517558ee6c1ba454572a20ee57fadf285 (patch) | |
tree | 1fa713f21dbf1e7327e1f3a1c775bbc9a914b16c /spec | |
parent | 13f10a96e345535cc8946bf73f85de0063d80739 (diff) | |
download | gitlab-ce-c970702517558ee6c1ba454572a20ee57fadf285.tar.gz |
Fix job play service specs related to protected actions
Diffstat (limited to 'spec')
-rw-r--r-- | spec/services/ci/play_build_service_spec.rb | 17 |
1 files changed, 13 insertions, 4 deletions
diff --git a/spec/services/ci/play_build_service_spec.rb b/spec/services/ci/play_build_service_spec.rb index d6f9fa42045..ea211de1f82 100644 --- a/spec/services/ci/play_build_service_spec.rb +++ b/spec/services/ci/play_build_service_spec.rb @@ -13,8 +13,11 @@ describe Ci::PlayBuildService, '#execute', :services do context 'when project does not have repository yet' do let(:project) { create(:empty_project) } - it 'allows user with master role to play build' do - project.add_master(user) + it 'allows user to play build if protected branch rules are met' do + project.add_developer(user) + + create(:protected_branch, :developers_can_merge, + name: build.ref, project: project) service.execute(build) @@ -45,7 +48,10 @@ describe Ci::PlayBuildService, '#execute', :services do let(:build) { create(:ci_build, :manual, pipeline: pipeline) } before do - project.add_master(user) + project.add_developer(user) + + create(:protected_branch, :developers_can_merge, + name: build.ref, project: project) end it 'enqueues the build' do @@ -64,7 +70,10 @@ describe Ci::PlayBuildService, '#execute', :services do let(:build) { create(:ci_build, when: :manual, pipeline: pipeline) } before do - project.add_master(user) + project.add_developer(user) + + create(:protected_branch, :developers_can_merge, + name: build.ref, project: project) end it 'duplicates the build' do |