From: Shaun McDonald <shaun@shaunmcdonald.me.uk>
Date: Wed, 6 May 2009 14:58:48 +0000 (+0000)
Subject: More trace tests
X-Git-Tag: live~7433
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/d10931effaabdbdd063029394981a26893a47db5?hp=30bbca0f3b05834dc9fd899e75e01b1994c0333b

More trace tests
---

diff --git a/test/fixtures/gpx_files.yml b/test/fixtures/gpx_files.yml
index 3ab74c806..ef0c004b4 100644
--- a/test/fixtures/gpx_files.yml
+++ b/test/fixtures/gpx_files.yml
@@ -1,4 +1,4 @@
-first_trace_file:
+public_trace_file:
   id: 1
   user_id: 1
   visible: true
@@ -10,3 +10,16 @@ first_trace_file:
   public: true
   description: This is a trace
   inserted: true
+  
+anon_trace_file:
+  id: 2
+  user_id: 2
+  visible: false
+  name: Private Trace.gpx
+  size: 123
+  latitude: 51.3
+  longitude: -0.56
+  timestamp: "2009-05-06 13:34:34"
+  public: false
+  description: This is an anonymous trace
+  inserted: true
diff --git a/test/functional/trace_controller_test.rb b/test/functional/trace_controller_test.rb
index 6b46dbced..475d3eb3e 100644
--- a/test/functional/trace_controller_test.rb
+++ b/test/functional/trace_controller_test.rb
@@ -1,8 +1,75 @@
 require File.dirname(__FILE__) + '/../test_helper'
 
 class TraceControllerTest < ActionController::TestCase
-  # Replace this with your real tests.
-  def test_truth
-    assert true
+  fixtures :users, :gpx_files
+  set_fixture_class :gpx_files => 'Trace'
+
+  
+  # Check that the list of changesets is displayed
+  def test_list
+    get :list
+    assert_response :success
+    assert_template 'list'
+  end
+  
+  # Check that the rss loads
+  def test_rss
+    get :georss
+    assert_rss_success
+    
+    get :georss, :display_name => users(:normal_user).display_name
+    assert_rss_success
+  end
+  
+  def assert_rss_success 
+    assert_response :success
+    assert_template nil
+    assert_equal "application/rss+xml", @response.content_type
+  end
+  
+  # Check getting a specific trace through the api
+  def test_api_details
+    # First with no auth
+    get :api_details, :id => gpx_files(:public_trace_file).id
+    assert_response :unauthorized
+    
+    # Now with some other user, which should work since the trace is public
+    basic_authorization(users(:public_user).display_name, "test")
+    get :api_details, :id => gpx_files(:public_trace_file).id
+    assert_response :success
+    
+    # And finally we should be able to do it with the owner of the trace
+    basic_authorization(users(:normal_user).display_name, "test")
+    get :api_details, :id => gpx_files(:public_trace_file).id
+    assert_response :success
+  end
+  
+  # Check an anoymous trace can't be specifically fetched by another user
+  def test_api_details_anon
+    # Furst with no auth
+    get :api_details, :id => gpx_files(:anon_trace_file).id
+    assert_response :unauthorized
+    
+    # Now try with another user, which shouldn't work since the trace is anon
+    basic_authorization(users(:normal_user).display_name, "test")
+    get :api_details, :id => gpx_files(:anon_trace_file).id
+    assert_response :forbidden
+    
+    # And finally we should be able to get the trace details with the trace owner
+    basic_authorization(users(:public_user).display_name, "test")
+    get :api_details, :id => gpx_files(:anon_trace_file).id
+    assert_response :success
+  end
+  
+  # Check the api details for a trace that doesn't exist
+  def test_api_details_not_found
+    # Try first with no auth, as it should requure it
+    get :api_details, :id => 0
+    assert_response :unauthorized
+    
+    # Login, and try again
+    basic_authorization(users(:public_user).display_name, "test")
+    get :api_details, :id => 0
+    assert_response :not_found
   end
 end