jro_apps/redmine Commit - r14328:c63da2c0a5b5

149

end

149

end

150

151

test "PUT /groups/:id.xml with valid parameters should update the group" do

151

test "PUT /groups/:id.xml with valid parameters should update the group" do

152

put '/groups/10.xml', {:group => {:name => 'New name', :user_ids => [2, 3]}}, credentials('admin')

152

group = Group.generate!

153

put "/groups/#{group.id}.xml", {:group => {:name => 'New name', :user_ids => [2, 3]}}, credentials('admin')

153

assert_response :ok

154

assert_response :ok

154

assert_equal '', @response.body

155

assert_equal '', @response.body

155

156

group = Group.find(10)

157

assert_equal 'New name', group.reload.name

157

assert_equal 'New name', group.name

158

assert_equal [2, 3], group.users.map(&:id).sort

158

assert_equal [2, 3], group.users.map(&:id).sort

159

end

159

end

160

161

test "PUT /groups/:id.xml with invalid parameters should return errors" do

161

test "PUT /groups/:id.xml with invalid parameters should return errors" do

162

put '/groups/10.xml', {:group => {:name => ''}}, credentials('admin')

162

group = Group.generate!

163

put "/groups/#{group.id}.xml", {:group => {:name => ''}}, credentials('admin')

163

assert_response :unprocessable_entity

164

assert_response :unprocessable_entity

164

assert_equal 'application/xml', response.content_type

165

assert_equal 'application/xml', response.content_type

165

166

169

end

170

end

170

171

test "DELETE /groups/:id.xml should delete the group" do

172

test "DELETE /groups/:id.xml should delete the group" do

173

group = Group.generate!

172

assert_difference 'Group.count', -1 do

174

assert_difference 'Group.count', -1 do

173

delete ~~'/groups/10.xml'~~, {}, credentials('admin')

175

delete "/groups/#{group.id}.xml", {}, credentials('admin')

174

assert_response :ok

176

assert_response :ok

175

assert_equal '', @response.body

177

assert_equal '', @response.body

176

end

178

end

177

end

179

end

178

180

179

test "POST /groups/:id/users.xml should add user to the group" do

181

test "POST /groups/:id/users.xml should add user to the group" do

180

assert_difference 'Group.find(10).users.count' do

182

group = Group.generate!

181

post '/groups/10/users.xml', {:user_id => 5}, credentials('admin')

183

assert_difference 'group.reload.users.count' do

184

post "/groups/#{group.id}/users.xml", {:user_id => 5}, credentials('admin')

182

assert_response :ok

185

assert_response :ok

183

assert_equal '', @response.body

186

assert_equal '', @response.body

184

end

187

end

185

assert_include User.find(5), Group.~~find~~(10).users

188

assert_include User.find(5), group.reload.users

186

end

189

end

187

190

188

test "POST /groups/:id/users.xml should not add the user if already added" do

191

test "POST /groups/:id/users.xml should not add the user if already added" do

189

Group.find(10).users << User.find(5)

192

group = Group.generate!

193

group.users << User.find(5)

190

194

191

assert_no_difference '~~Group.find(10)~~.users.count' do

195

assert_no_difference 'group.reload.users.count' do

192

post ~~'/groups/10/users.xml'~~, {:user_id => 5}, credentials('admin')

196

post "/groups/#{group.id}/users.xml", {:user_id => 5}, credentials('admin')

193

assert_response :unprocessable_entity

197

assert_response :unprocessable_entity

194

end

198

end

195

199

end

203

end

200

204

201

test "DELETE /groups/:id/users/:user_id.xml should remove user from the group" do

205

test "DELETE /groups/:id/users/:user_id.xml should remove user from the group" do

202

assert_difference 'Group.find(10).users.count', -1 do

206

group = Group.generate!

203

delete '/groups/10/users/8.xml', {}, credentials('admin')

207

group.users << User.find(8)

208

209

assert_difference 'group.reload.users.count', -1 do

210

delete "/groups/#{group.id}/users/8.xml", {}, credentials('admin')

204

assert_response :ok

211

assert_response :ok

205

assert_equal '', @response.body

212

assert_equal '', @response.body

206

end

213

end

207

assert_not_include User.find(8), Group.~~find~~(10).users

214

assert_not_include User.find(8), group.reload.users

208

end

215

end

209

end

216

end

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

               end
               test "PUT /groups/:id.xml with valid parameters should update the group" do
-                put '/groups/10.xml', {:group => {:name => 'New name', :user_ids => [2, 3]}}, credentials('admin')
+                group = Group.generate!
+                put "/groups/#{group.id}.xml", {:group => {:name => 'New name', :user_ids => [2, 3]}}, credentials('admin')
                 assert_response :ok
                 assert_equal '', @response.body
-                group = Group.find(10)
+                assert_equal 'New name', group.reload.name
-                assert_equal 'New name', group.name
                 assert_equal [2, 3], group.users.map(&:id).sort
               end
               test "PUT /groups/:id.xml with invalid parameters should return errors" do
-                put '/groups/10.xml', {:group => {:name => ''}}, credentials('admin')
+                group = Group.generate!
+                put "/groups/#{group.id}.xml", {:group => {:name => ''}}, credentials('admin')
                 assert_response :unprocessable_entity
                 assert_equal 'application/xml', response.content_type
               end
               test "DELETE /groups/:id.xml should delete the group" do
+                group = Group.generate!
                 assert_difference 'Group.count', -1 do
-                  delete '/groups/10.xml', {}, credentials('admin')
+                  delete "/groups/#{group.id}.xml", {}, credentials('admin')
                   assert_response :ok
                   assert_equal '', @response.body
                 end
               end
               test "POST /groups/:id/users.xml should add user to the group" do
-                assert_difference 'Group.find(10).users.count' do
+                group = Group.generate!
-                  post '/groups/10/users.xml', {:user_id => 5}, credentials('admin')
+                assert_difference 'group.reload.users.count' do
+                  post "/groups/#{group.id}/users.xml", {:user_id => 5}, credentials('admin')
                   assert_response :ok
                   assert_equal '', @response.body
                 end
-                assert_include User.find(5), Group.find(10).users
+                assert_include User.find(5), group.reload.users
               end
               test "POST /groups/:id/users.xml should not add the user if already added" do
-                Group.find(10).users << User.find(5)
+                group = Group.generate!
+                group.users << User.find(5)
-                assert_no_difference 'Group.find(10).users.count' do
+                assert_no_difference 'group.reload.users.count' do
-                  post '/groups/10/users.xml', {:user_id => 5}, credentials('admin')
+                  post "/groups/#{group.id}/users.xml", {:user_id => 5}, credentials('admin')
                   assert_response :unprocessable_entity
                 end
               end
               test "DELETE /groups/:id/users/:user_id.xml should remove user from the group" do
-                assert_difference 'Group.find(10).users.count', -1 do
+                group = Group.generate!
-                  delete '/groups/10/users/8.xml', {}, credentials('admin')
+                group.users << User.find(8)
+                assert_difference 'group.reload.users.count', -1 do
+                  delete "/groups/#{group.id}/users/8.xml", {}, credentials('admin')
                   assert_response :ok
                   assert_equal '', @response.body
                 end
-                assert_not_include User.find(8), Group.find(10).users
+                assert_not_include User.find(8), group.reload.users
               end
             end