diff options
author | Shinya Maeda <shinya@gitlab.com> | 2017-11-07 16:45:55 +0300 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2017-11-07 16:45:55 +0300 |
commit | d8fb903ef7db26566cfacf76ce10e9e7917b8f38 (patch) | |
tree | e5cb79ea279bf4aba78f00ea9e21cb7597fecb84 /spec/policies/ci | |
parent | d89c18901bde510da2668e676b3bf2f1e21deef2 (diff) |
Improve spec
Diffstat (limited to 'spec/policies/ci')
-rw-r--r-- | spec/policies/ci/build_policy_spec.rb | 36 |
1 files changed, 27 insertions, 9 deletions
diff --git a/spec/policies/ci/build_policy_spec.rb b/spec/policies/ci/build_policy_spec.rb index edf8d63a4c6..298a9d16425 100644 --- a/spec/policies/ci/build_policy_spec.rb +++ b/spec/policies/ci/build_policy_spec.rb @@ -152,7 +152,7 @@ describe Ci::BuildPolicy do end describe 'rules for erase build' do - let(:project) { create(:project) } + let(:project) { create(:project, :repository) } let(:build) { create(:ci_build, pipeline: pipeline, ref: 'some-ref', user: owner) } context 'when a developer erases a build' do @@ -162,7 +162,7 @@ describe Ci::BuildPolicy do context 'when developers can push to the branch' do before do - create(:protected_branch, :developers_can_merge, + create(:protected_branch, :developers_can_push, name: build.ref, project: project) end @@ -183,7 +183,7 @@ describe Ci::BuildPolicy do let(:owner) { user } before do - create(:protected_branch, :no_one_can_push, + create(:protected_branch, :no_one_can_push, :no_one_can_merge, name: build.ref, project: project) end @@ -196,16 +196,34 @@ describe Ci::BuildPolicy do project.add_master(user) end - context 'when the build was created by the master' do - let(:owner) { user } + context 'when masters can push to the branch' do + before do + create(:protected_branch, :masters_can_push, + name: build.ref, project: project) + end + + context 'when the build was created by the master' do + let(:owner) { user } + + it { expect(policy).to be_allowed :erase_build } + end + + context 'when the build was created by the other' do + let(:owner) { create(:user) } - it { expect(policy).to be_allowed :erase_build } + it { expect(policy).to be_allowed :erase_build } + end end - context 'when the build was created by the other' do - let(:owner) { create(:user) } + context 'when no one can push or merge to the branch' do + let(:owner) { user } - it { expect(policy).to be_allowed :erase_build } + before do + create(:protected_branch, :no_one_can_push, :no_one_can_merge, + name: build.ref, project: project) + end + + it { expect(policy).to be_disallowed :erase_build } end end end |