SSH Keys API implemented
This commit is contained in:
parent
caef9ed112
commit
c23eb40829
4 changed files with 106 additions and 0 deletions
|
@ -17,5 +17,6 @@ module Gitlab
|
|||
mount Projects
|
||||
mount Issues
|
||||
mount Milestones
|
||||
mount Keys
|
||||
end
|
||||
end
|
||||
|
|
|
@ -48,5 +48,11 @@ module Gitlab
|
|||
expose :assignee, :author, using: Entities::UserBasic
|
||||
expose :closed, :updated_at, :created_at
|
||||
end
|
||||
|
||||
class Key < Grape::Entity
|
||||
expose :id,
|
||||
:title,
|
||||
:key
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
44
lib/api/keys.rb
Normal file
44
lib/api/keys.rb
Normal file
|
@ -0,0 +1,44 @@
|
|||
module Gitlab
|
||||
# Keys API
|
||||
class Keys < Grape::API
|
||||
before { authenticate! }
|
||||
resource :keys do
|
||||
# Get currently authenticated user's keys
|
||||
#
|
||||
# Example Request:
|
||||
# GET /keys
|
||||
get do
|
||||
present current_user.keys, with: Entities::Key
|
||||
end
|
||||
# Add new ssh key to currently authenticated user
|
||||
#
|
||||
# Parameters:
|
||||
# key (required) - New SSH Key
|
||||
# title (required) - New SSH Key's title
|
||||
# Example Request:
|
||||
# POST /keys
|
||||
post do
|
||||
key = current_user.keys.new(
|
||||
title: params[:title],
|
||||
key: params[:key]
|
||||
)
|
||||
if key.save
|
||||
present key, with: Entities::Key
|
||||
else
|
||||
not_found!
|
||||
end
|
||||
end
|
||||
# Delete existed ssh key of currently authenticated user
|
||||
#
|
||||
# Parameters:
|
||||
# id (required) - SSH Key ID
|
||||
# Example Request:
|
||||
# DELETE /keys/:id
|
||||
delete "/:id" do
|
||||
key = current_user.keys.find params[:id]
|
||||
key.delete
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
55
spec/requests/api/ssh_keys_spec.rb
Normal file
55
spec/requests/api/ssh_keys_spec.rb
Normal file
|
@ -0,0 +1,55 @@
|
|||
require 'spec_helper'
|
||||
|
||||
describe Gitlab::Keys do
|
||||
include ApiHelpers
|
||||
let(:user) {
|
||||
user = Factory.create :user
|
||||
user.reset_authentication_token!
|
||||
user
|
||||
}
|
||||
let(:key) { Factory.create :key, { user: user}}
|
||||
|
||||
describe "GET /keys" do
|
||||
context "when unauthenticated" do
|
||||
it "should return authentication error" do
|
||||
get api("/keys")
|
||||
response.status.should == 401
|
||||
end
|
||||
end
|
||||
context "when authenticated" do
|
||||
it "should return array of ssh keys" do
|
||||
user.keys << key
|
||||
user.save
|
||||
get api("/keys", user)
|
||||
response.status.should == 200
|
||||
json_response.should be_an Array
|
||||
json_response.first["title"].should == key.title
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
describe "POST /keys" do
|
||||
it "should not create invalid ssh key" do
|
||||
post api("/keys", user), { title: "invalid key" }
|
||||
response.status.should == 404
|
||||
end
|
||||
it "should create ssh key" do
|
||||
key_attrs = Factory.attributes :key
|
||||
expect {
|
||||
post api("/keys", user), key_attrs
|
||||
}.to change{ user.keys.count }.by(1)
|
||||
end
|
||||
end
|
||||
|
||||
describe "DELETE /keys/:id" do
|
||||
it "should delete existed key" do
|
||||
user.keys << key
|
||||
user.save
|
||||
expect {
|
||||
delete api("/keys/#{key.id}", user)
|
||||
}.to change{user.keys.count}.by(-1)
|
||||
end
|
||||
end
|
||||
|
||||
end
|
||||
|
Loading…
Reference in a new issue