REST Api for Groups (#8981).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@9758 e93f8b46-1217-0410-a6f0-8f06a7374b81

test/integration/api_test/groups_test.rb

@@ -0,0 +1,199 @@
+# Redmine - project management software
+# Copyright (C) 2006-2012  Jean-Philippe Lang
+#
+# This program is free software; you can redistribute it and/or
+# modify it under the terms of the GNU General Public License
+# as published by the Free Software Foundation; either version 2
+# of the License, or (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program; if not, write to the Free Software
+# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
+
+require File.expand_path('../../../test_helper', __FILE__)
+
+class ApiTest::GroupsTest < ActionController::IntegrationTest
+  fixtures :users, :groups_users
+
+  def setup
+    Setting.rest_api_enabled = '1'
+  end
+
+  context "GET /groups" do
+    context ".xml" do
+      should "require authentication" do
+        get '/groups.xml'
+        assert_response 401
+      end
+
+      should "return groups" do
+        get '/groups.xml', {}, credentials('admin')
+        assert_response :success
+        assert_equal 'application/xml', response.content_type
+
+        assert_select 'groups' do
+          assert_select 'group' do
+            assert_select 'name', :text => 'A Team'
+            assert_select 'id', :text => '10'
+          end
+        end
+      end
+    end
+
+    context ".json" do
+      should "require authentication" do
+        get '/groups.json'
+        assert_response 401
+      end
+
+      should "return groups" do
+        get '/groups.json', {}, credentials('admin')
+        assert_response :success
+        assert_equal 'application/json', response.content_type
+
+        json = MultiJson.load(response.body)
+        groups = json['groups']
+        assert_kind_of Array, groups
+        group = groups.detect {|g| g['name'] == 'A Team'}
+        assert_not_nil group
+        assert_equal({'id' => 10, 'name' => 'A Team'}, group)
+      end
+    end
+  end
+
+  context "GET /groups/:id" do
+    context ".xml" do
+      should "return the group with its users" do
+        get '/groups/10.xml', {}, credentials('admin')
+        assert_response :success
+        assert_equal 'application/xml', response.content_type
+
+        assert_select 'group' do
+          assert_select 'name', :text => 'A Team'
+          assert_select 'id', :text => '10'
+          assert_select 'users' do
+            assert_select 'user', Group.find(10).users.count
+            assert_select 'user[id=8]'
+          end
+        end
+      end
+
+      should "include memberships if requested" do
+        get '/groups/10.xml?include=memberships', {}, credentials('admin')
+        assert_response :success
+        assert_equal 'application/xml', response.content_type
+
+        assert_select 'group' do
+          assert_select 'memberships'
+        end
+      end
+    end
+  end
+
+  context "POST /groups" do
+    context "with valid parameters" do
+      context ".xml" do
+        should "create groups" do
+          assert_difference('Group.count') do
+            post '/groups.xml', {:group => {:name => 'Test', :user_ids => [2, 3]}}, credentials('admin')
+            assert_response :created
+            assert_equal 'application/xml', response.content_type
+          end
+  
+          group = Group.order('id DESC').first
+          assert_equal 'Test', group.name
+          assert_equal [2, 3], group.users.map(&:id).sort
+
+          assert_select 'group' do
+            assert_select 'name', :text => 'Test'
+          end
+        end
+      end
+    end
+
+    context "with invalid parameters" do
+      context ".xml" do
+        should "return errors" do
+          assert_no_difference('Group.count') do
+            post '/groups.xml', {:group => {:name => ''}}, credentials('admin')
+          end
+          assert_response :unprocessable_entity
+          assert_equal 'application/xml', response.content_type
+
+          assert_select 'errors' do
+            assert_select 'error', :text => /Name can't be blank/
+          end
+        end
+      end
+    end
+  end
+
+  context "PUT /groups/:id" do
+    context "with valid parameters" do
+      context ".xml" do
+        should "update the group" do
+          put '/groups/10.xml', {:group => {:name => 'New name', :user_ids => [2, 3]}}, credentials('admin')
+          assert_response :ok
+  
+          group = Group.find(10)
+          assert_equal 'New name', group.name
+          assert_equal [2, 3], group.users.map(&:id).sort
+        end
+      end
+    end
+
+    context "with invalid parameters" do
+      context ".xml" do
+        should "return errors" do
+          put '/groups/10.xml', {:group => {:name => ''}}, credentials('admin')
+          assert_response :unprocessable_entity
+          assert_equal 'application/xml', response.content_type
+
+          assert_select 'errors' do
+            assert_select 'error', :text => /Name can't be blank/
+          end
+        end
+      end
+    end
+  end
+
+  context "DELETE /groups/:id" do
+    context ".xml" do
+      should "delete the group" do
+        assert_difference 'Group.count', -1 do
+          delete '/groups/10.xml', {}, credentials('admin')
+          assert_response :ok
+        end
+      end
+    end
+  end
+
+  context "POST /groups/:id/users" do
+    context ".xml" do
+      should "add user to the group" do
+        assert_difference 'Group.find(10).users.count' do
+          post '/groups/10/users.xml', {:user_id => 5}, credentials('admin')
+          assert_response :ok
+        end
+        assert_include User.find(5), Group.find(10).users
+      end
+    end
+  end
+
+  context "DELETE /groups/:id/users/:user_id" do
+    context ".xml" do
+      should "remove user from the group" do
+        assert_difference 'Group.find(10).users.count', -1 do
+          delete '/groups/10/users/8.xml', {}, credentials('admin')
+          assert_response :ok
+        end
+        assert_not_include User.find(8), Group.find(10).users
+      end
+    end
+  end
+end

test/integration/routing/groups_test.rb

@@ -39,48 +39,37 @@ class RoutingGroupsTest < ActionController::IntegrationTest
         { :method => 'get', :path => "/groups/new" },
         { :controller => 'groups', :action => 'new' }
       )
