X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/f2d13c075660efb9582d7fb600bd6798d7a382ca..4cd2d22788cbc8acf411c9f9436eee85a4590c5f:/test/controllers/site_controller_test.rb diff --git a/test/controllers/site_controller_test.rb b/test/controllers/site_controller_test.rb index 86a92490a..681df7945 100644 --- a/test/controllers/site_controller_test.rb +++ b/test/controllers/site_controller_test.rb @@ -1,4 +1,4 @@ -require 'test_helper' +require "test_helper" class SiteControllerTest < ActionController::TestCase api_fixtures @@ -6,8 +6,10 @@ class SiteControllerTest < ActionController::TestCase ## # setup oauth keys def setup - Object.const_set("ID_KEY", client_applications(:oauth_web_app).key) - Object.const_set("POTLATCH2_KEY", client_applications(:oauth_web_app).key) + Object.const_set("ID_KEY", create(:client_application).key) + Object.const_set("POTLATCH2_KEY", create(:client_application).key) + + stub_hostip_requests end ## @@ -86,56 +88,71 @@ class SiteControllerTest < ActionController::TestCase def test_index get :index assert_response :success - assert_template 'index' + assert_template "index" end # Test the index page redirects def test_index_redirect + get :index, :node => 123 + assert_redirected_to :controller => :browse, :action => :node, :id => 123 + + get :index, :way => 123 + assert_redirected_to :controller => :browse, :action => :way, :id => 123 + + get :index, :relation => 123 + assert_redirected_to :controller => :browse, :action => :relation, :id => 123 + + get :index, :note => 123 + assert_redirected_to :controller => :browse, :action => :note, :id => 123 + + get :index, :query => "test" + assert_redirected_to :controller => :geocoder, :action => :search, :query => "test" + get :index, :lat => 4, :lon => 5 - assert_redirected_to :controller => :site, :action => :index, :anchor => 'map=5/4/5' + assert_redirected_to :controller => :site, :action => :index, :anchor => "map=5/4/5" get :index, :lat => 4, :lon => 5, :zoom => 3 - assert_redirected_to :controller => :site, :action => :index, :anchor => 'map=3/4/5' + assert_redirected_to :controller => :site, :action => :index, :anchor => "map=3/4/5" - get :index, :layers => 'T' - assert_redirected_to :controller => :site, :action => :index, :anchor => 'layers=T' + get :index, :layers => "T" + assert_redirected_to :controller => :site, :action => :index, :anchor => "layers=T" - get :index, :notes => 'yes' - assert_redirected_to :controller => :site, :action => :index, :anchor => 'layers=N' + get :index, :notes => "yes" + assert_redirected_to :controller => :site, :action => :index, :anchor => "layers=N" - get :index, :lat => 4, :lon => 5, :zoom => 3, :layers => 'T' - assert_redirected_to :controller => :site, :action => :index, :anchor => 'map=3/4/5&layers=T' + get :index, :lat => 4, :lon => 5, :zoom => 3, :layers => "T" + assert_redirected_to :controller => :site, :action => :index, :anchor => "map=3/4/5&layers=T" end # Test the permalink redirect def test_permalink - get :permalink, :code => 'wBz3--' + get :permalink, :code => "wBz3--" assert_response :redirect - assert_redirected_to :controller => :site, :action => :index, :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :site, :action => :index, :anchor => "map=3/4.8779296875/3.955078125" - get :permalink, :code => 'wBz3--', :m => '' + get :permalink, :code => "wBz3--", :m => "" assert_response :redirect - assert_redirected_to :controller => :site, :action => :index, :mlat => '4.8779296875', :mlon => '3.955078125', :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :site, :action => :index, :mlat => "4.8779296875", :mlon => "3.955078125", :anchor => "map=3/4.8779296875/3.955078125" - get :permalink, :code => 'wBz3--', :layers => 'T' + get :permalink, :code => "wBz3--", :layers => "T" assert_response :redirect - assert_redirected_to :controller => :site, :action => :index, :layers => 'T', :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :site, :action => :index, :anchor => "map=3/4.8779296875/3.955078125&layers=T" - get :permalink, :code => 'wBz3--', :node => 1 + get :permalink, :code => "wBz3--", :node => 1 assert_response :redirect - assert_redirected_to :controller => :browse, :action => :node, :id => 1, :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :browse, :action => :node, :id => 1, :anchor => "map=3/4.8779296875/3.955078125" - get :permalink, :code => 'wBz3--', :way => 2 + get :permalink, :code => "wBz3--", :way => 2 assert_response :redirect - assert_redirected_to :controller => :browse, :action => :way, :id => 2, :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :browse, :action => :way, :id => 2, :anchor => "map=3/4.8779296875/3.955078125" - get :permalink, :code => 'wBz3--', :relation => 3 + get :permalink, :code => "wBz3--", :relation => 3 assert_response :redirect - assert_redirected_to :controller => :browse, :action => :relation, :id => 3, :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :browse, :action => :relation, :id => 3, :anchor => "map=3/4.8779296875/3.955078125" - get :permalink, :code => 'wBz3--', :changeset => 4 + get :permalink, :code => "wBz3--", :changeset => 4 assert_response :redirect - assert_redirected_to :controller => :browse, :action => :changeset, :id => 4, :anchor => 'map=3/4.8779296875/3.955078125' + assert_redirected_to :controller => :browse, :action => :changeset, :id => 4, :anchor => "map=3/4.8779296875/3.955078125" end # Test the key page @@ -155,7 +172,7 @@ class SiteControllerTest < ActionController::TestCase # Test the right editor gets used when the user hasn't set a preference def test_edit_without_preference - get :edit, nil, { :user => users(:public_user).id } + get :edit, nil, :user => create(:user) assert_response :success assert_template "edit" assert_template :partial => "_#{DEFAULT_EDITOR}", :count => 1 @@ -163,48 +180,70 @@ class SiteControllerTest < ActionController::TestCase # Test the right editor gets used when the user has set a preference def test_edit_with_preference - user = users(:public_user) + user = create(:user) user.preferred_editor = "id" user.save! - get :edit, nil, { :user => user.id } + get :edit, nil, :user => user assert_response :success assert_template "edit" assert_template :partial => "_id", :count => 1 - user = users(:public_user) + user = create(:user) user.preferred_editor = "potlatch2" user.save! - get :edit, nil, { :user => user.id } + get :edit, nil, :user => user assert_response :success assert_template "edit" assert_template :partial => "_potlatch2", :count => 1 - user = users(:public_user) + user = create(:user) user.preferred_editor = "potlatch" user.save! - get :edit, nil, { :user => user.id } + get :edit, nil, :user => user assert_response :success assert_template "edit" assert_template :partial => "_potlatch", :count => 1 - user = users(:public_user) + user = create(:user) user.preferred_editor = "remote" user.save! - get :edit, nil, { :user => user.id } + get :edit, nil, :user => user + assert_response :success + assert_template "index" + end + + # Test the right editor gets used when the URL has an override + def test_edit_with_override + get :edit, { :editor => "id" }, { :user => create(:user) } + assert_response :success + assert_template "edit" + assert_template :partial => "_id", :count => 1 + + get :edit, { :editor => "potlatch2" }, { :user => create(:user) } + assert_response :success + assert_template "edit" + assert_template :partial => "_potlatch2", :count => 1 + + get :edit, { :editor => "potlatch" }, { :user => create(:user) } + assert_response :success + assert_template "edit" + assert_template :partial => "_potlatch", :count => 1 + + get :edit, { :editor => "remote" }, { :user => create(:user) } assert_response :success assert_template "index" end # Test editing a specific node def test_edit_with_node - user = users(:public_user) - node = current_nodes(:visible_node) + user = create(:user) + node = create(:node, :lat => 1.0, :lon => 1.0) - get :edit, { :node => node.id }, { :user => user.id } + get :edit, { :node => node.id }, { :user => user } assert_response :success assert_template "edit" assert_equal 1.0, assigns(:lat) @@ -214,10 +253,12 @@ class SiteControllerTest < ActionController::TestCase # Test editing a specific way def test_edit_with_way - user = users(:public_user) - way = current_ways(:visible_way) + user = create(:user) + node = create(:node, :lat => 3, :lon => 3) + way = create(:way) + create(:way_node, :node => node, :way => way) - get :edit, { :way => way.id }, { :user => user.id } + get :edit, { :way => way.id }, { :user => user } assert_response :success assert_template "edit" assert_equal 3.0, assigns(:lat) @@ -227,10 +268,12 @@ class SiteControllerTest < ActionController::TestCase # Test editing a specific note def test_edit_with_note - user = users(:public_user) - note = notes(:open_note) + user = create(:user) + note = create(:note) do |n| + n.comments.create(:author_id => user.id) + end - get :edit, { :note => note.id }, { :user => user.id } + get :edit, { :note => note.id }, { :user => user } assert_response :success assert_template "edit" assert_equal 1.0, assigns(:lat) @@ -240,10 +283,10 @@ class SiteControllerTest < ActionController::TestCase # Test editing a specific GPX trace def test_edit_with_gpx - user = users(:public_user) - gpx = gpx_files(:public_trace_file) + user = create(:user) + gpx = create(:trace, :latitude => 1, :longitude => 1) - get :edit, { :gpx => gpx.id }, { :user => user.id } + get :edit, { :gpx => gpx.id }, { :user => user } assert_response :success assert_template "edit" assert_equal 1.0, assigns(:lat) @@ -254,13 +297,13 @@ class SiteControllerTest < ActionController::TestCase # Test the edit page redirects def test_edit_redirect get :edit, :lat => 4, :lon => 5 - assert_redirected_to :controller => :site, :action => :edit, :anchor => 'map=5/4/5' + assert_redirected_to :controller => :site, :action => :edit, :anchor => "map=5/4/5" get :edit, :lat => 4, :lon => 5, :zoom => 3 - assert_redirected_to :controller => :site, :action => :edit, :anchor => 'map=3/4/5' + assert_redirected_to :controller => :site, :action => :edit, :anchor => "map=3/4/5" - get :edit, :lat => 4, :lon => 5, :zoom => 3, :editor => 'id' - assert_redirected_to :controller => :site, :action => :edit, :editor => 'id', :anchor => 'map=3/4/5' + get :edit, :lat => 4, :lon => 5, :zoom => 3, :editor => "id" + assert_redirected_to :controller => :site, :action => :edit, :editor => "id", :anchor => "map=3/4/5" end # Test the copyright page @@ -276,7 +319,7 @@ class SiteControllerTest < ActionController::TestCase assert_response :redirect assert_redirected_to :controller => :user, :action => :login, :referer => "/welcome" - get :welcome, nil, { :user => users(:public_user).id } + get :welcome, nil, :user => create(:user) assert_response :success assert_template "welcome" end @@ -336,7 +379,7 @@ class SiteControllerTest < ActionController::TestCase # Test the id frame def test_id - get :id, nil, { :user => users(:public_user).id } + get :id, nil, :user => create(:user) assert_response :success assert_template "id" assert_template :layout => false