feat(group): membership management APIs

Gemfile

@@ -31,6 +31,7 @@ gem 'slim-rails'
 gem 'turbolinks'
 gem 'uglifier'
 gem 'whenever', require: false
+gem 'kaminari'
 
 group :development, :test do
   gem 'capybara'

app/controllers/api/v1/groups_controller.rb

@@ -1,4 +1,9 @@
 class ::Api::V1::GroupsController < ::Api::V1::BaseController
+  def index
+    groups = Group.order(:id).page(params[:page]).per(params[:per_page])
+    render json: groups, status: :ok
+  end
+
   def create
     if current_user.admin?
       @group = Group.new(group_params)
@@ -40,6 +45,33 @@ def add_user
     head :no_content
   end
 
+  def remove_user
+    @group = Group.find_by(id: params[:id])
+    return head :not_found unless @group.present?
+
+    return raise_unauthorized unless current_user.admin? || @group.admin?(current_user)
+
+    user = User.find_by(id: params[:user_id])
+    return head :unprocessable_entity unless user.present?
+
+    @group.remove_user(params[:user_id])
+    head :no_content
+  end
+
+  def list_admins
+    group = Group.find(params[:id])
+    users = group.group_admins.joins(:user).
+      select('users.id, users.email, users.name, users.active, group_admins.created_at as join_date').
+      where('users.active = ?', true)
+    render json: users, status: :ok
+  end
+
+  def associated_vpns
+    group = Group.find(params[:id])
+    vpns = group.vpns
+    render json: vpns, status: :ok
+  end
+
   private
 
   def group_params

app/controllers/api/v1/vpns_controller.rb

@@ -1,6 +1,17 @@
 class ::Api::V1::VpnsController < ::Api::V1::BaseController
   before_action :set_vpn, only: [:assign_group]
 
+  def index
+    vpns = Vpn.order(:id).page(params[:page]).per(params[:per_page])
+    render json: vpns, status: :ok
+  end
+
+  def associated_groups
+    vpn = Vpn.find(params[:id])
+    groups = vpn.groups
+    render json: groups, status: :ok
+  end
+
   def create
     if current_user.admin?
       @vpn = Vpn.new(vpn_params)

config/routes.rb

@@ -87,9 +87,13 @@
       post 'endpoints' => 'endpoints#create', format: :json, :constraints => { format: 'json' }
       post 'endpoints/:id/add_group' => 'endpoints#add_group', format: :json, constraints: { format: 'json' }
       post 'groups/:id/users' => 'groups#add_user', format: :json, constraints: { format: 'json' }
-
-      resources :groups, only: [:create], format: :json
-      resources :vpns, only: [:create], format: :json do
+      delete 'groups/:id/users/:user_id' => 'groups#remove_user', format: :json, constraints: { format: 'json' }
+      get 'groups/:id/admins' => 'groups#list_admins', format: :json, constraints: { format: 'json' }
+      get 'groups/:id/vpns' => 'groups#associated_vpns', format: :json, constraints: { format: 'json' }
+      get 'vpns/:id/groups' => 'vpns#associated_groups', format: :json, constraints: { format: 'json' }
+
+      resources :groups, only: [:create, :index], format: :json
+      resources :vpns, only: [:create, :index], format: :json do
         member do
           post 'assign_group'
         end