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
|