test/controllers/traces_controller_test.rb

   1 require "test_helper"
   2 require "digest"
   3 require "minitest/mock"
   4
   5 class TracesControllerTest < ActionController::TestCase
   6   def setup
   7     @gpx_trace_dir = Object.send("remove_const", "GPX_TRACE_DIR")
   8     Object.const_set("GPX_TRACE_DIR", Rails.root.join("test", "gpx", "traces"))
   9
  10     @gpx_image_dir = Object.send("remove_const", "GPX_IMAGE_DIR")
  11     Object.const_set("GPX_IMAGE_DIR", Rails.root.join("test", "gpx", "images"))
  12   end
  13
  14   def teardown
  15     File.unlink(*Dir.glob(File.join(GPX_TRACE_DIR, "*.gpx")))
  16     File.unlink(*Dir.glob(File.join(GPX_IMAGE_DIR, "*.gif")))
  17
  18     Object.send("remove_const", "GPX_TRACE_DIR")
  19     Object.const_set("GPX_TRACE_DIR", @gpx_trace_dir)
  20
  21     Object.send("remove_const", "GPX_IMAGE_DIR")
  22     Object.const_set("GPX_IMAGE_DIR", @gpx_image_dir)
  23   end
  24
  25   ##
  26   # test all routes which lead to this controller
  27   def test_routes
  28     assert_routing(
  29       { :path => "/api/0.6/gpx/create", :method => :post },
  30       { :controller => "traces", :action => "api_create" }
  31     )
  32     assert_routing(
  33       { :path => "/api/0.6/gpx/1", :method => :get },
  34       { :controller => "traces", :action => "api_read", :id => "1" }
  35     )
  36     assert_routing(
  37       { :path => "/api/0.6/gpx/1", :method => :put },
  38       { :controller => "traces", :action => "api_update", :id => "1" }
  39     )
  40     assert_routing(
  41       { :path => "/api/0.6/gpx/1", :method => :delete },
  42       { :controller => "traces", :action => "api_delete", :id => "1" }
  43     )
  44     assert_recognizes(
  45       { :controller => "traces", :action => "api_read", :id => "1" },
  46       { :path => "/api/0.6/gpx/1/details", :method => :get }
  47     )
  48     assert_routing(
  49       { :path => "/api/0.6/gpx/1/data", :method => :get },
  50       { :controller => "traces", :action => "api_data", :id => "1" }
  51     )
  52     assert_routing(
  53       { :path => "/api/0.6/gpx/1/data.xml", :method => :get },
  54       { :controller => "traces", :action => "api_data", :id => "1", :format => "xml" }
  55     )
  56
  57     assert_routing(
  58       { :path => "/traces", :method => :get },
  59       { :controller => "traces", :action => "list" }
  60     )
  61     assert_routing(
  62       { :path => "/traces/page/1", :method => :get },
  63       { :controller => "traces", :action => "list", :page => "1" }
  64     )
  65     assert_routing(
  66       { :path => "/traces/tag/tagname", :method => :get },
  67       { :controller => "traces", :action => "list", :tag => "tagname" }
  68     )
  69     assert_routing(
  70       { :path => "/traces/tag/tagname/page/1", :method => :get },
  71       { :controller => "traces", :action => "list", :tag => "tagname", :page => "1" }
  72     )
  73     assert_routing(
  74       { :path => "/user/username/traces", :method => :get },
  75       { :controller => "traces", :action => "list", :display_name => "username" }
  76     )
  77     assert_routing(
  78       { :path => "/user/username/traces/page/1", :method => :get },
  79       { :controller => "traces", :action => "list", :display_name => "username", :page => "1" }
  80     )
  81     assert_routing(
  82       { :path => "/user/username/traces/tag/tagname", :method => :get },
  83       { :controller => "traces", :action => "list", :display_name => "username", :tag => "tagname" }
  84     )
  85     assert_routing(
  86       { :path => "/user/username/traces/tag/tagname/page/1", :method => :get },
  87       { :controller => "traces", :action => "list", :display_name => "username", :tag => "tagname", :page => "1" }
  88     )
  89
  90     assert_routing(
  91       { :path => "/traces/mine", :method => :get },
  92       { :controller => "traces", :action => "mine" }
  93     )
  94     assert_routing(
  95       { :path => "/traces/mine/page/1", :method => :get },
  96       { :controller => "traces", :action => "mine", :page => "1" }
  97     )
  98     assert_routing(
  99       { :path => "/traces/mine/tag/tagname", :method => :get },
 100       { :controller => "traces", :action => "mine", :tag => "tagname" }
 101     )
 102     assert_routing(
 103       { :path => "/traces/mine/tag/tagname/page/1", :method => :get },
 104       { :controller => "traces", :action => "mine", :tag => "tagname", :page => "1" }
 105     )
 106
 107     assert_routing(
 108       { :path => "/traces/rss", :method => :get },
 109       { :controller => "traces", :action => "georss", :format => :rss }
 110     )
 111     assert_routing(
 112       { :path => "/traces/tag/tagname/rss", :method => :get },
 113       { :controller => "traces", :action => "georss", :tag => "tagname", :format => :rss }
 114     )
 115     assert_routing(
 116       { :path => "/user/username/traces/rss", :method => :get },
 117       { :controller => "traces", :action => "georss", :display_name => "username", :format => :rss }
 118     )
 119     assert_routing(
 120       { :path => "/user/username/traces/tag/tagname/rss", :method => :get },
 121       { :controller => "traces", :action => "georss", :display_name => "username", :tag => "tagname", :format => :rss }
 122     )
 123
 124     assert_routing(
 125       { :path => "/user/username/traces/1", :method => :get },
 126       { :controller => "traces", :action => "view", :display_name => "username", :id => "1" }
 127     )
 128     assert_routing(
 129       { :path => "/user/username/traces/1/picture", :method => :get },
 130       { :controller => "traces", :action => "picture", :display_name => "username", :id => "1" }
 131     )
 132     assert_routing(
 133       { :path => "/user/username/traces/1/icon", :method => :get },
 134       { :controller => "traces", :action => "icon", :display_name => "username", :id => "1" }
 135     )
 136
 137     assert_routing(
 138       { :path => "/traces/new", :method => :get },
 139       { :controller => "traces", :action => "new" }
 140     )
 141     assert_routing(
 142       { :path => "/traces", :method => :post },
 143       { :controller => "traces", :action => "create" }
 144     )
 145     assert_routing(
 146       { :path => "/trace/1/data", :method => :get },
 147       { :controller => "traces", :action => "data", :id => "1" }
 148     )
 149     assert_routing(
 150       { :path => "/trace/1/data.xml", :method => :get },
 151       { :controller => "traces", :action => "data", :id => "1", :format => "xml" }
 152     )
 153     assert_routing(
 154       { :path => "/trace/1/edit", :method => :get },
 155       { :controller => "traces", :action => "edit", :id => "1" }
 156     )
 157     assert_routing(
 158       { :path => "/trace/1/edit", :method => :post },
 159       { :controller => "traces", :action => "edit", :id => "1" }
 160     )
 161     assert_routing(
 162       { :path => "/trace/1/delete", :method => :post },
 163       { :controller => "traces", :action => "delete", :id => "1" }
 164     )
 165   end
 166
 167   # Check that the list of traces is displayed
 168   def test_list
 169     user = create(:user)
 170     # The fourth test below is surpisingly sensitive to timestamp ordering when the timestamps are equal.
 171     trace_a = create(:trace, :visibility => "public", :timestamp => 4.seconds.ago) do |trace|
 172       create(:tracetag, :trace => trace, :tag => "London")
 173     end
 174     trace_b = create(:trace, :visibility => "public", :timestamp => 3.seconds.ago) do |trace|
 175       create(:tracetag, :trace => trace, :tag => "Birmingham")
 176     end
 177     trace_c = create(:trace, :visibility => "private", :user => user, :timestamp => 2.seconds.ago) do |trace|
 178       create(:tracetag, :trace => trace, :tag => "London")
 179     end
 180     trace_d = create(:trace, :visibility => "private", :user => user, :timestamp => 1.second.ago) do |trace|
 181       create(:tracetag, :trace => trace, :tag => "Birmingham")
 182     end
 183
 184     # First with the public list
 185     get :list
 186     check_trace_list [trace_b, trace_a]
 187
 188     # Restrict traces to those with a given tag
 189     get :list, :params => { :tag => "London" }
 190     check_trace_list [trace_a]
 191
 192     # Should see more when we are logged in
 193     get :list, :session => { :user => user }
 194     check_trace_list [trace_d, trace_c, trace_b, trace_a]
 195
 196     # Again, we should see more when we are logged in
 197     get :list, :params => { :tag => "London" }, :session => { :user => user }
 198     check_trace_list [trace_c, trace_a]
 199   end
 200
 201   # Check that I can get mine
 202   def test_list_mine
 203     user = create(:user)
 204     create(:trace, :visibility => "public") do |trace|
 205       create(:tracetag, :trace => trace, :tag => "Birmingham")
 206     end
 207     trace_b = create(:trace, :visibility => "private", :user => user) do |trace|
 208       create(:tracetag, :trace => trace, :tag => "London")
 209     end
 210
 211     # First try to get it when not logged in
 212     get :mine
 213     assert_redirected_to :controller => "user", :action => "login", :referer => "/traces/mine"
 214
 215     # Now try when logged in
 216     get :mine, :session => { :user => user }
 217     assert_redirected_to :action => "list", :display_name => user.display_name
 218
 219     # Fetch the actual list
 220     get :list, :params => { :display_name => user.display_name }, :session => { :user => user }
 221     check_trace_list [trace_b]
 222   end
 223
 224   # Check the list of traces for a specific user
 225   def test_list_user
 226     user = create(:user)
 227     second_user = create(:user)
 228     third_user = create(:user)
 229     create(:trace)
 230     trace_b = create(:trace, :visibility => "public", :user => user)
 231     trace_c = create(:trace, :visibility => "private", :user => user) do |trace|
 232       create(:tracetag, :trace => trace, :tag => "London")
 233     end
 234
 235     # Test a user with no traces
 236     get :list, :params => { :display_name => second_user.display_name }
 237     check_trace_list []
 238
 239     # Test the user with the traces - should see only public ones
 240     get :list, :params => { :display_name => user.display_name }
 241     check_trace_list [trace_b]
 242
 243     # Should still see only public ones when authenticated as another user
 244     get :list, :params => { :display_name => user.display_name }, :session => { :user => third_user }
 245     check_trace_list [trace_b]
 246
 247     # Should see all traces when authenticated as the target user
 248     get :list, :params => { :display_name => user.display_name }, :session => { :user => user }
 249     check_trace_list [trace_c, trace_b]
 250
 251     # Should only see traces with the correct tag when a tag is specified
 252     get :list, :params => { :display_name => user.display_name, :tag => "London" }, :session => { :user => user }
 253     check_trace_list [trace_c]
 254
 255     # Should get an error if the user does not exist
 256     get :list, :params => { :display_name => "UnknownUser" }
 257     assert_response :not_found
 258     assert_template "user/no_such_user"
 259   end
 260
 261   # Check a multi-page list
 262   def test_list_paged
 263     # Create several pages worth of traces
 264     create_list(:trace, 50)
 265
 266     # Try and get the list
 267     get :list
 268     assert_response :success
 269     assert_select "table#trace_list tbody", :count => 1 do
 270       assert_select "tr", :count => 20
 271     end
 272
 273     # Try and get the second page
 274     get :list, :params => { :page => 2 }
 275     assert_response :success
 276     assert_select "table#trace_list tbody", :count => 1 do
 277       assert_select "tr", :count => 20
 278     end
 279   end
 280
 281   # Check that the rss loads
 282   def test_rss
 283     user = create(:user)
 284
 285     # First with the public feed
 286     get :georss, :params => { :format => :rss }
 287     check_trace_feed Trace.visible_to_all
 288
 289     # Restrict traces to those with a given tag
 290     get :georss, :params => { :tag => "London", :format => :rss }
 291     check_trace_feed Trace.tagged("London").visible_to_all
 292
 293     # Restrict traces to those for a given user
 294     get :georss, :params => { :display_name => user.display_name, :format => :rss }
 295     check_trace_feed user.traces.visible_to_all
 296
 297     # Restrict traces to those for a given user with a tiven tag
 298     get :georss, :params => { :display_name => user.display_name, :tag => "Birmingham", :format => :rss }
 299     check_trace_feed user.traces.tagged("Birmingham").visible_to_all
 300   end
 301
 302   # Test viewing a trace
 303   def test_view
 304     public_trace_file = create(:trace, :visibility => "public")
 305
 306     # First with no auth, which should work since the trace is public
 307     get :view, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 308     check_trace_view public_trace_file
 309
 310     # Now with some other user, which should work since the trace is public
 311     get :view, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 312     check_trace_view public_trace_file
 313
 314     # And finally we should be able to do it with the owner of the trace
 315     get :view, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 316     check_trace_view public_trace_file
 317   end
 318
 319   # Check an anonymous trace can't be viewed by another user
 320   def test_view_anon
 321     anon_trace_file = create(:trace, :visibility => "private")
 322
 323     # First with no auth
 324     get :view, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }
 325     assert_response :redirect
 326     assert_redirected_to :action => :list
 327
 328     # Now with some other user, which should not work since the trace is anon
 329     get :view, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => create(:user) }
 330     assert_response :redirect
 331     assert_redirected_to :action => :list
 332
 333     # And finally we should be able to do it with the owner of the trace
 334     get :view, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => anon_trace_file.user }
 335     check_trace_view anon_trace_file
 336   end
 337
 338   # Test viewing a trace that doesn't exist
 339   def test_view_not_found
 340     deleted_trace_file = create(:trace, :deleted)
 341
 342     # First with a trace that has never existed
 343     get :view, :params => { :display_name => create(:user).display_name, :id => 0 }
 344     assert_response :redirect
 345     assert_redirected_to :action => :list
 346
 347     # Now with a trace that has been deleted
 348     get :view, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 349     assert_response :redirect
 350     assert_redirected_to :action => :list
 351   end
 352
 353   # Test downloading a trace
 354   def test_data
 355     public_trace_file = create(:trace, :visibility => "public", :fixture => "a")
 356
 357     # First with no auth, which should work since the trace is public
 358     get :data, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 359     check_trace_data public_trace_file, "848caa72f2f456d1bd6a0fdf228aa1b9"
 360
 361     # Now with some other user, which should work since the trace is public
 362     get :data, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 363     check_trace_data public_trace_file, "848caa72f2f456d1bd6a0fdf228aa1b9"
 364
 365     # And finally we should be able to do it with the owner of the trace
 366     get :data, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 367     check_trace_data public_trace_file, "848caa72f2f456d1bd6a0fdf228aa1b9"
 368   end
 369
 370   # Test downloading a compressed trace
 371   def test_data_compressed
 372     identifiable_trace_file = create(:trace, :visibility => "identifiable", :fixture => "d")
 373
 374     # First get the data as is
 375     get :data, :params => { :display_name => identifiable_trace_file.user.display_name, :id => identifiable_trace_file.id }
 376     check_trace_data identifiable_trace_file, "c6422a3d8750faae49ed70e7e8a51b93", "application/x-gzip", "gpx.gz"
 377
 378     # Now ask explicitly for XML format
 379     get :data, :params => { :display_name => identifiable_trace_file.user.display_name, :id => identifiable_trace_file.id, :format => "xml" }
 380     check_trace_data identifiable_trace_file, "abd6675fdf3024a84fc0a1deac147c0d", "application/xml", "xml"
 381
 382     # Now ask explicitly for GPX format
 383     get :data, :params => { :display_name => identifiable_trace_file.user.display_name, :id => identifiable_trace_file.id, :format => "gpx" }
 384     check_trace_data identifiable_trace_file, "abd6675fdf3024a84fc0a1deac147c0d"
 385   end
 386
 387   # Check an anonymous trace can't be downloaded by another user
 388   def test_data_anon
 389     anon_trace_file = create(:trace, :visibility => "private", :fixture => "b")
 390
 391     # First with no auth
 392     get :data, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }
 393     assert_response :not_found
 394
 395     # Now with some other user, which shouldn't work since the trace is anon
 396     get :data, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => create(:user) }
 397     assert_response :not_found
 398
 399     # And finally we should be able to do it with the owner of the trace
 400     get :data, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => anon_trace_file.user }
 401     check_trace_data anon_trace_file, "66179ca44f1e93d8df62e2b88cbea732"
 402   end
 403
 404   # Test downloading a trace that doesn't exist
 405   def test_data_not_found
 406     deleted_trace_file = create(:trace, :deleted)
 407
 408     # First with a trace that has never existed
 409     get :data, :params => { :display_name => create(:user).display_name, :id => 0 }
 410     assert_response :not_found
 411
 412     # Now with a trace that has been deleted
 413     get :data, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 414     assert_response :not_found
 415   end
 416
 417   # Test downloading the picture for a trace
 418   def test_picture
 419     public_trace_file = create(:trace, :visibility => "public", :fixture => "a")
 420
 421     # First with no auth, which should work since the trace is public
 422     get :picture, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 423     check_trace_picture public_trace_file
 424
 425     # Now with some other user, which should work since the trace is public
 426     get :picture, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 427     check_trace_picture public_trace_file
 428
 429     # And finally we should be able to do it with the owner of the trace
 430     get :picture, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 431     check_trace_picture public_trace_file
 432   end
 433
 434   # Check the picture for an anonymous trace can't be downloaded by another user
 435   def test_picture_anon
 436     anon_trace_file = create(:trace, :visibility => "private", :fixture => "b")
 437
 438     # First with no auth
 439     get :picture, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }
 440     assert_response :forbidden
 441
 442     # Now with some other user, which shouldn't work since the trace is anon
 443     get :picture, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => create(:user) }
 444     assert_response :forbidden
 445
 446     # And finally we should be able to do it with the owner of the trace
 447     get :picture, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => anon_trace_file.user }
 448     check_trace_picture anon_trace_file
 449   end
 450
 451   # Test downloading the picture for a trace that doesn't exist
 452   def test_picture_not_found
 453     deleted_trace_file = create(:trace, :deleted)
 454
 455     # First with a trace that has never existed
 456     get :picture, :params => { :display_name => create(:user).display_name, :id => 0 }
 457     assert_response :not_found
 458
 459     # Now with a trace that has been deleted
 460     get :picture, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 461     assert_response :not_found
 462   end
 463
 464   # Test downloading the icon for a trace
 465   def test_icon
 466     public_trace_file = create(:trace, :visibility => "public", :fixture => "a")
 467
 468     # First with no auth, which should work since the trace is public
 469     get :icon, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 470     check_trace_icon public_trace_file
 471
 472     # Now with some other user, which should work since the trace is public
 473     get :icon, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 474     check_trace_icon public_trace_file
 475
 476     # And finally we should be able to do it with the owner of the trace
 477     get :icon, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 478     check_trace_icon public_trace_file
 479   end
 480
 481   # Check the icon for an anonymous trace can't be downloaded by another user
 482   def test_icon_anon
 483     anon_trace_file = create(:trace, :visibility => "private", :fixture => "b")
 484
 485     # First with no auth
 486     get :icon, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }
 487     assert_response :forbidden
 488
 489     # Now with some other user, which shouldn't work since the trace is anon
 490     get :icon, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => create(:user) }
 491     assert_response :forbidden
 492
 493     # And finally we should be able to do it with the owner of the trace
 494     get :icon, :params => { :display_name => anon_trace_file.user.display_name, :id => anon_trace_file.id }, :session => { :user => anon_trace_file.user }
 495     check_trace_icon anon_trace_file
 496   end
 497
 498   # Test downloading the icon for a trace that doesn't exist
 499   def test_icon_not_found
 500     deleted_trace_file = create(:trace, :deleted)
 501
 502     # First with a trace that has never existed
 503     get :icon, :params => { :display_name => create(:user).display_name, :id => 0 }
 504     assert_response :not_found
 505
 506     # Now with a trace that has been deleted
 507     get :icon, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 508     assert_response :not_found
 509   end
 510
 511   # Test fetching the new trace page
 512   def test_new_get
 513     # First with no auth
 514     get :new
 515     assert_response :redirect
 516     assert_redirected_to :controller => :user, :action => :login, :referer => new_trace_path
 517
 518     # Now authenticated as a user with gps.trace.visibility set
 519     user = create(:user)
 520     create(:user_preference, :user => user, :k => "gps.trace.visibility", :v => "identifiable")
 521     get :new, :session => { :user => user }
 522     assert_response :success
 523     assert_template :new
 524     assert_select "select#trace_visibility option[value=identifiable][selected]", 1
 525
 526     # Now authenticated as a user with gps.trace.public set
 527     second_user = create(:user)
 528     create(:user_preference, :user => second_user, :k => "gps.trace.public", :v => "default")
 529     get :new, :session => { :user => second_user }
 530     assert_response :success
 531     assert_template :new
 532     assert_select "select#trace_visibility option[value=public][selected]", 1
 533
 534     # Now authenticated as a user with no preferences
 535     third_user = create(:user)
 536     get :new, :session => { :user => third_user }
 537     assert_response :success
 538     assert_template :new
 539     assert_select "select#trace_visibility option[value=private][selected]", 1
 540   end
 541
 542   # Test creating a trace
 543   def test_create_post
 544     # Get file to use
 545     fixture = Rails.root.join("test", "gpx", "fixtures", "a.gpx")
 546     file = Rack::Test::UploadedFile.new(fixture, "application/gpx+xml")
 547     user = create(:user)
 548
 549     # First with no auth
 550     post :create, :params => { :trace => { :gpx_file => file, :description => "New Trace", :tagstring => "new,trace", :visibility => "trackable" } }
 551     assert_response :forbidden
 552
 553     # Rewind the file
 554     file.rewind
 555
 556     # Now authenticated
 557     create(:user_preference, :user => user, :k => "gps.trace.visibility", :v => "identifiable")
 558     assert_not_equal "trackable", user.preferences.where(:k => "gps.trace.visibility").first.v
 559     post :create, :params => { :trace => { :gpx_file => file, :description => "New Trace", :tagstring => "new,trace", :visibility => "trackable" } }, :session => { :user => user }
 560     assert_response :redirect
 561     assert_redirected_to :action => :list, :display_name => user.display_name
 562     assert_match /file has been uploaded/, flash[:notice]
 563     trace = Trace.order(:id => :desc).first
 564     assert_equal "a.gpx", trace.name
 565     assert_equal "New Trace", trace.description
 566     assert_equal %w[new trace], trace.tags.order(:tag).collect(&:tag)
 567     assert_equal "trackable", trace.visibility
 568     assert_equal false, trace.inserted
 569     assert_equal File.new(fixture).read, File.new(trace.trace_name).read
 570     trace.destroy
 571     assert_equal "trackable", user.preferences.where(:k => "gps.trace.visibility").first.v
 572   end
 573
 574   # Test creating a trace with validation errors
 575   def test_create_post_with_validation_errors
 576     # Get file to use
 577     fixture = Rails.root.join("test", "gpx", "fixtures", "a.gpx")
 578     file = Rack::Test::UploadedFile.new(fixture, "application/gpx+xml")
 579     user = create(:user)
 580
 581     # Now authenticated
 582     create(:user_preference, :user => user, :k => "gps.trace.visibility", :v => "identifiable")
 583     assert_not_equal "trackable", user.preferences.where(:k => "gps.trace.visibility").first.v
 584     post :create, :params => { :trace => { :gpx_file => file, :description => "", :tagstring => "new,trace", :visibility => "trackable" } }, :session => { :user => user }
 585     assert_template :new
 586     assert_match "Description is too short (minimum is 1 character)", response.body
 587   end
 588
 589   # Test fetching the edit page for a trace using GET
 590   def test_edit_get
 591     public_trace_file = create(:trace, :visibility => "public")
 592     deleted_trace_file = create(:trace, :deleted)
 593
 594     # First with no auth
 595     get :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 596     assert_response :redirect
 597     assert_redirected_to :controller => :user, :action => :login, :referer => trace_edit_path(:display_name => public_trace_file.user.display_name, :id => public_trace_file.id)
 598
 599     # Now with some other user, which should fail
 600     get :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 601     assert_response :forbidden
 602
 603     # Now with a trace which doesn't exist
 604     get :edit, :params => { :display_name => create(:user).display_name, :id => 0 }, :session => { :user => create(:user) }
 605     assert_response :not_found
 606
 607     # Now with a trace which has been deleted
 608     get :edit, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 609     assert_response :not_found
 610
 611     # Finally with a trace that we are allowed to edit
 612     get :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 613     assert_response :success
 614   end
 615
 616   # Test fetching the edit page for a trace using POST
 617   def test_edit_post_no_details
 618     public_trace_file = create(:trace, :visibility => "public")
 619     deleted_trace_file = create(:trace, :deleted)
 620
 621     # First with no auth
 622     post :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 623     assert_response :forbidden
 624
 625     # Now with some other user, which should fail
 626     post :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 627     assert_response :forbidden
 628
 629     # Now with a trace which doesn't exist
 630     post :edit, :params => { :display_name => create(:user).display_name, :id => 0 }, :session => { :user => create(:user) }
 631     assert_response :not_found
 632
 633     # Now with a trace which has been deleted
 634     post :edit, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 635     assert_response :not_found
 636
 637     # Finally with a trace that we are allowed to edit
 638     post :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 639     assert_response :success
 640   end
 641
 642   # Test saving edits to a trace
 643   def test_edit_post_with_details
 644     public_trace_file = create(:trace, :visibility => "public")
 645     deleted_trace_file = create(:trace, :deleted)
 646
 647     # New details
 648     new_details = { :description => "Changed description", :tagstring => "new_tag", :visibility => "private" }
 649
 650     # First with no auth
 651     post :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id, :trace => new_details }
 652     assert_response :forbidden
 653
 654     # Now with some other user, which should fail
 655     post :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id, :trace => new_details }, :session => { :user => create(:user) }
 656     assert_response :forbidden
 657
 658     # Now with a trace which doesn't exist
 659     post :edit, :params => { :display_name => create(:user).display_name, :id => 0 }, :session => { :user => create(:user), :trace => new_details }
 660     assert_response :not_found
 661
 662     # Now with a trace which has been deleted
 663     post :edit, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id, :trace => new_details }, :session => { :user => deleted_trace_file.user }
 664     assert_response :not_found
 665
 666     # Finally with a trace that we are allowed to edit
 667     post :edit, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id, :trace => new_details }, :session => { :user => public_trace_file.user }
 668     assert_response :redirect
 669     assert_redirected_to :action => :view, :display_name => public_trace_file.user.display_name
 670     trace = Trace.find(public_trace_file.id)
 671     assert_equal new_details[:description], trace.description
 672     assert_equal new_details[:tagstring], trace.tagstring
 673     assert_equal new_details[:visibility], trace.visibility
 674   end
 675
 676   # Test deleting a trace
 677   def test_delete
 678     public_trace_file = create(:trace, :visibility => "public")
 679     deleted_trace_file = create(:trace, :deleted)
 680
 681     # First with no auth
 682     post :delete, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }
 683     assert_response :forbidden
 684
 685     # Now with some other user, which should fail
 686     post :delete, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => create(:user) }
 687     assert_response :forbidden
 688
 689     # Now with a trace which doesn't exist
 690     post :delete, :params => { :display_name => create(:user).display_name, :id => 0 }, :session => { :user => create(:user) }
 691     assert_response :not_found
 692
 693     # Now with a trace has already been deleted
 694     post :delete, :params => { :display_name => deleted_trace_file.user.display_name, :id => deleted_trace_file.id }, :session => { :user => deleted_trace_file.user }
 695     assert_response :not_found
 696
 697     # Now with a trace that we are allowed to delete
 698     post :delete, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => public_trace_file.user }
 699     assert_response :redirect
 700     assert_redirected_to :action => :list, :display_name => public_trace_file.user.display_name
 701     trace = Trace.find(public_trace_file.id)
 702     assert_equal false, trace.visible
 703
 704     # Finally with a trace that is deleted by an admin
 705     public_trace_file = create(:trace, :visibility => "public")
 706     admin = create(:administrator_user)
 707
 708     post :delete, :params => { :display_name => public_trace_file.user.display_name, :id => public_trace_file.id }, :session => { :user => admin }
 709     assert_response :redirect
 710     assert_redirected_to :action => :list, :display_name => public_trace_file.user.display_name
 711     trace = Trace.find(public_trace_file.id)
 712     assert_equal false, trace.visible
 713   end
 714
 715   # Check getting a specific trace through the api
 716   def test_api_read
 717     public_trace_file = create(:trace, :visibility => "public")
 718
 719     # First with no auth
 720     get :api_read, :params => { :id => public_trace_file.id }
 721     assert_response :unauthorized
 722
 723     # Now with some other user, which should work since the trace is public
 724     basic_authorization create(:user).display_name, "test"
 725     get :api_read, :params => { :id => public_trace_file.id }
 726     assert_response :success
 727
 728     # And finally we should be able to do it with the owner of the trace
 729     basic_authorization public_trace_file.user.display_name, "test"
 730     get :api_read, :params => { :id => public_trace_file.id }
 731     assert_response :success
 732   end
 733
 734   # Check an anoymous trace can't be specifically fetched by another user
 735   def test_api_read_anon
 736     anon_trace_file = create(:trace, :visibility => "private")
 737
 738     # First with no auth
 739     get :api_read, :params => { :id => anon_trace_file.id }
 740     assert_response :unauthorized
 741
 742     # Now try with another user, which shouldn't work since the trace is anon
 743     basic_authorization create(:user).display_name, "test"
 744     get :api_read, :params => { :id => anon_trace_file.id }
 745     assert_response :forbidden
 746
 747     # And finally we should be able to get the trace details with the trace owner
 748     basic_authorization anon_trace_file.user.display_name, "test"
 749     get :api_read, :params => { :id => anon_trace_file.id }
 750     assert_response :success
 751   end
 752
 753   # Check the api details for a trace that doesn't exist
 754   def test_api_read_not_found
 755     deleted_trace_file = create(:trace, :deleted)
 756
 757     # Try first with no auth, as it should require it
 758     get :api_read, :params => { :id => 0 }
 759     assert_response :unauthorized
 760
 761     # Login, and try again
 762     basic_authorization deleted_trace_file.user.display_name, "test"
 763     get :api_read, :params => { :id => 0 }
 764     assert_response :not_found
 765
 766     # Now try a trace which did exist but has been deleted
 767     basic_authorization deleted_trace_file.user.display_name, "test"
 768     get :api_read, :params => { :id => deleted_trace_file.id }
 769     assert_response :not_found
 770   end
 771
 772   # Test downloading a trace through the api
 773   def test_api_data
 774     public_trace_file = create(:trace, :visibility => "public", :fixture => "a")
 775
 776     # First with no auth
 777     get :api_data, :params => { :id => public_trace_file.id }
 778     assert_response :unauthorized
 779
 780     # Now with some other user, which should work since the trace is public
 781     basic_authorization create(:user).display_name, "test"
 782     get :api_data, :params => { :id => public_trace_file.id }
 783     check_trace_data public_trace_file, "848caa72f2f456d1bd6a0fdf228aa1b9"
 784
 785     # And finally we should be able to do it with the owner of the trace
 786     basic_authorization public_trace_file.user.display_name, "test"
 787     get :api_data, :params => { :id => public_trace_file.id }
 788     check_trace_data public_trace_file, "848caa72f2f456d1bd6a0fdf228aa1b9"
 789   end
 790
 791   # Test downloading a compressed trace through the api
 792   def test_api_data_compressed
 793     identifiable_trace_file = create(:trace, :visibility => "identifiable", :fixture => "d")
 794
 795     # Authenticate as the owner of the trace we will be using
 796     basic_authorization identifiable_trace_file.user.display_name, "test"
 797
 798     # First get the data as is
 799     get :api_data, :params => { :id => identifiable_trace_file.id }
 800     check_trace_data identifiable_trace_file, "c6422a3d8750faae49ed70e7e8a51b93", "application/x-gzip", "gpx.gz"
 801
 802     # Now ask explicitly for XML format
 803     get :api_data, :params => { :id => identifiable_trace_file.id, :format => "xml" }
 804     check_trace_data identifiable_trace_file, "abd6675fdf3024a84fc0a1deac147c0d", "application/xml", "xml"
 805
 806     # Now ask explicitly for GPX format
 807     get :api_data, :params => { :id => identifiable_trace_file.id, :format => "gpx" }
 808     check_trace_data identifiable_trace_file, "abd6675fdf3024a84fc0a1deac147c0d"
 809   end
 810
 811   # Check an anonymous trace can't be downloaded by another user through the api
 812   def test_api_data_anon
 813     anon_trace_file = create(:trace, :visibility => "private", :fixture => "b")
 814
 815     # First with no auth
 816     get :api_data, :params => { :id => anon_trace_file.id }
 817     assert_response :unauthorized
 818
 819     # Now with some other user, which shouldn't work since the trace is anon
 820     basic_authorization create(:user).display_name, "test"
 821     get :api_data, :params => { :id => anon_trace_file.id }
 822     assert_response :forbidden
 823
 824     # And finally we should be able to do it with the owner of the trace
 825     basic_authorization anon_trace_file.user.display_name, "test"
 826     get :api_data, :params => { :id => anon_trace_file.id }
 827     check_trace_data anon_trace_file, "66179ca44f1e93d8df62e2b88cbea732"
 828   end
 829
 830   # Test downloading a trace that doesn't exist through the api
 831   def test_api_data_not_found
 832     deleted_trace_file = create(:trace, :deleted)
 833
 834     # Try first with no auth, as it should require it
 835     get :api_data, :params => { :id => 0 }
 836     assert_response :unauthorized
 837
 838     # Login, and try again
 839     basic_authorization create(:user).display_name, "test"
 840     get :api_data, :params => { :id => 0 }
 841     assert_response :not_found
 842
 843     # Now try a trace which did exist but has been deleted
 844     basic_authorization deleted_trace_file.user.display_name, "test"
 845     get :api_data, :params => { :id => deleted_trace_file.id }
 846     assert_response :not_found
 847   end
 848
 849   # Test creating a trace through the api
 850   def test_api_create
 851     # Get file to use
 852     fixture = Rails.root.join("test", "gpx", "fixtures", "a.gpx")
 853     file = Rack::Test::UploadedFile.new(fixture, "application/gpx+xml")
 854     user = create(:user)
 855
 856     # First with no auth
 857     post :api_create, :params => { :file => file, :description => "New Trace", :tags => "new,trace", :visibility => "trackable" }
 858     assert_response :unauthorized
 859
 860     # Rewind the file
 861     file.rewind
 862
 863     # Now authenticated
 864     create(:user_preference, :user => user, :k => "gps.trace.visibility", :v => "identifiable")
 865     assert_not_equal "trackable", user.preferences.where(:k => "gps.trace.visibility").first.v
 866     basic_authorization user.display_name, "test"
 867     post :api_create, :params => { :file => file, :description => "New Trace", :tags => "new,trace", :visibility => "trackable" }
 868     assert_response :success
 869     trace = Trace.find(response.body.to_i)
 870     assert_equal "a.gpx", trace.name
 871     assert_equal "New Trace", trace.description
 872     assert_equal %w[new trace], trace.tags.order(:tag).collect(&:tag)
 873     assert_equal "trackable", trace.visibility
 874     assert_equal false, trace.inserted
 875     assert_equal File.new(fixture).read, File.new(trace.trace_name).read
 876     trace.destroy
 877     assert_equal "trackable", user.preferences.where(:k => "gps.trace.visibility").first.v
 878
 879     # Rewind the file
 880     file.rewind
 881
 882     # Now authenticated, with the legacy public flag
 883     assert_not_equal "public", user.preferences.where(:k => "gps.trace.visibility").first.v
 884     basic_authorization user.display_name, "test"
 885     post :api_create, :params => { :file => file, :description => "New Trace", :tags => "new,trace", :public => 1 }
 886     assert_response :success
 887     trace = Trace.find(response.body.to_i)
 888     assert_equal "a.gpx", trace.name
 889     assert_equal "New Trace", trace.description
 890     assert_equal %w[new trace], trace.tags.order(:tag).collect(&:tag)
 891     assert_equal "public", trace.visibility
 892     assert_equal false, trace.inserted
 893     assert_equal File.new(fixture).read, File.new(trace.trace_name).read
 894     trace.destroy
 895     assert_equal "public", user.preferences.where(:k => "gps.trace.visibility").first.v
 896
 897     # Rewind the file
 898     file.rewind
 899
 900     # Now authenticated, with the legacy private flag
 901     second_user = create(:user)
 902     assert_nil second_user.preferences.where(:k => "gps.trace.visibility").first
 903     basic_authorization second_user.display_name, "test"
 904     post :api_create, :params => { :file => file, :description => "New Trace", :tags => "new,trace", :public => 0 }
 905     assert_response :success
 906     trace = Trace.find(response.body.to_i)
 907     assert_equal "a.gpx", trace.name
 908     assert_equal "New Trace", trace.description
 909     assert_equal %w[new trace], trace.tags.order(:tag).collect(&:tag)
 910     assert_equal "private", trace.visibility
 911     assert_equal false, trace.inserted
 912     assert_equal File.new(fixture).read, File.new(trace.trace_name).read
 913     trace.destroy
 914     assert_equal "private", second_user.preferences.where(:k => "gps.trace.visibility").first.v
 915   end
 916
 917   # Check updating a trace through the api
 918   def test_api_update
 919     public_trace_file = create(:trace, :visibility => "public", :fixture => "a")
 920     deleted_trace_file = create(:trace, :deleted)
 921     anon_trace_file = create(:trace, :visibility => "private")
 922
 923     # First with no auth
 924     content public_trace_file.to_xml
 925     put :api_update, :params => { :id => public_trace_file.id }
 926     assert_response :unauthorized
 927
 928     # Now with some other user, which should fail
 929     basic_authorization create(:user).display_name, "test"
 930     content public_trace_file.to_xml
 931     put :api_update, :params => { :id => public_trace_file.id }
 932     assert_response :forbidden
 933
 934     # Now with a trace which doesn't exist
 935     basic_authorization create(:user).display_name, "test"
 936     content public_trace_file.to_xml
 937     put :api_update, :params => { :id => 0 }
 938     assert_response :not_found
 939
 940     # Now with a trace which did exist but has been deleted
 941     basic_authorization deleted_trace_file.user.display_name, "test"
 942     content deleted_trace_file.to_xml
 943     put :api_update, :params => { :id => deleted_trace_file.id }
 944     assert_response :not_found
 945
 946     # Now try an update with the wrong ID
 947     basic_authorization public_trace_file.user.display_name, "test"
 948     content anon_trace_file.to_xml
 949     put :api_update, :params => { :id => public_trace_file.id }
 950     assert_response :bad_request,
 951                     "should not be able to update a trace with a different ID from the XML"
 952
 953     # And finally try an update that should work
 954     basic_authorization public_trace_file.user.display_name, "test"
 955     t = public_trace_file
 956     t.description = "Changed description"
 957     t.visibility = "private"
 958     content t.to_xml
 959     put :api_update, :params => { :id => t.id }
 960     assert_response :success
 961     nt = Trace.find(t.id)
 962     assert_equal nt.description, t.description
 963     assert_equal nt.visibility, t.visibility
 964   end
 965
 966   # Test that updating a trace doesn't duplicate the tags
 967   def test_api_update_tags
 968     tracetag = create(:tracetag)
 969     trace = tracetag.trace
 970     basic_authorization trace.user.display_name, "test"
 971
 972     content trace.to_xml
 973     put :api_update, :params => { :id => trace.id }
 974     assert_response :success
 975
 976     updated = Trace.find(trace.id)
 977     # Ensure there's only one tag in the database after updating
 978     assert_equal Tracetag.count, 1
 979     # The new tag object might have a different id, so check the string representation
 980     assert_equal trace.tagstring, updated.tagstring
 981   end
 982
 983   # Check deleting a trace through the api
 984   def test_api_delete
 985     public_trace_file = create(:trace, :visibility => "public")
 986
 987     # First with no auth
 988     delete :api_delete, :params => { :id => public_trace_file.id }
 989     assert_response :unauthorized
 990
 991     # Now with some other user, which should fail
 992     basic_authorization create(:user).display_name, "test"
 993     delete :api_delete, :params => { :id => public_trace_file.id }
 994     assert_response :forbidden
 995
 996     # Now with a trace which doesn't exist
 997     basic_authorization create(:user).display_name, "test"
 998     delete :api_delete, :params => { :id => 0 }
 999     assert_response :not_found
