annotate test/unit/project_members_inheritance_test.rb @ 1561:6074fffd8a1d feature_1136

No, a bare repo is better
author Chris Cannam
date Thu, 14 Jan 2016 12:03:06 +0000
parents e248c7af89ec
children
rev   line source
Chris@1464 1 # Redmine - project management software
Chris@1494 2 # Copyright (C) 2006-2014 Jean-Philippe Lang
Chris@1464 3 #
Chris@1464 4 # This program is free software; you can redistribute it and/or
Chris@1464 5 # modify it under the terms of the GNU General Public License
Chris@1464 6 # as published by the Free Software Foundation; either version 2
Chris@1464 7 # of the License, or (at your option) any later version.
Chris@1464 8 #
Chris@1464 9 # This program is distributed in the hope that it will be useful,
Chris@1464 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
Chris@1464 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Chris@1464 12 # GNU General Public License for more details.
Chris@1464 13 #
Chris@1464 14 # You should have received a copy of the GNU General Public License
Chris@1464 15 # along with this program; if not, write to the Free Software
Chris@1464 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
Chris@1464 17
Chris@1464 18 require File.expand_path('../../test_helper', __FILE__)
Chris@1464 19
Chris@1464 20 class ProjectMembersInheritanceTest < ActiveSupport::TestCase
Chris@1464 21 fixtures :roles, :users
Chris@1464 22
Chris@1464 23 def setup
Chris@1464 24 @parent = Project.generate!
Chris@1464 25 @member = Member.create!(:principal => User.find(2), :project => @parent, :role_ids => [1, 2])
Chris@1464 26 assert_equal 2, @member.reload.roles.size
Chris@1464 27 end
Chris@1464 28
Chris@1464 29 def test_project_created_with_inherit_members_disabled_should_not_inherit_members
Chris@1464 30 assert_no_difference 'Member.count' do
Chris@1464 31 project = Project.generate_with_parent!(@parent, :inherit_members => false)
Chris@1464 32
Chris@1464 33 assert_equal 0, project.memberships.count
Chris@1464 34 end
Chris@1464 35 end
Chris@1464 36
Chris@1464 37 def test_project_created_with_inherit_members_should_inherit_members
Chris@1464 38 assert_difference 'Member.count', 1 do
Chris@1464 39 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 40 project.reload
Chris@1464 41
Chris@1464 42 assert_equal 1, project.memberships.count
Chris@1464 43 member = project.memberships.first
Chris@1464 44 assert_equal @member.principal, member.principal
Chris@1464 45 assert_equal @member.roles.sort, member.roles.sort
Chris@1464 46 end
Chris@1464 47 end
Chris@1464 48
Chris@1464 49 def test_turning_on_inherit_members_should_inherit_members
Chris@1464 50 Project.generate_with_parent!(@parent, :inherit_members => false)
Chris@1464 51
Chris@1464 52 assert_difference 'Member.count', 1 do
Chris@1464 53 project = Project.order('id desc').first
Chris@1464 54 project.inherit_members = true
Chris@1464 55 project.save!
Chris@1464 56 project.reload
Chris@1464 57
Chris@1464 58 assert_equal 1, project.memberships.count
Chris@1464 59 member = project.memberships.first
Chris@1464 60 assert_equal @member.principal, member.principal
Chris@1464 61 assert_equal @member.roles.sort, member.roles.sort
Chris@1464 62 end
Chris@1464 63 end
Chris@1464 64
Chris@1464 65 def test_turning_off_inherit_members_should_remove_inherited_members
Chris@1464 66 Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 67
Chris@1464 68 assert_difference 'Member.count', -1 do
Chris@1464 69 project = Project.order('id desc').first
Chris@1464 70 project.inherit_members = false
Chris@1464 71 project.save!
Chris@1464 72 project.reload
Chris@1464 73
Chris@1464 74 assert_equal 0, project.memberships.count
Chris@1464 75 end
Chris@1464 76 end
Chris@1464 77
Chris@1464 78 def test_moving_a_root_project_under_a_parent_should_inherit_members
Chris@1464 79 Project.generate!(:inherit_members => true)
Chris@1464 80 project = Project.order('id desc').first
Chris@1464 81
Chris@1464 82 assert_difference 'Member.count', 1 do
Chris@1464 83 project.set_parent!(@parent)
Chris@1464 84 project.reload
Chris@1464 85
Chris@1464 86 assert_equal 1, project.memberships.count
Chris@1464 87 member = project.memberships.first
Chris@1464 88 assert_equal @member.principal, member.principal
Chris@1464 89 assert_equal @member.roles.sort, member.roles.sort
Chris@1464 90 end
Chris@1464 91 end
Chris@1464 92
Chris@1464 93 def test_moving_a_subproject_as_root_should_loose_inherited_members
Chris@1464 94 Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 95 project = Project.order('id desc').first
Chris@1464 96
Chris@1464 97 assert_difference 'Member.count', -1 do
Chris@1464 98 project.set_parent!(nil)
Chris@1464 99 project.reload
Chris@1464 100
Chris@1464 101 assert_equal 0, project.memberships.count
Chris@1464 102 end
Chris@1464 103 end
Chris@1464 104
Chris@1464 105 def test_moving_a_subproject_to_another_parent_should_change_inherited_members
Chris@1464 106 other_parent = Project.generate!
Chris@1464 107 other_member = Member.create!(:principal => User.find(4), :project => other_parent, :role_ids => [3])
Chris@1464 108 other_member.reload
Chris@1464 109
Chris@1464 110 Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 111 project = Project.order('id desc').first
Chris@1464 112 project.set_parent!(other_parent.reload)
Chris@1464 113 project.reload
Chris@1464 114
Chris@1464 115 assert_equal 1, project.memberships.count
Chris@1464 116 member = project.memberships.first
Chris@1464 117 assert_equal other_member.principal, member.principal
Chris@1464 118 assert_equal other_member.roles.sort, member.roles.sort
Chris@1464 119 end
Chris@1464 120
Chris@1464 121 def test_inheritance_should_propagate_to_subprojects
Chris@1464 122 project = Project.generate_with_parent!(@parent, :inherit_members => false)
Chris@1464 123 subproject = Project.generate_with_parent!(project, :inherit_members => true)
Chris@1464 124 project.reload
Chris@1464 125
Chris@1464 126 assert_difference 'Member.count', 2 do
Chris@1464 127 project.inherit_members = true
Chris@1464 128 project.save
Chris@1464 129 project.reload
Chris@1464 130 subproject.reload
Chris@1464 131
Chris@1464 132 assert_equal 1, project.memberships.count
Chris@1464 133 assert_equal 1, subproject.memberships.count
Chris@1464 134 member = subproject.memberships.first
Chris@1464 135 assert_equal @member.principal, member.principal
Chris@1464 136 assert_equal @member.roles.sort, member.roles.sort
Chris@1464 137 end
Chris@1464 138 end
Chris@1464 139
Chris@1464 140 def test_inheritance_removal_should_propagate_to_subprojects
Chris@1464 141 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 142 subproject = Project.generate_with_parent!(project, :inherit_members => true)
Chris@1464 143 project.reload
Chris@1464 144
Chris@1464 145 assert_difference 'Member.count', -2 do
Chris@1464 146 project.inherit_members = false
Chris@1464 147 project.save
Chris@1464 148 project.reload
Chris@1464 149 subproject.reload
Chris@1464 150
Chris@1464 151 assert_equal 0, project.memberships.count
Chris@1464 152 assert_equal 0, subproject.memberships.count
Chris@1464 153 end
Chris@1464 154 end
Chris@1464 155
Chris@1464 156 def test_adding_a_member_should_propagate
Chris@1464 157 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 158
Chris@1464 159 assert_difference 'Member.count', 2 do
Chris@1464 160 member = Member.create!(:principal => User.find(4), :project => @parent, :role_ids => [1, 3])
Chris@1464 161 member.reload
Chris@1464 162
Chris@1464 163 inherited_member = project.memberships.order('id desc').first
Chris@1464 164 assert_equal member.principal, inherited_member.principal
Chris@1464 165 assert_equal member.roles.sort, inherited_member.roles.sort
Chris@1464 166 end
Chris@1464 167 end
Chris@1464 168
Chris@1464 169 def test_adding_a_member_should_not_propagate_if_child_does_not_inherit
Chris@1464 170 project = Project.generate_with_parent!(@parent, :inherit_members => false)
Chris@1464 171
Chris@1464 172 assert_difference 'Member.count', 1 do
Chris@1464 173 member = Member.create!(:principal => User.find(4), :project => @parent, :role_ids => [1, 3])
Chris@1464 174
Chris@1464 175 assert_nil project.reload.memberships.detect {|m| m.principal == member.principal}
Chris@1464 176 end
Chris@1464 177 end
Chris@1464 178
Chris@1464 179 def test_removing_a_member_should_propagate
Chris@1464 180 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 181
Chris@1464 182 assert_difference 'Member.count', -2 do
Chris@1464 183 @member.reload.destroy
Chris@1464 184 project.reload
Chris@1464 185
Chris@1464 186 assert_equal 0, project.memberships.count
Chris@1464 187 end
Chris@1464 188 end
Chris@1464 189
Chris@1464 190 def test_adding_a_group_member_should_propagate_with_its_users
Chris@1464 191 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 192 group = Group.generate!
Chris@1464 193 user = User.find(4)
Chris@1464 194 group.users << user
Chris@1464 195
Chris@1464 196 assert_difference 'Member.count', 4 do
Chris@1464 197 assert_difference 'MemberRole.count', 8 do
Chris@1464 198 member = Member.create!(:principal => group, :project => @parent, :role_ids => [1, 3])
Chris@1464 199 project.reload
Chris@1464 200 member.reload
Chris@1464 201
Chris@1464 202 inherited_group_member = project.memberships.detect {|m| m.principal == group}
Chris@1464 203 assert_not_nil inherited_group_member
Chris@1464 204 assert_equal member.roles.sort, inherited_group_member.roles.sort
Chris@1464 205
Chris@1464 206 inherited_user_member = project.memberships.detect {|m| m.principal == user}
Chris@1464 207 assert_not_nil inherited_user_member
Chris@1464 208 assert_equal member.roles.sort, inherited_user_member.roles.sort
Chris@1464 209 end
Chris@1464 210 end
Chris@1464 211 end
Chris@1464 212
Chris@1464 213 def test_removing_a_group_member_should_propagate
Chris@1464 214 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 215 group = Group.generate!
Chris@1464 216 user = User.find(4)
Chris@1464 217 group.users << user
Chris@1464 218 member = Member.create!(:principal => group, :project => @parent, :role_ids => [1, 3])
Chris@1464 219
Chris@1464 220 assert_difference 'Member.count', -4 do
Chris@1464 221 assert_difference 'MemberRole.count', -8 do
Chris@1464 222 member.destroy
Chris@1464 223 project.reload
Chris@1464 224
Chris@1464 225 inherited_group_member = project.memberships.detect {|m| m.principal == group}
Chris@1464 226 assert_nil inherited_group_member
Chris@1464 227
Chris@1464 228 inherited_user_member = project.memberships.detect {|m| m.principal == user}
Chris@1464 229 assert_nil inherited_user_member
Chris@1464 230 end
Chris@1464 231 end
Chris@1464 232 end
Chris@1464 233
Chris@1464 234 def test_adding_user_who_use_is_already_a_member_to_parent_project_should_merge_roles
Chris@1464 235 project = Project.generate_with_parent!(@parent, :inherit_members => true)
Chris@1464 236 user = User.find(4)
Chris@1464 237 Member.create!(:principal => user, :project => project, :role_ids => [1, 2])
Chris@1464 238
Chris@1464 239 assert_difference 'Member.count', 1 do
Chris@1464 240 Member.create!(:principal => User.find(4), :project => @parent.reload, :role_ids => [1, 3])
Chris@1464 241
Chris@1464 242 member = project.reload.memberships.detect {|m| m.principal == user}
Chris@1464 243 assert_not_nil member
Chris@1464 244 assert_equal [1, 2, 3], member.roles.uniq.sort.map(&:id)
Chris@1464 245 end
Chris@1464 246 end
Chris@1464 247
Chris@1464 248 def test_turning_on_inheritance_with_user_who_is_already_a_member_should_merge_roles
Chris@1464 249 project = Project.generate_with_parent!(@parent)
Chris@1464 250 user = @member.user
Chris@1464 251 Member.create!(:principal => user, :project => project, :role_ids => [1, 3])
Chris@1464 252 project.reload
Chris@1464 253
Chris@1464 254 assert_no_difference 'Member.count' do
Chris@1464 255 project.inherit_members = true
Chris@1464 256 project.save!
Chris@1464 257
Chris@1464 258 member = project.reload.memberships.detect {|m| m.principal == user}
Chris@1464 259 assert_not_nil member
Chris@1464 260 assert_equal [1, 2, 3], member.roles.uniq.sort.map(&:id)
Chris@1464 261 end
Chris@1464 262 end
Chris@1464 263 end