Add a /api/0.6/user/NNNN call to the API
[rails.git] / test / functional / user_controller_test.rb
1 require File.dirname(__FILE__) + '/../test_helper'
2
3 class UserControllerTest < ActionController::TestCase
4   fixtures :users
5   
6   ##
7   # test all routes which lead to this controller
8   def test_routes
9     assert_routing(
10       { :path => "/api/0.6/user/1", :method => :get },
11       { :controller => "user", :action => "api_read", :id => "1" }
12     )
13     assert_routing(
14       { :path => "/api/0.6/user/details", :method => :get },
15       { :controller => "user", :action => "api_details" }
16     )
17     assert_routing(
18       { :path => "/api/0.6/user/gpx_files", :method => :get },
19       { :controller => "user", :action => "api_gpx_files" }
20     )
21
22     assert_routing(
23       { :path => "/login", :method => :get },
24       { :controller => "user", :action => "login" }
25     )
26     assert_routing(
27       { :path => "/login", :method => :post },
28       { :controller => "user", :action => "login" }
29     )
30     assert_recognizes(
31       { :controller => "user", :action => "login", :format => "html" },
32       { :path => "/login.html", :method => :get }
33     )
34
35     assert_routing(
36       { :path => "/logout", :method => :get },
37       { :controller => "user", :action => "logout" }
38     )
39     assert_routing(
40       { :path => "/logout", :method => :post },
41       { :controller => "user", :action => "logout" }
42     )
43     assert_recognizes(
44       { :controller => "user", :action => "logout", :format => "html" },
45       { :path => "/logout.html", :method => :get }
46     )
47
48     assert_routing(
49       { :path => "/user/new", :method => :get },
50       { :controller => "user", :action => "new" }
51     )
52     assert_recognizes(
53       { :controller => "user", :action => "new" },
54       { :path => "/create-account.html", :method => :get }
55     )
56
57     assert_routing(
58       { :path => "/user/terms", :method => :get },
59       { :controller => "user", :action => "terms" }
60     )
61     assert_routing(
62       { :path => "/user/terms", :method => :post },
63       { :controller => "user", :action => "terms" }
64     )
65
66     assert_routing(
67       { :path => "/user/save", :method => :post },
68       { :controller => "user", :action => "save" }
69     )
70
71     assert_routing(
72       { :path => "/user/username/confirm", :method => :get },
73       { :controller => "user", :action => "confirm", :display_name => "username" }
74     )
75     assert_routing(
76       { :path => "/user/username/confirm", :method => :post },
77       { :controller => "user", :action => "confirm", :display_name => "username" }
78     )
79     assert_routing(
80       { :path => "/user/username/confirm/resend", :method => :get },
81       { :controller => "user", :action => "confirm_resend", :display_name => "username" }
82     )
83
84     assert_routing(
85       { :path => "/user/confirm", :method => :get },
86       { :controller => "user", :action => "confirm" }
87     )
88     assert_routing(
89       { :path => "/user/confirm", :method => :post },
90       { :controller => "user", :action => "confirm" }
91     )
92     assert_routing(
93       { :path => "/user/confirm-email", :method => :get },
94       { :controller => "user", :action => "confirm_email" }
95     )
96     assert_routing(
97       { :path => "/user/confirm-email", :method => :post },
98       { :controller => "user", :action => "confirm_email" }
99     )
100
101     assert_routing(
102       { :path => "/user/go_public", :method => :post },
103       { :controller => "user", :action => "go_public" }
104     )
105
106     assert_routing(
107       { :path => "/user/forgot-password", :method => :get },
108       { :controller => "user", :action => "lost_password" }
109     )
110     assert_routing(
111       { :path => "/user/forgot-password", :method => :post },
112       { :controller => "user", :action => "lost_password" }
113     )
114     assert_recognizes(
115       { :controller => "user", :action => "lost_password" },
116       { :path => "/forgot-password.html", :method => :get }
117     )
118     assert_routing(
119       { :path => "/user/reset-password", :method => :get },
120       { :controller => "user", :action => "reset_password" }
121     )
122     assert_routing(
123       { :path => "/user/reset-password", :method => :post },
124       { :controller => "user", :action => "reset_password" }
125     )
126
127     assert_routing(
128       { :path => "/user/suspended", :method => :get },
129       { :controller => "user", :action => "suspended" }
130     )
131
132     assert_routing(
133       { :path => "/user/username", :method => :get },
134       { :controller => "user", :action => "view", :display_name => "username" }
135     )
136
137     assert_routing(
138       { :path => "/user/username/account", :method => :get },
139       { :controller => "user", :action => "account", :display_name => "username" }
140     )
141     assert_routing(
142       { :path => "/user/username/account", :method => :post },
143       { :controller => "user", :action => "account", :display_name => "username" }
144     )
145
146     assert_routing(
147       { :path => "/user/username/make_friend", :method => :get },
148       { :controller => "user", :action => "make_friend", :display_name => "username" }
149     )
150     assert_routing(
151       { :path => "/user/username/make_friend", :method => :post },
152       { :controller => "user", :action => "make_friend", :display_name => "username" }
153     )
154     assert_routing(
155       { :path => "/user/username/remove_friend", :method => :get },
156       { :controller => "user", :action => "remove_friend", :display_name => "username" }
157     )
158     assert_routing(
159       { :path => "/user/username/remove_friend", :method => :post },
160       { :controller => "user", :action => "remove_friend", :display_name => "username" }
161     )
162
163     assert_routing(
164       { :path => "/user/username/set_status", :method => :get },
165       { :controller => "user", :action => "set_status", :display_name => "username" }
166     )
167     assert_routing(
168       { :path => "/user/username/delete", :method => :get },
169       { :controller => "user", :action => "delete", :display_name => "username" }
170     )
171
172     assert_routing(
173       { :path => "/users", :method => :get },
174       { :controller => "user", :action => "list" }
175     )
176     assert_routing(
177       { :path => "/users", :method => :post },
178       { :controller => "user", :action => "list" }
179     )
180     assert_routing(
181       { :path => "/users/status", :method => :get },
182       { :controller => "user", :action => "list", :status => "status" }
183     )
184     assert_routing(
185       { :path => "/users/status", :method => :post },
186       { :controller => "user", :action => "list", :status => "status" }
187     )
188   end
189
190   # The user creation page loads
191   def test_user_create_view
192     get :new
193     assert_response :success
194     
195     assert_select "html", :count => 1 do
196       assert_select "head", :count => 1 do
197         assert_select "title", :text => /Create account/, :count => 1
198       end
199       assert_select "body", :count => 1 do
200         assert_select "div#content", :count => 1 do
201           assert_select "form[action='/user/terms'][method=post]", :count => 1 do
202             assert_select "input[id=user_email]", :count => 1
203             assert_select "input[id=user_email_confirmation]", :count => 1
204             assert_select "input[id=user_display_name]", :count => 1
205             assert_select "input[id=user_pass_crypt][type=password]", :count => 1
206             assert_select "input[id=user_pass_crypt_confirmation][type=password]", :count => 1
207             assert_select "input[type=submit][value=Continue]", :count => 1
208           end
209         end
210       end
211     end
212   end
213   
214   def test_user_create_success
215     new_email = "newtester@osm.org"
216     display_name = "new_tester"
217     assert_difference('User.count') do
218       assert_difference('ActionMailer::Base.deliveries.size') do
219         post :save, {:user => { :email => new_email, :email_confirmation => new_email, :display_name => display_name, :pass_crypt => "testtest", :pass_crypt_confirmation => "testtest"}}
220       end
221     end
222       
223     # Check the e-mail
224     register_email = ActionMailer::Base.deliveries.first
225   
226     assert_equal register_email.to[0], new_email
227     assert_match /#{@url}/, register_email.body.to_s
228
229     # Check the page
230     assert_redirected_to :action => 'login', :referer => nil
231       
232     ActionMailer::Base.deliveries.clear
233   end
234   
235   def test_user_create_submit_duplicate_email
236     email = users(:public_user).email
237     display_name = "new_tester"
238     assert_difference('User.count', 0) do
239       assert_difference('ActionMailer::Base.deliveries.size', 0) do
240         post :save, :user => { :email => email, :email_confirmation => email, :display_name => display_name, :pass_crypt => "testtest", :pass_crypt_confirmation => "testtest"}
241       end
242     end
243     assert_response :success                                                                       
244     assert_template 'new'
245     assert_select "div#errorExplanation"
246     assert_select "table#signupForm > tr > td > div[class=field_with_errors] > input#user_email"
247   end
248   
249   def test_user_create_submit_duplicate_email_uppercase
250     email = users(:public_user).email.upcase
251     display_name = "new_tester"
252     assert_difference('User.count', 0) do
253       assert_difference('ActionMailer::Base.deliveries.size', 0) do
254         post :save, :user => { :email => email, :email_confirmation => email, :display_name => display_name, :pass_crypt => "testtest", :pass_crypt_confirmation => "testtest"}
255       end
256     end
257     assert_response :success                                                                       
258     assert_template 'new'
259     assert_select "div#errorExplanation"
260     assert_select "table#signupForm > tr > td > div[class=field_with_errors] > input#user_email"
261   end
262     
263   def test_user_create_submit_duplicate_name
264     email = "new_tester@example.com"
265     display_name = users(:public_user).display_name
266     assert_difference('User.count', 0) do
267       assert_difference('ActionMailer::Base.deliveries.size', 0) do
268         post :save, :user => { :email => email, :email_confirmation => email, :display_name => display_name, :pass_crypt => "testtest", :pass_crypt_confirmation => "testtest"}
269       end
270     end
271     assert_response :success                                                                       
272     assert_template 'new'
273     assert_select "div#errorExplanation"
274     assert_select "table#signupForm > tr > td > div[class=field_with_errors] > input#user_display_name"
275   end
276   
277   def test_user_create_submit_duplicate_name_uppercase
278     email = "new_tester@example.com"
279     display_name = users(:public_user).display_name.upcase
280     assert_difference('User.count', 0) do
281       assert_difference('ActionMailer::Base.deliveries.size', 0) do
282         post :save, :user => { :email => email, :email_confirmation => email, :display_name => display_name, :pass_crypt => "testtest", :pass_crypt_confirmation => "testtest"}
283       end
284     end
285     assert_response :success                                                                       
286     assert_template 'new'
287     assert_select "div#errorExplanation"
288     assert_select "table#signupForm > tr > td > div[class=field_with_errors] > input#user_display_name"
289   end
290
291   def test_user_lost_password
292     # Test fetching the lost password page
293     get :lost_password
294     assert_response :success
295     assert_template :lost_password
296     assert_select "div#notice", false
297
298     # Test resetting using the address as recorded for a user that has an
299     # address which is duplicated in a different case by another user
300     assert_difference('ActionMailer::Base.deliveries.size', 1) do
301       post :lost_password, :user => { :email => users(:normal_user).email }
302     end
303     assert_response :redirect
304     assert_redirected_to :action => :login
305     assert_match /^Sorry you lost it/, flash[:notice]
306     assert_equal users(:normal_user).email, ActionMailer::Base.deliveries.last.to[0]
307
308     # Test resetting using an address that matches a different user
309     # that has the same address in a different case
310     assert_difference('ActionMailer::Base.deliveries.size', 1) do
311       post :lost_password, :user => { :email => users(:normal_user).email.upcase }
312     end
313     assert_response :redirect
314     assert_redirected_to :action => :login
315     assert_match /^Sorry you lost it/, flash[:notice]
316     assert_equal users(:uppercase_user).email, ActionMailer::Base.deliveries.last.to[0]
317
318     # Test resetting using an address that is a case insensitive match
319     # for more than one user but not an exact match for either
320     assert_difference('ActionMailer::Base.deliveries.size', 0) do
321       post :lost_password, :user => { :email => users(:normal_user).email.titlecase }
322     end
323     assert_response :success
324     assert_template :lost_password
325     assert_select "div#error", /^Could not find that email address/
326
327     # Test resetting using the address as recorded for a user that has an
328     # address which is case insensitively unique
329     assert_difference('ActionMailer::Base.deliveries.size', 1) do
330       post :lost_password, :user => { :email => users(:public_user).email }
331     end
332     assert_response :redirect
333     assert_redirected_to :action => :login
334     assert_match /^Sorry you lost it/, flash[:notice]
335     assert_equal users(:public_user).email, ActionMailer::Base.deliveries.last.to[0]
336
337     # Test resetting using an address that matches a user that has the
338     # same (case insensitively unique) address in a different case
339     assert_difference('ActionMailer::Base.deliveries.size', 1) do
340       post :lost_password, :user => { :email => users(:public_user).email.upcase }
341     end
342     assert_response :redirect
343     assert_redirected_to :action => :login
344     assert_match /^Sorry you lost it/, flash[:notice]
345     assert_equal users(:public_user).email, ActionMailer::Base.deliveries.last.to[0]
346   end
347
348   def test_user_update
349     # Get a user to work with - note that this user deliberately
350     # conflicts with uppercase_user in the email and display name
351     # fields to test that we can change other fields without any
352     # validation errors being reported
353     user = users(:normal_user)
354
355     # Set the username cookie
356     @request.cookies["_osm_username"] = user.display_name
357
358     # Make sure that you are redirected to the login page when
359     # you are not logged in
360     get :account, { :display_name => user.display_name }
361     assert_response :redirect
362     assert_redirected_to :controller => :user, :action => "login", :referer => "/user/test/account"
363
364     # Make sure that you are redirected to the login page when
365     # you are not logged in as the right user
366     get :account, { :display_name => user.display_name }, { "user" => users(:public_user).id }
367     assert_response :redirect
368     assert_redirected_to :controller => :user, :action => "login", :referer => "/user/test/account"
369
370     # Make sure we get the page when we are logged in as the right user
371     get :account, { :display_name => user.display_name }, { "user" => user }
372     assert_response :success
373     assert_template :account
374
375     # Updating the description should work
376     user.description = "new description"
377     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
378     assert_response :success
379     assert_template :account
380     assert_select "div#errorExplanation", false
381     assert_select "div#notice", /^User information updated successfully/
382     assert_select "table#accountForm > tr > td > div#user_description_container > div#user_description_content > textarea#user_description", user.description
383
384     # Changing name to one that exists should fail
385     user.display_name = users(:public_user).display_name
386     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
387     assert_response :success
388     assert_template :account
389     assert_select "div#notice", false
390     assert_select "div#errorExplanation"
391     assert_select "table#accountForm > tr > td > div[class=field_with_errors] > input#user_display_name"
392
393     # Changing name to one that exists should fail, regardless of case
394     user.display_name = users(:public_user).display_name.upcase
395     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
396     assert_response :success
397     assert_template :account
398     assert_select "div#notice", false
399     assert_select "div#errorExplanation"
400     assert_select "table#accountForm > tr > td > div[class=field_with_errors] > input#user_display_name"
401
402     # Changing name to one that doesn't exist should work
403     user.display_name = "new tester"
404     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
405     assert_response :success
406     assert_template :account
407     assert_select "div#errorExplanation", false
408     assert_select "div#notice", /^User information updated successfully/
409     assert_select "table#accountForm > tr > td > input#user_display_name[value=?]", user.display_name
410
411     # Need to update cookies now to stay valid
412     @request.cookies["_osm_username"] = user.display_name
413
414     # Changing email to one that exists should fail
415     user.new_email = users(:public_user).email
416     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
417     assert_response :success
418     assert_template :account
419     assert_select "div#notice", false
420     assert_select "div#errorExplanation"
421     assert_select "table#accountForm > tr > td > div[class=field_with_errors] > input#user_new_email"
422
423     # Changing email to one that exists should fail, regardless of case
424     user.new_email = users(:public_user).email.upcase
425     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
426     assert_response :success
427     assert_template :account
428     assert_select "div#notice", false
429     assert_select "div#errorExplanation"
430     assert_select "table#accountForm > tr > td > div[class=field_with_errors] > input#user_new_email"
431
432     # Changing email to one that doesn't exist should work
433     user.new_email = "new_tester@example.com"
434     post :account, { :display_name => user.display_name, :user => user.attributes }, { "user" => user.id }
435     assert_response :success
436     assert_template :account
437     assert_select "div#errorExplanation", false
438     assert_select "div#notice", /^User information updated successfully/
439     assert_select "table#accountForm > tr > td > input#user_new_email[value=?]", user.new_email
440   end
441   
442   # Check that the user account page will display and contains some relevant
443   # information for the user
444   def test_user_view_account
445     # Test a non-existent user
446     get :view, {:display_name => "unknown"}
447     assert_response :not_found
448     
449     # Test a normal user
450     get :view, {:display_name => "test"}
451     assert_response :success
452     assert_select "div#userinformation" do
453       assert_select "a[href=/user/test/edits]", 1
454       assert_select "a[href=/user/test/traces]", 1
455       assert_select "a[href=/user/test/diary]", 1
456       assert_select "a[href=/user/test/diary/comments]", 1
457       assert_select "a[href=/user/test/account]", 0
458       assert_select "a[href=/user/test/blocks]", 0
459       assert_select "a[href=/user/test/blocks_by]", 0
460       assert_select "a[href=/blocks/new/test]", 0
461     end
462     
463     # Test a user who has been blocked
464     get :view, {:display_name => "blocked"}
465     assert_response :success
466     assert_select "div#userinformation" do
467       assert_select "a[href=/user/blocked/edits]", 1
468       assert_select "a[href=/user/blocked/traces]", 1
469       assert_select "a[href=/user/blocked/diary]", 1
470       assert_select "a[href=/user/blocked/diary/comments]", 1
471       assert_select "a[href=/user/blocked/account]", 0
472       assert_select "a[href=/user/blocked/blocks]", 1
473       assert_select "a[href=/user/blocked/blocks_by]", 0
474       assert_select "a[href=/blocks/new/blocked]", 0
475     end
476     
477     # Test a moderator who has applied blocks
478     get :view, {:display_name => "moderator"}
479     assert_response :success
480     assert_select "div#userinformation" do
481       assert_select "a[href=/user/moderator/edits]", 1
482       assert_select "a[href=/user/moderator/traces]", 1
483       assert_select "a[href=/user/moderator/diary]", 1
484       assert_select "a[href=/user/moderator/diary/comments]", 1
485       assert_select "a[href=/user/moderator/account]", 0
486       assert_select "a[href=/user/moderator/blocks]", 0
487       assert_select "a[href=/user/moderator/blocks_by]", 1
488       assert_select "a[href=/blocks/new/moderator]", 0
489     end
490
491     # Login as a normal user
492     session[:user] = users(:normal_user).id
493     cookies["_osm_username"] = users(:normal_user).display_name
494
495     # Test the normal user
496     get :view, {:display_name => "test"}
497     assert_response :success
498     assert_select "div#userinformation" do
499       assert_select "a[href=/user/test/edits]", 1
500       assert_select "a[href=/traces/mine]", 1
501       assert_select "a[href=/user/test/diary]", 1
502       assert_select "a[href=/user/test/diary/comments]", 1
503       assert_select "a[href=/user/test/account]", 1
504       assert_select "a[href=/user/test/blocks]", 0
505       assert_select "a[href=/user/test/blocks_by]", 0
506       assert_select "a[href=/blocks/new/test]", 0
507     end
508
509     # Login as a moderator
510     session[:user] = users(:moderator_user).id
511     cookies["_osm_username"] = users(:moderator_user).display_name
512
513     # Test the normal user
514     get :view, {:display_name => "test"}
515     assert_response :success
516     assert_select "div#userinformation" do
517       assert_select "a[href=/user/test/edits]", 1
518       assert_select "a[href=/user/test/traces]", 1
519       assert_select "a[href=/user/test/diary]", 1
520       assert_select "a[href=/user/test/diary/comments]", 1
521       assert_select "a[href=/user/test/account]", 0
522       assert_select "a[href=/user/test/blocks]", 0
523       assert_select "a[href=/user/test/blocks_by]", 0
524       assert_select "a[href=/blocks/new/test]", 1
525     end
526   end
527
528   def test_user_api_read
529     # check that a visible user is returned properly
530     get :api_read, :id => users(:normal_user).id
531     assert_response :success
532
533     # check that we aren't revealing private information
534     assert_select "home", false
535     assert_select "languages", false
536
537     # check that a suspended user is not returned
538     get :api_read, :id => users(:suspended_user).id
539     assert_response :gone
540
541     # check that a deleted user is not returned
542     get :api_read, :id => users(:deleted_user).id
543     assert_response :gone
544
545     # check that a non-existent user is not returned
546     get :api_read, :id => 0
547     assert_response :not_found
548   end
549
550   def test_user_api_details
551     get :api_details
552     assert_response :unauthorized
553     
554     basic_authorization(users(:normal_user).email, "test")
555     get :api_details
556     assert_response :success
557   end
558
559   def test_user_make_friend
560     # Get users to work with
561     user = users(:normal_user)
562     friend = users(:second_public_user)
563
564     # Check that the users aren't already friends
565     assert_nil Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
566
567     # Set the username cookie
568     @request.cookies["_osm_username"] = user.display_name
569
570     # When not logged in a GET should ask us to login
571     get :make_friend, {:display_name => friend.display_name}
572     assert_redirected_to :controller => :user, :action => "login", :referer => make_friend_path(:display_name => friend.display_name)
573
574     # When not logged in a POST should error
575     post :make_friend, {:display_name => friend.display_name}
576     assert_response :forbidden
577     assert_nil Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
578
579     # When logged in a GET should get a confirmation page
580     get :make_friend, {:display_name => friend.display_name}, {"user" => user}
581     assert_response :success
582     assert_template :make_friend
583     assert_select "form" do
584       assert_select "input[type=hidden][name=referer]", 0
585       assert_select "input[type=submit]", 1
586     end
587     assert_nil Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
588
589     # The GET should preserve any referer
590     get :make_friend, {:display_name => friend.display_name, :referer => "/test"}, {"user" => user}
591     assert_response :success
592     assert_template :make_friend
593     assert_select "form" do
594       assert_select "input[type=hidden][name=referer][value=/test]", 1
595       assert_select "input[type=submit]", 1
596     end
597     assert_nil Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
598
599     # When logged in a POST should add the friendship
600     post :make_friend, {:display_name => friend.display_name}, {"user" => user}
601     assert_redirected_to user_path(:display_name => friend.display_name)
602     assert_match /is now your friend/, flash[:notice]
603     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
604
605     # A second POST should report that the friendship already exists
606     post :make_friend, {:display_name => friend.display_name}, {"user" => user}
607     assert_redirected_to user_path(:display_name => friend.display_name)
608     assert_match /You are already friends with/, flash[:warning]
609     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
610   end
611
612   def test_user_remove_friend
613     # Get users to work with
614     user = users(:normal_user)
615     friend = users(:public_user)
616
617     # Check that the users are friends
618     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
619
620     # Set the username cookie
621     @request.cookies["_osm_username"] = user.display_name
622
623     # When not logged in a GET should ask us to login
624     get :remove_friend, {:display_name => friend.display_name}
625     assert_redirected_to :controller => :user, :action => "login", :referer => remove_friend_path(:display_name => friend.display_name)
626
627     # When not logged in a POST should error
628     post :remove_friend, {:display_name => friend.display_name}
629     assert_response :forbidden
630     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
631
632     # When logged in a GET should get a confirmation page
633     get :remove_friend, {:display_name => friend.display_name}, {"user" => user}
634     assert_response :success
635     assert_template :remove_friend
636     assert_select "form" do
637       assert_select "input[type=hidden][name=referer]", 0
638       assert_select "input[type=submit]", 1
639     end
640     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
641
642     # The GET should preserve any referer
643     get :remove_friend, {:display_name => friend.display_name, :referer => "/test"}, {"user" => user}
644     assert_response :success
645     assert_template :remove_friend
646     assert_select "form" do
647       assert_select "input[type=hidden][name=referer][value=/test]", 1
648       assert_select "input[type=submit]", 1
649     end
650     assert Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
651
652     # When logged in a POST should remove the friendship
653     post :remove_friend, {:display_name => friend.display_name}, {"user" => user}
654     assert_redirected_to user_path(:display_name => friend.display_name)
655     assert_match /was removed from your friends/, flash[:notice]
656     assert_nil Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
657
658     # A second POST should report that the friendship does not exist
659     post :remove_friend, {:display_name => friend.display_name}, {"user" => user}
660     assert_redirected_to user_path(:display_name => friend.display_name)
661     assert_match /is not one of your friends/, flash[:error]
662     assert_nil Friend.where(:user_id => user.id, :friend_user_id => friend.id).first
663   end
664 end