1000
1001     # And finally we should be able to do it with the owner of the trace
1002     basic_authorization public_trace_file.user.display_name, "test"
1003     delete :api_delete, :params => { :id => public_trace_file.id }
1004     assert_response :success
1005
1006     # Try it a second time, which should fail
1007     basic_authorization public_trace_file.user.display_name, "test"
1008     delete :api_delete, :params => { :id => public_trace_file.id }
1009     assert_response :not_found
1010   end
1011
1012   private
1013
1014   def check_trace_feed(traces)
1015     assert_response :success
1016     assert_template "georss"
1017     assert_equal "application/rss+xml", @response.content_type
1018     assert_select "rss", :count => 1 do
1019       assert_select "channel", :count => 1 do
1020         assert_select "title"
1021         assert_select "description"
1022         assert_select "link"
1023         assert_select "image"
1024         assert_select "item", :count => traces.visible.count do |items|
1025           traces.visible.order("timestamp DESC").zip(items).each do |trace, item|
1026             assert_select item, "title", trace.name
1027             assert_select item, "link", "http://test.host/user/#{trace.user.display_name}/traces/#{trace.id}"
1028             assert_select item, "guid", "http://test.host/user/#{trace.user.display_name}/traces/#{trace.id}"
1029             assert_select item, "description"
1030             # assert_select item, "dc:creator", trace.user.display_name
1031             assert_select item, "pubDate", trace.timestamp.rfc822
1032           end
1033         end
1034       end
1035     end
1036   end
1037
1038   def check_trace_list(traces)
1039     assert_response :success
1040     assert_template "list"
1041
1042     if !traces.empty?
1043       assert_select "table#trace_list tbody", :count => 1 do
1044         assert_select "tr", :count => traces.length do |rows|
1045           traces.zip(rows).each do |trace, row|
1046             assert_select row, "a", Regexp.new(Regexp.escape(trace.name))
1047             assert_select row, "span.trace_summary", Regexp.new(Regexp.escape("(#{trace.size} points)")) if trace.inserted?
1048             assert_select row, "td", Regexp.new(Regexp.escape(trace.description))
1049             assert_select row, "td", Regexp.new(Regexp.escape("by #{trace.user.display_name}"))
1050           end
1051         end
1052       end
1053     else
1054       assert_select "h4", /Nothing here yet/
1055     end
1056   end
1057
1058   def check_trace_view(trace)
1059     assert_response :success
1060     assert_template "view"
1061
1062     assert_select "table", :count => 1 do
1063       assert_select "td", /^#{Regexp.quote(trace.name)} /
1064       assert_select "td", trace.user.display_name
1065       assert_select "td", trace.description
1066     end
1067   end
1068
1069   def check_trace_data(trace, digest, content_type = "application/gpx+xml", extension = "gpx")
1070     assert_response :success
1071     assert_equal digest, Digest::MD5.hexdigest(response.body)
1072     assert_equal content_type, response.content_type
1073     assert_equal "attachment; filename=\"#{trace.id}.#{extension}\"", @response.header["Content-Disposition"]
1074   end
1075
1076   def check_trace_picture(trace)
1077     assert_response :success
1078     assert_equal "image/gif", response.content_type
1079     assert_equal trace.large_picture, response.body
1080   end
1081
1082   def check_trace_icon(trace)
1083     assert_response :success
1084     assert_equal "image/gif", response.content_type
1085     assert_equal trace.icon_picture, response.body
1086   end
1087 end