add users API
This commit is contained in:
parent
4aca61e8a6
commit
4ad91d3c11
5 changed files with 91 additions and 1 deletions
31
lib/api.rb
31
lib/api.rb
|
@ -1,2 +1,31 @@
|
||||||
class Gitlab::API < Grape::API
|
require 'api/entities'
|
||||||
|
require 'api/helpers'
|
||||||
|
|
||||||
|
module Gitlab
|
||||||
|
class API < Grape::API
|
||||||
|
format :json
|
||||||
|
helpers APIHelpers
|
||||||
|
|
||||||
|
resource :users do
|
||||||
|
before { authenticate! }
|
||||||
|
|
||||||
|
# GET /users
|
||||||
|
get do
|
||||||
|
@users = User.all
|
||||||
|
present @users, :with => Entities::User
|
||||||
|
end
|
||||||
|
|
||||||
|
# GET /users/:id
|
||||||
|
get ":id" do
|
||||||
|
@user = User.find(params[:id])
|
||||||
|
present @user, :with => Entities::User
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
# GET /user
|
||||||
|
get "/user" do
|
||||||
|
authenticate!
|
||||||
|
present @current_user, :with => Entities::User
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
8
lib/api/entities.rb
Normal file
8
lib/api/entities.rb
Normal file
|
@ -0,0 +1,8 @@
|
||||||
|
module Gitlab
|
||||||
|
module Entities
|
||||||
|
class User < Grape::Entity
|
||||||
|
expose :id, :email, :name, :bio, :skype, :linkedin, :twitter,
|
||||||
|
:dark_scheme, :theme_id, :blocked, :created_at
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
11
lib/api/helpers.rb
Normal file
11
lib/api/helpers.rb
Normal file
|
@ -0,0 +1,11 @@
|
||||||
|
module Gitlab
|
||||||
|
module APIHelpers
|
||||||
|
def current_user
|
||||||
|
@current_user ||= User.find_by_authentication_token(params[:private_token])
|
||||||
|
end
|
||||||
|
|
||||||
|
def authenticate!
|
||||||
|
error!('401 Unauthorized', 401) unless current_user
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
38
spec/api/users_spec.rb
Normal file
38
spec/api/users_spec.rb
Normal file
|
@ -0,0 +1,38 @@
|
||||||
|
require 'spec_helper'
|
||||||
|
|
||||||
|
describe Gitlab::API do
|
||||||
|
let(:user) { Factory :user }
|
||||||
|
|
||||||
|
describe "GET /users" do
|
||||||
|
it "should return authentication error" do
|
||||||
|
get "/api/users"
|
||||||
|
response.status.should == 401
|
||||||
|
end
|
||||||
|
|
||||||
|
describe "authenticated GET /users" do
|
||||||
|
it "should return an array of users" do
|
||||||
|
get "/api/users?private_token=#{user.private_token}"
|
||||||
|
response.status.should == 200
|
||||||
|
json = JSON.parse(response.body)
|
||||||
|
json.should be_an Array
|
||||||
|
json.first['email'].should == user.email
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
describe "GET /users/:id" do
|
||||||
|
it "should return a user by id" do
|
||||||
|
get "/api/users/#{user.id}?private_token=#{user.private_token}"
|
||||||
|
response.status.should == 200
|
||||||
|
JSON.parse(response.body)['email'].should == user.email
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
describe "GET /user" do
|
||||||
|
it "should return current user" do
|
||||||
|
get "/api/user?private_token=#{user.private_token}"
|
||||||
|
response.status.should == 200
|
||||||
|
JSON.parse(response.body)['email'].should == user.email
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
|
@ -58,4 +58,8 @@ RSpec.configure do |config|
|
||||||
config.after do
|
config.after do
|
||||||
DatabaseCleaner.clean
|
DatabaseCleaner.clean
|
||||||
end
|
end
|
||||||
|
|
||||||
|
config.include RSpec::Rails::RequestExampleGroup, :type => :request, :example_group => {
|
||||||
|
:file_path => /spec\/api/
|
||||||
|
}
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue