Merge branch 'comments'

[rails.git] / test / controllers / changeset_controller_test.rb

diff --git a/test/controllers/changeset_controller_test.rb b/test/controllers/changeset_controller_test.rb
index c3fd0fadc923cea4b52855604cdd22aa06e17260..4617383303aeb34203414db00a2b09e39f0ad4d5 100644 (file)
--- a/test/controllers/changeset_controller_test.rb
+++ b/test/controllers/changeset_controller_test.rb
@@ -3,7 +3,7 @@ require 'changeset_controller'
 
 class ChangesetControllerTest < ActionController::TestCase
   api_fixtures
-  fixtures :changeset_comments
+  fixtures :changeset_comments, :changesets_subscribers
 
   ##
   # test all routes which lead to this controller
@@ -29,41 +29,41 @@ class ChangesetControllerTest < ActionController::TestCase
       { :controller => "changeset", :action => "read", :id => "1" }
     )
     assert_routing(
-      { :path => "/api/0.6/changeset/1", :method => :put },
-      { :controller => "changeset", :action => "update", :id => "1" }
-    )
-    assert_routing(
-      { :path => "/api/0.6/changeset/1/close", :method => :put },
-      { :controller => "changeset", :action => "close", :id => "1" }
+      { :path => "/api/0.6/changeset/1/subscribe", :method => :post },
+      { :controller => "changeset", :action => "subscribe", :id => "1" }
     )
     assert_routing(
-        { :path => "/api/0.6/changeset/1/subscribe", :method => :post },
-        { :controller => "changeset", :action => "subscribe", :id => "1" }
+      { :path => "/api/0.6/changeset/1/unsubscribe", :method => :post },
+      { :controller => "changeset", :action => "unsubscribe", :id => "1" }
     )
     assert_routing(
-        { :path => "/api/0.6/changeset/1/unsubscribe", :method => :post },
-        { :controller => "changeset", :action => "unsubscribe", :id => "1" }
+      { :path => "/api/0.6/changeset/1", :method => :put },
+      { :controller => "changeset", :action => "update", :id => "1" }
     )
     assert_routing(
-        { :path => "/api/0.6/changeset/comment/1/hide", :method => :post },
-        { :controller => "changeset", :action => "hide_comment", :id => "1" }
+      { :path => "/api/0.6/changeset/1/close", :method => :put },
+      { :controller => "changeset", :action => "close", :id => "1" }
     )
     assert_routing(
-        { :path => "/api/0.6/changeset/comment/1/unhide", :method => :post },
-        { :controller => "changeset", :action => "unhide_comment", :id => "1" }
+      { :path => "/api/0.6/changeset/1/comment", :method => :post },
+      { :controller => "changeset", :action => "comment", :id => "1" }
     )
     assert_routing(
-        { :path => "/api/0.6/changeset/comments/feed", :method => :get },
-        { :controller => "changeset", :action => "comments_feed", :format =>"rss" }
+      { :path => "/api/0.6/changeset/comment/1/hide", :method => :post },
+      { :controller => "changeset", :action => "hide_comment", :id => "1" }
     )
     assert_routing(
-        { :path => "/api/0.6/changeset/1/comments/feed", :method => :get },
-        { :controller => "changeset", :action => "comments_feed", :id => "1", :format =>"rss" }
+      { :path => "/api/0.6/changeset/comment/1/unhide", :method => :post },
+      { :controller => "changeset", :action => "unhide_comment", :id => "1" }
     )
     assert_routing(
       { :path => "/api/0.6/changesets", :method => :get },
       { :controller => "changeset", :action => "query" }
     )
+    assert_routing(
+      { :path => "/changeset/1/comments/feed", :method => :get },
+      { :controller => "changeset", :action => "comments_feed", :id => "1", :format =>"rss" }
+    )
     assert_routing(
       { :path => "/user/name/history", :method => :get },
       { :controller => "changeset", :action => "list", :display_name => "name" }
@@ -88,6 +88,10 @@ class ChangesetControllerTest < ActionController::TestCase
       { :path => "/history/feed", :method => :get },
       { :controller => "changeset", :action => "feed", :format => :atom }
     )
+    assert_routing(
+      { :path => "/history/comments/feed", :method => :get },
+      { :controller => "changeset", :action => "comments_feed", :format =>"rss" }
+    )
   end
 
   # -----------------------
@@ -177,14 +181,18 @@ class ChangesetControllerTest < ActionController::TestCase
   # document structure.
   def test_read
     changeset_id = changesets(:normal_user_first_change).id
-    get :read, :id => changeset_id, :format => :xml
+    get :read, :id => changeset_id
     assert_response :success, "cannot get first changeset"
 
     assert_select "osm[version=#{API_VERSION}][generator=\"OpenStreetMap server\"]", 1
     assert_select "osm>changeset[id=#{changeset_id}]", 1
     assert_select "osm>changeset>discussion", 0
 
-    get :read, :id => changeset_id, :format => :xml, :include_discussion => true
+    get :read, :id => changeset_id, :include_discussion => true
+    assert_response :success, "cannot get first changeset with comments"
+
+    assert_select "osm[version=#{API_VERSION}][generator=\"OpenStreetMap server\"]", 1
+    assert_select "osm>changeset[id=#{changeset_id}]", 1
     assert_select "osm>changeset>discussion", 1
   end
 
@@ -1414,8 +1422,7 @@ EOF
     end
 
     # get the bounding box back from the changeset
-    get :read, :id => changeset_id, :format => :xml
-
+    get :read, :id => changeset_id
     assert_response :success, "Couldn't read back changeset."
     assert_select "osm>changeset[min_lon=1.0]", 1
     assert_select "osm>changeset[max_lon=1.0]", 1
@@ -1430,7 +1437,7 @@ EOF
     end
 
     # get the bounding box back from the changeset
-    get :read, :id => changeset_id, :format => :xml
+    get :read, :id => changeset_id
     assert_response :success, "Couldn't read back changeset for the second time."
     assert_select "osm>changeset[min_lon=1.0]", 1
     assert_select "osm>changeset[max_lon=2.0]", 1
@@ -1446,7 +1453,7 @@ EOF
     end
 
     # get the bounding box back from the changeset
-    get :read, :id => changeset_id, :format => 'xml'
+    get :read, :id => changeset_id
     assert_response :success, "Couldn't read back changeset for the third time."
     # note that the 3.1 here is because of the bbox overexpansion
     assert_select "osm>changeset[min_lon=1.0]", 1
@@ -1535,11 +1542,11 @@ EOF
     basic_authorization "test@openstreetmap.org", "test"
     get :query, :user => users(:normal_user).id
     assert_response :success, "can't get changesets by user ID"
-    assert_changesets [1,3,6]
+    assert_changesets [1,3,6,8]
 
     get :query, :display_name => users(:normal_user).display_name
     assert_response :success, "can't get changesets by user name"
-    assert_changesets [1,3,6]
+    assert_changesets [1,3,6,8]
 
     # check that the correct error is given when we provide both UID and name
     get :query, :user => users(:normal_user).id, :display_name => users(:normal_user).display_name
@@ -1567,11 +1574,11 @@ EOF
 
     get :query, :closed => 'true'
     assert_response :success, "can't get changesets by closed-ness"
-    assert_changesets [3,5,6,7]
+    assert_changesets [3,5,6,7,8]
 
     get :query, :closed => 'true', :user => users(:normal_user).id
     assert_response :success, "can't get changesets by closed-ness and user"
-    assert_changesets [3,6]
+    assert_changesets [3,6,8]
 
     get :query, :closed => 'true', :user => users(:public_user).id
     assert_response :success, "can't get changesets by closed-ness and user"
@@ -1879,7 +1886,7 @@ EOF
     basic_authorization(users(:public_user).email, 'test')
 
     assert_difference('ChangesetComment.count') do
-      post :comment, { :id => changesets(:normal_user_closed_change).id, :text => 'This is a comment', :format => :xml }
+      post :comment, { :id => changesets(:normal_user_closed_change).id, :text => 'This is a comment' }
     end
     assert_response :success
   end
@@ -1925,7 +1932,7 @@ EOF
     changeset = changesets(:normal_user_closed_change)
 
     assert_difference('changeset.subscribers.count') do
-      post :subscribe, { :id => changeset.id, :format => :xml }
+      post :subscribe, { :id => changeset.id }
     end
     assert_response :success
   end
@@ -1933,33 +1940,32 @@ EOF
   ##
   # test subscribe fail
   def test_subscribe_fail
+    # unauthorized
     changeset = changesets(:normal_user_closed_change)
     assert_no_difference('changeset.subscribers.count') do
-      post :subscribe, { :id => changeset.id, :format => :xml }
+      post :subscribe, { :id => changeset.id }
     end
     assert_response :unauthorized
 
     basic_authorization(users(:public_user).email, 'test')
 
+    # bad changeset id
     assert_no_difference('changeset.subscribers.count') do
       post :subscribe, { :id => 999111 }
     end
     assert_response :not_found
 
+    # not closed changeset
     changeset = changesets(:normal_user_first_change)
     assert_no_difference('changeset.subscribers.count') do
       post :subscribe, { :id => changeset.id }
     end
     assert_response :conflict
 
-    # subscribing
-    changeset = changesets(:normal_user_closed_change)
-    post :subscribe, { :id => changeset.id, :format => :xml }
-    assert_response :success
-
-    # trying to subsrcirbe one more time
+    # trying to subscribe when already subscribed
+    changeset = changesets(:normal_user_subscribed_change)
     assert_no_difference('changeset.subscribers.count') do
-      post :subscribe, { :id => changeset.id, :format => :xml }
+      post :subscribe, { :id => changeset.id }
     end
     assert_response :conflict
   end
@@ -1968,11 +1974,10 @@ EOF
   # test unsubscribe success
   def test_unsubscribe_success
     basic_authorization(users(:public_user).email, 'test')
-    changeset = changesets(:normal_user_closed_change)
-    post :subscribe, { :id => changeset.id, :format => :xml }
-    # unsubscribe
+    changeset = changesets(:normal_user_subscribed_change)
+
     assert_difference('changeset.subscribers.count', -1) do
-      post :unsubscribe, { :id => changeset.id, :format => :xml }
+      post :unsubscribe, { :id => changeset.id }
     end
     assert_response :success
   end
@@ -1980,6 +1985,7 @@ EOF
   ##
   # test unsubscribe fail
   def test_unsubscribe_fail
+    # unauthorized
     changeset = changesets(:normal_user_closed_change)
     assert_no_difference('changeset.subscribers.count') do
       post :unsubscribe, { :id => changeset.id }
@@ -1988,11 +1994,13 @@ EOF
 
     basic_authorization(users(:public_user).email, 'test')
 
+    # bad changeset id
     assert_no_difference('changeset.subscribers.count', -1) do
       post :unsubscribe, { :id => 999111 }
     end
     assert_response :not_found
 
+    # not closed changeset
     changeset = changesets(:normal_user_first_change)
     assert_no_difference('changeset.subscribers.count', -1) do
       post :unsubscribe, { :id => changeset.id }
@@ -2010,23 +2018,25 @@ EOF
   ##
   # test hide comment fail
   def test_hide_comment_fail
+    # unauthorized
     comment = changeset_comments(:normal_comment_1)
     assert('comment.visible') do
-      post :hide_comment, { :id => comment.id, :format => "xml" }
+      post :hide_comment, { :id => comment.id }
       assert_response :unauthorized
     end
-    
 
     basic_authorization(users(:public_user).email, 'test')
 
+    # not a moderator
     assert('comment.visible') do
-      post :hide_comment, { :id => comment.id, :format => "xml" }
+      post :hide_comment, { :id => comment.id }
       assert_response :forbidden
     end
 
     basic_authorization(users(:moderator_user).email, 'test')
 
-    post :hide_comment, { :id => 999111, :format => "xml" }
+    # bad comment id
+    post :hide_comment, { :id => 999111 }
     assert_response :not_found
   end
 
@@ -2038,7 +2048,7 @@ EOF
     basic_authorization(users(:moderator_user).email, 'test')
 
     assert('!comment.visible') do
-      post :hide_comment, { :id => comment.id, :format => "xml" }
+      post :hide_comment, { :id => comment.id }
     end
     assert_response :success
   end
@@ -2046,23 +2056,25 @@ EOF
   ##
   # test unhide comment fail
   def test_unhide_comment_fail
+    # unauthorized
     comment = changeset_comments(:normal_comment_1)
     assert('comment.visible') do
-      post :unhide_comment, { :id => comment.id, :format => "xml" }
+      post :unhide_comment, { :id => comment.id }
       assert_response :unauthorized
     end
     
-
     basic_authorization(users(:public_user).email, 'test')
 
+    # not a moderator
     assert('comment.visible') do
-      post :unhide_comment, { :id => comment.id, :format => "xml" }
+      post :unhide_comment, { :id => comment.id }
       assert_response :forbidden
     end
 
     basic_authorization(users(:moderator_user).email, 'test')
 
-    post :unhide_comment, { :id => 999111, :format => "xml" }
+    # bad comment id
+    post :unhide_comment, { :id => 999111 }
     assert_response :not_found
   end
 
@@ -2074,7 +2086,7 @@ EOF
     basic_authorization(users(:moderator_user).email, 'test')
 
     assert('!comment.visible') do
-      post :unhide_comment, { :id => comment.id, :format => "xml" }
+      post :unhide_comment, { :id => comment.id }
     end
     assert_response :success
   end