Merge branch 'bugfix/deploy_keys__can_push__usage_in_api' into 'master'

Add missing `can_push` parameter to POST /v3/deploy_keys

See merge request !11607
This commit is contained in:
Rémy Coutable 2017-06-01 13:11:11 +00:00
commit f07aee72be
3 changed files with 14 additions and 0 deletions

View file

@ -0,0 +1,4 @@
---
title: "Fixed handling of the `can_push` attribute in the v3 deploy_keys api"
merge_request: 11607
author: Richard Clamp

View file

@ -41,6 +41,7 @@ module API
params do
requires :key, type: String, desc: 'The new deploy key'
requires :title, type: String, desc: 'The name of the deploy key'
optional :can_push, type: Boolean, desc: "Can deploy key push to the project's repository"
end
post ":id/#{path}" do
params[:key].strip!

View file

@ -105,6 +105,15 @@ describe API::V3::DeployKeys do
expect(response).to have_http_status(201)
end
it 'accepts can_push parameter' do
key_attrs = attributes_for :write_access_key
post v3_api("/projects/#{project.id}/#{path}", admin), key_attrs
expect(response).to have_http_status(201)
expect(json_response['can_push']).to eq(true)
end
end
describe "DELETE /projects/:id/#{path}/:key_id" do