]> git.openstreetmap.org Git - rails.git/commitdiff
Create a fresh user when requesting non-existent traces.
authorAndy Allan <git@gravitystorm.co.uk>
Thu, 9 Mar 2017 14:50:02 +0000 (14:50 +0000)
committerAndy Allan <git@gravitystorm.co.uk>
Thu, 9 Mar 2017 14:50:02 +0000 (14:50 +0000)
test/controllers/trace_controller_test.rb

index 908e4c89b59f11ebb0fba77ee21662f9c4aac315..8390b6e67738e6d200e8c7915f0444f7de3cc900 100644 (file)
@@ -323,12 +323,12 @@ class TraceControllerTest < ActionController::TestCase
     deleted_trace_file = create(:trace, :deleted)
 
     # First with no auth
     deleted_trace_file = create(:trace, :deleted)
 
     # First with no auth
-    get :view, :display_name => deleted_trace_file.user.display_name, :id => 0
+    get :view, :display_name => create(:user).display_name, :id => 0
     assert_response :redirect
     assert_redirected_to :action => :list
 
     # Now with some other user
     assert_response :redirect
     assert_redirected_to :action => :list
 
     # Now with some other user
-    get :view, { :display_name => deleted_trace_file.user.display_name, :id => 0 }, { :user => create(:user) }
+    get :view, { :display_name => create(:user).display_name, :id => 0 }, { :user => create(:user) }
     assert_response :redirect
     assert_redirected_to :action => :list
 
     assert_response :redirect
     assert_redirected_to :action => :list
 
@@ -394,11 +394,11 @@ class TraceControllerTest < ActionController::TestCase
     deleted_trace_file = create(:trace, :deleted)
 
     # First with no auth and a trace that has never existed
     deleted_trace_file = create(:trace, :deleted)
 
     # First with no auth and a trace that has never existed
-    get :data, :display_name => deleted_trace_file.user.display_name, :id => 0
+    get :data, :display_name => create(:user).display_name, :id => 0
     assert_response :not_found
 
     # Now with a trace that has never existed
     assert_response :not_found
 
     # Now with a trace that has never existed
-    get :data, { :display_name => deleted_trace_file.user.display_name, :id => 0 }, { :user => deleted_trace_file.user }
+    get :data, { :display_name => create(:user).display_name, :id => 0 }, { :user => deleted_trace_file.user }
     assert_response :not_found
 
     # Now with a trace that has been deleted
     assert_response :not_found
 
     # Now with a trace that has been deleted