diff options
author | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2011-07-03 11:01:08 +0000 |
---|---|---|
committer | Jean-Philippe Lang <jp_lang@yahoo.fr> | 2011-07-03 11:01:08 +0000 |
commit | 8914d323ee14c660c169ef143800343f87af33da (patch) | |
tree | 44ed9cc964f4b7888a84d7124fcf6d143f2db357 /test | |
parent | aede35d2268dbe25258344ffacf35487cc6e2ae7 (diff) | |
download | redmine-8914d323ee14c660c169ef143800343f87af33da.tar.gz redmine-8914d323ee14c660c169ef143800343f87af33da.zip |
Fixed: private queries should not be accessible to other users (#8729).
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@6163 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'test')
-rw-r--r-- | test/functional/issues_controller_test.rb | 27 |
1 files changed, 24 insertions, 3 deletions
diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb index eddb5493c..31e6ae11e 100644 --- a/test/functional/issues_controller_test.rb +++ b/test/functional/issues_controller_test.rb @@ -18,9 +18,6 @@ require File.expand_path('../../test_helper', __FILE__) require 'issues_controller' -# Re-raise errors caught by the controller. -class IssuesController; def rescue_action(e) raise e end; end - class IssuesControllerTest < ActionController::TestCase fixtures :projects, :users, @@ -193,6 +190,30 @@ class IssuesControllerTest < ActionController::TestCase assert_not_nil assigns(:issues) assert_not_nil assigns(:issue_count_by_group) end + + def test_private_query_should_not_be_available_to_other_users + q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil) + @request.session[:user_id] = 3 + + get :index, :query_id => q.id + assert_response 403 + end + + def test_private_query_should_be_available_to_its_user + q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil) + @request.session[:user_id] = 2 + + get :index, :query_id => q.id + assert_response :success + end + + def test_public_query_should_be_available_to_other_users + q = Query.create!(:name => "private", :user => User.find(2), :is_public => true, :project => nil) + @request.session[:user_id] = 3 + + get :index, :query_id => q.id + assert_response :success + end def test_index_sort_by_field_not_included_in_columns Setting.issue_list_default_columns = %w(subject author) |