Skip to content
Snippets Groups Projects
Verified Commit 3dcc9c1b authored by Matija Čupić's avatar Matija Čupić
Browse files

Gate MR head_pipeline behind read_pipeline ability

parent 3c240b7a
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -731,7 +731,9 @@ module API
merge_request.metrics&.pipeline
end
 
expose :head_pipeline, using: 'API::Entities::Pipeline'
expose :head_pipeline, using: 'API::Entities::Pipeline', if: -> (_, options) do
Ability.allowed?(options[:current_user], :read_pipeline, options[:project])
end
 
expose :diff_refs, using: Entities::DiffRefs
 
Loading
Loading
Loading
Loading
@@ -830,6 +830,31 @@ describe API::MergeRequests do
end
end
 
context 'head_pipeline' do
before do
merge_request.update(head_pipeline: create(:ci_pipeline))
merge_request.project.project_feature.update(builds_access_level: 10)
end
context 'when user can read the pipeline' do
it 'exposes pipeline information' do
get api("/projects/#{project.id}/merge_requests/#{merge_request.iid}", user)
expect(json_response).to include('head_pipeline')
end
end
context 'when user can not read the pipeline' do
let(:guest) { create(:user) }
it 'does not expose pipeline information' do
get api("/projects/#{project.id}/merge_requests/#{merge_request.iid}", guest)
expect(json_response).not_to include('head_pipeline')
end
end
end
it 'returns the commits behind the target branch when include_diverged_commits_count is present' do
allow_any_instance_of(merge_request.class).to receive(:diverged_commits_count).and_return(1)
 
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment