test/controllers/browse_controller_test.rb

   1 require "test_helper"
   2
   3 class BrowseControllerTest < ActionDispatch::IntegrationTest
   4   ##
   5   # test all routes which lead to this controller
   6   def test_routes
   7     assert_routing(
   8       { :path => "/node/1", :method => :get },
   9       { :controller => "browse", :action => "node", :id => "1" }
  10     )
  11     assert_routing(
  12       { :path => "/node/1/history", :method => :get },
  13       { :controller => "browse", :action => "node_history", :id => "1" }
  14     )
  15     assert_routing(
  16       { :path => "/way/1", :method => :get },
  17       { :controller => "browse", :action => "way", :id => "1" }
  18     )
  19     assert_routing(
  20       { :path => "/way/1/history", :method => :get },
  21       { :controller => "browse", :action => "way_history", :id => "1" }
  22     )
  23     assert_routing(
  24       { :path => "/relation/1", :method => :get },
  25       { :controller => "browse", :action => "relation", :id => "1" }
  26     )
  27     assert_routing(
  28       { :path => "/relation/1/history", :method => :get },
  29       { :controller => "browse", :action => "relation_history", :id => "1" }
  30     )
  31     assert_routing(
  32       { :path => "/changeset/1", :method => :get },
  33       { :controller => "browse", :action => "changeset", :id => "1" }
  34     )
  35     assert_routing(
  36       { :path => "/note/1", :method => :get },
  37       { :controller => "browse", :action => "note", :id => "1" }
  38     )
  39     assert_routing(
  40       { :path => "/note/new", :method => :get },
  41       { :controller => "browse", :action => "new_note" }
  42     )
  43     assert_routing(
  44       { :path => "/query", :method => :get },
  45       { :controller => "browse", :action => "query" }
  46     )
  47   end
  48
  49   def test_read_relation
  50     browse_check :relation_path, create(:relation).id, "browse/feature"
  51   end
  52
  53   def test_read_relation_history
  54     browse_check :relation_history_path, create(:relation, :with_history).id, "browse/history"
  55   end
  56
  57   def test_read_way
  58     browse_check :way_path, create(:way).id, "browse/feature"
  59   end
  60
  61   def test_read_way_history
  62     browse_check :way_history_path, create(:way, :with_history).id, "browse/history"
  63   end
  64
  65   def test_read_node
  66     node = create :node
  67     browse_check :node_path, node.id, "browse/feature"
  68     assert_select "a[href='#{api_node_path node}']", :count => 1
  69   end
  70
  71   def test_read_deleted_node
  72     node = create :node, :visible => false
  73     browse_check :node_path, node.id, "browse/feature"
  74     assert_select "a[href='#{api_node_path node}']", :count => 0
  75   end
  76
  77   def test_read_node_history
  78     browse_check :node_history_path, create(:node, :with_history).id, "browse/history"
  79   end
  80
  81   def test_read_changeset
  82     user = create(:user)
  83     changeset = create(:changeset, :user => user)
  84     create(:changeset, :user => user)
  85     browse_check :changeset_path, changeset.id, "browse/changeset"
  86   end
  87
  88   def test_read_private_changeset
  89     user = create(:user)
  90     changeset = create(:changeset, :user => create(:user, :data_public => false))
  91     create(:changeset, :user => user)
  92     browse_check :changeset_path, changeset.id, "browse/changeset"
  93   end
  94
  95   def test_read_changeset_hidden_comments
  96     changeset = create(:changeset)
  97     create_list(:changeset_comment, 3, :changeset => changeset)
  98     create(:changeset_comment, :visible => false, :changeset => changeset)
  99
 100     browse_check :changeset_path, changeset.id, "browse/changeset"
 101     assert_select "div.changeset-comments ul li", :count => 3
 102
 103     session_for(create(:moderator_user))
 104
 105     browse_check :changeset_path, changeset.id, "browse/changeset"
 106     assert_select "div.changeset-comments ul li", :count => 4
 107   end
 108
 109   def test_read_note
 110     open_note = create(:note_with_comments)
 111
 112     browse_check :browse_note_path, open_note.id, "browse/note"
 113   end
 114
 115   def test_read_hidden_note
 116     hidden_note_with_comment = create(:note_with_comments, :status => "hidden")
 117
 118     get browse_note_path(:id => hidden_note_with_comment)
 119     assert_response :not_found
 120     assert_template "browse/not_found"
 121     assert_template :layout => "map"
 122
 123     get browse_note_path(:id => hidden_note_with_comment), :xhr => true
 124     assert_response :not_found
 125     assert_template "browse/not_found"
 126     assert_template :layout => "xhr"
 127
 128     session_for(create(:moderator_user))
 129
 130     browse_check :browse_note_path, hidden_note_with_comment.id, "browse/note"
 131   end
 132
 133   def test_read_note_hidden_comments
 134     note_with_hidden_comment = create(:note_with_comments, :comments_count => 2) do |note|
 135       create(:note_comment, :note => note, :visible => false)
 136     end
 137
 138     browse_check :browse_note_path, note_with_hidden_comment.id, "browse/note"
 139     assert_select "div.note-comments ul li", :count => 1
 140
 141     session_for(create(:moderator_user))
 142
 143     browse_check :browse_note_path, note_with_hidden_comment.id, "browse/note"
 144     assert_select "div.note-comments ul li", :count => 2
 145   end
 146
 147   def test_read_note_hidden_user_comment
 148     hidden_user = create(:user, :deleted)
 149     note_with_hidden_user_comment = create(:note_with_comments, :comments_count => 2) do |note|
 150       create(:note_comment, :note => note, :author => hidden_user)
 151     end
 152
 153     browse_check :browse_note_path, note_with_hidden_user_comment.id, "browse/note"
 154     assert_select "div.note-comments ul li", :count => 1
 155
 156     session_for(create(:moderator_user))
 157
 158     browse_check :browse_note_path, note_with_hidden_user_comment.id, "browse/note"
 159     assert_select "div.note-comments ul li", :count => 1
 160   end
 161
 162   def test_read_closed_note
 163     user = create(:user)
 164     closed_note = create(:note_with_comments, :status => "closed", :closed_at => Time.now.utc, :comments_count => 2) do |note|
 165       create(:note_comment, :event => "closed", :note => note, :author => user)
 166     end
 167
 168     browse_check :browse_note_path, closed_note.id, "browse/note"
 169     assert_select "div.note-comments ul li", :count => 2
 170     assert_select "div.details", /Resolved by #{user.display_name}/
 171
 172     user.soft_destroy!
 173
 174     reset!
 175
 176     browse_check :browse_note_path, closed_note.id, "browse/note"
 177     assert_select "div.note-comments ul li", :count => 1
 178     assert_select "div.details", /Resolved by deleted/
 179   end
 180
 181   ##
 182   #  Methods to check redaction.
 183   #
 184   # note that these are presently highly reliant on the structure of the
 185   # page for the selection tests, which doesn't work out particularly
 186   # well if that structure changes. so... if you change the page layout
 187   # then please make it more easily (and robustly) testable!
 188   ##
 189   def test_redacted_node
 190     node = create(:node, :with_history, :deleted, :version => 2)
 191     node_v1 = node.old_nodes.find_by(:version => 1)
 192     node_v1.redact!(create(:redaction))
 193
 194     get node_path(:id => node)
 195     assert_response :success
 196     assert_template "feature"
 197
 198     # check that we don't show lat/lon for a redacted node.
 199     assert_select ".browse-section", 1
 200     assert_select ".browse-section.browse-node", 1
 201     assert_select ".browse-section.browse-node .latitude", 0
 202     assert_select ".browse-section.browse-node .longitude", 0
 203   end
 204
 205   def test_redacted_node_history
 206     node = create(:node, :with_history, :deleted, :version => 2)
 207     node_v1 = node.old_nodes.find_by(:version => 1)
 208     node_v1.redact!(create(:redaction))
 209
 210     get node_history_path(:id => node)
 211     assert_response :success
 212     assert_template "browse/history"
 213
 214     # there are 2 revisions of the redacted node, but only one
 215     # should be showing details here.
 216     assert_select ".browse-section", 2
 217     assert_select ".browse-section.browse-redacted", 1
 218     assert_select ".browse-section.browse-node", 1
 219     assert_select ".browse-section.browse-node .latitude", 0
 220     assert_select ".browse-section.browse-node .longitude", 0
 221   end
 222
 223   def test_redacted_way_history
 224     way = create(:way, :with_history, :version => 4)
 225     way_v1 = way.old_ways.find_by(:version => 1)
 226     way_v1.redact!(create(:redaction))
 227     way_v3 = way.old_ways.find_by(:version => 3)
 228     way_v3.redact!(create(:redaction))
 229
 230     get way_history_path(:id => way)
 231     assert_response :success
 232     assert_template "browse/history"
 233
 234     # there are 4 revisions of the redacted way, but only 2
 235     # should be showing details here.
 236     assert_select ".browse-section", 4
 237     assert_select ".browse-section.browse-redacted", 2
 238     assert_select ".browse-section.browse-way", 2
 239   end
 240
 241   def test_redacted_relation_history
 242     relation = create(:relation, :with_history, :version => 4)
 243     relation_v1 = relation.old_relations.find_by(:version => 1)
 244     relation_v1.redact!(create(:redaction))
 245     relation_v3 = relation.old_relations.find_by(:version => 3)
 246     relation_v3.redact!(create(:redaction))
 247
 248     get relation_history_path(:id => relation)
 249     assert_response :success
 250     assert_template "browse/history"
 251
 252     # there are 4 revisions of the redacted relation, but only 2
 253     # should be showing details here.
 254     assert_select ".browse-section", 4
 255     assert_select ".browse-section.browse-redacted", 2
 256     assert_select ".browse-section.browse-relation", 2
 257   end
 258
 259   def test_new_note
 260     get note_new_path
 261     assert_response :success
 262     assert_template "browse/new_note"
 263   end
 264
 265   def test_query
 266     get query_path
 267     assert_response :success
 268     assert_template "browse/query"
 269   end
 270
 271   private
 272
 273   # This is a convenience method for most of the above checks
 274   # First we check that when we don't have an id, it will correctly return a 404
 275   # then we check that we get the correct 404 when a non-existant id is passed
 276   # then we check that it will get a successful response, when we do pass an id
 277   def browse_check(path, id, template)
 278     path_method = method(path)
 279
 280     assert_raise ActionController::UrlGenerationError do
 281       get path_method.call
 282     end
 283
 284     assert_raise ActionController::UrlGenerationError do
 285       get path_method.call(:id => -10) # we won't have an id that's negative
 286     end
 287
 288     get path_method.call(:id => 0)
 289     assert_response :not_found
 290     assert_template "browse/not_found"
 291     assert_template :layout => "map"
 292
 293     get path_method.call(:id => 0), :xhr => true
 294     assert_response :not_found
 295     assert_template "browse/not_found"
 296     assert_template :layout => "xhr"
 297
 298     get path_method.call(:id => id)
 299     assert_response :success
 300     assert_template template
 301     assert_template :layout => "map"
 302
 303     get path_method.call(:id => id), :xhr => true
 304     assert_response :success
 305     assert_template template
 306     assert_template :layout => "xhr"
 307   end
 308 end