-    assert_routing(
-        { :method => 'get', :path => "/groups/new.xml" },
-        { :controller => 'groups', :action => 'new', :format => 'xml' }
-      )
     assert_routing(
         { :method => 'get', :path => "/groups/1/edit" },
         { :controller => 'groups', :action => 'edit', :id => '1' }
       )
     assert_routing(
         { :method => 'get', :path => "/groups/1/autocomplete_for_user" },
-        { :controller => 'groups', :action => 'autocomplete_for_user',
-          :id => '1' }
+        { :controller => 'groups', :action => 'autocomplete_for_user', :id => '1' }
       )
     assert_routing(
         { :method => 'get', :path => "/groups/1" },
-        { :controller => 'groups', :action => 'show',
-          :id => '1' }
+        { :controller => 'groups', :action => 'show', :id => '1' }
       )
     assert_routing(
         { :method => 'get', :path => "/groups/1.xml" },
-        { :controller => 'groups', :action => 'show',
-          :format => 'xml', :id => '1' }
+        { :controller => 'groups', :action => 'show', :id => '1', :format => 'xml' }
       )
     assert_routing(
         { :method => 'put', :path => "/groups/1" },
-        { :controller => 'groups', :action => 'update',
-          :id => '1' }
+        { :controller => 'groups', :action => 'update', :id => '1' }
       )
     assert_routing(
         { :method => 'put', :path => "/groups/1.xml" },
-        { :controller => 'groups', :action => 'update',
-          :format => 'xml', :id => '1' }
+        { :controller => 'groups', :action => 'update', :id => '1', :format => 'xml' }
       )
     assert_routing(
         { :method => 'delete', :path => "/groups/1" },
-        { :controller => 'groups', :action => 'destroy',
-          :id => '1' }
+        { :controller => 'groups', :action => 'destroy', :id => '1' }
       )
     assert_routing(
         { :method => 'delete', :path => "/groups/1.xml" },
-        { :controller => 'groups', :action => 'destroy',
-          :format => 'xml', :id => '1' }
+        { :controller => 'groups', :action => 'destroy', :id => '1', :format => 'xml' }
       )
   end
 
@@ -89,10 +78,17 @@ class RoutingGroupsTest < ActionController::IntegrationTest
         { :method => 'post', :path => "/groups/567/users" },
         { :controller => 'groups', :action => 'add_users', :id => '567' }
       )
+    assert_routing(
+        { :method => 'post', :path => "/groups/567/users.xml" },
+        { :controller => 'groups', :action => 'add_users', :id => '567', :format => 'xml' }
+      )
     assert_routing(
         { :method => 'delete', :path => "/groups/567/users/12" },
-        { :controller => 'groups', :action => 'remove_user', :id => '567',
-          :user_id => '12' }
+        { :controller => 'groups', :action => 'remove_user', :id => '567', :user_id => '12' }
+      )
+    assert_routing(
+        { :method => 'delete', :path => "/groups/567/users/12.xml" },
+        { :controller => 'groups', :action => 'remove_user', :id => '567', :user_id => '12', :format => 'xml' }
       )
     assert_routing(
         { :method => 'post', :path => "/groups/destroy_membership/567" },

test/unit/group_test.rb

@@ -53,7 +53,7 @@ class GroupTest < ActiveSupport::TestCase
     assert_include str, g.errors.full_messages
   end
 
-  def test_roles_given_to_new_user
+  def test_group_roles_should_be_given_to_added_user
     group = Group.find(11)
     user = User.find(9)
     project = Project.first
@@ -63,7 +63,7 @@ class GroupTest < ActiveSupport::TestCase
     assert user.member_of?(project)
   end
 
-  def test_roles_given_to_existing_user
+  def test_new_roles_should_be_given_to_existing_user
     group = Group.find(11)
     user = User.find(9)
     project = Project.first
@@ -73,7 +73,22 @@ class GroupTest < ActiveSupport::TestCase
     assert user.member_of?(project)
   end
 
-  def test_roles_updated
+  def test_user_roles_should_updated_when_updating_user_ids
+    group = Group.find(11)
+    user = User.find(9)
+    project = Project.first
+
+    Member.create!(:principal => group, :project => project, :role_ids => [1, 2])
+    group.user_ids = [user.id]
+    group.save!
+    assert User.find(9).member_of?(project)
+
+    group.user_ids = [1]
+    group.save!
+    assert !User.find(9).member_of?(project)
+  end
+
+  def test_user_roles_should_updated_when_updating_group_roles
     group = Group.find(11)
     user = User.find(9)
     project = Project.first
@@ -91,13 +106,13 @@ class GroupTest < ActiveSupport::TestCase
     assert_equal [1], user.reload.roles_for_project(project).collect(&:id).sort
   end
 
-  def test_roles_removed_when_removing_group_membership
+  def test_user_memberships_should_be_removed_when_removing_group_membership
     assert User.find(8).member_of?(Project.find(5))
     Member.find_by_project_id_and_user_id(5, 10).destroy
     assert !User.find(8).member_of?(Project.find(5))
   end
 
-  def test_roles_removed_when_removing_user_from_group
+  def test_user_roles_should_be_removed_when_removing_user_from_group
     assert User.find(8).member_of?(Project.find(5))
     User.find(8).groups.clear
     assert !User.find(8).member_of?(Project.find(5))