Chris@0: # Redmine - project management software Chris@441: # Copyright (C) 2006-2011 Jean-Philippe Lang Chris@0: # Chris@0: # This program is free software; you can redistribute it and/or Chris@0: # modify it under the terms of the GNU General Public License Chris@0: # as published by the Free Software Foundation; either version 2 Chris@0: # of the License, or (at your option) any later version. Chris@441: # Chris@0: # This program is distributed in the hope that it will be useful, Chris@0: # but WITHOUT ANY WARRANTY; without even the implied warranty of Chris@0: # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the Chris@0: # GNU General Public License for more details. Chris@441: # Chris@0: # You should have received a copy of the GNU General Public License Chris@0: # along with this program; if not, write to the Free Software Chris@0: # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. Chris@0: Chris@119: require File.expand_path('../../test_helper', __FILE__) Chris@0: require 'issues_controller' Chris@0: Chris@0: class IssuesControllerTest < ActionController::TestCase Chris@0: fixtures :projects, Chris@0: :users, Chris@0: :roles, Chris@0: :members, Chris@0: :member_roles, Chris@0: :issues, Chris@0: :issue_statuses, Chris@0: :versions, Chris@0: :trackers, Chris@0: :projects_trackers, Chris@0: :issue_categories, Chris@0: :enabled_modules, Chris@0: :enumerations, Chris@0: :attachments, Chris@0: :workflows, Chris@0: :custom_fields, Chris@0: :custom_values, Chris@0: :custom_fields_projects, Chris@0: :custom_fields_trackers, Chris@0: :time_entries, Chris@0: :journals, Chris@0: :journal_details, Chris@0: :queries Chris@441: Chris@909: include Redmine::I18n Chris@909: Chris@0: def setup Chris@0: @controller = IssuesController.new Chris@0: @request = ActionController::TestRequest.new Chris@0: @response = ActionController::TestResponse.new Chris@0: User.current = nil Chris@0: end Chris@441: Chris@0: def test_index Chris@0: Setting.default_language = 'en' Chris@441: Chris@0: get :index Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_nil assigns(:project) Chris@0: assert_tag :tag => 'a', :content => /Can't print recipes/ Chris@0: assert_tag :tag => 'a', :content => /Subproject issue/ Chris@0: # private projects hidden Chris@0: assert_no_tag :tag => 'a', :content => /Issue of a private subproject/ Chris@0: assert_no_tag :tag => 'a', :content => /Issue on project 2/ Chris@0: # project column Chris@0: assert_tag :tag => 'th', :content => /Project/ Chris@0: end Chris@441: Chris@0: def test_index_should_not_list_issues_when_module_disabled Chris@0: EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1") Chris@0: get :index Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_nil assigns(:project) Chris@0: assert_no_tag :tag => 'a', :content => /Can't print recipes/ Chris@0: assert_tag :tag => 'a', :content => /Subproject issue/ Chris@0: end Chris@441: Chris@441: def test_index_should_list_visible_issues_only Chris@441: get :index, :per_page => 100 Chris@441: assert_response :success Chris@441: assert_not_nil assigns(:issues) Chris@441: assert_nil assigns(:issues).detect {|issue| !issue.visible?} Chris@441: end Chris@441: Chris@0: def test_index_with_project Chris@0: Setting.display_subprojects_issues = 0 Chris@0: get :index, :project_id => 1 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_tag :tag => 'a', :content => /Can't print recipes/ Chris@0: assert_no_tag :tag => 'a', :content => /Subproject issue/ Chris@0: end Chris@441: Chris@0: def test_index_with_project_and_subprojects Chris@0: Setting.display_subprojects_issues = 1 Chris@0: get :index, :project_id => 1 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_tag :tag => 'a', :content => /Can't print recipes/ Chris@0: assert_tag :tag => 'a', :content => /Subproject issue/ Chris@0: assert_no_tag :tag => 'a', :content => /Issue of a private subproject/ Chris@0: end Chris@441: Chris@0: def test_index_with_project_and_subprojects_should_show_private_subprojects Chris@0: @request.session[:user_id] = 2 Chris@0: Setting.display_subprojects_issues = 1 Chris@0: get :index, :project_id => 1 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_tag :tag => 'a', :content => /Can't print recipes/ Chris@0: assert_tag :tag => 'a', :content => /Subproject issue/ Chris@0: assert_tag :tag => 'a', :content => /Issue of a private subproject/ Chris@0: end Chris@441: chris@37: def test_index_with_project_and_default_filter Chris@0: get :index, :project_id => 1, :set_filter => 1 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@441: chris@37: query = assigns(:query) chris@37: assert_not_nil query chris@37: # default filter chris@37: assert_equal({'status_id' => {:operator => 'o', :values => ['']}}, query.filters) chris@37: end Chris@441: chris@37: def test_index_with_project_and_filter Chris@441: get :index, :project_id => 1, :set_filter => 1, Chris@441: :f => ['tracker_id'], Chris@441: :op => {'tracker_id' => '='}, Chris@441: :v => {'tracker_id' => ['1']} chris@37: assert_response :success Chris@909: assert_template 'index' chris@37: assert_not_nil assigns(:issues) Chris@441: chris@37: query = assigns(:query) chris@37: assert_not_nil query chris@37: assert_equal({'tracker_id' => {:operator => '=', :values => ['1']}}, query.filters) chris@37: end Chris@441: Chris@909: def test_index_with_short_filters Chris@909: to_test = { Chris@909: 'status_id' => { Chris@909: 'o' => { :op => 'o', :values => [''] }, Chris@909: 'c' => { :op => 'c', :values => [''] }, Chris@909: '7' => { :op => '=', :values => ['7'] }, Chris@909: '7|3|4' => { :op => '=', :values => ['7', '3', '4'] }, Chris@909: '=7' => { :op => '=', :values => ['7'] }, Chris@909: '!3' => { :op => '!', :values => ['3'] }, Chris@909: '!7|3|4' => { :op => '!', :values => ['7', '3', '4'] }}, Chris@909: 'subject' => { Chris@909: 'This is a subject' => { :op => '=', :values => ['This is a subject'] }, Chris@909: 'o' => { :op => '=', :values => ['o'] }, Chris@909: '~This is part of a subject' => { :op => '~', :values => ['This is part of a subject'] }, Chris@909: '!~This is part of a subject' => { :op => '!~', :values => ['This is part of a subject'] }}, Chris@909: 'tracker_id' => { Chris@909: '3' => { :op => '=', :values => ['3'] }, Chris@909: '=3' => { :op => '=', :values => ['3'] }}, Chris@909: 'start_date' => { Chris@909: '2011-10-12' => { :op => '=', :values => ['2011-10-12'] }, Chris@909: '=2011-10-12' => { :op => '=', :values => ['2011-10-12'] }, Chris@909: '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] }, Chris@909: '<=2011-10-12' => { :op => '<=', :values => ['2011-10-12'] }, Chris@909: '><2011-10-01|2011-10-30' => { :op => '><', :values => ['2011-10-01', '2011-10-30'] }, Chris@909: ' { :op => ' ['2'] }, Chris@909: '>t+2' => { :op => '>t+', :values => ['2'] }, Chris@909: 't+2' => { :op => 't+', :values => ['2'] }, Chris@909: 't' => { :op => 't', :values => [''] }, Chris@909: 'w' => { :op => 'w', :values => [''] }, Chris@909: '>t-2' => { :op => '>t-', :values => ['2'] }, Chris@909: ' { :op => ' ['2'] }, Chris@909: 't-2' => { :op => 't-', :values => ['2'] }}, Chris@909: 'created_on' => { Chris@909: '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] }, Chris@929: ' { :op => ' ['2'] }, Chris@929: '>t-2' => { :op => '>t-', :values => ['2'] }, Chris@929: 't-2' => { :op => 't-', :values => ['2'] }}, Chris@909: 'cf_1' => { Chris@909: 'c' => { :op => '=', :values => ['c'] }, Chris@909: '!c' => { :op => '!', :values => ['c'] }, Chris@909: '!*' => { :op => '!*', :values => [''] }, Chris@909: '*' => { :op => '*', :values => [''] }}, Chris@909: 'estimated_hours' => { Chris@909: '=13.4' => { :op => '=', :values => ['13.4'] }, Chris@909: '>=45' => { :op => '>=', :values => ['45'] }, Chris@909: '<=125' => { :op => '<=', :values => ['125'] }, Chris@909: '><10.5|20.5' => { :op => '><', :values => ['10.5', '20.5'] }, Chris@909: '!*' => { :op => '!*', :values => [''] }, Chris@909: '*' => { :op => '*', :values => [''] }} Chris@909: } Chris@909: Chris@909: default_filter = { 'status_id' => {:operator => 'o', :values => [''] }} Chris@909: Chris@909: to_test.each do |field, expression_and_expected| Chris@909: expression_and_expected.each do |filter_expression, expected| Chris@909: Chris@909: get :index, :set_filter => 1, field => filter_expression Chris@909: Chris@909: assert_response :success Chris@909: assert_template 'index' Chris@909: assert_not_nil assigns(:issues) Chris@909: Chris@909: query = assigns(:query) Chris@909: assert_not_nil query Chris@909: assert query.has_filter?(field) Chris@909: assert_equal(default_filter.merge({field => {:operator => expected[:op], :values => expected[:values]}}), query.filters) Chris@909: end Chris@909: end Chris@909: end Chris@909: chris@37: def test_index_with_project_and_empty_filters chris@37: get :index, :project_id => 1, :set_filter => 1, :fields => [''] chris@37: assert_response :success Chris@909: assert_template 'index' chris@37: assert_not_nil assigns(:issues) Chris@441: chris@37: query = assigns(:query) chris@37: assert_not_nil query chris@37: # no filter chris@37: assert_equal({}, query.filters) Chris@0: end Chris@441: Chris@0: def test_index_with_query Chris@0: get :index, :project_id => 1, :query_id => 5 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_nil assigns(:issue_count_by_group) Chris@0: end Chris@441: Chris@0: def test_index_with_query_grouped_by_tracker Chris@0: get :index, :project_id => 1, :query_id => 6 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_not_nil assigns(:issue_count_by_group) Chris@0: end Chris@441: Chris@0: def test_index_with_query_grouped_by_list_custom_field Chris@0: get :index, :project_id => 1, :query_id => 9 Chris@0: assert_response :success Chris@909: assert_template 'index' Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_not_nil assigns(:issue_count_by_group) Chris@0: end Chris@909: Chris@909: def test_index_with_query_id_and_project_id_should_set_session_query Chris@909: get :index, :project_id => 1, :query_id => 4 Chris@909: assert_response :success Chris@909: assert_kind_of Hash, session[:query] Chris@909: assert_equal 4, session[:query][:id] Chris@909: assert_equal 1, session[:query][:project_id] Chris@909: end Chris@909: Chris@909: def test_index_with_cross_project_query_in_session_should_show_project_issues Chris@909: q = Query.create!(:name => "test", :user_id => 2, :is_public => false, :project => nil) Chris@909: @request.session[:query] = {:id => q.id, :project_id => 1} Chris@909: Chris@909: with_settings :display_subprojects_issues => '0' do Chris@909: get :index, :project_id => 1 Chris@909: end Chris@909: assert_response :success Chris@909: assert_not_nil assigns(:query) Chris@909: assert_equal q.id, assigns(:query).id Chris@909: assert_equal 1, assigns(:query).project_id Chris@909: assert_equal [1], assigns(:issues).map(&:project_id).uniq Chris@909: end Chris@909: Chris@507: def test_private_query_should_not_be_available_to_other_users Chris@507: q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil) Chris@507: @request.session[:user_id] = 3 Chris@909: Chris@507: get :index, :query_id => q.id Chris@507: assert_response 403 Chris@507: end Chris@909: Chris@507: def test_private_query_should_be_available_to_its_user Chris@507: q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil) Chris@507: @request.session[:user_id] = 2 Chris@909: Chris@507: get :index, :query_id => q.id Chris@507: assert_response :success Chris@507: end Chris@441: Chris@909: def test_public_query_should_be_available_to_other_users Chris@909: q = Query.create!(:name => "private", :user => User.find(2), :is_public => true, :project => nil) Chris@909: @request.session[:user_id] = 3 Chris@909: Chris@909: get :index, :query_id => q.id Chris@909: assert_response :success Chris@0: end Chris@441: Chris@909: def test_index_csv Chris@0: get :index, :format => 'csv' Chris@0: assert_response :success Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_equal 'text/csv', @response.content_type Chris@0: assert @response.body.starts_with?("#,") Chris@909: lines = @response.body.chomp.split("\n") Chris@909: assert_equal assigns(:query).columns.size + 1, lines[0].split(',').size Chris@909: end Chris@0: Chris@909: def test_index_csv_with_project Chris@0: get :index, :project_id => 1, :format => 'csv' Chris@0: assert_response :success Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_equal 'text/csv', @response.content_type Chris@0: end Chris@441: Chris@909: def test_index_csv_with_description Chris@909: get :index, :format => 'csv', :description => '1' Chris@0: assert_response :success Chris@0: assert_not_nil assigns(:issues) Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: assert @response.body.starts_with?("#,") Chris@909: lines = @response.body.chomp.split("\n") Chris@909: assert_equal assigns(:query).columns.size + 2, lines[0].split(',').size Chris@909: end Chris@441: Chris@909: def test_index_csv_with_all_columns Chris@909: get :index, :format => 'csv', :columns => 'all' Chris@0: assert_response :success Chris@0: assert_not_nil assigns(:issues) Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: assert @response.body.starts_with?("#,") Chris@909: lines = @response.body.chomp.split("\n") Chris@909: assert_equal assigns(:query).available_columns.size + 1, lines[0].split(',').size Chris@909: end Chris@441: Chris@909: def test_index_csv_big_5 Chris@909: with_settings :default_language => "zh-TW" do Chris@909: str_utf8 = "\xe4\xb8\x80\xe6\x9c\x88" Chris@909: str_big5 = "\xa4@\xa4\xeb" Chris@909: if str_utf8.respond_to?(:force_encoding) Chris@909: str_utf8.force_encoding('UTF-8') Chris@909: str_big5.force_encoding('Big5') Chris@909: end Chris@909: issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3, Chris@909: :status_id => 1, :priority => IssuePriority.all.first, Chris@909: :subject => str_utf8) Chris@909: assert issue.save Chris@909: Chris@909: get :index, :project_id => 1, Chris@909: :f => ['subject'], Chris@909: :op => '=', :values => [str_utf8], Chris@909: :format => 'csv' Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: lines = @response.body.chomp.split("\n") Chris@909: s1 = "\xaa\xac\xbaA" Chris@909: if str_utf8.respond_to?(:force_encoding) Chris@909: s1.force_encoding('Big5') Chris@909: end Chris@909: assert lines[0].include?(s1) Chris@909: assert lines[1].include?(str_big5) Chris@909: end Chris@909: end Chris@909: Chris@909: def test_index_csv_cannot_convert_should_be_replaced_big_5 Chris@909: with_settings :default_language => "zh-TW" do Chris@909: str_utf8 = "\xe4\xbb\xa5\xe5\x86\x85" Chris@909: if str_utf8.respond_to?(:force_encoding) Chris@909: str_utf8.force_encoding('UTF-8') Chris@909: end Chris@909: issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3, Chris@909: :status_id => 1, :priority => IssuePriority.all.first, Chris@909: :subject => str_utf8) Chris@909: assert issue.save Chris@909: Chris@909: get :index, :project_id => 1, Chris@909: :f => ['subject'], Chris@909: :op => '=', :values => [str_utf8], Chris@909: :c => ['status', 'subject'], Chris@909: :format => 'csv', Chris@909: :set_filter => 1 Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: lines = @response.body.chomp.split("\n") Chris@909: s1 = "\xaa\xac\xbaA" # status Chris@909: if str_utf8.respond_to?(:force_encoding) Chris@909: s1.force_encoding('Big5') Chris@909: end Chris@909: assert lines[0].include?(s1) Chris@909: s2 = lines[1].split(",")[2] Chris@909: if s1.respond_to?(:force_encoding) Chris@909: s3 = "\xa5H?" # subject Chris@909: s3.force_encoding('Big5') Chris@909: assert_equal s3, s2 Chris@909: elsif RUBY_PLATFORM == 'java' Chris@909: assert_equal "??", s2 Chris@909: else Chris@909: assert_equal "\xa5H???", s2 Chris@909: end Chris@909: end Chris@909: end Chris@909: Chris@909: def test_index_csv_tw Chris@909: with_settings :default_language => "zh-TW" do Chris@909: str1 = "test_index_csv_tw" Chris@909: issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3, Chris@909: :status_id => 1, :priority => IssuePriority.all.first, Chris@909: :subject => str1, :estimated_hours => '1234.5') Chris@909: assert issue.save Chris@909: assert_equal 1234.5, issue.estimated_hours Chris@909: Chris@909: get :index, :project_id => 1, Chris@909: :f => ['subject'], Chris@909: :op => '=', :values => [str1], Chris@909: :c => ['estimated_hours', 'subject'], Chris@909: :format => 'csv', Chris@909: :set_filter => 1 Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: lines = @response.body.chomp.split("\n") Chris@909: assert_equal "#{issue.id},1234.5,#{str1}", lines[1] Chris@909: Chris@909: str_tw = "Traditional Chinese (\xe7\xb9\x81\xe9\xab\x94\xe4\xb8\xad\xe6\x96\x87)" Chris@909: if str_tw.respond_to?(:force_encoding) Chris@909: str_tw.force_encoding('UTF-8') Chris@909: end Chris@909: assert_equal str_tw, l(:general_lang_name) Chris@909: assert_equal ',', l(:general_csv_separator) Chris@909: assert_equal '.', l(:general_csv_decimal_separator) Chris@909: end Chris@909: end Chris@909: Chris@909: def test_index_csv_fr Chris@909: with_settings :default_language => "fr" do Chris@909: str1 = "test_index_csv_fr" Chris@909: issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3, Chris@909: :status_id => 1, :priority => IssuePriority.all.first, Chris@909: :subject => str1, :estimated_hours => '1234.5') Chris@909: assert issue.save Chris@909: assert_equal 1234.5, issue.estimated_hours Chris@909: Chris@909: get :index, :project_id => 1, Chris@909: :f => ['subject'], Chris@909: :op => '=', :values => [str1], Chris@909: :c => ['estimated_hours', 'subject'], Chris@909: :format => 'csv', Chris@909: :set_filter => 1 Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: lines = @response.body.chomp.split("\n") Chris@909: assert_equal "#{issue.id};1234,5;#{str1}", lines[1] Chris@909: Chris@909: str_fr = "Fran\xc3\xa7ais" Chris@909: if str_fr.respond_to?(:force_encoding) Chris@909: str_fr.force_encoding('UTF-8') Chris@909: end Chris@909: assert_equal str_fr, l(:general_lang_name) Chris@909: assert_equal ';', l(:general_csv_separator) Chris@909: assert_equal ',', l(:general_csv_decimal_separator) Chris@909: end Chris@909: end Chris@909: Chris@909: def test_index_pdf Chris@909: ["en", "zh", "zh-TW", "ja", "ko"].each do |lang| Chris@909: with_settings :default_language => lang do Chris@909: Chris@909: get :index Chris@909: assert_response :success Chris@909: assert_template 'index' Chris@909: Chris@909: if lang == "ja" Chris@909: if RUBY_PLATFORM != 'java' Chris@909: assert_equal "CP932", l(:general_pdf_encoding) Chris@909: end Chris@909: if RUBY_PLATFORM == 'java' && l(:general_pdf_encoding) == "CP932" Chris@909: next Chris@909: end Chris@909: end Chris@909: Chris@909: get :index, :format => 'pdf' Chris@909: assert_response :success Chris@909: assert_not_nil assigns(:issues) Chris@909: assert_equal 'application/pdf', @response.content_type Chris@909: Chris@909: get :index, :project_id => 1, :format => 'pdf' Chris@909: assert_response :success Chris@909: assert_not_nil assigns(:issues) Chris@909: assert_equal 'application/pdf', @response.content_type Chris@909: Chris@909: get :index, :project_id => 1, :query_id => 6, :format => 'pdf' Chris@909: assert_response :success Chris@909: assert_not_nil assigns(:issues) Chris@909: assert_equal 'application/pdf', @response.content_type Chris@909: end Chris@909: end Chris@0: end Chris@441: Chris@0: def test_index_pdf_with_query_grouped_by_list_custom_field Chris@0: get :index, :project_id => 1, :query_id => 9, :format => 'pdf' Chris@0: assert_response :success Chris@0: assert_not_nil assigns(:issues) Chris@0: assert_not_nil assigns(:issue_count_by_group) Chris@0: assert_equal 'application/pdf', @response.content_type Chris@0: end Chris@441: Chris@0: def test_index_sort Chris@0: get :index, :sort => 'tracker,id:desc' Chris@0: assert_response :success Chris@441: Chris@0: sort_params = @request.session['issues_index_sort'] Chris@0: assert sort_params.is_a?(String) Chris@0: assert_equal 'tracker,id:desc', sort_params Chris@441: Chris@0: issues = assigns(:issues) Chris@0: assert_not_nil issues Chris@0: assert !issues.empty? Chris@0: assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id) Chris@0: end Chris@441: Chris@909: def test_index_sort_by_field_not_included_in_columns Chris@909: Setting.issue_list_default_columns = %w(subject author) Chris@909: get :index, :sort => 'tracker' Chris@909: end Chris@909: Chris@909: def test_index_sort_by_assigned_to Chris@909: get :index, :sort => 'assigned_to' Chris@909: assert_response :success Chris@909: assignees = assigns(:issues).collect(&:assigned_to).compact Chris@909: assert_equal assignees.sort, assignees Chris@909: end Chris@909: Chris@909: def test_index_sort_by_assigned_to_desc Chris@909: get :index, :sort => 'assigned_to:desc' Chris@909: assert_response :success Chris@909: assignees = assigns(:issues).collect(&:assigned_to).compact Chris@909: assert_equal assignees.sort.reverse, assignees Chris@909: end Chris@909: Chris@909: def test_index_group_by_assigned_to Chris@909: get :index, :group_by => 'assigned_to', :sort => 'priority' Chris@909: assert_response :success Chris@909: end Chris@909: Chris@909: def test_index_sort_by_author Chris@909: get :index, :sort => 'author' Chris@909: assert_response :success Chris@909: authors = assigns(:issues).collect(&:author) Chris@909: assert_equal authors.sort, authors Chris@909: end Chris@909: Chris@909: def test_index_sort_by_author_desc Chris@909: get :index, :sort => 'author:desc' Chris@909: assert_response :success Chris@909: authors = assigns(:issues).collect(&:author) Chris@909: assert_equal authors.sort.reverse, authors Chris@909: end Chris@909: Chris@909: def test_index_group_by_author Chris@909: get :index, :group_by => 'author', :sort => 'priority' Chris@909: assert_response :success Chris@909: end Chris@909: Chris@0: def test_index_with_columns Chris@0: columns = ['tracker', 'subject', 'assigned_to'] Chris@441: get :index, :set_filter => 1, :c => columns Chris@0: assert_response :success Chris@441: Chris@0: # query should use specified columns Chris@0: query = assigns(:query) Chris@0: assert_kind_of Query, query Chris@0: assert_equal columns, query.column_names.map(&:to_s) Chris@441: Chris@0: # columns should be stored in session Chris@0: assert_kind_of Hash, session[:query] Chris@0: assert_kind_of Array, session[:query][:column_names] Chris@0: assert_equal columns, session[:query][:column_names].map(&:to_s) Chris@507: Chris@507: # ensure only these columns are kept in the selected columns list Chris@507: assert_tag :tag => 'select', :attributes => { :id => 'selected_columns' }, Chris@507: :children => { :count => 3 } Chris@507: assert_no_tag :tag => 'option', :attributes => { :value => 'project' }, Chris@507: :parent => { :tag => 'select', :attributes => { :id => "selected_columns" } } Chris@0: end Chris@0: Chris@909: def test_index_without_project_should_implicitly_add_project_column_to_default_columns Chris@909: Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to'] Chris@909: get :index, :set_filter => 1 Chris@909: Chris@909: # query should use specified columns Chris@909: query = assigns(:query) Chris@909: assert_kind_of Query, query Chris@909: assert_equal [:project, :tracker, :subject, :assigned_to], query.columns.map(&:name) Chris@909: end Chris@909: Chris@909: def test_index_without_project_and_explicit_default_columns_should_not_add_project_column Chris@909: Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to'] Chris@909: columns = ['tracker', 'subject', 'assigned_to'] Chris@909: get :index, :set_filter => 1, :c => columns Chris@909: Chris@909: # query should use specified columns Chris@909: query = assigns(:query) Chris@909: assert_kind_of Query, query Chris@909: assert_equal columns.map(&:to_sym), query.columns.map(&:name) Chris@909: end Chris@909: Chris@441: def test_index_with_custom_field_column Chris@441: columns = %w(tracker subject cf_2) Chris@441: get :index, :set_filter => 1, :c => columns Chris@441: assert_response :success Chris@441: Chris@441: # query should use specified columns Chris@441: query = assigns(:query) Chris@441: assert_kind_of Query, query Chris@441: assert_equal columns, query.column_names.map(&:to_s) Chris@441: Chris@441: assert_tag :td, Chris@441: :attributes => {:class => 'cf_2 string'}, Chris@441: :ancestor => {:tag => 'table', :attributes => {:class => /issues/}} Chris@441: end Chris@441: Chris@909: def test_index_with_date_column Chris@909: Issue.find(1).update_attribute :start_date, '1987-08-24' Chris@909: Chris@909: with_settings :date_format => '%d/%m/%Y' do Chris@909: get :index, :set_filter => 1, :c => %w(start_date) Chris@909: assert_tag 'td', :attributes => {:class => /start_date/}, :content => '24/08/1987' Chris@909: end Chris@909: end Chris@909: Chris@909: def test_index_with_done_ratio Chris@909: Issue.find(1).update_attribute :done_ratio, 40 Chris@909: Chris@909: get :index, :set_filter => 1, :c => %w(done_ratio) Chris@909: assert_tag 'td', :attributes => {:class => /done_ratio/}, Chris@909: :child => {:tag => 'table', :attributes => {:class => 'progress'}, Chris@909: :descendant => {:tag => 'td', :attributes => {:class => 'closed', :style => 'width: 40%;'}} Chris@909: } Chris@909: end Chris@909: Chris@909: def test_index_with_fixed_version Chris@909: get :index, :set_filter => 1, :c => %w(fixed_version) Chris@909: assert_tag 'td', :attributes => {:class => /fixed_version/}, Chris@909: :child => {:tag => 'a', :content => '1.0', :attributes => {:href => '/versions/2'}} Chris@909: end Chris@909: Chris@909: def test_index_send_html_if_query_is_invalid Chris@909: get :index, :f => ['start_date'], :op => {:start_date => '='} Chris@909: assert_equal 'text/html', @response.content_type Chris@909: assert_template 'index' Chris@909: end Chris@909: Chris@909: def test_index_send_nothing_if_query_is_invalid Chris@909: get :index, :f => ['start_date'], :op => {:start_date => '='}, :format => 'csv' Chris@909: assert_equal 'text/csv', @response.content_type Chris@909: assert @response.body.blank? Chris@909: end Chris@909: Chris@0: def test_show_by_anonymous Chris@0: get :show, :id => 1 Chris@0: assert_response :success Chris@909: assert_template 'show' Chris@0: assert_not_nil assigns(:issue) Chris@0: assert_equal Issue.find(1), assigns(:issue) Chris@441: Chris@0: # anonymous role is allowed to add a note Chris@0: assert_tag :tag => 'form', Chris@0: :descendant => { :tag => 'fieldset', Chris@441: :child => { :tag => 'legend', Chris@0: :content => /Notes/ } } Chris@909: assert_tag :tag => 'title', Chris@909: :content => "Bug #1: Can't print recipes - eCookbook - Redmine" Chris@0: end Chris@441: Chris@0: def test_show_by_manager Chris@0: @request.session[:user_id] = 2 Chris@0: get :show, :id => 1 Chris@0: assert_response :success Chris@441: Chris@119: assert_tag :tag => 'a', Chris@119: :content => /Quote/ Chris@441: Chris@0: assert_tag :tag => 'form', Chris@0: :descendant => { :tag => 'fieldset', Chris@441: :child => { :tag => 'legend', Chris@0: :content => /Change properties/ } }, Chris@0: :descendant => { :tag => 'fieldset', Chris@441: :child => { :tag => 'legend', Chris@0: :content => /Log time/ } }, Chris@0: :descendant => { :tag => 'fieldset', Chris@441: :child => { :tag => 'legend', Chris@0: :content => /Notes/ } } Chris@0: end Chris@441: Chris@909: def test_update_form_should_not_display_inactive_enumerations Chris@909: @request.session[:user_id] = 2 Chris@909: get :show, :id => 1 Chris@909: assert_response :success Chris@909: Chris@909: assert ! IssuePriority.find(15).active? Chris@909: assert_no_tag :option, :attributes => {:value => '15'}, Chris@909: :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} } Chris@909: end Chris@909: Chris@909: def test_update_form_should_allow_attachment_upload Chris@909: @request.session[:user_id] = 2 Chris@909: get :show, :id => 1 Chris@909: Chris@909: assert_tag :tag => 'form', Chris@909: :attributes => {:id => 'issue-form', :method => 'post', :enctype => 'multipart/form-data'}, Chris@909: :descendant => { Chris@909: :tag => 'input', Chris@909: :attributes => {:type => 'file', :name => 'attachments[1][file]'} Chris@909: } Chris@909: end Chris@909: Chris@0: def test_show_should_deny_anonymous_access_without_permission Chris@0: Role.anonymous.remove_permission!(:view_issues) Chris@0: get :show, :id => 1 Chris@0: assert_response :redirect Chris@0: end Chris@441: Chris@441: def test_show_should_deny_anonymous_access_to_private_issue Chris@441: Issue.update_all(["is_private = ?", true], "id = 1") Chris@441: get :show, :id => 1 Chris@441: assert_response :redirect Chris@441: end Chris@441: Chris@0: def test_show_should_deny_non_member_access_without_permission Chris@0: Role.non_member.remove_permission!(:view_issues) Chris@0: @request.session[:user_id] = 9 Chris@0: get :show, :id => 1 Chris@0: assert_response 403 Chris@0: end Chris@441: Chris@441: def test_show_should_deny_non_member_access_to_private_issue Chris@441: Issue.update_all(["is_private = ?", true], "id = 1") Chris@441: @request.session[:user_id] = 9 Chris@441: get :show, :id => 1 Chris@441: assert_response 403 Chris@441: end Chris@441: Chris@0: def test_show_should_deny_member_access_without_permission Chris@0: Role.find(1).remove_permission!(:view_issues) Chris@0: @request.session[:user_id] = 2 Chris@0: get :show, :id => 1 Chris@0: assert_response 403 Chris@0: end Chris@441: Chris@441: def test_show_should_deny_member_access_to_private_issue_without_permission Chris@441: Issue.update_all(["is_private = ?", true], "id = 1") Chris@441: @request.session[:user_id] = 3 Chris@441: get :show, :id => 1 Chris@441: assert_response 403 Chris@441: end Chris@441: Chris@441: def test_show_should_allow_author_access_to_private_issue Chris@441: Issue.update_all(["is_private = ?, author_id = 3", true], "id = 1") Chris@441: @request.session[:user_id] = 3 Chris@441: get :show, :id => 1 Chris@441: assert_response :success Chris@441: end Chris@441: Chris@441: def test_show_should_allow_assignee_access_to_private_issue Chris@441: Issue.update_all(["is_private = ?, assigned_to_id = 3", true], "id = 1") Chris@441: @request.session[:user_id] = 3 Chris@441: get :show, :id => 1 Chris@441: assert_response :success Chris@441: end Chris@441: Chris@441: def test_show_should_allow_member_access_to_private_issue_with_permission Chris@441: Issue.update_all(["is_private = ?", true], "id = 1") Chris@441: User.find(3).roles_for_project(Project.find(1)).first.update_attribute :issues_visibility, 'all' Chris@441: @request.session[:user_id] = 3 Chris@441: get :show, :id => 1 Chris@441: assert_response :success Chris@441: end Chris@441: Chris@0: def test_show_should_not_disclose_relations_to_invisible_issues Chris@0: Setting.cross_project_issue_relations = '1' Chris@0: IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates') Chris@0: # Relation to a private project issue Chris@0: IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates') Chris@441: Chris@0: get :show, :id => 1 Chris@0: assert_response :success Chris@441: Chris@0: assert_tag :div, :attributes => { :id => 'relations' }, Chris@0: :descendant => { :tag => 'a', :content => /#2$/ } Chris@0: assert_no_tag :div, :attributes => { :id => 'relations' }, Chris@0: :descendant => { :tag => 'a', :content => /#4$/ } Chris@0: end Chris@441: Chris@0: def test_show_atom Chris@0: get :show, :id => 2, :format => 'atom' Chris@0: assert_response :success Chris@909: assert_template 'journals/index' Chris@0: # Inline image Chris@0: assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10')) Chris@0: end Chris@441: Chris@0: def test_show_export_to_pdf Chris@0: get :show, :id => 3, :format => 'pdf' Chris@0: assert_response :success Chris@0: assert_equal 'application/pdf', @response.content_type Chris@0: assert @response.body.starts_with?('%PDF') Chris@0: assert_not_nil assigns(:issue) Chris@0: end Chris@0: Chris@0: def test_get_new Chris@0: @request.session[:user_id] = 2 Chris@0: get :new, :project_id => 1, :tracker_id => 1 Chris@0: assert_response :success Chris@0: assert_template 'new' Chris@441: Chris@0: assert_tag :tag => 'input', :attributes => { :name => 'issue[custom_field_values][2]', Chris@0: :value => 'Default string' } Chris@909: Chris@909: # Be sure we don't display inactive IssuePriorities Chris@909: assert ! IssuePriority.find(15).active? Chris@909: assert_no_tag :option, :attributes => {:value => '15'}, Chris@909: :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} } Chris@909: end Chris@909: Chris@909: def test_get_new_without_default_start_date_is_creation_date Chris@909: Setting.default_issue_start_date_to_creation_date = 0 Chris@909: Chris@909: @request.session[:user_id] = 2 Chris@909: get :new, :project_id => 1, :tracker_id => 1 Chris@909: assert_response :success Chris@909: assert_template 'new' Chris@909: Chris@909: assert_tag :tag => 'input', :attributes => { :name => 'issue[start_date]', Chris@909: :value => nil } Chris@909: end Chris@909: Chris@909: def test_get_new_with_default_start_date_is_creation_date Chris@909: Setting.default_issue_start_date_to_creation_date = 1 Chris@909: Chris@909: @request.session[:user_id] = 2 Chris@909: get :new, :project_id => 1, :tracker_id => 1 Chris@909: assert_response :success Chris@909: assert_template 'new' Chris@909: Chris@909: assert_tag :tag => 'input', :attributes => { :name => 'issue[start_date]', Chris@909: :value => Date.today.to_s } Chris@909: end Chris@909: Chris@909: def test_get_new_form_should_allow_attachment_upload Chris@909: @request.session[:user_id] = 2 Chris@909: get :new, :project_id => 1, :tracker_id => 1 Chris@909: Chris@909: assert_tag :tag => 'form', Chris@909: :attributes => {:id => 'issue-form', :method => 'post', :enctype => 'multipart/form-data'}, Chris@909: :descendant => { Chris@909: :tag => 'input', Chris@909: :attributes => {:type => 'file', :name => 'attachments[1][file]'} Chris@909: } Chris@0: end Chris@0: Chris@0: def test_get_new_without_tracker_id Chris@0: @request.session[:user_id] = 2 Chris@0: get :new, :project_id => 1 Chris@0: assert_response :success Chris@0: assert_template 'new' Chris@441: Chris@0: issue = assigns(:issue) Chris@0: assert_not_nil issue Chris@0: assert_equal Project.find(1).trackers.first, issue.tracker Chris@0: end Chris@441: Chris@0: def test_get_new_with_no_default_status_should_display_an_error Chris@0: @request.session[:user_id] = 2 Chris@0: IssueStatus.delete_all Chris@441: Chris@0: get :new, :project_id => 1 Chris@0: assert_response 500 chris@37: assert_error_tag :content => /No default issue/ Chris@0: end Chris@441: Chris@0: def test_get_new_with_no_tracker_should_display_an_error Chris@0: @request.session[:user_id] = 2 Chris@0: Tracker.delete_all Chris@441: Chris@0: get :new, :project_id => 1 Chris@0: assert_response 500 chris@37: assert_error_tag :content => /No tracker/ Chris@0: end Chris@441: Chris@0: def test_update_new_form Chris@0: @request.session[:user_id] = 2 Chris@14: xhr :post, :new, :project_id => 1, Chris@441: :issue => {:tracker_id => 2, Chris@0: :subject => 'This is the test_new issue', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5} Chris@0: assert_response :success Chris@0: assert_template 'attributes' Chris@441: Chris@0: issue = assigns(:issue) Chris@0: assert_kind_of Issue, issue Chris@0: assert_equal 1, issue.project_id Chris@0: assert_equal 2, issue.tracker_id Chris@0: assert_equal 'This is the test_new issue', issue.subject Chris@0: end Chris@441: Chris@0: def test_post_create Chris@0: @request.session[:user_id] = 2 Chris@0: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 3, Chris@0: :status_id => 2, Chris@0: :subject => 'This is the test_new issue', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5, chris@37: :start_date => '2010-11-07', Chris@0: :estimated_hours => '', Chris@0: :custom_field_values => {'2' => 'Value for field 2'}} Chris@0: end Chris@0: assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id Chris@441: Chris@0: issue = Issue.find_by_subject('This is the test_new issue') Chris@0: assert_not_nil issue Chris@0: assert_equal 2, issue.author_id Chris@0: assert_equal 3, issue.tracker_id Chris@0: assert_equal 2, issue.status_id chris@37: assert_equal Date.parse('2010-11-07'), issue.start_date Chris@0: assert_nil issue.estimated_hours Chris@0: v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2}) Chris@0: assert_not_nil v Chris@0: assert_equal 'Value for field 2', v.value Chris@0: end Chris@441: Chris@909: def test_post_new_with_group_assignment Chris@909: group = Group.find(11) Chris@909: project = Project.find(1) Chris@929: project.members << Member.new(:principal => group, :roles => [Role.givable.first]) Chris@909: Chris@909: with_settings :issue_group_assignment => '1' do Chris@909: @request.session[:user_id] = 2 Chris@909: assert_difference 'Issue.count' do Chris@909: post :create, :project_id => project.id, Chris@909: :issue => {:tracker_id => 3, Chris@909: :status_id => 1, Chris@909: :subject => 'This is the test_new_with_group_assignment issue', Chris@909: :assigned_to_id => group.id} Chris@909: end Chris@909: end Chris@909: assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id Chris@909: Chris@909: issue = Issue.find_by_subject('This is the test_new_with_group_assignment issue') Chris@909: assert_not_nil issue Chris@909: assert_equal group, issue.assigned_to Chris@909: end Chris@909: Chris@909: def test_post_create_without_start_date_and_default_start_date_is_not_creation_date Chris@909: Setting.default_issue_start_date_to_creation_date = 0 Chris@909: chris@37: @request.session[:user_id] = 2 chris@37: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, chris@37: :issue => {:tracker_id => 3, chris@37: :status_id => 2, chris@37: :subject => 'This is the test_new issue', chris@37: :description => 'This is the description', chris@37: :priority_id => 5, chris@37: :estimated_hours => '', chris@37: :custom_field_values => {'2' => 'Value for field 2'}} chris@37: end chris@37: assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id Chris@441: chris@37: issue = Issue.find_by_subject('This is the test_new issue') chris@37: assert_not_nil issue chris@37: assert_nil issue.start_date chris@37: end Chris@441: Chris@909: def test_post_create_without_start_date_and_default_start_date_is_creation_date Chris@909: Setting.default_issue_start_date_to_creation_date = 1 Chris@909: Chris@909: @request.session[:user_id] = 2 Chris@909: assert_difference 'Issue.count' do Chris@909: post :create, :project_id => 1, Chris@909: :issue => {:tracker_id => 3, Chris@909: :status_id => 2, Chris@909: :subject => 'This is the test_new issue', Chris@909: :description => 'This is the description', Chris@909: :priority_id => 5, Chris@909: :estimated_hours => '', Chris@909: :custom_field_values => {'2' => 'Value for field 2'}} Chris@909: end Chris@909: assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id Chris@909: Chris@909: issue = Issue.find_by_subject('This is the test_new issue') Chris@909: assert_not_nil issue Chris@909: assert_equal Date.today, issue.start_date Chris@909: end Chris@909: Chris@0: def test_post_create_and_continue Chris@0: @request.session[:user_id] = 2 Chris@909: assert_difference 'Issue.count' do Chris@909: post :create, :project_id => 1, Chris@909: :issue => {:tracker_id => 3, :subject => 'This is first issue', :priority_id => 5}, Chris@909: :continue => '' Chris@909: end Chris@909: Chris@909: issue = Issue.first(:order => 'id DESC') Chris@909: assert_redirected_to :controller => 'issues', :action => 'new', :project_id => 'ecookbook', :issue => {:tracker_id => 3} Chris@909: assert_not_nil flash[:notice], "flash was not set" Chris@909: assert flash[:notice].include?("##{issue.id}"), "issue link not found in flash: #{flash[:notice]}" Chris@0: end Chris@441: Chris@0: def test_post_create_without_custom_fields_param Chris@0: @request.session[:user_id] = 2 Chris@0: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: :subject => 'This is the test_new issue', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5} Chris@0: end Chris@0: assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id Chris@0: end Chris@0: Chris@0: def test_post_create_with_required_custom_field_and_without_custom_fields_param Chris@0: field = IssueCustomField.find_by_name('Database') Chris@0: field.update_attribute(:is_required, true) Chris@0: Chris@0: @request.session[:user_id] = 2 Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: :subject => 'This is the test_new issue', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5} Chris@0: assert_response :success Chris@0: assert_template 'new' Chris@0: issue = assigns(:issue) Chris@0: assert_not_nil issue Chris@0: assert_equal I18n.translate('activerecord.errors.messages.invalid'), issue.errors.on(:custom_values) Chris@0: end Chris@441: Chris@0: def test_post_create_with_watchers Chris@0: @request.session[:user_id] = 2 Chris@0: ActionMailer::Base.deliveries.clear Chris@441: Chris@0: assert_difference 'Watcher.count', 2 do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: :subject => 'This is a new issue with watchers', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5, Chris@0: :watcher_user_ids => ['2', '3']} Chris@0: end Chris@0: issue = Issue.find_by_subject('This is a new issue with watchers') Chris@0: assert_not_nil issue Chris@0: assert_redirected_to :controller => 'issues', :action => 'show', :id => issue Chris@441: Chris@0: # Watchers added Chris@0: assert_equal [2, 3], issue.watcher_user_ids.sort Chris@0: assert issue.watched_by?(User.find(3)) Chris@0: # Watchers notified Chris@0: mail = ActionMailer::Base.deliveries.last Chris@0: assert_kind_of TMail::Mail, mail Chris@0: assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail) Chris@0: end Chris@441: Chris@0: def test_post_create_subissue Chris@0: @request.session[:user_id] = 2 Chris@441: Chris@0: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: :subject => 'This is a child issue', Chris@0: :parent_issue_id => 2} Chris@0: end Chris@0: issue = Issue.find_by_subject('This is a child issue') Chris@0: assert_not_nil issue Chris@0: assert_equal Issue.find(2), issue.parent Chris@0: end Chris@119: Chris@119: def test_post_create_subissue_with_non_numeric_parent_id Chris@119: @request.session[:user_id] = 2 Chris@441: Chris@119: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@119: :issue => {:tracker_id => 1, Chris@119: :subject => 'This is a child issue', Chris@119: :parent_issue_id => 'ABC'} Chris@119: end Chris@119: issue = Issue.find_by_subject('This is a child issue') Chris@119: assert_not_nil issue Chris@119: assert_nil issue.parent Chris@119: end Chris@909: Chris@507: def test_post_create_private Chris@507: @request.session[:user_id] = 2 Chris@507: Chris@507: assert_difference 'Issue.count' do Chris@507: post :create, :project_id => 1, Chris@507: :issue => {:tracker_id => 1, Chris@507: :subject => 'This is a private issue', Chris@507: :is_private => '1'} Chris@507: end Chris@507: issue = Issue.first(:order => 'id DESC') Chris@507: assert issue.is_private? Chris@507: end Chris@909: Chris@507: def test_post_create_private_with_set_own_issues_private_permission Chris@507: role = Role.find(1) Chris@507: role.remove_permission! :set_issues_private Chris@507: role.add_permission! :set_own_issues_private Chris@909: Chris@507: @request.session[:user_id] = 2 Chris@507: Chris@507: assert_difference 'Issue.count' do Chris@507: post :create, :project_id => 1, Chris@507: :issue => {:tracker_id => 1, Chris@507: :subject => 'This is a private issue', Chris@507: :is_private => '1'} Chris@507: end Chris@507: issue = Issue.first(:order => 'id DESC') Chris@507: assert issue.is_private? Chris@507: end Chris@441: Chris@0: def test_post_create_should_send_a_notification Chris@0: ActionMailer::Base.deliveries.clear Chris@0: @request.session[:user_id] = 2 Chris@0: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 3, Chris@0: :subject => 'This is the test_new issue', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5, Chris@0: :estimated_hours => '', Chris@0: :custom_field_values => {'2' => 'Value for field 2'}} Chris@0: end Chris@0: assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id Chris@441: Chris@0: assert_equal 1, ActionMailer::Base.deliveries.size Chris@0: end Chris@441: Chris@0: def test_post_create_should_preserve_fields_values_on_validation_failure Chris@0: @request.session[:user_id] = 2 Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: # empty subject Chris@0: :subject => '', Chris@0: :description => 'This is a description', Chris@0: :priority_id => 6, Chris@0: :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}} Chris@0: assert_response :success Chris@0: assert_template 'new' Chris@441: Chris@0: assert_tag :textarea, :attributes => { :name => 'issue[description]' }, Chris@0: :content => 'This is a description' Chris@0: assert_tag :select, :attributes => { :name => 'issue[priority_id]' }, Chris@0: :child => { :tag => 'option', :attributes => { :selected => 'selected', Chris@0: :value => '6' }, Chris@441: :content => 'High' } Chris@0: # Custom fields Chris@0: assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' }, Chris@0: :child => { :tag => 'option', :attributes => { :selected => 'selected', Chris@0: :value => 'Oracle' }, Chris@441: :content => 'Oracle' } Chris@0: assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]', Chris@0: :value => 'Value for field 2'} Chris@0: end Chris@441: Chris@0: def test_post_create_should_ignore_non_safe_attributes Chris@0: @request.session[:user_id] = 2 Chris@0: assert_nothing_raised do Chris@0: post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" } Chris@0: end Chris@0: end Chris@441: Chris@909: def test_post_create_with_attachment Chris@909: set_tmp_attachments_directory Chris@909: @request.session[:user_id] = 2 Chris@909: Chris@909: assert_difference 'Issue.count' do Chris@909: assert_difference 'Attachment.count' do Chris@909: post :create, :project_id => 1, Chris@909: :issue => { :tracker_id => '1', :subject => 'With attachment' }, Chris@909: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}} Chris@909: end Chris@909: end Chris@909: Chris@909: issue = Issue.first(:order => 'id DESC') Chris@909: attachment = Attachment.first(:order => 'id DESC') Chris@909: Chris@909: assert_equal issue, attachment.container Chris@909: assert_equal 2, attachment.author_id Chris@909: assert_equal 'testfile.txt', attachment.filename Chris@909: assert_equal 'text/plain', attachment.content_type Chris@909: assert_equal 'test file', attachment.description Chris@909: assert_equal 59, attachment.filesize Chris@909: assert File.exists?(attachment.diskfile) Chris@909: assert_equal 59, File.size(attachment.diskfile) Chris@909: end Chris@909: Chris@0: context "without workflow privilege" do Chris@0: setup do Chris@0: Workflow.delete_all(["role_id = ?", Role.anonymous.id]) chris@37: Role.anonymous.add_permission! :add_issues, :add_issue_notes Chris@0: end Chris@441: Chris@0: context "#new" do Chris@0: should "propose default status only" do Chris@0: get :new, :project_id => 1 Chris@0: assert_response :success Chris@0: assert_template 'new' Chris@0: assert_tag :tag => 'select', Chris@0: :attributes => {:name => 'issue[status_id]'}, Chris@0: :children => {:count => 1}, Chris@0: :child => {:tag => 'option', :attributes => {:value => IssueStatus.default.id.to_s}} Chris@0: end Chris@441: Chris@0: should "accept default status" do Chris@0: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: :subject => 'This is an issue', Chris@0: :status_id => 1} Chris@0: end Chris@0: issue = Issue.last(:order => 'id') Chris@0: assert_equal IssueStatus.default, issue.status Chris@0: end Chris@441: Chris@0: should "ignore unauthorized status" do Chris@0: assert_difference 'Issue.count' do Chris@441: post :create, :project_id => 1, Chris@0: :issue => {:tracker_id => 1, Chris@0: :subject => 'This is an issue', Chris@0: :status_id => 3} Chris@0: end Chris@0: issue = Issue.last(:order => 'id') Chris@0: assert_equal IssueStatus.default, issue.status Chris@0: end Chris@0: end Chris@441: chris@37: context "#update" do chris@37: should "ignore status change" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3} chris@37: end chris@37: assert_equal 1, Issue.find(1).status_id chris@37: end Chris@441: chris@37: should "ignore attributes changes" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2} chris@37: end chris@37: issue = Issue.find(1) chris@37: assert_equal "Can't print recipes", issue.subject chris@37: assert_nil issue.assigned_to chris@37: end chris@37: end chris@37: end Chris@441: chris@37: context "with workflow privilege" do chris@37: setup do chris@37: Workflow.delete_all(["role_id = ?", Role.anonymous.id]) chris@37: Workflow.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3) chris@37: Workflow.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 4) chris@37: Role.anonymous.add_permission! :add_issues, :add_issue_notes chris@37: end Chris@441: chris@37: context "#update" do chris@37: should "accept authorized status" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3} chris@37: end chris@37: assert_equal 3, Issue.find(1).status_id chris@37: end Chris@441: chris@37: should "ignore unauthorized status" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2} chris@37: end chris@37: assert_equal 1, Issue.find(1).status_id chris@37: end Chris@441: chris@37: should "accept authorized attributes changes" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:assigned_to_id => 2} chris@37: end chris@37: issue = Issue.find(1) chris@37: assert_equal 2, issue.assigned_to_id chris@37: end Chris@441: chris@37: should "ignore unauthorized attributes changes" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed'} chris@37: end chris@37: issue = Issue.find(1) chris@37: assert_equal "Can't print recipes", issue.subject chris@37: end chris@37: end Chris@441: chris@37: context "and :edit_issues permission" do chris@37: setup do chris@37: Role.anonymous.add_permission! :add_issues, :edit_issues chris@37: end chris@37: chris@37: should "accept authorized status" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3} chris@37: end chris@37: assert_equal 3, Issue.find(1).status_id chris@37: end Chris@441: chris@37: should "ignore unauthorized status" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2} chris@37: end chris@37: assert_equal 1, Issue.find(1).status_id chris@37: end Chris@441: chris@37: should "accept authorized attributes changes" do chris@37: assert_difference 'Journal.count' do chris@37: put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2} chris@37: end chris@37: issue = Issue.find(1) chris@37: assert_equal "changed", issue.subject chris@37: assert_equal 2, issue.assigned_to_id chris@37: end chris@37: end Chris@0: end Chris@441: Chris@0: def test_copy_issue Chris@0: @request.session[:user_id] = 2 Chris@0: get :new, :project_id => 1, :copy_from => 1 Chris@0: assert_template 'new' Chris@0: assert_not_nil assigns(:issue) Chris@0: orig = Issue.find(1) Chris@0: assert_equal orig.subject, assigns(:issue).subject Chris@0: end Chris@441: Chris@0: def test_get_edit Chris@0: @request.session[:user_id] = 2 Chris@0: get :edit, :id => 1 Chris@0: assert_response :success Chris@0: assert_template 'edit' Chris@0: assert_not_nil assigns(:issue) Chris@0: assert_equal Issue.find(1), assigns(:issue) Chris@909: Chris@909: # Be sure we don't display inactive IssuePriorities Chris@909: assert ! IssuePriority.find(15).active? Chris@909: assert_no_tag :option, :attributes => {:value => '15'}, Chris@909: :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} } Chris@909: end Chris@909: Chris@909: def test_get_edit_should_display_the_time_entry_form_with_log_time_permission Chris@909: @request.session[:user_id] = 2 Chris@909: Role.find_by_name('Manager').update_attribute :permissions, [:view_issues, :edit_issues, :log_time] Chris@909: Chris@909: get :edit, :id => 1 Chris@909: assert_tag 'input', :attributes => {:name => 'time_entry[hours]'} Chris@909: end Chris@909: Chris@909: def test_get_edit_should_not_display_the_time_entry_form_without_log_time_permission Chris@909: @request.session[:user_id] = 2 Chris@909: Role.find_by_name('Manager').remove_permission! :log_time Chris@909: Chris@909: get :edit, :id => 1 Chris@909: assert_no_tag 'input', :attributes => {:name => 'time_entry[hours]'} Chris@0: end Chris@441: Chris@0: def test_get_edit_with_params Chris@0: @request.session[:user_id] = 2 chris@37: get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 }, chris@37: :time_entry => { :hours => '2.5', :comments => 'test_get_edit_with_params', :activity_id => TimeEntryActivity.first.id } Chris@0: assert_response :success Chris@0: assert_template 'edit' Chris@441: Chris@0: issue = assigns(:issue) Chris@0: assert_not_nil issue Chris@441: Chris@0: assert_equal 5, issue.status_id Chris@0: assert_tag :select, :attributes => { :name => 'issue[status_id]' }, Chris@441: :child => { :tag => 'option', Chris@0: :content => 'Closed', Chris@0: :attributes => { :selected => 'selected' } } Chris@441: Chris@0: assert_equal 7, issue.priority_id Chris@0: assert_tag :select, :attributes => { :name => 'issue[priority_id]' }, Chris@441: :child => { :tag => 'option', Chris@0: :content => 'Urgent', Chris@0: :attributes => { :selected => 'selected' } } chris@37: chris@37: assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => '2.5' } chris@37: assert_tag :select, :attributes => { :name => 'time_entry[activity_id]' }, chris@37: :child => { :tag => 'option', chris@37: :attributes => { :selected => 'selected', :value => TimeEntryActivity.first.id } } chris@37: assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => 'test_get_edit_with_params' } Chris@0: end Chris@0: Chris@0: def test_update_edit_form Chris@0: @request.session[:user_id] = 2 Chris@14: xhr :post, :new, :project_id => 1, Chris@0: :id => 1, Chris@441: :issue => {:tracker_id => 2, Chris@0: :subject => 'This is the test_new issue', Chris@0: :description => 'This is the description', Chris@0: :priority_id => 5} Chris@0: assert_response :success Chris@0: assert_template 'attributes' Chris@441: Chris@0: issue = assigns(:issue) Chris@0: assert_kind_of Issue, issue Chris@0: assert_equal 1, issue.id Chris@0: assert_equal 1, issue.project_id Chris@0: assert_equal 2, issue.tracker_id Chris@0: assert_equal 'This is the test_new issue', issue.subject Chris@0: end Chris@441: Chris@0: def test_update_using_invalid_http_verbs Chris@0: @request.session[:user_id] = 2 Chris@0: subject = 'Updated by an invalid http verb' Chris@0: Chris@0: get :update, :id => 1, :issue => {:subject => subject} Chris@0: assert_not_equal subject, Issue.find(1).subject Chris@0: Chris@0: post :update, :id => 1, :issue => {:subject => subject} Chris@0: assert_not_equal subject, Issue.find(1).subject Chris@0: Chris@0: delete :update, :id => 1, :issue => {:subject => subject} Chris@0: assert_not_equal subject, Issue.find(1).subject Chris@0: end Chris@0: Chris@0: def test_put_update_without_custom_fields_param Chris@0: @request.session[:user_id] = 2 Chris@0: ActionMailer::Base.deliveries.clear Chris@441: Chris@0: issue = Issue.find(1) Chris@0: assert_equal '125', issue.custom_value_for(2).value Chris@0: old_subject = issue.subject Chris@0: new_subject = 'Subject modified by IssuesControllerTest#test_post_edit' Chris@441: Chris@0: assert_difference('Journal.count') do Chris@0: assert_difference('JournalDetail.count', 2) do Chris@0: put :update, :id => 1, :issue => {:subject => new_subject, Chris@0: :priority_id => '6', Chris@0: :category_id => '1' # no change Chris@0: } Chris@0: end Chris@0: end Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@0: issue.reload Chris@0: assert_equal new_subject, issue.subject Chris@0: # Make sure custom fields were not cleared Chris@0: assert_equal '125', issue.custom_value_for(2).value Chris@441: Chris@0: mail = ActionMailer::Base.deliveries.last Chris@0: assert_kind_of TMail::Mail, mail Chris@0: assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]") Chris@0: assert mail.body.include?("Subject changed from #{old_subject} to #{new_subject}") Chris@0: end Chris@441: Chris@0: def test_put_update_with_custom_field_change Chris@0: @request.session[:user_id] = 2 Chris@0: issue = Issue.find(1) Chris@0: assert_equal '125', issue.custom_value_for(2).value Chris@441: Chris@0: assert_difference('Journal.count') do Chris@0: assert_difference('JournalDetail.count', 3) do Chris@0: put :update, :id => 1, :issue => {:subject => 'Custom field change', Chris@0: :priority_id => '6', Chris@0: :category_id => '1', # no change Chris@0: :custom_field_values => { '2' => 'New custom value' } Chris@0: } Chris@0: end Chris@0: end Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@0: issue.reload Chris@0: assert_equal 'New custom value', issue.custom_value_for(2).value Chris@441: Chris@0: mail = ActionMailer::Base.deliveries.last Chris@0: assert_kind_of TMail::Mail, mail Chris@0: assert mail.body.include?("Searchable field changed from 125 to New custom value") Chris@0: end Chris@441: Chris@0: def test_put_update_with_status_and_assignee_change Chris@0: issue = Issue.find(1) Chris@0: assert_equal 1, issue.status_id Chris@0: @request.session[:user_id] = 2 Chris@0: assert_difference('TimeEntry.count', 0) do Chris@0: put :update, Chris@0: :id => 1, Chris@0: :issue => { :status_id => 2, :assigned_to_id => 3 }, Chris@0: :notes => 'Assigned to dlopper', Chris@0: :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first } Chris@0: end Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@0: issue.reload Chris@0: assert_equal 2, issue.status_id Chris@0: j = Journal.find(:first, :order => 'id DESC') Chris@0: assert_equal 'Assigned to dlopper', j.notes Chris@0: assert_equal 2, j.details.size Chris@441: Chris@0: mail = ActionMailer::Base.deliveries.last Chris@0: assert mail.body.include?("Status changed from New to Assigned") Chris@0: # subject should contain the new status Chris@0: assert mail.subject.include?("(#{ IssueStatus.find(2).name })") Chris@0: end Chris@441: Chris@0: def test_put_update_with_note_only Chris@0: notes = 'Note added by IssuesControllerTest#test_update_with_note_only' Chris@0: # anonymous user Chris@0: put :update, Chris@0: :id => 1, Chris@0: :notes => notes Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@0: j = Journal.find(:first, :order => 'id DESC') Chris@0: assert_equal notes, j.notes Chris@0: assert_equal 0, j.details.size Chris@0: assert_equal User.anonymous, j.user Chris@441: Chris@0: mail = ActionMailer::Base.deliveries.last Chris@0: assert mail.body.include?(notes) Chris@0: end Chris@441: Chris@0: def test_put_update_with_note_and_spent_time Chris@0: @request.session[:user_id] = 2 Chris@0: spent_hours_before = Issue.find(1).spent_hours Chris@0: assert_difference('TimeEntry.count') do Chris@0: put :update, Chris@0: :id => 1, Chris@0: :notes => '2.5 hours added', Chris@0: :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id } Chris@0: end Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@441: Chris@0: issue = Issue.find(1) Chris@441: Chris@0: j = Journal.find(:first, :order => 'id DESC') Chris@0: assert_equal '2.5 hours added', j.notes Chris@0: assert_equal 0, j.details.size Chris@441: Chris@0: t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time') Chris@0: assert_not_nil t Chris@0: assert_equal 2.5, t.hours Chris@0: assert_equal spent_hours_before + 2.5, issue.spent_hours Chris@0: end Chris@441: Chris@0: def test_put_update_with_attachment_only Chris@0: set_tmp_attachments_directory Chris@441: Chris@0: # Delete all fixtured journals, a race condition can occur causing the wrong Chris@0: # journal to get fetched in the next find. Chris@0: Journal.delete_all Chris@0: Chris@0: # anonymous user Chris@909: assert_difference 'Attachment.count' do Chris@909: put :update, :id => 1, Chris@909: :notes => '', Chris@909: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}} Chris@909: end Chris@909: Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@0: j = Issue.find(1).journals.find(:first, :order => 'id DESC') Chris@0: assert j.notes.blank? Chris@0: assert_equal 1, j.details.size Chris@0: assert_equal 'testfile.txt', j.details.first.value Chris@0: assert_equal User.anonymous, j.user Chris@441: Chris@909: attachment = Attachment.first(:order => 'id DESC') Chris@909: assert_equal Issue.find(1), attachment.container Chris@909: assert_equal User.anonymous, attachment.author Chris@909: assert_equal 'testfile.txt', attachment.filename Chris@909: assert_equal 'text/plain', attachment.content_type Chris@909: assert_equal 'test file', attachment.description Chris@909: assert_equal 59, attachment.filesize Chris@909: assert File.exists?(attachment.diskfile) Chris@909: assert_equal 59, File.size(attachment.diskfile) Chris@909: Chris@0: mail = ActionMailer::Base.deliveries.last Chris@0: assert mail.body.include?('testfile.txt') Chris@0: end Chris@0: Chris@0: def test_put_update_with_attachment_that_fails_to_save Chris@0: set_tmp_attachments_directory Chris@441: Chris@0: # Delete all fixtured journals, a race condition can occur causing the wrong Chris@0: # journal to get fetched in the next find. Chris@0: Journal.delete_all Chris@0: Chris@0: # Mock out the unsaved attachment Chris@0: Attachment.any_instance.stubs(:create).returns(Attachment.new) Chris@441: Chris@0: # anonymous user Chris@0: put :update, Chris@0: :id => 1, Chris@0: :notes => '', Chris@0: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}} Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@0: assert_equal '1 file(s) could not be saved.', flash[:warning] Chris@0: Chris@0: end if Object.const_defined?(:Mocha) Chris@0: Chris@0: def test_put_update_with_no_change Chris@0: issue = Issue.find(1) Chris@0: issue.journals.clear Chris@0: ActionMailer::Base.deliveries.clear Chris@441: Chris@0: put :update, Chris@0: :id => 1, Chris@0: :notes => '' Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@441: Chris@0: issue.reload Chris@0: assert issue.journals.empty? Chris@0: # No email should be sent Chris@0: assert ActionMailer::Base.deliveries.empty? Chris@0: end Chris@0: Chris@0: def test_put_update_should_send_a_notification Chris@0: @request.session[:user_id] = 2 Chris@0: ActionMailer::Base.deliveries.clear Chris@0: issue = Issue.find(1) Chris@0: old_subject = issue.subject Chris@0: new_subject = 'Subject modified by IssuesControllerTest#test_post_edit' Chris@441: Chris@0: put :update, :id => 1, :issue => {:subject => new_subject, Chris@0: :priority_id => '6', Chris@0: :category_id => '1' # no change Chris@0: } Chris@0: assert_equal 1, ActionMailer::Base.deliveries.size Chris@0: end Chris@441: Chris@441: def test_put_update_with_invalid_spent_time_hours_only Chris@0: @request.session[:user_id] = 2 Chris@0: notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time' Chris@441: Chris@0: assert_no_difference('Journal.count') do Chris@0: put :update, Chris@0: :id => 1, Chris@0: :notes => notes, Chris@0: :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"} Chris@0: end Chris@0: assert_response :success Chris@0: assert_template 'edit' Chris@441: Chris@441: assert_error_tag :descendant => {:content => /Activity can't be blank/} Chris@441: assert_tag :textarea, :attributes => { :name => 'notes' }, :content => notes Chris@0: assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" } Chris@0: end Chris@441: Chris@441: def test_put_update_with_invalid_spent_time_comments_only Chris@441: @request.session[:user_id] = 2 Chris@441: notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time' Chris@441: Chris@441: assert_no_difference('Journal.count') do Chris@441: put :update, Chris@441: :id => 1, Chris@441: :notes => notes, Chris@441: :time_entry => {"comments"=>"this is my comment", "activity_id"=>"", "hours"=>""} Chris@441: end Chris@441: assert_response :success Chris@441: assert_template 'edit' Chris@441: Chris@441: assert_error_tag :descendant => {:content => /Activity can't be blank/} Chris@441: assert_error_tag :descendant => {:content => /Hours can't be blank/} Chris@441: assert_tag :textarea, :attributes => { :name => 'notes' }, :content => notes Chris@441: assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => "this is my comment" } Chris@441: end Chris@441: Chris@0: def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject Chris@0: issue = Issue.find(2) Chris@0: @request.session[:user_id] = 2 Chris@0: Chris@0: put :update, Chris@0: :id => issue.id, Chris@0: :issue => { Chris@0: :fixed_version_id => 4 Chris@0: } Chris@0: Chris@0: assert_response :redirect Chris@0: issue.reload Chris@0: assert_equal 4, issue.fixed_version_id Chris@0: assert_not_equal issue.project_id, issue.fixed_version.project_id Chris@0: end Chris@0: Chris@0: def test_put_update_should_redirect_back_using_the_back_url_parameter Chris@0: issue = Issue.find(2) Chris@0: @request.session[:user_id] = 2 Chris@0: Chris@0: put :update, Chris@0: :id => issue.id, Chris@0: :issue => { Chris@0: :fixed_version_id => 4 Chris@0: }, Chris@0: :back_url => '/issues' Chris@0: Chris@0: assert_response :redirect Chris@0: assert_redirected_to '/issues' Chris@0: end Chris@441: Chris@0: def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host Chris@0: issue = Issue.find(2) Chris@0: @request.session[:user_id] = 2 Chris@0: Chris@0: put :update, Chris@0: :id => issue.id, Chris@0: :issue => { Chris@0: :fixed_version_id => 4 Chris@0: }, Chris@0: :back_url => 'http://google.com' Chris@0: Chris@0: assert_response :redirect Chris@0: assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id Chris@0: end Chris@441: Chris@0: def test_get_bulk_edit Chris@0: @request.session[:user_id] = 2 Chris@0: get :bulk_edit, :ids => [1, 2] Chris@0: assert_response :success Chris@0: assert_template 'bulk_edit' Chris@441: Chris@441: assert_tag :input, :attributes => {:name => 'issue[parent_issue_id]'} Chris@441: Chris@0: # Project specific custom field, date type Chris@0: field = CustomField.find(9) Chris@0: assert !field.is_for_all? Chris@0: assert_equal 'date', field.field_format Chris@0: assert_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'} Chris@441: Chris@0: # System wide custom field Chris@0: assert CustomField.find(1).is_for_all? Chris@0: assert_tag :select, :attributes => {:name => 'issue[custom_field_values][1]'} Chris@909: Chris@909: # Be sure we don't display inactive IssuePriorities Chris@909: assert ! IssuePriority.find(15).active? Chris@909: assert_no_tag :option, :attributes => {:value => '15'}, Chris@909: :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} } Chris@0: end Chris@0: chris@37: def test_get_bulk_edit_on_different_projects chris@37: @request.session[:user_id] = 2 chris@37: get :bulk_edit, :ids => [1, 2, 6] chris@37: assert_response :success chris@37: assert_template 'bulk_edit' Chris@441: Chris@441: # Can not set issues from different projects as children of an issue Chris@441: assert_no_tag :input, :attributes => {:name => 'issue[parent_issue_id]'} Chris@441: chris@37: # Project specific custom field, date type chris@37: field = CustomField.find(9) chris@37: assert !field.is_for_all? chris@37: assert !field.project_ids.include?(Issue.find(6).project_id) chris@37: assert_no_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'} chris@37: end chris@37: Chris@441: def test_get_bulk_edit_with_user_custom_field Chris@441: field = IssueCustomField.create!(:name => 'Tester', :field_format => 'user', :is_for_all => true) Chris@441: Chris@441: @request.session[:user_id] = 2 Chris@441: get :bulk_edit, :ids => [1, 2] Chris@441: assert_response :success Chris@441: assert_template 'bulk_edit' Chris@441: Chris@441: assert_tag :select, Chris@441: :attributes => {:name => "issue[custom_field_values][#{field.id}]"}, Chris@441: :children => { Chris@441: :only => {:tag => 'option'}, Chris@441: :count => Project.find(1).users.count + 1 Chris@441: } Chris@441: end Chris@441: Chris@441: def test_get_bulk_edit_with_version_custom_field Chris@441: field = IssueCustomField.create!(:name => 'Affected version', :field_format => 'version', :is_for_all => true) Chris@441: Chris@441: @request.session[:user_id] = 2 Chris@441: get :bulk_edit, :ids => [1, 2] Chris@441: assert_response :success Chris@441: assert_template 'bulk_edit' Chris@441: Chris@441: assert_tag :select, Chris@441: :attributes => {:name => "issue[custom_field_values][#{field.id}]"}, Chris@441: :children => { Chris@441: :only => {:tag => 'option'}, Chris@909: :count => Project.find(1).shared_versions.count + 1 Chris@441: } Chris@441: end Chris@441: Chris@14: def test_bulk_update Chris@0: @request.session[:user_id] = 2 Chris@0: # update issues priority Chris@14: post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing', Chris@0: :issue => {:priority_id => 7, Chris@0: :assigned_to_id => '', Chris@0: :custom_field_values => {'2' => ''}} Chris@441: Chris@0: assert_response 302 Chris@0: # check that the issues were updated Chris@0: assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id} Chris@441: Chris@0: issue = Issue.find(1) Chris@0: journal = issue.journals.find(:first, :order => 'created_on DESC') Chris@0: assert_equal '125', issue.custom_value_for(2).value Chris@0: assert_equal 'Bulk editing', journal.notes Chris@0: assert_equal 1, journal.details.size Chris@0: end Chris@0: Chris@909: def test_bulk_update_with_group_assignee Chris@909: group = Group.find(11) Chris@909: project = Project.find(1) Chris@929: project.members << Member.new(:principal => group, :roles => [Role.givable.first]) Chris@909: Chris@909: @request.session[:user_id] = 2 Chris@909: # update issues assignee Chris@909: post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing', Chris@909: :issue => {:priority_id => '', Chris@909: :assigned_to_id => group.id, Chris@909: :custom_field_values => {'2' => ''}} Chris@909: Chris@909: assert_response 302 Chris@909: assert_equal [group, group], Issue.find_all_by_id([1, 2]).collect {|i| i.assigned_to} Chris@909: end Chris@909: chris@37: def test_bulk_update_on_different_projects chris@37: @request.session[:user_id] = 2 chris@37: # update issues priority chris@37: post :bulk_update, :ids => [1, 2, 6], :notes => 'Bulk editing', chris@37: :issue => {:priority_id => 7, chris@37: :assigned_to_id => '', chris@37: :custom_field_values => {'2' => ''}} Chris@441: chris@37: assert_response 302 chris@37: # check that the issues were updated chris@37: assert_equal [7, 7, 7], Issue.find([1,2,6]).map(&:priority_id) Chris@441: chris@37: issue = Issue.find(1) chris@37: journal = issue.journals.find(:first, :order => 'created_on DESC') chris@37: assert_equal '125', issue.custom_value_for(2).value chris@37: assert_equal 'Bulk editing', journal.notes chris@37: assert_equal 1, journal.details.size chris@37: end chris@37: chris@37: def test_bulk_update_on_different_projects_without_rights chris@37: @request.session[:user_id] = 3 chris@37: user = User.find(3) chris@37: action = { :controller => "issues", :action => "bulk_update" } chris@37: assert user.allowed_to?(action, Issue.find(1).project) chris@37: assert ! user.allowed_to?(action, Issue.find(6).project) chris@37: post :bulk_update, :ids => [1, 6], :notes => 'Bulk should fail', chris@37: :issue => {:priority_id => 7, chris@37: :assigned_to_id => '', chris@37: :custom_field_values => {'2' => ''}} chris@37: assert_response 403 chris@37: assert_not_equal "Bulk should fail", Journal.last.notes chris@37: end Chris@441: Chris@14: def test_bullk_update_should_send_a_notification Chris@0: @request.session[:user_id] = 2 Chris@0: ActionMailer::Base.deliveries.clear Chris@14: post(:bulk_update, Chris@0: { Chris@0: :ids => [1, 2], Chris@0: :notes => 'Bulk editing', Chris@0: :issue => { Chris@0: :priority_id => 7, Chris@0: :assigned_to_id => '', Chris@0: :custom_field_values => {'2' => ''} Chris@0: } Chris@0: }) Chris@0: Chris@0: assert_response 302 Chris@0: assert_equal 2, ActionMailer::Base.deliveries.size Chris@0: end Chris@0: Chris@14: def test_bulk_update_status Chris@0: @request.session[:user_id] = 2 Chris@0: # update issues priority Chris@14: post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing status', Chris@0: :issue => {:priority_id => '', Chris@0: :assigned_to_id => '', Chris@0: :status_id => '5'} Chris@441: Chris@0: assert_response 302 Chris@0: issue = Issue.find(1) Chris@0: assert issue.closed? Chris@0: end Chris@0: Chris@441: def test_bulk_update_parent_id Chris@441: @request.session[:user_id] = 2 Chris@441: post :bulk_update, :ids => [1, 3], Chris@441: :notes => 'Bulk editing parent', Chris@441: :issue => {:priority_id => '', :assigned_to_id => '', :status_id => '', :parent_issue_id => '2'} Chris@441: Chris@441: assert_response 302 Chris@441: parent = Issue.find(2) Chris@441: assert_equal parent.id, Issue.find(1).parent_id Chris@441: assert_equal parent.id, Issue.find(3).parent_id Chris@441: assert_equal [1, 3], parent.children.collect(&:id).sort Chris@441: end Chris@441: Chris@14: def test_bulk_update_custom_field Chris@0: @request.session[:user_id] = 2 Chris@0: # update issues priority Chris@14: post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing custom field', Chris@0: :issue => {:priority_id => '', Chris@0: :assigned_to_id => '', Chris@0: :custom_field_values => {'2' => '777'}} Chris@441: Chris@0: assert_response 302 Chris@441: Chris@0: issue = Issue.find(1) Chris@0: journal = issue.journals.find(:first, :order => 'created_on DESC') Chris@0: assert_equal '777', issue.custom_value_for(2).value Chris@0: assert_equal 1, journal.details.size Chris@0: assert_equal '125', journal.details.first.old_value Chris@0: assert_equal '777', journal.details.first.value Chris@0: end Chris@0: Chris@14: def test_bulk_update_unassign Chris@0: assert_not_nil Issue.find(2).assigned_to Chris@0: @request.session[:user_id] = 2 Chris@0: # unassign issues Chris@14: post :bulk_update, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'} Chris@0: assert_response 302 Chris@0: # check that the issues were updated Chris@0: assert_nil Issue.find(2).assigned_to Chris@0: end Chris@441: Chris@14: def test_post_bulk_update_should_allow_fixed_version_to_be_set_to_a_subproject Chris@0: @request.session[:user_id] = 2 Chris@0: Chris@14: post :bulk_update, :ids => [1,2], :issue => {:fixed_version_id => 4} Chris@0: Chris@0: assert_response :redirect Chris@0: issues = Issue.find([1,2]) Chris@0: issues.each do |issue| Chris@0: assert_equal 4, issue.fixed_version_id Chris@0: assert_not_equal issue.project_id, issue.fixed_version.project_id Chris@0: end Chris@0: end Chris@0: Chris@14: def test_post_bulk_update_should_redirect_back_using_the_back_url_parameter Chris@0: @request.session[:user_id] = 2 Chris@14: post :bulk_update, :ids => [1,2], :back_url => '/issues' Chris@0: Chris@0: assert_response :redirect Chris@0: assert_redirected_to '/issues' Chris@0: end Chris@0: Chris@14: def test_post_bulk_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host Chris@0: @request.session[:user_id] = 2 Chris@14: post :bulk_update, :ids => [1,2], :back_url => 'http://google.com' Chris@0: Chris@0: assert_response :redirect Chris@0: assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier Chris@0: end Chris@441: Chris@0: def test_destroy_issue_with_no_time_entries Chris@0: assert_nil TimeEntry.find_by_issue_id(2) Chris@0: @request.session[:user_id] = 2 Chris@0: post :destroy, :id => 2 Chris@0: assert_redirected_to :action => 'index', :project_id => 'ecookbook' Chris@0: assert_nil Issue.find_by_id(2) Chris@0: end Chris@0: Chris@0: def test_destroy_issues_with_time_entries Chris@0: @request.session[:user_id] = 2 Chris@0: post :destroy, :ids => [1, 3] Chris@0: assert_response :success Chris@0: assert_template 'destroy' Chris@0: assert_not_nil assigns(:hours) Chris@0: assert Issue.find_by_id(1) && Issue.find_by_id(3) Chris@0: end Chris@0: Chris@0: def test_destroy_issues_and_destroy_time_entries Chris@0: @request.session[:user_id] = 2 Chris@0: post :destroy, :ids => [1, 3], :todo => 'destroy' Chris@0: assert_redirected_to :action => 'index', :project_id => 'ecookbook' Chris@0: assert !(Issue.find_by_id(1) || Issue.find_by_id(3)) Chris@0: assert_nil TimeEntry.find_by_id([1, 2]) Chris@0: end Chris@0: Chris@0: def test_destroy_issues_and_assign_time_entries_to_project Chris@0: @request.session[:user_id] = 2 Chris@0: post :destroy, :ids => [1, 3], :todo => 'nullify' Chris@0: assert_redirected_to :action => 'index', :project_id => 'ecookbook' Chris@0: assert !(Issue.find_by_id(1) || Issue.find_by_id(3)) Chris@0: assert_nil TimeEntry.find(1).issue_id Chris@0: assert_nil TimeEntry.find(2).issue_id Chris@0: end Chris@441: Chris@0: def test_destroy_issues_and_reassign_time_entries_to_another_issue Chris@0: @request.session[:user_id] = 2 Chris@0: post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2 Chris@0: assert_redirected_to :action => 'index', :project_id => 'ecookbook' Chris@0: assert !(Issue.find_by_id(1) || Issue.find_by_id(3)) Chris@0: assert_equal 2, TimeEntry.find(1).issue_id Chris@0: assert_equal 2, TimeEntry.find(2).issue_id Chris@0: end Chris@441: chris@37: def test_destroy_issues_from_different_projects chris@37: @request.session[:user_id] = 2 chris@37: post :destroy, :ids => [1, 2, 6], :todo => 'destroy' chris@37: assert_redirected_to :controller => 'issues', :action => 'index' chris@37: assert !(Issue.find_by_id(1) || Issue.find_by_id(2) || Issue.find_by_id(6)) chris@37: end Chris@441: Chris@441: def test_destroy_parent_and_child_issues Chris@441: parent = Issue.generate!(:project_id => 1, :tracker_id => 1) Chris@441: child = Issue.generate!(:project_id => 1, :tracker_id => 1, :parent_issue_id => parent.id) Chris@441: assert child.is_descendant_of?(parent.reload) Chris@441: Chris@441: @request.session[:user_id] = 2 Chris@441: assert_difference 'Issue.count', -2 do Chris@441: post :destroy, :ids => [parent.id, child.id], :todo => 'destroy' Chris@441: end Chris@441: assert_response 302 Chris@441: end Chris@441: Chris@0: def test_default_search_scope Chris@0: get :index Chris@0: assert_tag :div, :attributes => {:id => 'quick-search'}, Chris@0: :child => {:tag => 'form', Chris@0: :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}} Chris@0: end Chris@0: end