summaryrefslogtreecommitdiffstats
path: root/test/functional
diff options
context:
space:
mode:
authorJean-Philippe Lang <jp_lang@yahoo.fr>2012-03-03 15:09:20 +0000
committerJean-Philippe Lang <jp_lang@yahoo.fr>2012-03-03 15:09:20 +0000
commitb3866b05c14bd0f1fff4c54051f522e03e2bec36 (patch)
treec7248ef2b6cd4d22ec7bfaae11cf6815afdc8f64 /test/functional
parentbf8f8545461e5aeb1d2b1ddc1b4bf861ee5c2f9c (diff)
downloadredmine-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.rb14
-rw-r--r--test/functional/projects_controller_test.rb12
-rw-r--r--test/functional/users_controller_test.rb7
-rw-r--r--test/functional/watchers_controller_test.rb6
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