diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-03-03 15:09:20 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2012-03-03 15:09:20 +0000 |
commit | b3866b05c14bd0f1fff4c54051f522e03e2bec36 (patch) | |
tree | c7248ef2b6cd4d22ec7bfaae11cf6815afdc8f64 /test/functional | |
parent | bf8f8545461e5aeb1d2b1ddc1b4bf861ee5c2f9c (diff) | |
download | redmine-b3866b05c14bd0f1fff4c54051f522e03e2bec36.tar.gz redmine-b3866b05c14bd0f1fff4c54051f522e03e2bec36.zip |
Removes all #verify calls in controllers. Verification is handled at routing level now that the default route is removed.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@9061 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'test/functional')
-rw-r--r-- | test/functional/issues_controller_test.rb | 14 | ||||
-rw-r--r-- | test/functional/projects_controller_test.rb | 12 | ||||
-rw-r--r-- | test/functional/users_controller_test.rb | 7 | ||||
-rw-r--r-- | test/functional/watchers_controller_test.rb | 6 |
4 files changed, 0 insertions, 39 deletions
diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index 585ee2fa9..13cd3679d 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -2138,20 +2138,6 @@ class IssuesControllerTest < ActionController::TestCase assert_equal 'This is the test_new issue', issue.subject end - def test_update_using_invalid_http_verbs - @request.session[:user_id] = 2 - subject = 'Updated by an invalid http verb' - - get :update, :id => 1, :issue => {:subject => subject} - assert_not_equal subject, Issue.find(1).subject - - post :update, :id => 1, :issue => {:subject => subject} - assert_not_equal subject, Issue.find(1).subject - - delete :update, :id => 1, :issue => {:subject => subject} - assert_not_equal subject, Issue.find(1).subject - end - def test_put_update_without_custom_fields_param @request.session[:user_id] = 2 ActionMailer::Base.deliveries.clear diff --git a/test/functional/projects_controller_test.rb b/test/functional/projects_controller_test.rb index 6a788e6ec..9b7fca3c0 100644 --- a/test/functional/projects_controller_test.rb +++ b/test/functional/projects_controller_test.rb @@ -311,12 +311,6 @@ class ProjectsControllerTest < ActionController::TestCase end end - def test_create_should_not_accept_get - @request.session[:user_id] = 1 - get :create - assert_response :method_not_allowed - end - def test_show_by_id get :show, :id => 1 assert_response :success @@ -412,12 +406,6 @@ class ProjectsControllerTest < ActionController::TestCase assert_equal ['documents', 'issue_tracking', 'repository'], Project.find(1).enabled_module_names.sort end - def test_modules_should_not_allow_get - @request.session[:user_id] = 1 - get :modules, :id => 1 - assert_response :method_not_allowed - end - def test_destroy_without_confirmation @request.session[:user_id] = 1 # admin delete :destroy, :id => 1 diff --git a/test/functional/users_controller_test.rb b/test/functional/users_controller_test.rb index 3fc224d01..6adfc2757 100644 --- a/test/functional/users_controller_test.rb +++ b/test/functional/users_controller_test.rb @@ -289,13 +289,6 @@ class UsersControllerTest < ActionController::TestCase assert_nil User.find_by_id(2) end - def test_destroy_should_not_accept_get_requests - assert_no_difference 'User.count' do - get :destroy, :id => 2 - end - assert_response 405 - end - def test_destroy_should_be_denied_for_non_admin_users @request.session[:user_id] = 3 diff --git a/test/functional/watchers_controller_test.rb b/test/functional/watchers_controller_test.rb index c8345a833..bfd93fe0d 100644 --- a/test/functional/watchers_controller_test.rb +++ b/test/functional/watchers_controller_test.rb @@ -32,12 +32,6 @@ class WatchersControllerTest < ActionController::TestCase User.current = nil end - def test_get_watch_should_be_invalid - @request.session[:user_id] = 3 - get :watch, :object_type => 'issue', :object_id => '1' - assert_response 405 - end - def test_watch @request.session[:user_id] = 3 assert_difference('Watcher.count') do |