2019-10-24 20:06:14 -04:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2017-01-23 08:49:13 -05:00
|
|
|
require 'spec_helper'
|
|
|
|
|
2020-06-24 02:09:01 -04:00
|
|
|
RSpec.describe Ci::BuildPolicy do
|
2017-01-23 08:49:13 -05:00
|
|
|
let(:user) { create(:user) }
|
|
|
|
let(:build) { create(:ci_build, pipeline: pipeline) }
|
|
|
|
let(:pipeline) { create(:ci_empty_pipeline, project: project) }
|
|
|
|
|
2017-04-06 17:09:58 -04:00
|
|
|
let(:policy) do
|
|
|
|
described_class.new(user, build)
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
shared_context 'public pipelines disabled' do
|
2017-06-14 14:18:56 -04:00
|
|
|
before do
|
|
|
|
project.update_attribute(:public_builds, false)
|
|
|
|
end
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
describe '#rules' do
|
|
|
|
context 'when user does not have access to the project' do
|
2017-08-02 15:55:11 -04:00
|
|
|
let(:project) { create(:project, :private) }
|
2017-01-23 08:49:13 -05:00
|
|
|
|
|
|
|
context 'when public builds are enabled' do
|
|
|
|
it 'does not include ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).not_to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when public builds are disabled' do
|
|
|
|
include_context 'public pipelines disabled'
|
|
|
|
|
|
|
|
it 'does not include ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).not_to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when anonymous user has access to the project' do
|
2017-08-02 15:55:11 -04:00
|
|
|
let(:project) { create(:project, :public) }
|
2017-01-23 08:49:13 -05:00
|
|
|
|
|
|
|
context 'when public builds are enabled' do
|
|
|
|
it 'includes ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when public builds are disabled' do
|
|
|
|
include_context 'public pipelines disabled'
|
|
|
|
|
|
|
|
it 'does not include ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).not_to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when team member has access to the project' do
|
2017-08-02 15:55:11 -04:00
|
|
|
let(:project) { create(:project, :public) }
|
2017-01-23 08:49:13 -05:00
|
|
|
|
|
|
|
context 'team member is a guest' do
|
2017-06-14 14:18:56 -04:00
|
|
|
before do
|
2017-12-22 03:18:28 -05:00
|
|
|
project.add_guest(user)
|
2017-06-14 14:18:56 -04:00
|
|
|
end
|
2017-01-23 08:49:13 -05:00
|
|
|
|
|
|
|
context 'when public builds are enabled' do
|
|
|
|
it 'includes ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when public builds are disabled' do
|
|
|
|
include_context 'public pipelines disabled'
|
|
|
|
|
|
|
|
it 'does not include ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).not_to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'team member is a reporter' do
|
2017-06-14 14:18:56 -04:00
|
|
|
before do
|
2017-12-22 03:18:28 -05:00
|
|
|
project.add_reporter(user)
|
2017-06-14 14:18:56 -04:00
|
|
|
end
|
2017-01-23 08:49:13 -05:00
|
|
|
|
|
|
|
context 'when public builds are enabled' do
|
|
|
|
it 'includes ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when public builds are disabled' do
|
|
|
|
include_context 'public pipelines disabled'
|
|
|
|
|
|
|
|
it 'does not include ability to read build' do
|
2017-04-06 17:09:58 -04:00
|
|
|
expect(policy).to be_allowed :read_build
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2018-05-15 04:18:22 -04:00
|
|
|
|
|
|
|
context 'when maintainer is allowed to push to pipeline branch' do
|
|
|
|
let(:project) { create(:project, :public) }
|
|
|
|
let(:owner) { user }
|
|
|
|
|
|
|
|
it 'enables update_build if user is maintainer' do
|
|
|
|
allow_any_instance_of(Project).to receive(:empty_repo?).and_return(false)
|
2018-05-22 21:54:57 -04:00
|
|
|
allow_any_instance_of(Project).to receive(:branch_allows_collaboration?).and_return(true)
|
2018-05-15 04:18:22 -04:00
|
|
|
|
|
|
|
expect(policy).to be_allowed :update_build
|
|
|
|
expect(policy).to be_allowed :update_commit_status
|
|
|
|
end
|
|
|
|
end
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
2017-04-12 05:26:18 -04:00
|
|
|
|
2017-07-18 10:04:22 -04:00
|
|
|
describe 'rules for protected ref' do
|
2017-08-01 14:51:52 -04:00
|
|
|
let(:project) { create(:project, :repository) }
|
2017-07-18 09:08:48 -04:00
|
|
|
let(:build) { create(:ci_build, ref: 'some-ref', pipeline: pipeline) }
|
2017-04-12 05:26:18 -04:00
|
|
|
|
|
|
|
before do
|
|
|
|
project.add_developer(user)
|
2017-07-03 17:01:05 -04:00
|
|
|
end
|
2017-04-12 05:26:18 -04:00
|
|
|
|
2017-07-03 17:01:05 -04:00
|
|
|
context 'when no one can push or merge to the branch' do
|
2017-07-18 09:08:48 -04:00
|
|
|
before do
|
|
|
|
create(:protected_branch, :no_one_can_push,
|
2017-07-18 10:04:22 -04:00
|
|
|
name: build.ref, project: project)
|
2017-07-18 09:08:48 -04:00
|
|
|
end
|
2017-04-12 05:26:18 -04:00
|
|
|
|
2017-07-03 17:01:05 -04:00
|
|
|
it 'does not include ability to update build' do
|
2017-07-03 17:20:44 -04:00
|
|
|
expect(policy).to be_disallowed :update_build
|
2017-04-12 05:26:18 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-07-03 17:01:05 -04:00
|
|
|
context 'when developers can push to the branch' do
|
2017-07-18 09:08:48 -04:00
|
|
|
before do
|
|
|
|
create(:protected_branch, :developers_can_merge,
|
2017-07-18 10:04:22 -04:00
|
|
|
name: build.ref, project: project)
|
2017-07-18 09:08:48 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'includes ability to update build' do
|
|
|
|
expect(policy).to be_allowed :update_build
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when no one can create the tag' do
|
|
|
|
before do
|
|
|
|
create(:protected_tag, :no_one_can_create,
|
2017-07-18 10:04:22 -04:00
|
|
|
name: build.ref, project: project)
|
2017-07-18 09:08:48 -04:00
|
|
|
|
2020-07-29 08:09:45 -04:00
|
|
|
build.update!(tag: true)
|
2017-07-18 09:08:48 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not include ability to update build' do
|
|
|
|
expect(policy).to be_disallowed :update_build
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when no one can create the tag but it is not a tag' do
|
|
|
|
before do
|
|
|
|
create(:protected_tag, :no_one_can_create,
|
2017-07-18 10:04:22 -04:00
|
|
|
name: build.ref, project: project)
|
2017-07-18 09:08:48 -04:00
|
|
|
end
|
2017-04-12 05:26:18 -04:00
|
|
|
|
2017-07-03 17:01:05 -04:00
|
|
|
it 'includes ability to update build' do
|
2017-07-03 17:20:44 -04:00
|
|
|
expect(policy).to be_allowed :update_build
|
2017-04-12 05:26:18 -04:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-11-06 08:20:44 -05:00
|
|
|
|
|
|
|
describe 'rules for erase build' do
|
2017-11-07 08:45:55 -05:00
|
|
|
let(:project) { create(:project, :repository) }
|
2017-11-06 12:47:05 -05:00
|
|
|
let(:build) { create(:ci_build, pipeline: pipeline, ref: 'some-ref', user: owner) }
|
2017-11-06 08:20:44 -05:00
|
|
|
|
2017-11-06 12:47:05 -05:00
|
|
|
context 'when a developer erases a build' do
|
2017-11-06 08:20:44 -05:00
|
|
|
before do
|
|
|
|
project.add_developer(user)
|
|
|
|
end
|
|
|
|
|
2017-11-06 12:47:05 -05:00
|
|
|
context 'when developers can push to the branch' do
|
|
|
|
context 'when the build was created by the developer' do
|
|
|
|
let(:owner) { user }
|
|
|
|
|
2020-05-12 11:10:33 -04:00
|
|
|
context 'when the build was created for a protected ref' do
|
|
|
|
before do
|
|
|
|
create(:protected_branch, :developers_can_push,
|
|
|
|
name: build.ref, project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { expect(policy).to be_disallowed :erase_build }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the build was created for an unprotected ref' do
|
|
|
|
it { expect(policy).to be_allowed :erase_build }
|
|
|
|
end
|
2017-11-06 12:47:05 -05:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the build was created by the other' do
|
|
|
|
let(:owner) { create(:user) }
|
|
|
|
|
|
|
|
it { expect(policy).to be_disallowed :erase_build }
|
|
|
|
end
|
2017-11-06 08:20:44 -05:00
|
|
|
end
|
|
|
|
|
2017-11-06 12:47:05 -05:00
|
|
|
context 'when no one can push or merge to the branch' do
|
|
|
|
let(:owner) { user }
|
|
|
|
|
|
|
|
before do
|
2017-11-07 08:45:55 -05:00
|
|
|
create(:protected_branch, :no_one_can_push, :no_one_can_merge,
|
2017-11-06 12:47:05 -05:00
|
|
|
name: build.ref, project: project)
|
|
|
|
end
|
2017-11-06 08:20:44 -05:00
|
|
|
|
|
|
|
it { expect(policy).to be_disallowed :erase_build }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2018-07-11 10:36:08 -04:00
|
|
|
context 'when a maintainer erases a build' do
|
2017-11-06 08:20:44 -05:00
|
|
|
before do
|
2018-07-11 10:36:08 -04:00
|
|
|
project.add_maintainer(user)
|
2017-11-06 08:20:44 -05:00
|
|
|
end
|
|
|
|
|
2018-07-11 10:36:08 -04:00
|
|
|
context 'when maintainers can push to the branch' do
|
2017-11-07 08:45:55 -05:00
|
|
|
before do
|
2018-07-11 10:36:08 -04:00
|
|
|
create(:protected_branch, :maintainers_can_push,
|
2017-11-07 08:45:55 -05:00
|
|
|
name: build.ref, project: project)
|
|
|
|
end
|
|
|
|
|
2018-07-11 10:36:08 -04:00
|
|
|
context 'when the build was created by the maintainer' do
|
2017-11-07 08:45:55 -05:00
|
|
|
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) }
|
2017-11-06 08:20:44 -05:00
|
|
|
|
2017-11-07 08:45:55 -05:00
|
|
|
it { expect(policy).to be_allowed :erase_build }
|
|
|
|
end
|
2017-11-06 08:20:44 -05:00
|
|
|
end
|
|
|
|
|
2017-11-07 08:45:55 -05:00
|
|
|
context 'when no one can push or merge to the branch' do
|
|
|
|
let(:owner) { user }
|
2017-11-06 08:20:44 -05:00
|
|
|
|
2017-11-07 08:45:55 -05:00
|
|
|
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 }
|
2017-11-06 08:20:44 -05:00
|
|
|
end
|
|
|
|
end
|
2020-07-30 14:09:39 -04:00
|
|
|
|
|
|
|
context 'when an admin erases a build', :enable_admin_mode do
|
|
|
|
let(:owner) { create(:user) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
user.update!(admin: true)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the build was created for a protected branch' do
|
|
|
|
before do
|
|
|
|
create(:protected_branch, :developers_can_push,
|
|
|
|
name: build.ref, project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { expect(policy).to be_allowed :erase_build }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the build was created for a protected tag' do
|
|
|
|
before do
|
|
|
|
create(:protected_tag, :developers_can_create,
|
|
|
|
name: build.ref, project: project)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { expect(policy).to be_allowed :erase_build }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the build was created for an unprotected ref' do
|
|
|
|
it { expect(policy).to be_allowed :erase_build }
|
|
|
|
end
|
|
|
|
end
|
2017-11-06 08:20:44 -05:00
|
|
|
end
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|
2020-05-25 11:07:58 -04:00
|
|
|
|
|
|
|
describe 'manage a web ide terminal' do
|
|
|
|
let(:build_permissions) { %i[read_web_ide_terminal create_build_terminal update_web_ide_terminal create_build_service_proxy] }
|
|
|
|
let_it_be(:maintainer) { create(:user) }
|
|
|
|
let(:owner) { create(:owner) }
|
|
|
|
let(:admin) { create(:admin) }
|
|
|
|
let(:maintainer) { create(:user) }
|
|
|
|
let(:developer) { create(:user) }
|
|
|
|
let(:reporter) { create(:user) }
|
|
|
|
let(:guest) { create(:user) }
|
|
|
|
let(:project) { create(:project, :public, namespace: owner.namespace) }
|
|
|
|
let(:pipeline) { create(:ci_empty_pipeline, project: project, source: :webide) }
|
|
|
|
let(:build) { create(:ci_build, pipeline: pipeline) }
|
|
|
|
|
|
|
|
before do
|
|
|
|
allow(build).to receive(:has_terminal?).and_return(true)
|
|
|
|
|
|
|
|
project.add_maintainer(maintainer)
|
|
|
|
project.add_developer(developer)
|
|
|
|
project.add_reporter(reporter)
|
|
|
|
project.add_guest(guest)
|
|
|
|
end
|
|
|
|
|
|
|
|
subject { described_class.new(current_user, build) }
|
|
|
|
|
|
|
|
context 'when create_web_ide_terminal access enabled' do
|
|
|
|
context 'with admin' do
|
|
|
|
let(:current_user) { admin }
|
|
|
|
|
|
|
|
context 'when admin mode enabled', :enable_admin_mode do
|
|
|
|
it { expect_allowed(*build_permissions) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when admin mode disabled' do
|
|
|
|
it { expect_disallowed(*build_permissions) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when build is not from a webide pipeline' do
|
|
|
|
let(:pipeline) { create(:ci_empty_pipeline, project: project, source: :chat) }
|
|
|
|
|
|
|
|
it { expect_disallowed(:read_web_ide_terminal, :update_web_ide_terminal, :create_build_service_proxy) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when build has no runner terminal' do
|
|
|
|
before do
|
|
|
|
allow(build).to receive(:has_terminal?).and_return(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when admin mode enabled', :enable_admin_mode do
|
|
|
|
it { expect_allowed(:read_web_ide_terminal, :update_web_ide_terminal) }
|
|
|
|
it { expect_disallowed(:create_build_terminal, :create_build_service_proxy) }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when admin mode disabled' do
|
|
|
|
it { expect_disallowed(:read_web_ide_terminal, :update_web_ide_terminal) }
|
|
|
|
it { expect_disallowed(:create_build_terminal, :create_build_service_proxy) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'feature flag "build_service_proxy" is disabled' do
|
|
|
|
before do
|
|
|
|
stub_feature_flags(build_service_proxy: false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { expect_disallowed(:create_build_service_proxy) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'allowed build owner access' do
|
|
|
|
it { expect_disallowed(*build_permissions) }
|
|
|
|
|
|
|
|
context 'when user is the owner of the job' do
|
|
|
|
let(:build) { create(:ci_build, pipeline: pipeline, user: current_user) }
|
|
|
|
|
|
|
|
it { expect_allowed(*build_permissions) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
shared_examples 'forbidden access' do
|
|
|
|
it { expect_disallowed(*build_permissions) }
|
|
|
|
|
|
|
|
context 'when user is the owner of the job' do
|
|
|
|
let(:build) { create(:ci_build, pipeline: pipeline, user: current_user) }
|
|
|
|
|
|
|
|
it { expect_disallowed(*build_permissions) }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with owner' do
|
|
|
|
let(:current_user) { owner }
|
|
|
|
|
|
|
|
it_behaves_like 'allowed build owner access'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with maintainer' do
|
|
|
|
let(:current_user) { maintainer }
|
|
|
|
|
|
|
|
it_behaves_like 'allowed build owner access'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with developer' do
|
|
|
|
let(:current_user) { developer }
|
|
|
|
|
|
|
|
it_behaves_like 'forbidden access'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with reporter' do
|
|
|
|
let(:current_user) { reporter }
|
|
|
|
|
|
|
|
it_behaves_like 'forbidden access'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with guest' do
|
|
|
|
let(:current_user) { guest }
|
|
|
|
|
|
|
|
it_behaves_like 'forbidden access'
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with non member' do
|
|
|
|
let(:current_user) { create(:user) }
|
|
|
|
|
|
|
|
it_behaves_like 'forbidden access'
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2017-01-23 08:49:13 -05:00
|
|
|
end
|