Chris@0: # Redmine - project management software Chris@1494: # Copyright (C) 2006-2014 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: 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@1115: :queries, Chris@1115: :repositories, Chris@1115: :changesets Chris@441: Chris@909: include Redmine::I18n Chris@909: Chris@0: def setup Chris@0: User.current = nil Chris@0: end Chris@441: Chris@0: def test_index Chris@1115: with_settings :default_language => "en" do Chris@1115: get :index Chris@1115: assert_response :success Chris@1115: assert_template 'index' Chris@1115: assert_not_nil assigns(:issues) Chris@1115: assert_nil assigns(:project) Chris@1115: Chris@1115: # links to visible issues Chris@1115: assert_select 'a[href=/issues/1]', :text => /Can't print recipes/ Chris@1115: assert_select 'a[href=/issues/5]', :text => /Subproject issue/ Chris@1115: # private projects hidden Chris@1115: assert_select 'a[href=/issues/6]', 0 Chris@1115: assert_select 'a[href=/issues/4]', 0 Chris@1115: # project column Chris@1115: assert_select 'th', :text => /Project/ Chris@1115: end 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@1115: Chris@1115: assert_select 'a[href=/issues/1]', 0 Chris@1115: assert_select 'a[href=/issues/5]', :text => /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@1115: Chris@1115: assert_select 'a[href=/issues/1]', :text => /Can't print recipes/ Chris@1115: assert_select 'a[href=/issues/5]', 0 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@1115: Chris@1115: assert_select 'a[href=/issues/1]', :text => /Can't print recipes/ Chris@1115: assert_select 'a[href=/issues/5]', :text => /Subproject issue/ Chris@1115: assert_select 'a[href=/issues/6]', 0 Chris@0: end Chris@441: Chris@1115: def test_index_with_project_and_subprojects_should_show_private_subprojects_with_permission 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@1115: Chris@1115: assert_select 'a[href=/issues/1]', :text => /Can't print recipes/ Chris@1115: assert_select 'a[href=/issues/5]', :text => /Subproject issue/ Chris@1115: assert_select 'a[href=/issues/6]', :text => /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@1115: def test_index_with_project_custom_field_filter Chris@1115: field = ProjectCustomField.create!(:name => 'Client', :is_filter => true, :field_format => 'string') Chris@1115: CustomValue.create!(:custom_field => field, :customized => Project.find(3), :value => 'Foo') Chris@1115: CustomValue.create!(:custom_field => field, :customized => Project.find(5), :value => 'Foo') Chris@1115: filter_name = "project.cf_#{field.id}" Chris@1115: @request.session[:user_id] = 1 Chris@1115: Chris@1115: get :index, :set_filter => 1, Chris@1115: :f => [filter_name], Chris@1115: :op => {filter_name => '='}, Chris@1115: :v => {filter_name => ['Foo']} Chris@1115: assert_response :success Chris@1115: assert_template 'index' Chris@1115: assert_equal [3, 5], assigns(:issues).map(&:project_id).uniq.sort Chris@1115: end Chris@1115: 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@1115: def test_index_with_query_grouped_by_user_custom_field Chris@1115: cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '') Chris@1115: Chris@1115: get :index, :project_id => 1, :set_filter => 1, :group_by => "cf_#{cf.id}" Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'tr.group', 3 Chris@1115: assert_select 'tr.group' do Chris@1115: assert_select 'a', :text => 'John Smith' Chris@1115: assert_select 'span.count', :text => '1' Chris@1115: end Chris@1115: assert_select 'tr.group' do Chris@1115: assert_select 'a', :text => 'Dave Lopper' Chris@1115: assert_select 'span.count', :text => '2' Chris@1115: end Chris@1115: end Chris@1115: Chris@1464: def test_index_with_query_grouped_by_tracker_in_normal_order Chris@1115: 3.times {|i| Issue.generate!(:tracker_id => (i + 1))} Chris@1115: Chris@1115: get :index, :set_filter => 1, :group_by => 'tracker', :sort => 'id:desc' Chris@1115: assert_response :success Chris@1115: Chris@1115: trackers = assigns(:issues).map(&:tracker).uniq Chris@1115: assert_equal [1, 2, 3], trackers.map(&:id) Chris@1115: end Chris@1115: Chris@1115: def test_index_with_query_grouped_by_tracker_in_reverse_order Chris@1115: 3.times {|i| Issue.generate!(:tracker_id => (i + 1))} Chris@1115: Chris@1115: get :index, :set_filter => 1, :group_by => 'tracker', :sort => 'id:desc,tracker:desc' Chris@1115: assert_response :success Chris@1115: Chris@1115: trackers = assigns(:issues).map(&:tracker).uniq Chris@1115: assert_equal [3, 2, 1], trackers.map(&:id) Chris@1115: end Chris@1115: 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@1115: def test_index_with_invalid_query_id_should_respond_404 Chris@1115: get :index, :project_id => 1, :query_id => 999 Chris@1115: assert_response 404 Chris@1115: end Chris@1115: Chris@909: def test_index_with_cross_project_query_in_session_should_show_project_issues Chris@1464: q = IssueQuery.create!(:name => "test", :user_id => 2, :visibility => IssueQuery::VISIBILITY_PRIVATE, :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@1464: q = IssueQuery.create!(:name => "private", :user => User.find(2), :visibility => IssueQuery::VISIBILITY_PRIVATE, :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@1464: q = IssueQuery.create!(:name => "private", :user => User.find(2), :visibility => IssueQuery::VISIBILITY_PRIVATE, :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@1464: q = IssueQuery.create!(:name => "private", :user => User.find(2), :visibility => IssueQuery::VISIBILITY_PUBLIC, :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@1115: def test_index_should_omit_page_param_in_export_links Chris@1115: get :index, :page => 2 Chris@1115: assert_response :success Chris@1115: assert_select 'a.atom[href=/issues.atom]' Chris@1115: assert_select 'a.csv[href=/issues.csv]' Chris@1115: assert_select 'a.pdf[href=/issues.pdf]' Chris@1115: assert_select 'form#csv-export-form[action=/issues.csv]' Chris@1115: end Chris@1115: 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@1115: assert_equal 'text/csv; header=present', @response.content_type Chris@0: assert @response.body.starts_with?("#,") Chris@909: lines = @response.body.chomp.split("\n") Chris@1464: assert_equal assigns(:query).columns.size, 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@1115: assert_equal 'text/csv; header=present', @response.content_type Chris@0: end Chris@441: Chris@909: def test_index_csv_with_description Chris@1464: Issue.generate!(:description => 'test_index_csv_with_description') Chris@1464: Chris@1464: with_settings :default_language => 'en' do Chris@1464: get :index, :format => 'csv', :description => '1' Chris@1464: assert_response :success Chris@1464: assert_not_nil assigns(:issues) Chris@1464: end Chris@1464: Chris@1464: assert_equal 'text/csv; header=present', response.content_type Chris@1464: headers = response.body.chomp.split("\n").first.split(',') Chris@1464: assert_include 'Description', headers Chris@1464: assert_include 'test_index_csv_with_description', response.body Chris@909: end Chris@441: Chris@1115: def test_index_csv_with_spent_time_column Chris@1115: issue = Issue.create!(:project_id => 1, :tracker_id => 1, :subject => 'test_index_csv_with_spent_time_column', :author_id => 2) Chris@1115: TimeEntry.create!(:project => issue.project, :issue => issue, :hours => 7.33, :user => User.find(2), :spent_on => Date.today) Chris@1115: Chris@1115: get :index, :format => 'csv', :set_filter => '1', :c => %w(subject spent_hours) Chris@1115: assert_response :success Chris@1115: assert_equal 'text/csv; header=present', @response.content_type Chris@1115: lines = @response.body.chomp.split("\n") Chris@1115: assert_include "#{issue.id},#{issue.subject},7.33", lines Chris@1115: end Chris@1115: 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@1115: assert_equal 'text/csv; header=present', @response.content_type Chris@1464: assert_match /\A#,/, response.body Chris@1464: lines = response.body.chomp.split("\n") Chris@1464: assert_equal assigns(:query).available_inline_columns.size, lines[0].split(',').size Chris@1115: end Chris@1115: Chris@1115: def test_index_csv_with_multi_column_field Chris@1115: CustomField.find(1).update_attribute :multiple, true Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {1 => ['MySQL', 'Oracle']} Chris@1115: issue.save! Chris@1115: Chris@1115: get :index, :format => 'csv', :columns => 'all' Chris@1115: assert_response :success Chris@1115: lines = @response.body.chomp.split("\n") Chris@1115: assert lines.detect {|line| line.include?('"MySQL, Oracle"')} Chris@909: end Chris@441: Chris@1464: def test_index_csv_should_format_float_custom_fields_with_csv_decimal_separator Chris@1464: field = IssueCustomField.create!(:name => 'Float', :is_for_all => true, :tracker_ids => [1], :field_format => 'float') Chris@1464: issue = Issue.generate!(:project_id => 1, :tracker_id => 1, :custom_field_values => {field.id => '185.6'}) Chris@1464: Chris@1464: with_settings :default_language => 'fr' do Chris@1464: get :index, :format => 'csv', :columns => 'all' Chris@1464: assert_response :success Chris@1464: issue_line = response.body.chomp.split("\n").map {|line| line.split(';')}.detect {|line| line[0]==issue.id.to_s} Chris@1464: assert_include '185,60', issue_line Chris@1464: end Chris@1464: Chris@1464: with_settings :default_language => 'en' do Chris@1464: get :index, :format => 'csv', :columns => 'all' Chris@1464: assert_response :success Chris@1464: issue_line = response.body.chomp.split("\n").map {|line| line.split(',')}.detect {|line| line[0]==issue.id.to_s} Chris@1464: assert_include '185.60', issue_line Chris@1464: end Chris@1464: end Chris@1464: 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@1115: issue = Issue.generate!(:subject => str_utf8) Chris@909: Chris@909: get :index, :project_id => 1, Chris@909: :f => ['subject'], Chris@909: :op => '=', :values => [str_utf8], Chris@909: :format => 'csv' Chris@1115: assert_equal 'text/csv; header=present', @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@1464: assert_include s1, lines[0] Chris@1464: assert_include str_big5, lines[1] 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@1115: issue = Issue.generate!(:subject => str_utf8) 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@1115: assert_equal 'text/csv; header=present', @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@1115: issue = Issue.generate!(:subject => str1, :estimated_hours => '1234.5') 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@1115: assert_equal 'text/csv; header=present', @response.content_type Chris@909: lines = @response.body.chomp.split("\n") Chris@1115: assert_equal "#{issue.id},1234.50,#{str1}", lines[1] 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@1115: issue = Issue.generate!(:subject => str1, :estimated_hours => '1234.5') 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@1115: assert_equal 'text/csv; header=present', @response.content_type Chris@909: lines = @response.body.chomp.split("\n") Chris@1115: assert_equal "#{issue.id};1234,50;#{str1}", lines[1] 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@1115: def test_index_atom Chris@1115: get :index, :project_id => 'ecookbook', :format => 'atom' Chris@1115: assert_response :success Chris@1115: assert_template 'common/feed' Chris@1115: assert_equal 'application/atom+xml', response.content_type Chris@1115: Chris@1115: assert_select 'feed' do Chris@1115: assert_select 'link[rel=self][href=?]', 'http://test.host/projects/ecookbook/issues.atom' Chris@1115: assert_select 'link[rel=alternate][href=?]', 'http://test.host/projects/ecookbook/issues' Chris@1115: assert_select 'entry link[href=?]', 'http://test.host/issues/1' Chris@1115: end Chris@1115: end Chris@1115: 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@1115: Chris@1115: def test_index_sort_by_spent_hours Chris@1115: get :index, :sort => 'spent_hours:desc' Chris@1115: assert_response :success Chris@1115: hours = assigns(:issues).collect(&:spent_hours) Chris@1115: assert_equal hours.sort.reverse, hours Chris@1115: end Chris@1115: Chris@1115: def test_index_sort_by_user_custom_field Chris@1115: cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '') Chris@1115: Chris@1115: get :index, :project_id => 1, :set_filter => 1, :sort => "cf_#{cf.id},id" Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_equal [2, 3, 1], assigns(:issues).select {|issue| issue.custom_field_value(cf).present?}.map(&:id) Chris@1115: 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@1464: assert_kind_of IssueQuery, 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@1115: assert_select 'select#selected_columns option' do Chris@1115: assert_select 'option', 3 Chris@1115: assert_select 'option[value=tracker]' Chris@1115: assert_select 'option[value=project]', 0 Chris@1115: end 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@1464: assert_kind_of IssueQuery, query Chris@1464: assert_equal [:id, :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@1464: columns = ['id', '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@1464: assert_kind_of IssueQuery, 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@1464: assert_kind_of IssueQuery, query Chris@441: assert_equal columns, query.column_names.map(&:to_s) Chris@441: Chris@1115: assert_select 'table.issues td.cf_2.string' Chris@441: end Chris@441: Chris@1115: def test_index_with_multi_custom_field_column Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {1 => ['MySQL', 'Oracle']} Chris@1115: issue.save! Chris@1115: Chris@1115: get :index, :set_filter => 1, :c => %w(tracker subject cf_1) Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'table.issues td.cf_1', :text => 'MySQL, Oracle' Chris@1115: end Chris@1115: Chris@1115: def test_index_with_multi_user_custom_field_column Chris@1115: field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true, Chris@1115: :tracker_ids => [1], :is_for_all => true) Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {field.id => ['2', '3']} Chris@1115: issue.save! Chris@1115: Chris@1115: get :index, :set_filter => 1, :c => ['tracker', 'subject', "cf_#{field.id}"] Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select "table.issues td.cf_#{field.id}" do Chris@1115: assert_select 'a', 2 Chris@1115: assert_select 'a[href=?]', '/users/2', :text => 'John Smith' Chris@1115: assert_select 'a[href=?]', '/users/3', :text => 'Dave Lopper' Chris@909: end Chris@909: end Chris@909: Chris@1115: def test_index_with_date_column Chris@1115: with_settings :date_format => '%d/%m/%Y' do Chris@1115: Issue.find(1).update_attribute :start_date, '1987-08-24' Chris@1115: get :index, :set_filter => 1, :c => %w(start_date) Chris@1115: assert_select "table.issues td.start_date", :text => '24/08/1987' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_index_with_done_ratio_column Chris@909: Issue.find(1).update_attribute :done_ratio, 40 Chris@909: get :index, :set_filter => 1, :c => %w(done_ratio) Chris@1115: assert_select 'table.issues td.done_ratio' do Chris@1115: assert_select 'table.progress' do Chris@1115: assert_select 'td.closed[style=?]', 'width: 40%;' Chris@1115: end Chris@1115: end Chris@909: end Chris@909: Chris@1115: def test_index_with_spent_hours_column Chris@1115: get :index, :set_filter => 1, :c => %w(subject spent_hours) Chris@1115: assert_select 'table.issues tr#issue-3 td.spent_hours', :text => '1.00' Chris@1115: end Chris@1115: Chris@1115: def test_index_should_not_show_spent_hours_column_without_permission Chris@1115: Role.anonymous.remove_permission! :view_time_entries Chris@1115: get :index, :set_filter => 1, :c => %w(subject spent_hours) Chris@1115: assert_select 'td.spent_hours', 0 Chris@1115: end Chris@1115: Chris@1115: def test_index_with_fixed_version_column Chris@909: get :index, :set_filter => 1, :c => %w(fixed_version) Chris@1115: assert_select 'table.issues td.fixed_version' do Chris@1115: assert_select 'a[href=?]', '/versions/2', :text => '1.0' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_index_with_relations_column Chris@1115: IssueRelation.delete_all Chris@1115: IssueRelation.create!(:relation_type => "relates", :issue_from => Issue.find(1), :issue_to => Issue.find(7)) Chris@1115: IssueRelation.create!(:relation_type => "relates", :issue_from => Issue.find(8), :issue_to => Issue.find(1)) Chris@1115: IssueRelation.create!(:relation_type => "blocks", :issue_from => Issue.find(1), :issue_to => Issue.find(11)) Chris@1115: IssueRelation.create!(:relation_type => "blocks", :issue_from => Issue.find(12), :issue_to => Issue.find(2)) Chris@1115: Chris@1115: get :index, :set_filter => 1, :c => %w(subject relations) Chris@1115: assert_response :success Chris@1115: assert_select "tr#issue-1 td.relations" do Chris@1115: assert_select "span", 3 Chris@1115: assert_select "span", :text => "Related to #7" Chris@1115: assert_select "span", :text => "Related to #8" Chris@1115: assert_select "span", :text => "Blocks #11" Chris@1115: end Chris@1115: assert_select "tr#issue-2 td.relations" do Chris@1115: assert_select "span", 1 Chris@1115: assert_select "span", :text => "Blocked by #12" Chris@1115: end Chris@1115: assert_select "tr#issue-3 td.relations" do Chris@1115: assert_select "span", 0 Chris@1115: end Chris@1115: Chris@1115: get :index, :set_filter => 1, :c => %w(relations), :format => 'csv' Chris@1115: assert_response :success Chris@1115: assert_equal 'text/csv; header=present', response.content_type Chris@1115: lines = response.body.chomp.split("\n") Chris@1115: assert_include '1,"Related to #7, Related to #8, Blocks #11"', lines Chris@1115: assert_include '2,Blocked by #12', lines Chris@1115: assert_include '3,""', lines Chris@1115: Chris@1115: get :index, :set_filter => 1, :c => %w(subject relations), :format => 'pdf' Chris@1115: assert_response :success Chris@1115: assert_equal 'application/pdf', response.content_type Chris@1115: end Chris@1115: Chris@1115: def test_index_with_description_column Chris@1115: get :index, :set_filter => 1, :c => %w(subject description) Chris@1115: Chris@1115: assert_select 'table.issues thead th', 3 # columns: chekbox + id + subject Chris@1115: assert_select 'td.description[colspan=3]', :text => 'Unable to print recipes' Chris@1115: Chris@1115: get :index, :set_filter => 1, :c => %w(subject description), :format => 'pdf' Chris@1115: assert_response :success Chris@1115: assert_equal 'application/pdf', response.content_type 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_equal Issue.find(1), assigns(:issue) Chris@1115: assert_select 'div.issue div.description', :text => /Unable to print recipes/ Chris@0: # anonymous role is allowed to add a note Chris@1115: assert_select 'form#issue-form' do Chris@1115: assert_select 'fieldset' do Chris@1115: assert_select 'legend', :text => 'Notes' Chris@1115: assert_select 'textarea[name=?]', 'issue[notes]' Chris@1115: end Chris@1115: end Chris@1115: assert_select 'title', :text => "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@1115: assert_select 'a', :text => /Quote/ Chris@1115: assert_select 'form#issue-form' do Chris@1115: assert_select 'fieldset' do Chris@1115: assert_select 'legend', :text => 'Change properties' Chris@1115: assert_select 'input[name=?]', 'issue[subject]' Chris@1115: end Chris@1115: assert_select 'fieldset' do Chris@1115: assert_select 'legend', :text => 'Log time' Chris@1115: assert_select 'input[name=?]', 'time_entry[hours]' Chris@1115: end Chris@1115: assert_select 'fieldset' do Chris@1115: assert_select 'legend', :text => 'Notes' Chris@1115: assert_select 'textarea[name=?]', 'issue[notes]' Chris@1115: end Chris@1115: end Chris@0: end Chris@441: Chris@1115: def test_show_should_display_update_form Chris@909: @request.session[:user_id] = 2 Chris@909: get :show, :id => 1 Chris@909: assert_response :success Chris@909: Chris@1464: assert_select 'form#issue-form' do Chris@1464: assert_select 'input[name=?]', 'issue[is_private]' Chris@1464: assert_select 'select[name=?]', 'issue[project_id]' Chris@1464: assert_select 'select[name=?]', 'issue[tracker_id]' Chris@1464: assert_select 'input[name=?]', 'issue[subject]' Chris@1464: assert_select 'textarea[name=?]', 'issue[description]' Chris@1464: assert_select 'select[name=?]', 'issue[status_id]' Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' Chris@1464: assert_select 'select[name=?]', 'issue[assigned_to_id]' Chris@1464: assert_select 'select[name=?]', 'issue[category_id]' Chris@1464: assert_select 'select[name=?]', 'issue[fixed_version_id]' Chris@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]' Chris@1464: assert_select 'input[name=?]', 'issue[start_date]' Chris@1464: assert_select 'input[name=?]', 'issue[due_date]' Chris@1464: assert_select 'select[name=?]', 'issue[done_ratio]' Chris@1464: assert_select 'input[name=?]', 'issue[custom_field_values][2]' Chris@1464: assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0 Chris@1464: assert_select 'textarea[name=?]', 'issue[notes]' Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_update_form_with_minimal_permissions Chris@1115: Role.find(1).update_attribute :permissions, [:view_issues, :add_issue_notes] Chris@1115: WorkflowTransition.delete_all :role_id => 1 Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1464: assert_select 'form#issue-form' do Chris@1464: assert_select 'input[name=?]', 'issue[is_private]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[project_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[tracker_id]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[subject]', 0 Chris@1464: assert_select 'textarea[name=?]', 'issue[description]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[status_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[assigned_to_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[category_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[fixed_version_id]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[start_date]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[due_date]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[done_ratio]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[custom_field_values][2]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0 Chris@1464: assert_select 'textarea[name=?]', 'issue[notes]' Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_update_form_with_workflow_permissions Chris@1115: Role.find(1).update_attribute :permissions, [:view_issues, :add_issue_notes] Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1464: assert_select 'form#issue-form' do Chris@1464: assert_select 'input[name=?]', 'issue[is_private]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[project_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[tracker_id]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[subject]', 0 Chris@1464: assert_select 'textarea[name=?]', 'issue[description]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[status_id]' Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[assigned_to_id]' Chris@1464: assert_select 'select[name=?]', 'issue[category_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[fixed_version_id]' Chris@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[start_date]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[due_date]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[done_ratio]' Chris@1464: assert_select 'input[name=?]', 'issue[custom_field_values][2]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0 Chris@1464: assert_select 'textarea[name=?]', 'issue[notes]' Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_not_display_update_form_without_permissions Chris@1115: Role.find(1).update_attribute :permissions, [:view_issues] Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'form#issue-form', 0 Chris@1115: end Chris@1115: Chris@1115: def test_update_form_should_not_display_inactive_enumerations Chris@1115: assert !IssuePriority.find(15).active? Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'form#issue-form' do Chris@1115: assert_select 'select[name=?]', 'issue[priority_id]' do Chris@1115: assert_select 'option[value=4]' Chris@1115: assert_select 'option[value=15]', 0 Chris@1115: end Chris@1115: end 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@1115: assert_select 'form#issue-form[method=post][enctype=multipart/form-data]' do Chris@1464: assert_select 'input[type=file][name=?]', 'attachments[dummy][file]' Chris@1115: end 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@1115: assert_select 'div#relations' do Chris@1115: assert_select 'a', :text => /#2$/ Chris@1115: assert_select 'a', :text => /#4$/, :count => 0 Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_list_subtasks Chris@1115: Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :parent_issue_id => 1, :subject => 'Child Issue') Chris@1115: Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'div#issue_tree' do Chris@1115: assert_select 'td.subject', :text => /Child Issue/ Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_list_parents Chris@1115: issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :parent_issue_id => 1, :subject => 'Child Issue') Chris@1115: Chris@1115: get :show, :id => issue.id Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'div.subject' do Chris@1115: assert_select 'h3', 'Child Issue' Chris@1115: assert_select 'a[href=/issues/1]' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_not_display_prev_next_links_without_query_in_session Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: assert_nil assigns(:prev_issue_id) Chris@1115: assert_nil assigns(:next_issue_id) Chris@1115: Chris@1115: assert_select 'div.next-prev-links', 0 Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_prev_next_links_with_query_in_session Chris@1115: @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => nil} Chris@1115: @request.session['issues_index_sort'] = 'id' Chris@1115: Chris@1115: with_settings :display_subprojects_issues => '0' do Chris@1115: get :show, :id => 3 Chris@1115: end Chris@1115: Chris@1115: assert_response :success Chris@1115: # Previous and next issues for all projects Chris@1115: assert_equal 2, assigns(:prev_issue_id) Chris@1115: assert_equal 5, assigns(:next_issue_id) Chris@1115: Chris@1115: count = Issue.open.visible.count Chris@1115: Chris@1115: assert_select 'div.next-prev-links' do Chris@1115: assert_select 'a[href=/issues/2]', :text => /Previous/ Chris@1115: assert_select 'a[href=/issues/5]', :text => /Next/ Chris@1115: assert_select 'span.position', :text => "3 of #{count}" Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_prev_next_links_with_saved_query_in_session Chris@1464: query = IssueQuery.create!(:name => 'test', :visibility => IssueQuery::VISIBILITY_PUBLIC, :user_id => 1, Chris@1115: :filters => {'status_id' => {:values => ['5'], :operator => '='}}, Chris@1115: :sort_criteria => [['id', 'asc']]) Chris@1115: @request.session[:query] = {:id => query.id, :project_id => nil} Chris@1115: Chris@1115: get :show, :id => 11 Chris@1115: Chris@1115: assert_response :success Chris@1115: assert_equal query, assigns(:query) Chris@1115: # Previous and next issues for all projects Chris@1115: assert_equal 8, assigns(:prev_issue_id) Chris@1115: assert_equal 12, assigns(:next_issue_id) Chris@1115: Chris@1115: assert_select 'div.next-prev-links' do Chris@1115: assert_select 'a[href=/issues/8]', :text => /Previous/ Chris@1115: assert_select 'a[href=/issues/12]', :text => /Next/ Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_prev_next_links_with_query_and_sort_on_association Chris@1115: @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => nil} Chris@1115: Chris@1115: %w(project tracker status priority author assigned_to category fixed_version).each do |assoc_sort| Chris@1115: @request.session['issues_index_sort'] = assoc_sort Chris@1115: Chris@1115: get :show, :id => 3 Chris@1115: assert_response :success, "Wrong response status for #{assoc_sort} sort" Chris@1115: Chris@1115: assert_select 'div.next-prev-links' do Chris@1115: assert_select 'a', :text => /(Previous|Next)/ Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_prev_next_links_with_project_query_in_session Chris@1115: @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => 1} Chris@1115: @request.session['issues_index_sort'] = 'id' Chris@1115: Chris@1115: with_settings :display_subprojects_issues => '0' do Chris@1115: get :show, :id => 3 Chris@1115: end Chris@1115: Chris@1115: assert_response :success Chris@1115: # Previous and next issues inside project Chris@1115: assert_equal 2, assigns(:prev_issue_id) Chris@1115: assert_equal 7, assigns(:next_issue_id) Chris@1115: Chris@1115: assert_select 'div.next-prev-links' do Chris@1115: assert_select 'a[href=/issues/2]', :text => /Previous/ Chris@1115: assert_select 'a[href=/issues/7]', :text => /Next/ Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_not_display_prev_link_for_first_issue Chris@1115: @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => 1} Chris@1115: @request.session['issues_index_sort'] = 'id' Chris@1115: Chris@1115: with_settings :display_subprojects_issues => '0' do Chris@1115: get :show, :id => 1 Chris@1115: end Chris@1115: Chris@1115: assert_response :success Chris@1115: assert_nil assigns(:prev_issue_id) Chris@1115: assert_equal 2, assigns(:next_issue_id) Chris@1115: Chris@1115: assert_select 'div.next-prev-links' do Chris@1115: assert_select 'a', :text => /Previous/, :count => 0 Chris@1115: assert_select 'a[href=/issues/2]', :text => /Next/ Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_not_display_prev_next_links_for_issue_not_in_query_results Chris@1115: @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'c'}}, :project_id => 1} Chris@1115: @request.session['issues_index_sort'] = 'id' Chris@1115: Chris@1115: get :show, :id => 1 Chris@1115: Chris@1115: assert_response :success Chris@1115: assert_nil assigns(:prev_issue_id) Chris@1115: assert_nil assigns(:next_issue_id) Chris@1115: Chris@1115: assert_select 'a', :text => /Previous/, :count => 0 Chris@1115: assert_select 'a', :text => /Next/, :count => 0 Chris@1115: end Chris@1115: Chris@1115: def test_show_show_should_display_prev_next_links_with_query_sort_by_user_custom_field Chris@1115: cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3') Chris@1115: CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '') Chris@1115: Chris@1464: query = IssueQuery.create!(:name => 'test', :visibility => IssueQuery::VISIBILITY_PUBLIC, :user_id => 1, :filters => {}, Chris@1115: :sort_criteria => [["cf_#{cf.id}", 'asc'], ['id', 'asc']]) Chris@1115: @request.session[:query] = {:id => query.id, :project_id => nil} Chris@1115: Chris@1115: get :show, :id => 3 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_equal 2, assigns(:prev_issue_id) Chris@1115: assert_equal 1, assigns(:next_issue_id) Chris@1115: Chris@1115: assert_select 'div.next-prev-links' do Chris@1115: assert_select 'a[href=/issues/2]', :text => /Previous/ Chris@1115: assert_select 'a[href=/issues/1]', :text => /Next/ Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_link_to_the_assignee Chris@1115: get :show, :id => 2 Chris@1115: assert_response :success Chris@1115: assert_select '.assigned-to' do Chris@1115: assert_select 'a[href=/users/3]' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_visible_changesets_from_other_projects Chris@1115: project = Project.find(2) Chris@1115: issue = project.issues.first Chris@1115: issue.changeset_ids = [102] Chris@1115: issue.save! Chris@1115: # changesets from other projects should be displayed even if repository Chris@1115: # is disabled on issue's project Chris@1115: project.disable_module! :repository Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :show, :id => issue.id Chris@1115: Chris@1115: assert_select 'a[href=?]', '/projects/ecookbook/repository/revisions/3' Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_watchers Chris@1115: @request.session[:user_id] = 2 Chris@1115: Issue.find(1).add_watcher User.find(2) Chris@1115: Chris@1115: get :show, :id => 1 Chris@1115: assert_select 'div#watchers ul' do Chris@1115: assert_select 'li' do Chris@1115: assert_select 'a[href=/users/2]' Chris@1115: assert_select 'a img[alt=Delete]' Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_watchers_with_gravatars Chris@1115: @request.session[:user_id] = 2 Chris@1115: Issue.find(1).add_watcher User.find(2) Chris@1115: Chris@1115: with_settings :gravatar_enabled => '1' do Chris@1115: get :show, :id => 1 Chris@1115: end Chris@1115: Chris@1115: assert_select 'div#watchers ul' do Chris@1115: assert_select 'li' do Chris@1115: assert_select 'img.gravatar' Chris@1115: assert_select 'a[href=/users/2]' Chris@1115: assert_select 'a img[alt=Delete]' Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_with_thumbnails_enabled_should_display_thumbnails Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: with_settings :thumbnails_enabled => '1' do Chris@1115: get :show, :id => 14 Chris@1115: assert_response :success Chris@1115: end Chris@1115: Chris@1115: assert_select 'div.thumbnails' do Chris@1115: assert_select 'a[href=/attachments/16/testfile.png]' do Chris@1115: assert_select 'img[src=/attachments/thumbnail/16]' Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_show_with_thumbnails_disabled_should_not_display_thumbnails Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: with_settings :thumbnails_enabled => '0' do Chris@1115: get :show, :id => 14 Chris@1115: assert_response :success Chris@1115: end Chris@1115: Chris@1115: assert_select 'div.thumbnails', 0 Chris@1115: end Chris@1115: Chris@1115: def test_show_with_multi_custom_field Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {1 => ['MySQL', 'Oracle']} Chris@1115: issue.save! Chris@1115: Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'td', :text => 'MySQL, Oracle' Chris@1115: end Chris@1115: Chris@1115: def test_show_with_multi_user_custom_field Chris@1115: field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true, Chris@1115: :tracker_ids => [1], :is_for_all => true) Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {field.id => ['2', '3']} Chris@1115: issue.save! Chris@1115: Chris@1115: get :show, :id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: # TODO: should display links Chris@1115: assert_select 'td', :text => 'Dave Lopper, John Smith' Chris@1115: end Chris@1115: Chris@1115: def test_show_should_display_private_notes_with_permission_only Chris@1115: journal = Journal.create!(:journalized => Issue.find(2), :notes => 'Privates notes', :private_notes => true, :user_id => 1) Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: get :show, :id => 2 Chris@1115: assert_response :success Chris@1115: assert_include journal, assigns(:journals) Chris@1115: Chris@1115: Role.find(1).remove_permission! :view_private_notes Chris@1115: get :show, :id => 2 Chris@1115: assert_response :success Chris@1115: assert_not_include journal, assigns(:journals) 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@1115: def test_show_export_to_pdf_with_ancestors Chris@1115: issue = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1) Chris@1115: Chris@1115: get :show, :id => issue.id, :format => 'pdf' Chris@1115: assert_response :success Chris@1115: assert_equal 'application/pdf', @response.content_type Chris@1115: assert @response.body.starts_with?('%PDF') Chris@1115: end Chris@1115: Chris@1115: def test_show_export_to_pdf_with_descendants Chris@1115: c1 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1) Chris@1115: c2 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1) Chris@1115: c3 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => c1.id) Chris@1115: Chris@1115: get :show, :id => 1, :format => 'pdf' Chris@1115: assert_response :success Chris@1115: assert_equal 'application/pdf', @response.content_type Chris@1115: assert @response.body.starts_with?('%PDF') Chris@1115: end Chris@1115: Chris@1115: def test_show_export_to_pdf_with_journals Chris@1115: get :show, :id => 1, :format => 'pdf' Chris@1115: assert_response :success Chris@1115: assert_equal 'application/pdf', @response.content_type Chris@1115: assert @response.body.starts_with?('%PDF') Chris@1115: end Chris@1115: Chris@1115: def test_show_export_to_pdf_with_changesets Chris@1115: Issue.find(3).changesets = Changeset.find_all_by_id(100, 101, 102) Chris@1115: Chris@1115: get :show, :id => 3, :format => 'pdf' Chris@1115: assert_response :success Chris@1115: assert_equal 'application/pdf', @response.content_type Chris@1115: assert @response.body.starts_with?('%PDF') Chris@1115: end Chris@1115: Chris@1464: def test_show_invalid_should_respond_with_404 Chris@1464: get :show, :id => 999 Chris@1464: assert_response 404 Chris@1464: end Chris@1464: 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@1464: assert_select 'form#issue-form' do Chris@1464: assert_select 'input[name=?]', 'issue[is_private]' Chris@1464: assert_select 'select[name=?]', 'issue[project_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[tracker_id]' Chris@1464: assert_select 'input[name=?]', 'issue[subject]' Chris@1464: assert_select 'textarea[name=?]', 'issue[description]' Chris@1464: assert_select 'select[name=?]', 'issue[status_id]' Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' Chris@1464: assert_select 'select[name=?]', 'issue[assigned_to_id]' Chris@1464: assert_select 'select[name=?]', 'issue[category_id]' Chris@1464: assert_select 'select[name=?]', 'issue[fixed_version_id]' Chris@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]' Chris@1464: assert_select 'input[name=?]', 'issue[start_date]' Chris@1464: assert_select 'input[name=?]', 'issue[due_date]' Chris@1464: assert_select 'select[name=?]', 'issue[done_ratio]' Chris@1464: assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Default string' Chris@1464: assert_select 'input[name=?]', 'issue[watcher_user_ids][]' Chris@1464: end Chris@909: Chris@909: # Be sure we don't display inactive IssuePriorities Chris@909: assert ! IssuePriority.find(15).active? Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' do Chris@1464: assert_select 'option[value=15]', 0 Chris@1464: end Chris@909: end Chris@909: Chris@1115: def test_get_new_with_minimal_permissions Chris@1115: Role.find(1).update_attribute :permissions, [:add_issues] Chris@1115: WorkflowTransition.delete_all :role_id => 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@1464: assert_select 'form#issue-form' do Chris@1464: assert_select 'input[name=?]', 'issue[is_private]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[project_id]', 0 Chris@1464: assert_select 'select[name=?]', 'issue[tracker_id]' Chris@1464: assert_select 'input[name=?]', 'issue[subject]' Chris@1464: assert_select 'textarea[name=?]', 'issue[description]' Chris@1464: assert_select 'select[name=?]', 'issue[status_id]' Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' Chris@1464: assert_select 'select[name=?]', 'issue[assigned_to_id]' Chris@1464: assert_select 'select[name=?]', 'issue[category_id]' Chris@1464: assert_select 'select[name=?]', 'issue[fixed_version_id]' Chris@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]', 0 Chris@1464: assert_select 'input[name=?]', 'issue[start_date]' Chris@1464: assert_select 'input[name=?]', 'issue[due_date]' Chris@1464: assert_select 'select[name=?]', 'issue[done_ratio]' Chris@1464: assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Default string' Chris@1464: assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0 Chris@1464: end Chris@909: end Chris@909: Chris@1115: def test_get_new_with_list_custom_field 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@1115: assert_select 'select.list_cf[name=?]', 'issue[custom_field_values][1]' do Chris@1115: assert_select 'option', 4 Chris@1115: assert_select 'option[value=MySQL]', :text => 'MySQL' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_get_new_with_multi_custom_field Chris@1115: field = IssueCustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :new, :project_id => 1, :tracker_id => 1 Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: Chris@1115: assert_select 'select[name=?][multiple=multiple]', 'issue[custom_field_values][1][]' do Chris@1115: assert_select 'option', 3 Chris@1115: assert_select 'option[value=MySQL]', :text => 'MySQL' Chris@1115: end Chris@1115: assert_select 'input[name=?][type=hidden][value=?]', 'issue[custom_field_values][1][]', '' Chris@1115: end Chris@1115: Chris@1115: def test_get_new_with_multi_user_custom_field Chris@1115: field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true, Chris@1115: :tracker_ids => [1], :is_for_all => true) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :new, :project_id => 1, :tracker_id => 1 Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: Chris@1115: assert_select 'select[name=?][multiple=multiple]', "issue[custom_field_values][#{field.id}][]" do Chris@1115: assert_select 'option', Project.find(1).users.count Chris@1115: assert_select 'option[value=2]', :text => 'John Smith' Chris@1115: end Chris@1115: assert_select 'input[name=?][type=hidden][value=?]', "issue[custom_field_values][#{field.id}][]", '' Chris@1115: end Chris@1115: Chris@1115: def test_get_new_with_date_custom_field Chris@1115: field = IssueCustomField.create!(:name => 'Date', :field_format => 'date', :tracker_ids => [1], :is_for_all => true) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :new, :project_id => 1, :tracker_id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'input[name=?]', "issue[custom_field_values][#{field.id}]" Chris@1115: end Chris@1115: Chris@1115: def test_get_new_with_text_custom_field Chris@1115: field = IssueCustomField.create!(:name => 'Text', :field_format => 'text', :tracker_ids => [1], :is_for_all => true) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :new, :project_id => 1, :tracker_id => 1 Chris@1115: assert_response :success Chris@1115: Chris@1115: assert_select 'textarea[name=?]', "issue[custom_field_values][#{field.id}]" Chris@1115: end Chris@1115: Chris@1115: def test_get_new_without_default_start_date_is_creation_date Chris@1464: with_settings :default_issue_start_date_to_creation_date => 0 do Chris@1464: @request.session[:user_id] = 2 Chris@1464: get :new, :project_id => 1, :tracker_id => 1 Chris@1464: assert_response :success Chris@1464: assert_template 'new' Chris@1464: assert_select 'input[name=?]', 'issue[start_date]' Chris@1464: assert_select 'input[name=?][value]', 'issue[start_date]', 0 Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_get_new_with_default_start_date_is_creation_date Chris@1464: with_settings :default_issue_start_date_to_creation_date => 1 do Chris@1464: @request.session[:user_id] = 2 Chris@1464: get :new, :project_id => 1, :tracker_id => 1 Chris@1464: assert_response :success Chris@1464: assert_template 'new' Chris@1464: assert_select 'input[name=?][value=?]', 'issue[start_date]', Chris@1464: Date.today.to_s Chris@1464: end 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@1115: assert_select 'form[id=issue-form][method=post][enctype=multipart/form-data]' do Chris@1464: assert_select 'input[name=?][type=file]', 'attachments[dummy][file]' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_get_new_should_prefill_the_form_from_params Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :new, :project_id => 1, Chris@1115: :issue => {:tracker_id => 3, :description => 'Prefilled', :custom_field_values => {'2' => 'Custom field value'}} Chris@1115: Chris@1115: issue = assigns(:issue) Chris@1115: assert_equal 3, issue.tracker_id Chris@1115: assert_equal 'Prefilled', issue.description Chris@1115: assert_equal 'Custom field value', issue.custom_field_value(2) Chris@1115: Chris@1115: assert_select 'select[name=?]', 'issue[tracker_id]' do Chris@1115: assert_select 'option[value=3][selected=selected]' Chris@1115: end Chris@1115: assert_select 'textarea[name=?]', 'issue[description]', :text => /Prefilled/ Chris@1115: assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Custom field value' Chris@1115: end Chris@1115: Chris@1115: def test_get_new_should_mark_required_fields Chris@1115: cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: WorkflowPermission.delete_all Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'required') Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'required') Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: get :new, :project_id => 1 Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: Chris@1115: assert_select 'label[for=issue_start_date]' do Chris@1115: assert_select 'span[class=required]', 0 Chris@1115: end Chris@1115: assert_select 'label[for=issue_due_date]' do Chris@1115: assert_select 'span[class=required]' Chris@1115: end Chris@1115: assert_select 'label[for=?]', "issue_custom_field_values_#{cf1.id}" do Chris@1115: assert_select 'span[class=required]', 0 Chris@1115: end Chris@1115: assert_select 'label[for=?]', "issue_custom_field_values_#{cf2.id}" do Chris@1115: assert_select 'span[class=required]' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_get_new_should_not_display_readonly_fields Chris@1115: cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: WorkflowPermission.delete_all Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'readonly') Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'readonly') Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: get :new, :project_id => 1 Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: Chris@1115: assert_select 'input[name=?]', 'issue[start_date]' Chris@1115: assert_select 'input[name=?]', 'issue[due_date]', 0 Chris@1115: assert_select 'input[name=?]', "issue[custom_field_values][#{cf1.id}]" Chris@1115: assert_select 'input[name=?]', "issue[custom_field_values][#{cf2.id}]", 0 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@1464: def test_update_form_for_new_issue Chris@0: @request.session[:user_id] = 2 Chris@1464: xhr :post, :update_form, :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@1115: assert_template 'update_form' Chris@1115: assert_template 'form' Chris@1115: assert_equal 'text/javascript', response.content_type 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@1464: def test_update_form_for_new_issue_should_propose_transitions_based_on_initial_status Chris@1115: @request.session[:user_id] = 2 Chris@1115: WorkflowTransition.delete_all Chris@1115: WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 2) Chris@1115: WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 5) Chris@1115: WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 5, :new_status_id => 4) Chris@1115: Chris@1464: xhr :post, :update_form, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :status_id => 5, Chris@1115: :subject => 'This is an issue'} Chris@1115: Chris@1115: assert_equal 5, assigns(:issue).status_id Chris@1115: assert_equal [1,2,5], assigns(:allowed_statuses).map(&:id).sort Chris@1115: end Chris@1115: 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@1464: v = issue.custom_values.where(:custom_field_id => 2).first 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@1464: with_settings :default_issue_start_date_to_creation_date => 0 do Chris@1464: @request.session[:user_id] = 2 Chris@1464: assert_difference 'Issue.count' do Chris@1464: 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@1464: end Chris@1464: assert_redirected_to :controller => 'issues', :action => 'show', Chris@1464: :id => Issue.last.id Chris@1464: issue = Issue.find_by_subject('This is the test_new issue') Chris@1464: assert_not_nil issue Chris@1464: assert_nil issue.start_date chris@37: end chris@37: end Chris@441: Chris@909: def test_post_create_without_start_date_and_default_start_date_is_creation_date Chris@1464: with_settings :default_issue_start_date_to_creation_date => 1 do Chris@1464: @request.session[:user_id] = 2 Chris@1464: assert_difference 'Issue.count' do Chris@1464: 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@1464: end Chris@1464: assert_redirected_to :controller => 'issues', :action => 'show', Chris@1464: :id => Issue.last.id Chris@1464: issue = Issue.find_by_subject('This is the test_new issue') Chris@1464: assert_not_nil issue Chris@1464: assert_equal Date.today, issue.start_date Chris@909: end 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@1115: assert_include %|##{issue.id}|, flash[:notice], "issue link not found in the flash message" 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@1115: def test_post_create_with_multi_custom_field Chris@1115: field = IssueCustomField.find_by_name('Database') Chris@1115: field.update_attribute(:multiple, true) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :subject => 'This is the test_new issue', Chris@1115: :description => 'This is the description', Chris@1115: :priority_id => 5, Chris@1115: :custom_field_values => {'1' => ['', 'MySQL', 'Oracle']}} Chris@1115: end Chris@1115: assert_response 302 Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal ['MySQL', 'Oracle'], issue.custom_field_value(1).sort Chris@1115: end Chris@1115: Chris@1115: def test_post_create_with_empty_multi_custom_field Chris@1115: field = IssueCustomField.find_by_name('Database') Chris@1115: field.update_attribute(:multiple, true) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :subject => 'This is the test_new issue', Chris@1115: :description => 'This is the description', Chris@1115: :priority_id => 5, Chris@1115: :custom_field_values => {'1' => ['']}} Chris@1115: end Chris@1115: assert_response 302 Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal [''], issue.custom_field_value(1).sort Chris@1115: end Chris@1115: Chris@1115: def test_post_create_with_multi_user_custom_field Chris@1115: field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true, Chris@1115: :tracker_ids => [1], :is_for_all => true) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :subject => 'This is the test_new issue', Chris@1115: :description => 'This is the description', Chris@1115: :priority_id => 5, Chris@1115: :custom_field_values => {field.id.to_s => ['', '2', '3']}} Chris@1115: end Chris@1115: assert_response 302 Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal ['2', '3'], issue.custom_field_value(field).sort Chris@1115: end Chris@1115: 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@1115: assert_no_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :subject => 'This is the test_new issue', Chris@1115: :description => 'This is the description', Chris@1115: :priority_id => 5} Chris@1115: end Chris@0: assert_response :success Chris@0: assert_template 'new' Chris@0: issue = assigns(:issue) Chris@0: assert_not_nil issue Chris@1115: assert_error_tag :content => /Database can't be blank/ Chris@1115: end Chris@1115: Chris@1115: def test_create_should_validate_required_fields Chris@1115: cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: WorkflowPermission.delete_all Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => 'due_date', :rule => 'required') Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'required') Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, :issue => { Chris@1115: :tracker_id => 2, Chris@1115: :status_id => 1, Chris@1115: :subject => 'Test', Chris@1115: :start_date => '', Chris@1115: :due_date => '', Chris@1115: :custom_field_values => {cf1.id.to_s => '', cf2.id.to_s => ''} Chris@1115: } Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: end Chris@1115: Chris@1115: assert_error_tag :content => /Due date can't be blank/i Chris@1115: assert_error_tag :content => /Bar can't be blank/i Chris@1115: end Chris@1115: Chris@1115: def test_create_should_ignore_readonly_fields Chris@1115: cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2]) Chris@1115: WorkflowPermission.delete_all Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => 'due_date', :rule => 'readonly') Chris@1115: WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'readonly') Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, :issue => { Chris@1115: :tracker_id => 2, Chris@1115: :status_id => 1, Chris@1115: :subject => 'Test', Chris@1115: :start_date => '2012-07-14', Chris@1115: :due_date => '2012-07-16', Chris@1115: :custom_field_values => {cf1.id.to_s => 'value1', cf2.id.to_s => 'value2'} Chris@1115: } Chris@1115: assert_response 302 Chris@1115: end Chris@1115: Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal Date.parse('2012-07-14'), issue.start_date Chris@1115: assert_nil issue.due_date Chris@1115: assert_equal 'value1', issue.custom_field_value(cf1) Chris@1115: assert_nil issue.custom_field_value(cf2) 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@1115: assert_not_nil 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@1115: :parent_issue_id => '2'} Chris@1115: assert_response 302 Chris@0: end Chris@1115: issue = Issue.order('id DESC').first Chris@0: assert_equal Issue.find(2), issue.parent Chris@0: end Chris@119: Chris@1115: def test_post_create_subissue_with_sharp_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@1115: :parent_issue_id => '#2'} Chris@1115: assert_response 302 Chris@119: end Chris@1115: issue = Issue.order('id DESC').first Chris@1115: assert_equal Issue.find(2), issue.parent Chris@1115: end Chris@1115: Chris@1115: def test_post_create_subissue_with_non_visible_parent_id_should_not_validate Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :subject => 'This is a child issue', Chris@1115: :parent_issue_id => '4'} Chris@1115: Chris@1115: assert_response :success Chris@1115: assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', '4' Chris@1115: assert_error_tag :content => /Parent task is invalid/i Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_post_create_subissue_with_non_numeric_parent_id_should_not_validate Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :subject => 'This is a child issue', Chris@1115: :parent_issue_id => '01ABC'} Chris@1115: Chris@1115: assert_response :success Chris@1115: assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', '01ABC' Chris@1115: assert_error_tag :content => /Parent task is invalid/i Chris@1115: end 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@1464: assert_select 'textarea[name=?]', 'issue[description]', :text => 'This is a description' Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' do Chris@1464: assert_select 'option[value=6][selected=selected]', :text => 'High' Chris@1464: end Chris@0: # Custom fields Chris@1464: assert_select 'select[name=?]', 'issue[custom_field_values][1]' do Chris@1464: assert_select 'option[value=Oracle][selected=selected]', :text => 'Oracle' Chris@1464: end Chris@1464: assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Value for field 2' Chris@0: end Chris@441: Chris@1115: def test_post_create_with_failure_should_preserve_watchers Chris@1115: assert !User.find(8).member_of?(Project.find(1)) Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => {:tracker_id => 1, Chris@1115: :watcher_user_ids => ['3', '8']} Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: Chris@1464: assert_select 'input[name=?][value=2]:not(checked)', 'issue[watcher_user_ids][]' Chris@1464: assert_select 'input[name=?][value=3][checked=checked]', 'issue[watcher_user_ids][]' Chris@1464: assert_select 'input[name=?][value=8][checked=checked]', 'issue[watcher_user_ids][]' Chris@1115: end Chris@1115: 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@1464: def test_post_create_with_attachment_should_notify_with_attachments Chris@1464: ActionMailer::Base.deliveries.clear Chris@1464: set_tmp_attachments_directory Chris@1464: @request.session[:user_id] = 2 Chris@1464: Chris@1464: with_settings :host_name => 'mydomain.foo', :protocol => 'http' do Chris@1464: assert_difference 'Issue.count' do Chris@1464: post :create, :project_id => 1, Chris@1464: :issue => { :tracker_id => '1', :subject => 'With attachment' }, Chris@1464: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}} Chris@1464: end Chris@1464: end Chris@1464: Chris@1464: assert_not_nil ActionMailer::Base.deliveries.last Chris@1464: assert_select_email do Chris@1464: assert_select 'a[href^=?]', 'http://mydomain.foo/attachments/download', 'testfile.txt' Chris@1464: end Chris@1464: end Chris@1464: Chris@1115: def test_post_create_with_failure_should_save_attachments Chris@1115: set_tmp_attachments_directory Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Issue.count' do Chris@1115: assert_difference 'Attachment.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => { :tracker_id => '1', :subject => '' }, Chris@1115: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}} Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: attachment = Attachment.first(:order => 'id DESC') Chris@1115: assert_equal 'testfile.txt', attachment.filename Chris@1115: assert File.exists?(attachment.diskfile) Chris@1115: assert_nil attachment.container Chris@1115: Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt' Chris@1115: end Chris@1115: Chris@1115: def test_post_create_with_failure_should_keep_saved_attachments Chris@1115: set_tmp_attachments_directory Chris@1115: attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2) Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Issue.count' do Chris@1115: assert_no_difference 'Attachment.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => { :tracker_id => '1', :subject => '' }, Chris@1115: :attachments => {'p0' => {'token' => attachment.token}} Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: end Chris@1115: end Chris@1115: Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt' Chris@1115: end Chris@1115: Chris@1115: def test_post_create_should_attach_saved_attachments Chris@1115: set_tmp_attachments_directory Chris@1115: attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2) Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count' do Chris@1115: assert_no_difference 'Attachment.count' do Chris@1115: post :create, :project_id => 1, Chris@1115: :issue => { :tracker_id => '1', :subject => 'Saved attachments' }, Chris@1115: :attachments => {'p0' => {'token' => attachment.token}} Chris@1115: assert_response 302 Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal 1, issue.attachments.count Chris@1115: Chris@1115: attachment.reload Chris@1115: assert_equal issue, attachment.container Chris@1115: end Chris@1115: Chris@0: context "without workflow privilege" do Chris@0: setup do Chris@1115: WorkflowTransition.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@1464: assert_select 'select[name=?]', 'issue[status_id]' do Chris@1464: assert_select 'option', 1 Chris@1464: assert_select 'option[value=?]', IssueStatus.default.id.to_s Chris@1464: end 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@1115: put :update, :id => 1, :issue => {:status_id => 3, :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:subject => 'changed', :assigned_to_id => 2, :notes => 'just trying'} 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@1115: WorkflowTransition.delete_all(["role_id = ?", Role.anonymous.id]) Chris@1115: WorkflowTransition.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3) Chris@1115: WorkflowTransition.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@1115: put :update, :id => 1, :issue => {:status_id => 3, :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:status_id => 2, :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:assigned_to_id => 2, :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:subject => 'changed', :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:status_id => 3, :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:status_id => 2, :notes => 'just trying'} 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@1115: put :update, :id => 1, :issue => {:subject => 'changed', :assigned_to_id => 2, :notes => 'just trying'} 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@1115: def test_new_as_copy Chris@0: @request.session[:user_id] = 2 Chris@0: get :new, :project_id => 1, :copy_from => 1 Chris@1115: Chris@1115: assert_response :success Chris@0: assert_template 'new' Chris@1115: Chris@0: assert_not_nil assigns(:issue) Chris@0: orig = Issue.find(1) Chris@1115: assert_equal 1, assigns(:issue).project_id Chris@0: assert_equal orig.subject, assigns(:issue).subject Chris@1115: assert assigns(:issue).copy? Chris@1115: Chris@1464: assert_select 'form[id=issue-form][action=/projects/ecookbook/issues]' do Chris@1464: assert_select 'select[name=?]', 'issue[project_id]' do Chris@1464: assert_select 'option[value=1][selected=selected]', :text => 'eCookbook' Chris@1464: assert_select 'option[value=2]:not([selected])', :text => 'OnlineStore' Chris@1464: end Chris@1464: assert_select 'input[name=copy_from][value=1]' Chris@1464: end Chris@1294: Chris@1294: # "New issue" menu item should not link to copy Chris@1294: assert_select '#main-menu a.new-issue[href=/projects/ecookbook/issues/new]' Chris@1115: end Chris@1115: Chris@1115: def test_new_as_copy_with_attachments_should_show_copy_attachments_checkbox Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.find(3) Chris@1115: assert issue.attachments.count > 0 Chris@1115: get :new, :project_id => 1, :copy_from => 3 Chris@1115: Chris@1464: assert_select 'input[name=copy_attachments][type=checkbox][checked=checked][value=1]' Chris@1115: end Chris@1115: Chris@1115: def test_new_as_copy_without_attachments_should_not_show_copy_attachments_checkbox Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.find(3) Chris@1115: issue.attachments.delete_all Chris@1115: get :new, :project_id => 1, :copy_from => 3 Chris@1115: Chris@1464: assert_select 'input[name=copy_attachments]', 0 Chris@1115: end Chris@1115: Chris@1115: def test_new_as_copy_with_subtasks_should_show_copy_subtasks_checkbox Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.generate_with_descendants! Chris@1115: get :new, :project_id => 1, :copy_from => issue.id Chris@1115: Chris@1115: assert_select 'input[type=checkbox][name=copy_subtasks][checked=checked][value=1]' Chris@1115: end Chris@1115: Chris@1115: def test_new_as_copy_with_invalid_issue_should_respond_with_404 Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :new, :project_id => 1, :copy_from => 99999 Chris@1115: assert_response 404 Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_on_different_project Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, :copy_from => 1, Chris@1115: :issue => {:project_id => '2', :tracker_id => '3', :status_id => '1', :subject => 'Copy'} Chris@1115: Chris@1115: assert_not_nil assigns(:issue) Chris@1115: assert assigns(:issue).copy? Chris@1115: end Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_redirected_to "/issues/#{issue.id}" Chris@1115: Chris@1115: assert_equal 2, issue.project_id Chris@1115: assert_equal 3, issue.tracker_id Chris@1115: assert_equal 'Copy', issue.subject Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_should_copy_attachments Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.find(3) Chris@1115: count = issue.attachments.count Chris@1115: assert count > 0 Chris@1115: assert_difference 'Issue.count' do Chris@1115: assert_difference 'Attachment.count', count do Chris@1464: assert_difference 'Journal.count', 2 do Chris@1115: post :create, :project_id => 1, :copy_from => 3, Chris@1464: :issue => {:project_id => '1', :tracker_id => '3', Chris@1464: :status_id => '1', :subject => 'Copy with attachments'}, Chris@1115: :copy_attachments => '1' Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: copy = Issue.first(:order => 'id DESC') Chris@1115: assert_equal count, copy.attachments.count Chris@1115: assert_equal issue.attachments.map(&:filename).sort, copy.attachments.map(&:filename).sort Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_without_copy_attachments_option_should_not_copy_attachments Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.find(3) Chris@1115: count = issue.attachments.count Chris@1115: assert count > 0 Chris@1115: assert_difference 'Issue.count' do Chris@1115: assert_no_difference 'Attachment.count' do Chris@1464: assert_difference 'Journal.count', 2 do Chris@1115: post :create, :project_id => 1, :copy_from => 3, Chris@1464: :issue => {:project_id => '1', :tracker_id => '3', Chris@1464: :status_id => '1', :subject => 'Copy with attachments'} Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: copy = Issue.first(:order => 'id DESC') Chris@1115: assert_equal 0, copy.attachments.count Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_with_attachments_should_add_new_files Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.find(3) Chris@1115: count = issue.attachments.count Chris@1115: assert count > 0 Chris@1115: assert_difference 'Issue.count' do Chris@1115: assert_difference 'Attachment.count', count + 1 do Chris@1464: assert_difference 'Journal.count', 2 do Chris@1115: post :create, :project_id => 1, :copy_from => 3, Chris@1464: :issue => {:project_id => '1', :tracker_id => '3', Chris@1464: :status_id => '1', :subject => 'Copy with attachments'}, Chris@1115: :copy_attachments => '1', Chris@1464: :attachments => {'1' => Chris@1464: {'file' => uploaded_test_file('testfile.txt', 'text/plain'), Chris@1464: 'description' => 'test file'}} Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: copy = Issue.first(:order => 'id DESC') Chris@1115: assert_equal count + 1, copy.attachments.count Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_should_add_relation_with_copied_issue Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count' do Chris@1115: assert_difference 'IssueRelation.count' do Chris@1115: post :create, :project_id => 1, :copy_from => 1, Chris@1464: :issue => {:project_id => '1', :tracker_id => '3', Chris@1464: :status_id => '1', :subject => 'Copy'} Chris@1115: end Chris@1115: end Chris@1115: copy = Issue.first(:order => 'id DESC') Chris@1115: assert_equal 1, copy.relations.size Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_should_copy_subtasks Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.generate_with_descendants! Chris@1115: count = issue.descendants.count Chris@1464: assert_difference 'Issue.count', count + 1 do Chris@1464: assert_difference 'Journal.count', (count + 1) * 2 do Chris@1115: post :create, :project_id => 1, :copy_from => issue.id, Chris@1464: :issue => {:project_id => '1', :tracker_id => '3', Chris@1464: :status_id => '1', :subject => 'Copy with subtasks'}, Chris@1115: :copy_subtasks => '1' Chris@1115: end Chris@1115: end Chris@1115: copy = Issue.where(:parent_id => nil).first(:order => 'id DESC') Chris@1115: assert_equal count, copy.descendants.count Chris@1115: assert_equal issue.descendants.map(&:subject).sort, copy.descendants.map(&:subject).sort Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_without_copy_subtasks_option_should_not_copy_subtasks Chris@1115: @request.session[:user_id] = 2 Chris@1115: issue = Issue.generate_with_descendants! Chris@1115: assert_difference 'Issue.count', 1 do Chris@1464: assert_difference 'Journal.count', 2 do Chris@1115: post :create, :project_id => 1, :copy_from => 3, Chris@1464: :issue => {:project_id => '1', :tracker_id => '3', Chris@1464: :status_id => '1', :subject => 'Copy with subtasks'} Chris@1115: end Chris@1115: end Chris@1115: copy = Issue.where(:parent_id => nil).first(:order => 'id DESC') Chris@1115: assert_equal 0, copy.descendants.count Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_with_failure Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :create, :project_id => 1, :copy_from => 1, Chris@1115: :issue => {:project_id => '2', :tracker_id => '3', :status_id => '1', :subject => ''} Chris@1115: Chris@1115: assert_response :success Chris@1115: assert_template 'new' Chris@1115: Chris@1115: assert_not_nil assigns(:issue) Chris@1115: assert assigns(:issue).copy? Chris@1115: Chris@1464: assert_select 'form#issue-form[action=/projects/ecookbook/issues]' do Chris@1464: assert_select 'select[name=?]', 'issue[project_id]' do Chris@1464: assert_select 'option[value=1]:not([selected])', :text => 'eCookbook' Chris@1464: assert_select 'option[value=2][selected=selected]', :text => 'OnlineStore' Chris@1464: end Chris@1464: assert_select 'input[name=copy_from][value=1]' Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_create_as_copy_on_project_without_permission_should_ignore_target_project Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert !User.find(2).member_of?(Project.find(4)) Chris@1115: Chris@1115: assert_difference 'Issue.count' do Chris@1115: post :create, :project_id => 1, :copy_from => 1, Chris@1115: :issue => {:project_id => '4', :tracker_id => '3', :status_id => '1', :subject => 'Copy'} Chris@1115: end Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal 1, issue.project_id 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@1464: assert_select 'select[name=?]', 'issue[priority_id]' do Chris@1464: assert_select 'option[value=15]', 0 Chris@1464: end 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@1464: assert_select 'input[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@1464: assert_select 'input[name=?]', 'time_entry[hours]', 0 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@1464: :time_entry => { :hours => '2.5', :comments => 'test_get_edit_with_params', :activity_id => 10 } 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@1464: assert_select 'select[name=?]', 'issue[status_id]' do Chris@1464: assert_select 'option[value=5][selected=selected]', :text => 'Closed' Chris@1464: end Chris@441: Chris@0: assert_equal 7, issue.priority_id Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' do Chris@1464: assert_select 'option[value=7][selected=selected]', :text => 'Urgent' Chris@1464: end Chris@1464: Chris@1464: assert_select 'input[name=?][value=2.5]', 'time_entry[hours]' Chris@1464: assert_select 'select[name=?]', 'time_entry[activity_id]' do Chris@1464: assert_select 'option[value=10][selected=selected]', :text => 'Development' Chris@1464: end Chris@1464: assert_select 'input[name=?][value=test_get_edit_with_params]', 'time_entry[comments]' Chris@0: end Chris@0: Chris@1115: def test_get_edit_with_multi_custom_field Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {1 => ['MySQL', 'Oracle']} Chris@1115: issue.save! Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :edit, :id => 1 Chris@1115: assert_response :success Chris@1115: assert_template 'edit' Chris@1115: Chris@1464: assert_select 'select[name=?][multiple=multiple]', 'issue[custom_field_values][1][]' do Chris@1464: assert_select 'option', 3 Chris@1464: assert_select 'option[value=MySQL][selected=selected]' Chris@1464: assert_select 'option[value=Oracle][selected=selected]' Chris@1464: assert_select 'option[value=PostgreSQL]:not([selected])' Chris@1464: end Chris@1464: end Chris@1464: Chris@1464: def test_update_form_for_existing_issue Chris@0: @request.session[:user_id] = 2 Chris@1464: xhr :put, :update_form, :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@1115: assert_equal 'text/javascript', response.content_type Chris@1115: assert_template 'update_form' Chris@1115: assert_template 'form' 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@1464: def test_update_form_for_existing_issue_should_keep_issue_author Chris@1115: @request.session[:user_id] = 3 Chris@1464: xhr :put, :update_form, :project_id => 1, :id => 1, :issue => {:subject => 'Changed'} Chris@1115: assert_response :success Chris@1115: assert_equal 'text/javascript', response.content_type Chris@1115: Chris@1115: issue = assigns(:issue) Chris@1115: assert_equal User.find(2), issue.author Chris@1115: assert_equal 2, issue.author_id Chris@1115: assert_not_equal User.current, issue.author Chris@1115: end Chris@1115: Chris@1464: def test_update_form_for_existing_issue_should_propose_transitions_based_on_initial_status Chris@0: @request.session[:user_id] = 2 Chris@1115: WorkflowTransition.delete_all Chris@1115: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 1) Chris@1115: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 5) Chris@1115: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 5, :new_status_id => 4) Chris@1115: Chris@1464: xhr :put, :update_form, :project_id => 1, Chris@1115: :id => 2, Chris@1115: :issue => {:tracker_id => 2, Chris@1115: :status_id => 5, Chris@1115: :subject => 'This is an issue'} Chris@1115: Chris@1115: assert_equal 5, assigns(:issue).status_id Chris@1115: assert_equal [1,2,5], assigns(:allowed_statuses).map(&:id).sort Chris@1115: end Chris@1115: Chris@1464: def test_update_form_for_existing_issue_with_project_change Chris@1115: @request.session[:user_id] = 2 Chris@1464: xhr :put, :update_form, :project_id => 1, Chris@1115: :id => 1, Chris@1115: :issue => {:project_id => 2, Chris@1115: :tracker_id => 2, Chris@1115: :subject => 'This is the test_new issue', Chris@1115: :description => 'This is the description', Chris@1115: :priority_id => 5} Chris@1115: assert_response :success Chris@1115: assert_template 'form' Chris@1115: Chris@1115: issue = assigns(:issue) Chris@1115: assert_kind_of Issue, issue Chris@1115: assert_equal 1, issue.id Chris@1115: assert_equal 2, issue.project_id Chris@1115: assert_equal 2, issue.tracker_id Chris@1115: assert_equal 'This is the test_new issue', issue.subject Chris@0: end Chris@0: Chris@1464: def test_update_form_should_propose_default_status_for_existing_issue Chris@1464: @request.session[:user_id] = 2 Chris@1464: WorkflowTransition.delete_all Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 3) Chris@1464: Chris@1464: xhr :put, :update_form, :project_id => 1, :id => 2 Chris@1464: assert_response :success Chris@1464: assert_equal [2,3], assigns(:allowed_statuses).map(&:id).sort Chris@1464: end Chris@1464: 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@1115: assert_not_nil mail Chris@0: assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]") Chris@1115: assert_mail_body_match "Subject changed from #{old_subject} to #{new_subject}", mail Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_project_change Chris@1115: @request.session[:user_id] = 2 Chris@1115: ActionMailer::Base.deliveries.clear Chris@1115: Chris@1115: assert_difference('Journal.count') do Chris@1115: assert_difference('JournalDetail.count', 3) do Chris@1115: put :update, :id => 1, :issue => {:project_id => '2', Chris@1115: :tracker_id => '1', # no change Chris@1115: :priority_id => '6', Chris@1115: :category_id => '3' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: assert_redirected_to :action => 'show', :id => '1' Chris@1115: issue = Issue.find(1) Chris@1115: assert_equal 2, issue.project_id Chris@1115: assert_equal 1, issue.tracker_id Chris@1115: assert_equal 6, issue.priority_id Chris@1115: assert_equal 3, issue.category_id Chris@1115: Chris@1115: mail = ActionMailer::Base.deliveries.last Chris@1115: assert_not_nil mail Chris@1115: assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]") Chris@1115: assert_mail_body_match "Project changed from eCookbook to OnlineStore", mail Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_tracker_change Chris@1115: @request.session[:user_id] = 2 Chris@1115: ActionMailer::Base.deliveries.clear Chris@1115: Chris@1115: assert_difference('Journal.count') do Chris@1115: assert_difference('JournalDetail.count', 2) do Chris@1115: put :update, :id => 1, :issue => {:project_id => '1', Chris@1115: :tracker_id => '2', Chris@1115: :priority_id => '6' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: assert_redirected_to :action => 'show', :id => '1' Chris@1115: issue = Issue.find(1) Chris@1115: assert_equal 1, issue.project_id Chris@1115: assert_equal 2, issue.tracker_id Chris@1115: assert_equal 6, issue.priority_id Chris@1115: assert_equal 1, issue.category_id Chris@1115: Chris@1115: mail = ActionMailer::Base.deliveries.last Chris@1115: assert_not_nil mail Chris@1115: assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]") Chris@1115: assert_mail_body_match "Tracker changed from Bug to Feature request", mail 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@1115: assert_not_nil mail Chris@1115: assert_mail_body_match "Searchable field changed from 125 to New custom value", mail Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_multi_custom_field_change Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: issue = Issue.find(1) Chris@1115: issue.custom_field_values = {1 => ['MySQL', 'Oracle']} Chris@1115: issue.save! Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference('Journal.count') do Chris@1115: assert_difference('JournalDetail.count', 3) do Chris@1115: put :update, :id => 1, Chris@1115: :issue => { Chris@1115: :subject => 'Custom field change', Chris@1115: :custom_field_values => { '1' => ['', 'Oracle', 'PostgreSQL'] } Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: assert_redirected_to :action => 'show', :id => '1' Chris@1115: assert_equal ['Oracle', 'PostgreSQL'], Issue.find(1).custom_field_value(1).sort 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@1115: :issue => { :status_id => 2, :assigned_to_id => 3, :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@1464: j = Journal.order('id DESC').first 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@1115: assert_mail_body_match "Status changed from New to Assigned", mail 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@1115: :issue => { :notes => notes } Chris@0: assert_redirected_to :action => 'show', :id => '1' Chris@1464: j = Journal.order('id DESC').first 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@1115: assert_mail_body_match notes, mail Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_private_note_only Chris@1115: notes = 'Private note' Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Journal.count' do Chris@1115: put :update, :id => 1, :issue => {:notes => notes, :private_notes => '1'} Chris@1115: assert_redirected_to :action => 'show', :id => '1' Chris@1115: end Chris@1115: Chris@1115: j = Journal.order('id DESC').first Chris@1115: assert_equal notes, j.notes Chris@1115: assert_equal true, j.private_notes Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_private_note_and_changes Chris@1115: notes = 'Private note' Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Journal.count', 2 do Chris@1115: put :update, :id => 1, :issue => {:subject => 'New subject', :notes => notes, :private_notes => '1'} Chris@1115: assert_redirected_to :action => 'show', :id => '1' Chris@1115: end Chris@1115: Chris@1115: j = Journal.order('id DESC').first Chris@1115: assert_equal notes, j.notes Chris@1115: assert_equal true, j.private_notes Chris@1115: assert_equal 0, j.details.count Chris@1115: Chris@1115: j = Journal.order('id DESC').offset(1).first Chris@1115: assert_nil j.notes Chris@1115: assert_equal false, j.private_notes Chris@1115: assert_equal 1, j.details.count 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@1115: :issue => { :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@1464: j = Journal.order('id DESC').first 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@1294: def test_put_update_should_preserve_parent_issue_even_if_not_visible Chris@1294: parent = Issue.generate!(:project_id => 1, :is_private => true) Chris@1294: issue = Issue.generate!(:parent_issue_id => parent.id) Chris@1294: assert !parent.visible?(User.find(3)) Chris@1294: @request.session[:user_id] = 3 Chris@1294: Chris@1294: get :edit, :id => issue.id Chris@1294: assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', parent.id.to_s Chris@1294: Chris@1294: put :update, :id => issue.id, :issue => {:subject => 'New subject', :parent_issue_id => parent.id.to_s} Chris@1294: assert_response 302 Chris@1294: assert_equal parent, issue.parent Chris@1294: end Chris@1294: 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@1115: :issue => {: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@1464: j = Issue.find(1).journals.reorder('id DESC').first 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@1115: assert_mail_body_match 'testfile.txt', mail Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_failure_should_save_attachments Chris@1115: set_tmp_attachments_directory Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Journal.count' do Chris@1115: assert_difference 'Attachment.count' do Chris@1115: put :update, :id => 1, Chris@1115: :issue => { :subject => '' }, Chris@1115: :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}} Chris@1115: assert_response :success Chris@1115: assert_template 'edit' Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: attachment = Attachment.first(:order => 'id DESC') Chris@1115: assert_equal 'testfile.txt', attachment.filename Chris@1115: assert File.exists?(attachment.diskfile) Chris@1115: assert_nil attachment.container Chris@1115: Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt' Chris@1115: end Chris@1115: Chris@1115: def test_put_update_with_failure_should_keep_saved_attachments Chris@1115: set_tmp_attachments_directory Chris@1115: attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2) Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_no_difference 'Journal.count' do Chris@1115: assert_no_difference 'Attachment.count' do Chris@1115: put :update, :id => 1, Chris@1115: :issue => { :subject => '' }, Chris@1115: :attachments => {'p0' => {'token' => attachment.token}} Chris@1115: assert_response :success Chris@1115: assert_template 'edit' Chris@1115: end Chris@1115: end Chris@1115: Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token Chris@1464: assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt' Chris@1115: end Chris@1115: Chris@1115: def test_put_update_should_attach_saved_attachments Chris@1115: set_tmp_attachments_directory Chris@1115: attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2) Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Journal.count' do Chris@1115: assert_difference 'JournalDetail.count' do Chris@1115: assert_no_difference 'Attachment.count' do Chris@1115: put :update, :id => 1, Chris@1115: :issue => {:notes => 'Attachment added'}, Chris@1115: :attachments => {'p0' => {'token' => attachment.token}} Chris@1115: assert_redirected_to '/issues/1' Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: attachment.reload Chris@1115: assert_equal Issue.find(1), attachment.container Chris@1115: Chris@1115: journal = Journal.first(:order => 'id DESC') Chris@1115: assert_equal 1, journal.details.size Chris@1115: assert_equal 'testfile.txt', journal.details.first.value 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@1115: :issue => {: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@1115: end 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@1115: :issue => {: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@1115: :issue => {: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@1115: assert_error_tag :descendant => {:content => /Activity can't be blank/} Chris@1464: assert_select 'textarea[name=?]', 'issue[notes]', :text => notes Chris@1464: assert_select 'input[name=?][value=?]', 'time_entry[hours]', '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@1115: :issue => {: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@1115: assert_error_tag :descendant => {:content => /Activity can't be blank/} Chris@1115: assert_error_tag :descendant => {:content => /Hours can't be blank/} Chris@1464: assert_select 'textarea[name=?]', 'issue[notes]', :text => notes Chris@1464: assert_select 'input[name=?][value=?]', 'time_entry[comments]', '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@1464: assert_select 'ul#bulk-selection' do Chris@1464: assert_select 'li', 2 Chris@1464: assert_select 'li a', :text => 'Bug #1' Chris@1464: end Chris@1464: Chris@1464: assert_select 'form#bulk_edit_form[action=?]', '/issues/bulk_update' do Chris@1464: assert_select 'input[name=?]', 'ids[]', 2 Chris@1464: assert_select 'input[name=?][value=1][type=hidden]', 'ids[]' Chris@1464: Chris@1464: assert_select 'select[name=?]', 'issue[project_id]' Chris@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]' Chris@1464: Chris@1464: # Project specific custom field, date type Chris@1464: field = CustomField.find(9) Chris@1464: assert !field.is_for_all? Chris@1464: assert_equal 'date', field.field_format Chris@1464: assert_select 'input[name=?]', 'issue[custom_field_values][9]' Chris@1464: Chris@1464: # System wide custom field Chris@1464: assert CustomField.find(1).is_for_all? Chris@1464: assert_select 'select[name=?]', 'issue[custom_field_values][1]' Chris@1464: Chris@1464: # Be sure we don't display inactive IssuePriorities Chris@1464: assert ! IssuePriority.find(15).active? Chris@1464: assert_select 'select[name=?]', 'issue[priority_id]' do Chris@1464: assert_select 'option[value=15]', 0 Chris@1464: end Chris@1464: end 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@1464: assert_select 'input[name=?]', 'issue[parent_issue_id]', 0 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@1464: assert_select 'input[name=?]', 'issue[custom_field_values][9]', 0 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@1464: assert_select 'select.user_cf[name=?]', "issue[custom_field_values][#{field.id}]" do Chris@1464: assert_select 'option', Project.find(1).users.count + 2 # "no change" + "none" options Chris@1464: end 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@1464: assert_select 'select.version_cf[name=?]', "issue[custom_field_values][#{field.id}]" do Chris@1464: assert_select 'option', Project.find(1).shared_versions.count + 2 # "no change" + "none" options Chris@1464: end Chris@441: end Chris@441: Chris@1115: def test_get_bulk_edit_with_multi_custom_field Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :bulk_edit, :ids => [1, 2] Chris@1115: assert_response :success Chris@1115: assert_template 'bulk_edit' Chris@1115: Chris@1464: assert_select 'select[name=?]', 'issue[custom_field_values][1][]' do Chris@1464: assert_select 'option', field.possible_values.size + 1 # "none" options Chris@1464: end Chris@1464: end Chris@1464: Chris@1464: def test_bulk_edit_should_propose_to_clear_text_custom_fields Chris@1464: @request.session[:user_id] = 2 Chris@1464: get :bulk_edit, :ids => [1, 3] Chris@1464: assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', '__none__' Chris@1115: end Chris@1115: Chris@1115: def test_bulk_edit_should_only_propose_statuses_allowed_for_all_issues Chris@1115: WorkflowTransition.delete_all Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, Chris@1464: :old_status_id => 1, :new_status_id => 1) Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, Chris@1464: :old_status_id => 1, :new_status_id => 3) Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, Chris@1464: :old_status_id => 1, :new_status_id => 4) Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, Chris@1464: :old_status_id => 2, :new_status_id => 1) Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, Chris@1464: :old_status_id => 2, :new_status_id => 3) Chris@1464: WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, Chris@1464: :old_status_id => 2, :new_status_id => 5) Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :bulk_edit, :ids => [1, 2] Chris@1115: Chris@1115: assert_response :success Chris@1115: statuses = assigns(:available_statuses) Chris@1115: assert_not_nil statuses Chris@1115: assert_equal [1, 3], statuses.map(&:id).sort Chris@1115: Chris@1464: assert_select 'select[name=?]', 'issue[status_id]' do Chris@1464: assert_select 'option', 3 # 2 statuses + "no change" option Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_edit_should_propose_target_project_open_shared_versions Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_edit, :ids => [1, 2, 6], :issue => {:project_id => 1} Chris@1115: assert_response :success Chris@1115: assert_template 'bulk_edit' Chris@1115: assert_equal Project.find(1).shared_versions.open.all.sort, assigns(:versions).sort Chris@1464: Chris@1464: assert_select 'select[name=?]', 'issue[fixed_version_id]' do Chris@1464: assert_select 'option', :text => '2.0' Chris@1464: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_edit_should_propose_target_project_categories Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_edit, :ids => [1, 2, 6], :issue => {:project_id => 1} Chris@1115: assert_response :success Chris@1115: assert_template 'bulk_edit' Chris@1115: assert_equal Project.find(1).issue_categories.sort, assigns(:categories).sort Chris@1464: Chris@1464: assert_select 'select[name=?]', 'issue[category_id]' do Chris@1464: assert_select 'option', :text => 'Recipes' Chris@1464: end Chris@1115: end Chris@1115: 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@1464: journal = issue.journals.reorder('created_on DESC').first 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@1464: journal = issue.journals.reorder('created_on DESC').first 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@1115: def test_bulk_update_project Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 2], :issue => {:project_id => '2'} Chris@1115: assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook' Chris@1115: # Issues moved to project 2 Chris@1115: assert_equal 2, Issue.find(1).project_id Chris@1115: assert_equal 2, Issue.find(2).project_id Chris@1115: # No tracker change Chris@1115: assert_equal 1, Issue.find(1).tracker_id Chris@1115: assert_equal 2, Issue.find(2).tracker_id Chris@1115: end Chris@1115: Chris@1115: def test_bulk_update_project_on_single_issue_should_follow_when_needed Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :id => 1, :issue => {:project_id => '2'}, :follow => '1' Chris@1115: assert_redirected_to '/issues/1' Chris@1115: end Chris@1115: Chris@1115: def test_bulk_update_project_on_multiple_issues_should_follow_when_needed Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :id => [1, 2], :issue => {:project_id => '2'}, :follow => '1' Chris@1115: assert_redirected_to '/projects/onlinestore/issues' Chris@1115: end Chris@1115: Chris@1115: def test_bulk_update_tracker Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 2], :issue => {:tracker_id => '2'} Chris@1115: assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook' Chris@1115: assert_equal 2, Issue.find(1).tracker_id Chris@1115: assert_equal 2, Issue.find(2).tracker_id Chris@1115: end Chris@1115: 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@1115: def test_bulk_update_priority Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 2], :issue => {:priority_id => 6} Chris@1115: Chris@1115: assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook' Chris@1115: assert_equal 6, Issue.find(1).priority_id Chris@1115: assert_equal 6, Issue.find(2).priority_id Chris@1115: end Chris@1115: Chris@1115: def test_bulk_update_with_notes Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 2], :notes => 'Moving two issues' Chris@1115: Chris@1115: assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook' Chris@1115: assert_equal 'Moving two issues', Issue.find(1).journals.sort_by(&:id).last.notes Chris@1115: assert_equal 'Moving two issues', Issue.find(2).journals.sort_by(&:id).last.notes Chris@1115: end Chris@1115: Chris@441: def test_bulk_update_parent_id Chris@1464: IssueRelation.delete_all Chris@441: @request.session[:user_id] = 2 Chris@441: post :bulk_update, :ids => [1, 3], Chris@441: :notes => 'Bulk editing parent', Chris@1464: :issue => {:priority_id => '', :assigned_to_id => '', Chris@1464: :status_id => '', :parent_issue_id => '2'} 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@1464: journal = issue.journals.reorder('created_on DESC').first 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@1115: def test_bulk_update_custom_field_to_blank Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 3], :notes => 'Bulk editing custom field', Chris@1115: :issue => {:priority_id => '', Chris@1115: :assigned_to_id => '', Chris@1115: :custom_field_values => {'1' => '__none__'}} Chris@1115: assert_response 302 Chris@1115: assert_equal '', Issue.find(1).custom_field_value(1) Chris@1115: assert_equal '', Issue.find(3).custom_field_value(1) Chris@1115: end Chris@1115: Chris@1115: def test_bulk_update_multi_custom_field Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 2, 3], :notes => 'Bulk editing multi custom field', Chris@1115: :issue => {:priority_id => '', Chris@1115: :assigned_to_id => '', Chris@1115: :custom_field_values => {'1' => ['MySQL', 'Oracle']}} Chris@1115: Chris@1115: assert_response 302 Chris@1115: Chris@1115: assert_equal ['MySQL', 'Oracle'], Issue.find(1).custom_field_value(1).sort Chris@1115: assert_equal ['MySQL', 'Oracle'], Issue.find(3).custom_field_value(1).sort Chris@1115: # the custom field is not associated with the issue tracker Chris@1115: assert_nil Issue.find(2).custom_field_value(1) Chris@1115: end Chris@1115: Chris@1115: def test_bulk_update_multi_custom_field_to_blank Chris@1115: field = CustomField.find(1) Chris@1115: field.update_attribute :multiple, true Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1, 3], :notes => 'Bulk editing multi custom field', Chris@1115: :issue => {:priority_id => '', Chris@1115: :assigned_to_id => '', Chris@1115: :custom_field_values => {'1' => ['__none__']}} Chris@1115: assert_response 302 Chris@1115: assert_equal [''], Issue.find(1).custom_field_value(1) Chris@1115: assert_equal [''], Issue.find(3).custom_field_value(1) Chris@1115: end Chris@1115: 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@1464: def test_bulk_update_with_all_failures_should_show_errors Chris@1115: @request.session[:user_id] = 2 Chris@1464: post :bulk_update, :ids => [1, 2], :issue => {:start_date => 'foo'} Chris@1464: Chris@1464: assert_response :success Chris@1464: assert_template 'bulk_edit' Chris@1464: assert_select '#errorExplanation span', :text => 'Failed to save 2 issue(s) on 2 selected: #1, #2.' Chris@1464: assert_select '#errorExplanation ul li', :text => 'Start date is not a valid date: #1, #2' Chris@1464: Chris@1464: assert_equal [1, 2], assigns[:issues].map(&:id) Chris@1464: end Chris@1464: Chris@1464: def test_bulk_update_with_some_failures_should_show_errors Chris@1464: issue1 = Issue.generate!(:start_date => '2013-05-12') Chris@1464: issue2 = Issue.generate!(:start_date => '2013-05-15') Chris@1464: issue3 = Issue.generate! Chris@1464: @request.session[:user_id] = 2 Chris@1464: post :bulk_update, :ids => [issue1.id, issue2.id, issue3.id], Chris@1464: :issue => {:due_date => '2013-05-01'} Chris@1464: assert_response :success Chris@1464: assert_template 'bulk_edit' Chris@1464: assert_select '#errorExplanation span', Chris@1464: :text => "Failed to save 2 issue(s) on 3 selected: ##{issue1.id}, ##{issue2.id}." Chris@1464: assert_select '#errorExplanation ul li', Chris@1464: :text => "Due date must be greater than start date: ##{issue1.id}, ##{issue2.id}" Chris@1464: assert_equal [issue1.id, issue2.id], assigns[:issues].map(&:id) Chris@1464: end Chris@1464: Chris@1464: def test_bulk_update_with_failure_should_preserved_form_values Chris@1464: @request.session[:user_id] = 2 Chris@1464: post :bulk_update, :ids => [1, 2], :issue => {:tracker_id => '2', :start_date => 'foo'} Chris@1464: Chris@1464: assert_response :success Chris@1464: assert_template 'bulk_edit' Chris@1464: assert_select 'select[name=?]', 'issue[tracker_id]' do Chris@1464: assert_select 'option[value=2][selected=selected]' Chris@1464: end Chris@1464: assert_select 'input[name=?][value=?]', 'issue[start_date]', 'foo' Chris@1115: end Chris@1115: Chris@1115: def test_get_bulk_copy Chris@1115: @request.session[:user_id] = 2 Chris@1115: get :bulk_edit, :ids => [1, 2, 3], :copy => '1' Chris@1115: assert_response :success Chris@1115: assert_template 'bulk_edit' Chris@1115: Chris@1115: issues = assigns(:issues) Chris@1115: assert_not_nil issues Chris@1115: assert_equal [1, 2, 3], issues.map(&:id).sort Chris@1115: Chris@1115: assert_select 'input[name=copy_attachments]' Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_to_another_project Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count', 2 do Chris@1115: assert_no_difference 'Project.find(1).issues.count' do Chris@1115: post :bulk_update, :ids => [1, 2], :issue => {:project_id => '2'}, :copy => '1' Chris@1115: end Chris@1115: end Chris@1115: assert_redirected_to '/projects/ecookbook/issues' Chris@1115: Chris@1115: copies = Issue.all(:order => 'id DESC', :limit => issues.size) Chris@1115: copies.each do |copy| Chris@1115: assert_equal 2, copy.project_id Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_not_changing_the_issue_attributes Chris@1115: @request.session[:user_id] = 2 Chris@1115: issues = [ Chris@1464: Issue.create!(:project_id => 1, :tracker_id => 1, :status_id => 1, Chris@1464: :priority_id => 2, :subject => 'issue 1', :author_id => 1, Chris@1464: :assigned_to_id => nil), Chris@1464: Issue.create!(:project_id => 2, :tracker_id => 3, :status_id => 2, Chris@1464: :priority_id => 1, :subject => 'issue 2', :author_id => 2, Chris@1464: :assigned_to_id => 3) Chris@1115: ] Chris@1115: assert_difference 'Issue.count', issues.size do Chris@1115: post :bulk_update, :ids => issues.map(&:id), :copy => '1', Chris@1115: :issue => { Chris@1115: :project_id => '', :tracker_id => '', :assigned_to_id => '', Chris@1115: :status_id => '', :start_date => '', :due_date => '' Chris@1115: } Chris@1115: end Chris@1115: Chris@1115: copies = Issue.all(:order => 'id DESC', :limit => issues.size) Chris@1115: issues.each do |orig| Chris@1115: copy = copies.detect {|c| c.subject == orig.subject} Chris@1115: assert_not_nil copy Chris@1115: assert_equal orig.project_id, copy.project_id Chris@1115: assert_equal orig.tracker_id, copy.tracker_id Chris@1115: assert_equal orig.status_id, copy.status_id Chris@1115: assert_equal orig.assigned_to_id, copy.assigned_to_id Chris@1115: assert_equal orig.priority_id, copy.priority_id Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_changing_the_issue_attributes Chris@1115: # Fixes random test failure with Mysql Chris@1464: # where Issue.where(:project_id => 2).limit(2).order('id desc') Chris@1115: # doesn't return the expected results Chris@1115: Issue.delete_all("project_id=2") Chris@1115: Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count', 2 do Chris@1115: assert_no_difference 'Project.find(1).issues.count' do Chris@1115: post :bulk_update, :ids => [1, 2], :copy => '1', Chris@1115: :issue => { Chris@1115: :project_id => '2', :tracker_id => '', :assigned_to_id => '4', Chris@1115: :status_id => '1', :start_date => '2009-12-01', :due_date => '2009-12-31' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: Chris@1464: copied_issues = Issue.where(:project_id => 2).limit(2).order('id desc').to_a Chris@1115: assert_equal 2, copied_issues.size Chris@1115: copied_issues.each do |issue| Chris@1115: assert_equal 2, issue.project_id, "Project is incorrect" Chris@1115: assert_equal 4, issue.assigned_to_id, "Assigned to is incorrect" Chris@1115: assert_equal 1, issue.status_id, "Status is incorrect" Chris@1115: assert_equal '2009-12-01', issue.start_date.to_s, "Start date is incorrect" Chris@1115: assert_equal '2009-12-31', issue.due_date.to_s, "Due date is incorrect" Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_adding_a_note Chris@1115: @request.session[:user_id] = 2 Chris@1115: assert_difference 'Issue.count', 1 do Chris@1115: post :bulk_update, :ids => [1], :copy => '1', Chris@1115: :notes => 'Copying one issue', Chris@1115: :issue => { Chris@1115: :project_id => '', :tracker_id => '', :assigned_to_id => '4', Chris@1115: :status_id => '3', :start_date => '2009-12-01', :due_date => '2009-12-31' Chris@1115: } Chris@1115: end Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_equal 1, issue.journals.size Chris@1115: journal = issue.journals.first Chris@1464: assert_equal 1, journal.details.size Chris@1115: assert_equal 'Copying one issue', journal.notes Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_not_copying_the_attachments Chris@1115: attachment_count = Issue.find(3).attachments.size Chris@1115: assert attachment_count > 0 Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', 1 do Chris@1115: assert_no_difference 'Attachment.count' do Chris@1115: post :bulk_update, :ids => [3], :copy => '1', Chris@1115: :issue => { Chris@1115: :project_id => '' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_copying_the_attachments Chris@1115: attachment_count = Issue.find(3).attachments.size Chris@1115: assert attachment_count > 0 Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', 1 do Chris@1115: assert_difference 'Attachment.count', attachment_count do Chris@1115: post :bulk_update, :ids => [3], :copy => '1', :copy_attachments => '1', Chris@1115: :issue => { Chris@1115: :project_id => '' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_add_relations_with_copied_issues Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', 2 do Chris@1115: assert_difference 'IssueRelation.count', 2 do Chris@1115: post :bulk_update, :ids => [1, 3], :copy => '1', Chris@1115: :issue => { Chris@1115: :project_id => '1' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_not_copying_the_subtasks Chris@1115: issue = Issue.generate_with_descendants! Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', 1 do Chris@1115: post :bulk_update, :ids => [issue.id], :copy => '1', Chris@1115: :issue => { Chris@1115: :project_id => '' Chris@1115: } Chris@1115: end Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_allow_copying_the_subtasks Chris@1115: issue = Issue.generate_with_descendants! Chris@1115: count = issue.descendants.count Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', count+1 do Chris@1115: post :bulk_update, :ids => [issue.id], :copy => '1', :copy_subtasks => '1', Chris@1115: :issue => { Chris@1115: :project_id => '' Chris@1115: } Chris@1115: end Chris@1115: copy = Issue.where(:parent_id => nil).order("id DESC").first Chris@1115: assert_equal count, copy.descendants.count Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_should_not_copy_selected_subtasks_twice Chris@1115: issue = Issue.generate_with_descendants! Chris@1115: count = issue.descendants.count Chris@1115: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', count+1 do Chris@1115: post :bulk_update, :ids => issue.self_and_descendants.map(&:id), :copy => '1', :copy_subtasks => '1', Chris@1115: :issue => { Chris@1115: :project_id => '' Chris@1115: } Chris@1115: end Chris@1115: copy = Issue.where(:parent_id => nil).order("id DESC").first Chris@1115: assert_equal count, copy.descendants.count Chris@1115: end Chris@1115: Chris@1115: def test_bulk_copy_to_another_project_should_follow_when_needed Chris@1115: @request.session[:user_id] = 2 Chris@1115: post :bulk_update, :ids => [1], :copy => '1', :issue => {:project_id => 2}, :follow => '1' Chris@1115: issue = Issue.first(:order => 'id DESC') Chris@1115: assert_redirected_to :controller => 'issues', :action => 'show', :id => issue Chris@1115: end Chris@1115: Chris@1464: def test_bulk_copy_with_all_failures_should_display_errors Chris@1464: @request.session[:user_id] = 2 Chris@1464: post :bulk_update, :ids => [1, 2], :copy => '1', :issue => {:start_date => 'foo'} Chris@1464: Chris@1464: assert_response :success Chris@1464: end Chris@1464: 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@1115: Chris@1115: assert_difference 'Issue.count', -1 do Chris@1115: delete :destroy, :id => 2 Chris@1115: end 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@1115: Chris@1115: assert_no_difference 'Issue.count' do Chris@1115: delete :destroy, :ids => [1, 3] Chris@1115: end 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@1464: Chris@1464: assert_select 'form' do Chris@1464: assert_select 'input[name=_method][value=delete]' Chris@1464: end Chris@0: end Chris@0: Chris@0: def test_destroy_issues_and_destroy_time_entries Chris@0: @request.session[:user_id] = 2 Chris@1115: Chris@1115: assert_difference 'Issue.count', -2 do Chris@1115: assert_difference 'TimeEntry.count', -3 do Chris@1115: delete :destroy, :ids => [1, 3], :todo => 'destroy' Chris@1115: end Chris@1115: end 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@1115: Chris@1115: assert_difference 'Issue.count', -2 do Chris@1115: assert_no_difference 'TimeEntry.count' do Chris@1115: delete :destroy, :ids => [1, 3], :todo => 'nullify' Chris@1115: end Chris@1115: end 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@1115: Chris@1115: assert_difference 'Issue.count', -2 do Chris@1115: assert_no_difference 'TimeEntry.count' do Chris@1115: delete :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2 Chris@1115: end Chris@1115: end 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@1115: Chris@1115: assert_difference 'Issue.count', -3 do Chris@1115: delete :destroy, :ids => [1, 2, 6], :todo => 'destroy' Chris@1115: end 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@1115: parent = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'Parent Issue') Chris@1115: child = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'Child Issue', :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@1115: delete :destroy, :ids => [parent.id, child.id], :todo => 'destroy' Chris@441: end Chris@441: assert_response 302 Chris@441: end Chris@441: Chris@1464: def test_destroy_invalid_should_respond_with_404 Chris@1464: @request.session[:user_id] = 2 Chris@1464: assert_no_difference 'Issue.count' do Chris@1464: delete :destroy, :id => 999 Chris@1464: end Chris@1464: assert_response 404 Chris@1464: end Chris@1464: Chris@0: def test_default_search_scope Chris@0: get :index Chris@1464: Chris@1464: assert_select 'div#quick-search form' do Chris@1464: assert_select 'input[name=issues][value=1][type=hidden]' Chris@1464: end Chris@0: end Chris@0: end