annotate .svn/pristine/70/70213c8e5dc3672aa042ac0c170938fbe19b6615.svn-base @ 1519:afce8026aaeb redmine-2.4-integration

Merge from branch "live"
author Chris Cannam
date Tue, 09 Sep 2014 09:34:53 +0100
parents e248c7af89ec
children
rev   line source
Chris@1494 1 # Redmine - project management software
Chris@1494 2 # Copyright (C) 2006-2014 Jean-Philippe Lang
Chris@1494 3 #
Chris@1494 4 # This program is free software; you can redistribute it and/or
Chris@1494 5 # modify it under the terms of the GNU General Public License
Chris@1494 6 # as published by the Free Software Foundation; either version 2
Chris@1494 7 # of the License, or (at your option) any later version.
Chris@1494 8 #
Chris@1494 9 # This program is distributed in the hope that it will be useful,
Chris@1494 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
Chris@1494 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Chris@1494 12 # GNU General Public License for more details.
Chris@1494 13 #
Chris@1494 14 # You should have received a copy of the GNU General Public License
Chris@1494 15 # along with this program; if not, write to the Free Software
Chris@1494 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
Chris@1494 17
Chris@1494 18 require File.expand_path('../../test_helper', __FILE__)
Chris@1494 19
Chris@1494 20 class IssuesControllerTest < ActionController::TestCase
Chris@1494 21 fixtures :projects,
Chris@1494 22 :users,
Chris@1494 23 :roles,
Chris@1494 24 :members,
Chris@1494 25 :member_roles,
Chris@1494 26 :issues,
Chris@1494 27 :issue_statuses,
Chris@1494 28 :versions,
Chris@1494 29 :trackers,
Chris@1494 30 :projects_trackers,
Chris@1494 31 :issue_categories,
Chris@1494 32 :enabled_modules,
Chris@1494 33 :enumerations,
Chris@1494 34 :attachments,
Chris@1494 35 :workflows,
Chris@1494 36 :custom_fields,
Chris@1494 37 :custom_values,
Chris@1494 38 :custom_fields_projects,
Chris@1494 39 :custom_fields_trackers,
Chris@1494 40 :time_entries,
Chris@1494 41 :journals,
Chris@1494 42 :journal_details,
Chris@1494 43 :queries,
Chris@1494 44 :repositories,
Chris@1494 45 :changesets
Chris@1494 46
Chris@1494 47 include Redmine::I18n
Chris@1494 48
Chris@1494 49 def setup
Chris@1494 50 User.current = nil
Chris@1494 51 end
Chris@1494 52
Chris@1494 53 def test_index
Chris@1494 54 with_settings :default_language => "en" do
Chris@1494 55 get :index
Chris@1494 56 assert_response :success
Chris@1494 57 assert_template 'index'
Chris@1494 58 assert_not_nil assigns(:issues)
Chris@1494 59 assert_nil assigns(:project)
Chris@1494 60
Chris@1494 61 # links to visible issues
Chris@1494 62 assert_select 'a[href=/issues/1]', :text => /Can&#x27;t print recipes/
Chris@1494 63 assert_select 'a[href=/issues/5]', :text => /Subproject issue/
Chris@1494 64 # private projects hidden
Chris@1494 65 assert_select 'a[href=/issues/6]', 0
Chris@1494 66 assert_select 'a[href=/issues/4]', 0
Chris@1494 67 # project column
Chris@1494 68 assert_select 'th', :text => /Project/
Chris@1494 69 end
Chris@1494 70 end
Chris@1494 71
Chris@1494 72 def test_index_should_not_list_issues_when_module_disabled
Chris@1494 73 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
Chris@1494 74 get :index
Chris@1494 75 assert_response :success
Chris@1494 76 assert_template 'index'
Chris@1494 77 assert_not_nil assigns(:issues)
Chris@1494 78 assert_nil assigns(:project)
Chris@1494 79
Chris@1494 80 assert_select 'a[href=/issues/1]', 0
Chris@1494 81 assert_select 'a[href=/issues/5]', :text => /Subproject issue/
Chris@1494 82 end
Chris@1494 83
Chris@1494 84 def test_index_should_list_visible_issues_only
Chris@1494 85 get :index, :per_page => 100
Chris@1494 86 assert_response :success
Chris@1494 87 assert_not_nil assigns(:issues)
Chris@1494 88 assert_nil assigns(:issues).detect {|issue| !issue.visible?}
Chris@1494 89 end
Chris@1494 90
Chris@1494 91 def test_index_with_project
Chris@1494 92 Setting.display_subprojects_issues = 0
Chris@1494 93 get :index, :project_id => 1
Chris@1494 94 assert_response :success
Chris@1494 95 assert_template 'index'
Chris@1494 96 assert_not_nil assigns(:issues)
Chris@1494 97
Chris@1494 98 assert_select 'a[href=/issues/1]', :text => /Can&#x27;t print recipes/
Chris@1494 99 assert_select 'a[href=/issues/5]', 0
Chris@1494 100 end
Chris@1494 101
Chris@1494 102 def test_index_with_project_and_subprojects
Chris@1494 103 Setting.display_subprojects_issues = 1
Chris@1494 104 get :index, :project_id => 1
Chris@1494 105 assert_response :success
Chris@1494 106 assert_template 'index'
Chris@1494 107 assert_not_nil assigns(:issues)
Chris@1494 108
Chris@1494 109 assert_select 'a[href=/issues/1]', :text => /Can&#x27;t print recipes/
Chris@1494 110 assert_select 'a[href=/issues/5]', :text => /Subproject issue/
Chris@1494 111 assert_select 'a[href=/issues/6]', 0
Chris@1494 112 end
Chris@1494 113
Chris@1494 114 def test_index_with_project_and_subprojects_should_show_private_subprojects_with_permission
Chris@1494 115 @request.session[:user_id] = 2
Chris@1494 116 Setting.display_subprojects_issues = 1
Chris@1494 117 get :index, :project_id => 1
Chris@1494 118 assert_response :success
Chris@1494 119 assert_template 'index'
Chris@1494 120 assert_not_nil assigns(:issues)
Chris@1494 121
Chris@1494 122 assert_select 'a[href=/issues/1]', :text => /Can&#x27;t print recipes/
Chris@1494 123 assert_select 'a[href=/issues/5]', :text => /Subproject issue/
Chris@1494 124 assert_select 'a[href=/issues/6]', :text => /Issue of a private subproject/
Chris@1494 125 end
Chris@1494 126
Chris@1494 127 def test_index_with_project_and_default_filter
Chris@1494 128 get :index, :project_id => 1, :set_filter => 1
Chris@1494 129 assert_response :success
Chris@1494 130 assert_template 'index'
Chris@1494 131 assert_not_nil assigns(:issues)
Chris@1494 132
Chris@1494 133 query = assigns(:query)
Chris@1494 134 assert_not_nil query
Chris@1494 135 # default filter
Chris@1494 136 assert_equal({'status_id' => {:operator => 'o', :values => ['']}}, query.filters)
Chris@1494 137 end
Chris@1494 138
Chris@1494 139 def test_index_with_project_and_filter
Chris@1494 140 get :index, :project_id => 1, :set_filter => 1,
Chris@1494 141 :f => ['tracker_id'],
Chris@1494 142 :op => {'tracker_id' => '='},
Chris@1494 143 :v => {'tracker_id' => ['1']}
Chris@1494 144 assert_response :success
Chris@1494 145 assert_template 'index'
Chris@1494 146 assert_not_nil assigns(:issues)
Chris@1494 147
Chris@1494 148 query = assigns(:query)
Chris@1494 149 assert_not_nil query
Chris@1494 150 assert_equal({'tracker_id' => {:operator => '=', :values => ['1']}}, query.filters)
Chris@1494 151 end
Chris@1494 152
Chris@1494 153 def test_index_with_short_filters
Chris@1494 154 to_test = {
Chris@1494 155 'status_id' => {
Chris@1494 156 'o' => { :op => 'o', :values => [''] },
Chris@1494 157 'c' => { :op => 'c', :values => [''] },
Chris@1494 158 '7' => { :op => '=', :values => ['7'] },
Chris@1494 159 '7|3|4' => { :op => '=', :values => ['7', '3', '4'] },
Chris@1494 160 '=7' => { :op => '=', :values => ['7'] },
Chris@1494 161 '!3' => { :op => '!', :values => ['3'] },
Chris@1494 162 '!7|3|4' => { :op => '!', :values => ['7', '3', '4'] }},
Chris@1494 163 'subject' => {
Chris@1494 164 'This is a subject' => { :op => '=', :values => ['This is a subject'] },
Chris@1494 165 'o' => { :op => '=', :values => ['o'] },
Chris@1494 166 '~This is part of a subject' => { :op => '~', :values => ['This is part of a subject'] },
Chris@1494 167 '!~This is part of a subject' => { :op => '!~', :values => ['This is part of a subject'] }},
Chris@1494 168 'tracker_id' => {
Chris@1494 169 '3' => { :op => '=', :values => ['3'] },
Chris@1494 170 '=3' => { :op => '=', :values => ['3'] }},
Chris@1494 171 'start_date' => {
Chris@1494 172 '2011-10-12' => { :op => '=', :values => ['2011-10-12'] },
Chris@1494 173 '=2011-10-12' => { :op => '=', :values => ['2011-10-12'] },
Chris@1494 174 '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] },
Chris@1494 175 '<=2011-10-12' => { :op => '<=', :values => ['2011-10-12'] },
Chris@1494 176 '><2011-10-01|2011-10-30' => { :op => '><', :values => ['2011-10-01', '2011-10-30'] },
Chris@1494 177 '<t+2' => { :op => '<t+', :values => ['2'] },
Chris@1494 178 '>t+2' => { :op => '>t+', :values => ['2'] },
Chris@1494 179 't+2' => { :op => 't+', :values => ['2'] },
Chris@1494 180 't' => { :op => 't', :values => [''] },
Chris@1494 181 'w' => { :op => 'w', :values => [''] },
Chris@1494 182 '>t-2' => { :op => '>t-', :values => ['2'] },
Chris@1494 183 '<t-2' => { :op => '<t-', :values => ['2'] },
Chris@1494 184 't-2' => { :op => 't-', :values => ['2'] }},
Chris@1494 185 'created_on' => {
Chris@1494 186 '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] },
Chris@1494 187 '<t-2' => { :op => '<t-', :values => ['2'] },
Chris@1494 188 '>t-2' => { :op => '>t-', :values => ['2'] },
Chris@1494 189 't-2' => { :op => 't-', :values => ['2'] }},
Chris@1494 190 'cf_1' => {
Chris@1494 191 'c' => { :op => '=', :values => ['c'] },
Chris@1494 192 '!c' => { :op => '!', :values => ['c'] },
Chris@1494 193 '!*' => { :op => '!*', :values => [''] },
Chris@1494 194 '*' => { :op => '*', :values => [''] }},
Chris@1494 195 'estimated_hours' => {
Chris@1494 196 '=13.4' => { :op => '=', :values => ['13.4'] },
Chris@1494 197 '>=45' => { :op => '>=', :values => ['45'] },
Chris@1494 198 '<=125' => { :op => '<=', :values => ['125'] },
Chris@1494 199 '><10.5|20.5' => { :op => '><', :values => ['10.5', '20.5'] },
Chris@1494 200 '!*' => { :op => '!*', :values => [''] },
Chris@1494 201 '*' => { :op => '*', :values => [''] }}
Chris@1494 202 }
Chris@1494 203
Chris@1494 204 default_filter = { 'status_id' => {:operator => 'o', :values => [''] }}
Chris@1494 205
Chris@1494 206 to_test.each do |field, expression_and_expected|
Chris@1494 207 expression_and_expected.each do |filter_expression, expected|
Chris@1494 208
Chris@1494 209 get :index, :set_filter => 1, field => filter_expression
Chris@1494 210
Chris@1494 211 assert_response :success
Chris@1494 212 assert_template 'index'
Chris@1494 213 assert_not_nil assigns(:issues)
Chris@1494 214
Chris@1494 215 query = assigns(:query)
Chris@1494 216 assert_not_nil query
Chris@1494 217 assert query.has_filter?(field)
Chris@1494 218 assert_equal(default_filter.merge({field => {:operator => expected[:op], :values => expected[:values]}}), query.filters)
Chris@1494 219 end
Chris@1494 220 end
Chris@1494 221 end
Chris@1494 222
Chris@1494 223 def test_index_with_project_and_empty_filters
Chris@1494 224 get :index, :project_id => 1, :set_filter => 1, :fields => ['']
Chris@1494 225 assert_response :success
Chris@1494 226 assert_template 'index'
Chris@1494 227 assert_not_nil assigns(:issues)
Chris@1494 228
Chris@1494 229 query = assigns(:query)
Chris@1494 230 assert_not_nil query
Chris@1494 231 # no filter
Chris@1494 232 assert_equal({}, query.filters)
Chris@1494 233 end
Chris@1494 234
Chris@1494 235 def test_index_with_project_custom_field_filter
Chris@1494 236 field = ProjectCustomField.create!(:name => 'Client', :is_filter => true, :field_format => 'string')
Chris@1494 237 CustomValue.create!(:custom_field => field, :customized => Project.find(3), :value => 'Foo')
Chris@1494 238 CustomValue.create!(:custom_field => field, :customized => Project.find(5), :value => 'Foo')
Chris@1494 239 filter_name = "project.cf_#{field.id}"
Chris@1494 240 @request.session[:user_id] = 1
Chris@1494 241
Chris@1494 242 get :index, :set_filter => 1,
Chris@1494 243 :f => [filter_name],
Chris@1494 244 :op => {filter_name => '='},
Chris@1494 245 :v => {filter_name => ['Foo']}
Chris@1494 246 assert_response :success
Chris@1494 247 assert_template 'index'
Chris@1494 248 assert_equal [3, 5], assigns(:issues).map(&:project_id).uniq.sort
Chris@1494 249 end
Chris@1494 250
Chris@1494 251 def test_index_with_query
Chris@1494 252 get :index, :project_id => 1, :query_id => 5
Chris@1494 253 assert_response :success
Chris@1494 254 assert_template 'index'
Chris@1494 255 assert_not_nil assigns(:issues)
Chris@1494 256 assert_nil assigns(:issue_count_by_group)
Chris@1494 257 end
Chris@1494 258
Chris@1494 259 def test_index_with_query_grouped_by_tracker
Chris@1494 260 get :index, :project_id => 1, :query_id => 6
Chris@1494 261 assert_response :success
Chris@1494 262 assert_template 'index'
Chris@1494 263 assert_not_nil assigns(:issues)
Chris@1494 264 assert_not_nil assigns(:issue_count_by_group)
Chris@1494 265 end
Chris@1494 266
Chris@1494 267 def test_index_with_query_grouped_by_list_custom_field
Chris@1494 268 get :index, :project_id => 1, :query_id => 9
Chris@1494 269 assert_response :success
Chris@1494 270 assert_template 'index'
Chris@1494 271 assert_not_nil assigns(:issues)
Chris@1494 272 assert_not_nil assigns(:issue_count_by_group)
Chris@1494 273 end
Chris@1494 274
Chris@1494 275 def test_index_with_query_grouped_by_user_custom_field
Chris@1494 276 cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user')
Chris@1494 277 CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2')
Chris@1494 278 CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3')
Chris@1494 279 CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3')
Chris@1494 280 CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '')
Chris@1494 281
Chris@1494 282 get :index, :project_id => 1, :set_filter => 1, :group_by => "cf_#{cf.id}"
Chris@1494 283 assert_response :success
Chris@1494 284
Chris@1494 285 assert_select 'tr.group', 3
Chris@1494 286 assert_select 'tr.group' do
Chris@1494 287 assert_select 'a', :text => 'John Smith'
Chris@1494 288 assert_select 'span.count', :text => '1'
Chris@1494 289 end
Chris@1494 290 assert_select 'tr.group' do
Chris@1494 291 assert_select 'a', :text => 'Dave Lopper'
Chris@1494 292 assert_select 'span.count', :text => '2'
Chris@1494 293 end
Chris@1494 294 end
Chris@1494 295
Chris@1494 296 def test_index_with_query_grouped_by_tracker_in_normal_order
Chris@1494 297 3.times {|i| Issue.generate!(:tracker_id => (i + 1))}
Chris@1494 298
Chris@1494 299 get :index, :set_filter => 1, :group_by => 'tracker', :sort => 'id:desc'
Chris@1494 300 assert_response :success
Chris@1494 301
Chris@1494 302 trackers = assigns(:issues).map(&:tracker).uniq
Chris@1494 303 assert_equal [1, 2, 3], trackers.map(&:id)
Chris@1494 304 end
Chris@1494 305
Chris@1494 306 def test_index_with_query_grouped_by_tracker_in_reverse_order
Chris@1494 307 3.times {|i| Issue.generate!(:tracker_id => (i + 1))}
Chris@1494 308
Chris@1494 309 get :index, :set_filter => 1, :group_by => 'tracker', :sort => 'id:desc,tracker:desc'
Chris@1494 310 assert_response :success
Chris@1494 311
Chris@1494 312 trackers = assigns(:issues).map(&:tracker).uniq
Chris@1494 313 assert_equal [3, 2, 1], trackers.map(&:id)
Chris@1494 314 end
Chris@1494 315
Chris@1494 316 def test_index_with_query_id_and_project_id_should_set_session_query
Chris@1494 317 get :index, :project_id => 1, :query_id => 4
Chris@1494 318 assert_response :success
Chris@1494 319 assert_kind_of Hash, session[:query]
Chris@1494 320 assert_equal 4, session[:query][:id]
Chris@1494 321 assert_equal 1, session[:query][:project_id]
Chris@1494 322 end
Chris@1494 323
Chris@1494 324 def test_index_with_invalid_query_id_should_respond_404
Chris@1494 325 get :index, :project_id => 1, :query_id => 999
Chris@1494 326 assert_response 404
Chris@1494 327 end
Chris@1494 328
Chris@1494 329 def test_index_with_cross_project_query_in_session_should_show_project_issues
Chris@1494 330 q = IssueQuery.create!(:name => "test", :user_id => 2, :visibility => IssueQuery::VISIBILITY_PRIVATE, :project => nil)
Chris@1494 331 @request.session[:query] = {:id => q.id, :project_id => 1}
Chris@1494 332
Chris@1494 333 with_settings :display_subprojects_issues => '0' do
Chris@1494 334 get :index, :project_id => 1
Chris@1494 335 end
Chris@1494 336 assert_response :success
Chris@1494 337 assert_not_nil assigns(:query)
Chris@1494 338 assert_equal q.id, assigns(:query).id
Chris@1494 339 assert_equal 1, assigns(:query).project_id
Chris@1494 340 assert_equal [1], assigns(:issues).map(&:project_id).uniq
Chris@1494 341 end
Chris@1494 342
Chris@1494 343 def test_private_query_should_not_be_available_to_other_users
Chris@1494 344 q = IssueQuery.create!(:name => "private", :user => User.find(2), :visibility => IssueQuery::VISIBILITY_PRIVATE, :project => nil)
Chris@1494 345 @request.session[:user_id] = 3
Chris@1494 346
Chris@1494 347 get :index, :query_id => q.id
Chris@1494 348 assert_response 403
Chris@1494 349 end
Chris@1494 350
Chris@1494 351 def test_private_query_should_be_available_to_its_user
Chris@1494 352 q = IssueQuery.create!(:name => "private", :user => User.find(2), :visibility => IssueQuery::VISIBILITY_PRIVATE, :project => nil)
Chris@1494 353 @request.session[:user_id] = 2
Chris@1494 354
Chris@1494 355 get :index, :query_id => q.id
Chris@1494 356 assert_response :success
Chris@1494 357 end
Chris@1494 358
Chris@1494 359 def test_public_query_should_be_available_to_other_users
Chris@1494 360 q = IssueQuery.create!(:name => "private", :user => User.find(2), :visibility => IssueQuery::VISIBILITY_PUBLIC, :project => nil)
Chris@1494 361 @request.session[:user_id] = 3
Chris@1494 362
Chris@1494 363 get :index, :query_id => q.id
Chris@1494 364 assert_response :success
Chris@1494 365 end
Chris@1494 366
Chris@1494 367 def test_index_should_omit_page_param_in_export_links
Chris@1494 368 get :index, :page => 2
Chris@1494 369 assert_response :success
Chris@1494 370 assert_select 'a.atom[href=/issues.atom]'
Chris@1494 371 assert_select 'a.csv[href=/issues.csv]'
Chris@1494 372 assert_select 'a.pdf[href=/issues.pdf]'
Chris@1494 373 assert_select 'form#csv-export-form[action=/issues.csv]'
Chris@1494 374 end
Chris@1494 375
Chris@1494 376 def test_index_csv
Chris@1494 377 get :index, :format => 'csv'
Chris@1494 378 assert_response :success
Chris@1494 379 assert_not_nil assigns(:issues)
Chris@1494 380 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 381 assert @response.body.starts_with?("#,")
Chris@1494 382 lines = @response.body.chomp.split("\n")
Chris@1494 383 assert_equal assigns(:query).columns.size, lines[0].split(',').size
Chris@1494 384 end
Chris@1494 385
Chris@1494 386 def test_index_csv_with_project
Chris@1494 387 get :index, :project_id => 1, :format => 'csv'
Chris@1494 388 assert_response :success
Chris@1494 389 assert_not_nil assigns(:issues)
Chris@1494 390 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 391 end
Chris@1494 392
Chris@1494 393 def test_index_csv_with_description
Chris@1494 394 Issue.generate!(:description => 'test_index_csv_with_description')
Chris@1494 395
Chris@1494 396 with_settings :default_language => 'en' do
Chris@1494 397 get :index, :format => 'csv', :description => '1'
Chris@1494 398 assert_response :success
Chris@1494 399 assert_not_nil assigns(:issues)
Chris@1494 400 end
Chris@1494 401
Chris@1494 402 assert_equal 'text/csv; header=present', response.content_type
Chris@1494 403 headers = response.body.chomp.split("\n").first.split(',')
Chris@1494 404 assert_include 'Description', headers
Chris@1494 405 assert_include 'test_index_csv_with_description', response.body
Chris@1494 406 end
Chris@1494 407
Chris@1494 408 def test_index_csv_with_spent_time_column
Chris@1494 409 issue = Issue.create!(:project_id => 1, :tracker_id => 1, :subject => 'test_index_csv_with_spent_time_column', :author_id => 2)
Chris@1494 410 TimeEntry.create!(:project => issue.project, :issue => issue, :hours => 7.33, :user => User.find(2), :spent_on => Date.today)
Chris@1494 411
Chris@1494 412 get :index, :format => 'csv', :set_filter => '1', :c => %w(subject spent_hours)
Chris@1494 413 assert_response :success
Chris@1494 414 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 415 lines = @response.body.chomp.split("\n")
Chris@1494 416 assert_include "#{issue.id},#{issue.subject},7.33", lines
Chris@1494 417 end
Chris@1494 418
Chris@1494 419 def test_index_csv_with_all_columns
Chris@1494 420 get :index, :format => 'csv', :columns => 'all'
Chris@1494 421 assert_response :success
Chris@1494 422 assert_not_nil assigns(:issues)
Chris@1494 423 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 424 assert_match /\A#,/, response.body
Chris@1494 425 lines = response.body.chomp.split("\n")
Chris@1494 426 assert_equal assigns(:query).available_inline_columns.size, lines[0].split(',').size
Chris@1494 427 end
Chris@1494 428
Chris@1494 429 def test_index_csv_with_multi_column_field
Chris@1494 430 CustomField.find(1).update_attribute :multiple, true
Chris@1494 431 issue = Issue.find(1)
Chris@1494 432 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
Chris@1494 433 issue.save!
Chris@1494 434
Chris@1494 435 get :index, :format => 'csv', :columns => 'all'
Chris@1494 436 assert_response :success
Chris@1494 437 lines = @response.body.chomp.split("\n")
Chris@1494 438 assert lines.detect {|line| line.include?('"MySQL, Oracle"')}
Chris@1494 439 end
Chris@1494 440
Chris@1494 441 def test_index_csv_should_format_float_custom_fields_with_csv_decimal_separator
Chris@1494 442 field = IssueCustomField.create!(:name => 'Float', :is_for_all => true, :tracker_ids => [1], :field_format => 'float')
Chris@1494 443 issue = Issue.generate!(:project_id => 1, :tracker_id => 1, :custom_field_values => {field.id => '185.6'})
Chris@1494 444
Chris@1494 445 with_settings :default_language => 'fr' do
Chris@1494 446 get :index, :format => 'csv', :columns => 'all'
Chris@1494 447 assert_response :success
Chris@1494 448 issue_line = response.body.chomp.split("\n").map {|line| line.split(';')}.detect {|line| line[0]==issue.id.to_s}
Chris@1494 449 assert_include '185,60', issue_line
Chris@1494 450 end
Chris@1494 451
Chris@1494 452 with_settings :default_language => 'en' do
Chris@1494 453 get :index, :format => 'csv', :columns => 'all'
Chris@1494 454 assert_response :success
Chris@1494 455 issue_line = response.body.chomp.split("\n").map {|line| line.split(',')}.detect {|line| line[0]==issue.id.to_s}
Chris@1494 456 assert_include '185.60', issue_line
Chris@1494 457 end
Chris@1494 458 end
Chris@1494 459
Chris@1494 460 def test_index_csv_big_5
Chris@1494 461 with_settings :default_language => "zh-TW" do
Chris@1494 462 str_utf8 = "\xe4\xb8\x80\xe6\x9c\x88"
Chris@1494 463 str_big5 = "\xa4@\xa4\xeb"
Chris@1494 464 if str_utf8.respond_to?(:force_encoding)
Chris@1494 465 str_utf8.force_encoding('UTF-8')
Chris@1494 466 str_big5.force_encoding('Big5')
Chris@1494 467 end
Chris@1494 468 issue = Issue.generate!(:subject => str_utf8)
Chris@1494 469
Chris@1494 470 get :index, :project_id => 1,
Chris@1494 471 :f => ['subject'],
Chris@1494 472 :op => '=', :values => [str_utf8],
Chris@1494 473 :format => 'csv'
Chris@1494 474 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 475 lines = @response.body.chomp.split("\n")
Chris@1494 476 s1 = "\xaa\xac\xbaA"
Chris@1494 477 if str_utf8.respond_to?(:force_encoding)
Chris@1494 478 s1.force_encoding('Big5')
Chris@1494 479 end
Chris@1494 480 assert_include s1, lines[0]
Chris@1494 481 assert_include str_big5, lines[1]
Chris@1494 482 end
Chris@1494 483 end
Chris@1494 484
Chris@1494 485 def test_index_csv_cannot_convert_should_be_replaced_big_5
Chris@1494 486 with_settings :default_language => "zh-TW" do
Chris@1494 487 str_utf8 = "\xe4\xbb\xa5\xe5\x86\x85"
Chris@1494 488 if str_utf8.respond_to?(:force_encoding)
Chris@1494 489 str_utf8.force_encoding('UTF-8')
Chris@1494 490 end
Chris@1494 491 issue = Issue.generate!(:subject => str_utf8)
Chris@1494 492
Chris@1494 493 get :index, :project_id => 1,
Chris@1494 494 :f => ['subject'],
Chris@1494 495 :op => '=', :values => [str_utf8],
Chris@1494 496 :c => ['status', 'subject'],
Chris@1494 497 :format => 'csv',
Chris@1494 498 :set_filter => 1
Chris@1494 499 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 500 lines = @response.body.chomp.split("\n")
Chris@1494 501 s1 = "\xaa\xac\xbaA" # status
Chris@1494 502 if str_utf8.respond_to?(:force_encoding)
Chris@1494 503 s1.force_encoding('Big5')
Chris@1494 504 end
Chris@1494 505 assert lines[0].include?(s1)
Chris@1494 506 s2 = lines[1].split(",")[2]
Chris@1494 507 if s1.respond_to?(:force_encoding)
Chris@1494 508 s3 = "\xa5H?" # subject
Chris@1494 509 s3.force_encoding('Big5')
Chris@1494 510 assert_equal s3, s2
Chris@1494 511 elsif RUBY_PLATFORM == 'java'
Chris@1494 512 assert_equal "??", s2
Chris@1494 513 else
Chris@1494 514 assert_equal "\xa5H???", s2
Chris@1494 515 end
Chris@1494 516 end
Chris@1494 517 end
Chris@1494 518
Chris@1494 519 def test_index_csv_tw
Chris@1494 520 with_settings :default_language => "zh-TW" do
Chris@1494 521 str1 = "test_index_csv_tw"
Chris@1494 522 issue = Issue.generate!(:subject => str1, :estimated_hours => '1234.5')
Chris@1494 523
Chris@1494 524 get :index, :project_id => 1,
Chris@1494 525 :f => ['subject'],
Chris@1494 526 :op => '=', :values => [str1],
Chris@1494 527 :c => ['estimated_hours', 'subject'],
Chris@1494 528 :format => 'csv',
Chris@1494 529 :set_filter => 1
Chris@1494 530 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 531 lines = @response.body.chomp.split("\n")
Chris@1494 532 assert_equal "#{issue.id},1234.50,#{str1}", lines[1]
Chris@1494 533 end
Chris@1494 534 end
Chris@1494 535
Chris@1494 536 def test_index_csv_fr
Chris@1494 537 with_settings :default_language => "fr" do
Chris@1494 538 str1 = "test_index_csv_fr"
Chris@1494 539 issue = Issue.generate!(:subject => str1, :estimated_hours => '1234.5')
Chris@1494 540
Chris@1494 541 get :index, :project_id => 1,
Chris@1494 542 :f => ['subject'],
Chris@1494 543 :op => '=', :values => [str1],
Chris@1494 544 :c => ['estimated_hours', 'subject'],
Chris@1494 545 :format => 'csv',
Chris@1494 546 :set_filter => 1
Chris@1494 547 assert_equal 'text/csv; header=present', @response.content_type
Chris@1494 548 lines = @response.body.chomp.split("\n")
Chris@1494 549 assert_equal "#{issue.id};1234,50;#{str1}", lines[1]
Chris@1494 550 end
Chris@1494 551 end
Chris@1494 552
Chris@1494 553 def test_index_pdf
Chris@1494 554 ["en", "zh", "zh-TW", "ja", "ko"].each do |lang|
Chris@1494 555 with_settings :default_language => lang do
Chris@1494 556
Chris@1494 557 get :index
Chris@1494 558 assert_response :success
Chris@1494 559 assert_template 'index'
Chris@1494 560
Chris@1494 561 if lang == "ja"
Chris@1494 562 if RUBY_PLATFORM != 'java'
Chris@1494 563 assert_equal "CP932", l(:general_pdf_encoding)
Chris@1494 564 end
Chris@1494 565 if RUBY_PLATFORM == 'java' && l(:general_pdf_encoding) == "CP932"
Chris@1494 566 next
Chris@1494 567 end
Chris@1494 568 end
Chris@1494 569
Chris@1494 570 get :index, :format => 'pdf'
Chris@1494 571 assert_response :success
Chris@1494 572 assert_not_nil assigns(:issues)
Chris@1494 573 assert_equal 'application/pdf', @response.content_type
Chris@1494 574
Chris@1494 575 get :index, :project_id => 1, :format => 'pdf'
Chris@1494 576 assert_response :success
Chris@1494 577 assert_not_nil assigns(:issues)
Chris@1494 578 assert_equal 'application/pdf', @response.content_type
Chris@1494 579
Chris@1494 580 get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
Chris@1494 581 assert_response :success
Chris@1494 582 assert_not_nil assigns(:issues)
Chris@1494 583 assert_equal 'application/pdf', @response.content_type
Chris@1494 584 end
Chris@1494 585 end
Chris@1494 586 end
Chris@1494 587
Chris@1494 588 def test_index_pdf_with_query_grouped_by_list_custom_field
Chris@1494 589 get :index, :project_id => 1, :query_id => 9, :format => 'pdf'
Chris@1494 590 assert_response :success
Chris@1494 591 assert_not_nil assigns(:issues)
Chris@1494 592 assert_not_nil assigns(:issue_count_by_group)
Chris@1494 593 assert_equal 'application/pdf', @response.content_type
Chris@1494 594 end
Chris@1494 595
Chris@1494 596 def test_index_atom
Chris@1494 597 get :index, :project_id => 'ecookbook', :format => 'atom'
Chris@1494 598 assert_response :success
Chris@1494 599 assert_template 'common/feed'
Chris@1494 600 assert_equal 'application/atom+xml', response.content_type
Chris@1494 601
Chris@1494 602 assert_select 'feed' do
Chris@1494 603 assert_select 'link[rel=self][href=?]', 'http://test.host/projects/ecookbook/issues.atom'
Chris@1494 604 assert_select 'link[rel=alternate][href=?]', 'http://test.host/projects/ecookbook/issues'
Chris@1494 605 assert_select 'entry link[href=?]', 'http://test.host/issues/1'
Chris@1494 606 end
Chris@1494 607 end
Chris@1494 608
Chris@1494 609 def test_index_sort
Chris@1494 610 get :index, :sort => 'tracker,id:desc'
Chris@1494 611 assert_response :success
Chris@1494 612
Chris@1494 613 sort_params = @request.session['issues_index_sort']
Chris@1494 614 assert sort_params.is_a?(String)
Chris@1494 615 assert_equal 'tracker,id:desc', sort_params
Chris@1494 616
Chris@1494 617 issues = assigns(:issues)
Chris@1494 618 assert_not_nil issues
Chris@1494 619 assert !issues.empty?
Chris@1494 620 assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
Chris@1494 621 end
Chris@1494 622
Chris@1494 623 def test_index_sort_by_field_not_included_in_columns
Chris@1494 624 Setting.issue_list_default_columns = %w(subject author)
Chris@1494 625 get :index, :sort => 'tracker'
Chris@1494 626 end
Chris@1494 627
Chris@1494 628 def test_index_sort_by_assigned_to
Chris@1494 629 get :index, :sort => 'assigned_to'
Chris@1494 630 assert_response :success
Chris@1494 631 assignees = assigns(:issues).collect(&:assigned_to).compact
Chris@1494 632 assert_equal assignees.sort, assignees
Chris@1494 633 end
Chris@1494 634
Chris@1494 635 def test_index_sort_by_assigned_to_desc
Chris@1494 636 get :index, :sort => 'assigned_to:desc'
Chris@1494 637 assert_response :success
Chris@1494 638 assignees = assigns(:issues).collect(&:assigned_to).compact
Chris@1494 639 assert_equal assignees.sort.reverse, assignees
Chris@1494 640 end
Chris@1494 641
Chris@1494 642 def test_index_group_by_assigned_to
Chris@1494 643 get :index, :group_by => 'assigned_to', :sort => 'priority'
Chris@1494 644 assert_response :success
Chris@1494 645 end
Chris@1494 646
Chris@1494 647 def test_index_sort_by_author
Chris@1494 648 get :index, :sort => 'author'
Chris@1494 649 assert_response :success
Chris@1494 650 authors = assigns(:issues).collect(&:author)
Chris@1494 651 assert_equal authors.sort, authors
Chris@1494 652 end
Chris@1494 653
Chris@1494 654 def test_index_sort_by_author_desc
Chris@1494 655 get :index, :sort => 'author:desc'
Chris@1494 656 assert_response :success
Chris@1494 657 authors = assigns(:issues).collect(&:author)
Chris@1494 658 assert_equal authors.sort.reverse, authors
Chris@1494 659 end
Chris@1494 660
Chris@1494 661 def test_index_group_by_author
Chris@1494 662 get :index, :group_by => 'author', :sort => 'priority'
Chris@1494 663 assert_response :success
Chris@1494 664 end
Chris@1494 665
Chris@1494 666 def test_index_sort_by_spent_hours
Chris@1494 667 get :index, :sort => 'spent_hours:desc'
Chris@1494 668 assert_response :success
Chris@1494 669 hours = assigns(:issues).collect(&:spent_hours)
Chris@1494 670 assert_equal hours.sort.reverse, hours
Chris@1494 671 end
Chris@1494 672
Chris@1494 673 def test_index_sort_by_user_custom_field
Chris@1494 674 cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user')
Chris@1494 675 CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2')
Chris@1494 676 CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3')
Chris@1494 677 CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3')
Chris@1494 678 CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '')
Chris@1494 679
Chris@1494 680 get :index, :project_id => 1, :set_filter => 1, :sort => "cf_#{cf.id},id"
Chris@1494 681 assert_response :success
Chris@1494 682
Chris@1494 683 assert_equal [2, 3, 1], assigns(:issues).select {|issue| issue.custom_field_value(cf).present?}.map(&:id)
Chris@1494 684 end
Chris@1494 685
Chris@1494 686 def test_index_with_columns
Chris@1494 687 columns = ['tracker', 'subject', 'assigned_to']
Chris@1494 688 get :index, :set_filter => 1, :c => columns
Chris@1494 689 assert_response :success
Chris@1494 690
Chris@1494 691 # query should use specified columns
Chris@1494 692 query = assigns(:query)
Chris@1494 693 assert_kind_of IssueQuery, query
Chris@1494 694 assert_equal columns, query.column_names.map(&:to_s)
Chris@1494 695
Chris@1494 696 # columns should be stored in session
Chris@1494 697 assert_kind_of Hash, session[:query]
Chris@1494 698 assert_kind_of Array, session[:query][:column_names]
Chris@1494 699 assert_equal columns, session[:query][:column_names].map(&:to_s)
Chris@1494 700
Chris@1494 701 # ensure only these columns are kept in the selected columns list
Chris@1494 702 assert_select 'select#selected_columns option' do
Chris@1494 703 assert_select 'option', 3
Chris@1494 704 assert_select 'option[value=tracker]'
Chris@1494 705 assert_select 'option[value=project]', 0
Chris@1494 706 end
Chris@1494 707 end
Chris@1494 708
Chris@1494 709 def test_index_without_project_should_implicitly_add_project_column_to_default_columns
Chris@1494 710 Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to']
Chris@1494 711 get :index, :set_filter => 1
Chris@1494 712
Chris@1494 713 # query should use specified columns
Chris@1494 714 query = assigns(:query)
Chris@1494 715 assert_kind_of IssueQuery, query
Chris@1494 716 assert_equal [:id, :project, :tracker, :subject, :assigned_to], query.columns.map(&:name)
Chris@1494 717 end
Chris@1494 718
Chris@1494 719 def test_index_without_project_and_explicit_default_columns_should_not_add_project_column
Chris@1494 720 Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to']
Chris@1494 721 columns = ['id', 'tracker', 'subject', 'assigned_to']
Chris@1494 722 get :index, :set_filter => 1, :c => columns
Chris@1494 723
Chris@1494 724 # query should use specified columns
Chris@1494 725 query = assigns(:query)
Chris@1494 726 assert_kind_of IssueQuery, query
Chris@1494 727 assert_equal columns.map(&:to_sym), query.columns.map(&:name)
Chris@1494 728 end
Chris@1494 729
Chris@1494 730 def test_index_with_custom_field_column
Chris@1494 731 columns = %w(tracker subject cf_2)
Chris@1494 732 get :index, :set_filter => 1, :c => columns
Chris@1494 733 assert_response :success
Chris@1494 734
Chris@1494 735 # query should use specified columns
Chris@1494 736 query = assigns(:query)
Chris@1494 737 assert_kind_of IssueQuery, query
Chris@1494 738 assert_equal columns, query.column_names.map(&:to_s)
Chris@1494 739
Chris@1494 740 assert_select 'table.issues td.cf_2.string'
Chris@1494 741 end
Chris@1494 742
Chris@1494 743 def test_index_with_multi_custom_field_column
Chris@1494 744 field = CustomField.find(1)
Chris@1494 745 field.update_attribute :multiple, true
Chris@1494 746 issue = Issue.find(1)
Chris@1494 747 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
Chris@1494 748 issue.save!
Chris@1494 749
Chris@1494 750 get :index, :set_filter => 1, :c => %w(tracker subject cf_1)
Chris@1494 751 assert_response :success
Chris@1494 752
Chris@1494 753 assert_select 'table.issues td.cf_1', :text => 'MySQL, Oracle'
Chris@1494 754 end
Chris@1494 755
Chris@1494 756 def test_index_with_multi_user_custom_field_column
Chris@1494 757 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
Chris@1494 758 :tracker_ids => [1], :is_for_all => true)
Chris@1494 759 issue = Issue.find(1)
Chris@1494 760 issue.custom_field_values = {field.id => ['2', '3']}
Chris@1494 761 issue.save!
Chris@1494 762
Chris@1494 763 get :index, :set_filter => 1, :c => ['tracker', 'subject', "cf_#{field.id}"]
Chris@1494 764 assert_response :success
Chris@1494 765
Chris@1494 766 assert_select "table.issues td.cf_#{field.id}" do
Chris@1494 767 assert_select 'a', 2
Chris@1494 768 assert_select 'a[href=?]', '/users/2', :text => 'John Smith'
Chris@1494 769 assert_select 'a[href=?]', '/users/3', :text => 'Dave Lopper'
Chris@1494 770 end
Chris@1494 771 end
Chris@1494 772
Chris@1494 773 def test_index_with_date_column
Chris@1494 774 with_settings :date_format => '%d/%m/%Y' do
Chris@1494 775 Issue.find(1).update_attribute :start_date, '1987-08-24'
Chris@1494 776 get :index, :set_filter => 1, :c => %w(start_date)
Chris@1494 777 assert_select "table.issues td.start_date", :text => '24/08/1987'
Chris@1494 778 end
Chris@1494 779 end
Chris@1494 780
Chris@1494 781 def test_index_with_done_ratio_column
Chris@1494 782 Issue.find(1).update_attribute :done_ratio, 40
Chris@1494 783 get :index, :set_filter => 1, :c => %w(done_ratio)
Chris@1494 784 assert_select 'table.issues td.done_ratio' do
Chris@1494 785 assert_select 'table.progress' do
Chris@1494 786 assert_select 'td.closed[style=?]', 'width: 40%;'
Chris@1494 787 end
Chris@1494 788 end
Chris@1494 789 end
Chris@1494 790
Chris@1494 791 def test_index_with_spent_hours_column
Chris@1494 792 get :index, :set_filter => 1, :c => %w(subject spent_hours)
Chris@1494 793 assert_select 'table.issues tr#issue-3 td.spent_hours', :text => '1.00'
Chris@1494 794 end
Chris@1494 795
Chris@1494 796 def test_index_should_not_show_spent_hours_column_without_permission
Chris@1494 797 Role.anonymous.remove_permission! :view_time_entries
Chris@1494 798 get :index, :set_filter => 1, :c => %w(subject spent_hours)
Chris@1494 799 assert_select 'td.spent_hours', 0
Chris@1494 800 end
Chris@1494 801
Chris@1494 802 def test_index_with_fixed_version_column
Chris@1494 803 get :index, :set_filter => 1, :c => %w(fixed_version)
Chris@1494 804 assert_select 'table.issues td.fixed_version' do
Chris@1494 805 assert_select 'a[href=?]', '/versions/2', :text => '1.0'
Chris@1494 806 end
Chris@1494 807 end
Chris@1494 808
Chris@1494 809 def test_index_with_relations_column
Chris@1494 810 IssueRelation.delete_all
Chris@1494 811 IssueRelation.create!(:relation_type => "relates", :issue_from => Issue.find(1), :issue_to => Issue.find(7))
Chris@1494 812 IssueRelation.create!(:relation_type => "relates", :issue_from => Issue.find(8), :issue_to => Issue.find(1))
Chris@1494 813 IssueRelation.create!(:relation_type => "blocks", :issue_from => Issue.find(1), :issue_to => Issue.find(11))
Chris@1494 814 IssueRelation.create!(:relation_type => "blocks", :issue_from => Issue.find(12), :issue_to => Issue.find(2))
Chris@1494 815
Chris@1494 816 get :index, :set_filter => 1, :c => %w(subject relations)
Chris@1494 817 assert_response :success
Chris@1494 818 assert_select "tr#issue-1 td.relations" do
Chris@1494 819 assert_select "span", 3
Chris@1494 820 assert_select "span", :text => "Related to #7"
Chris@1494 821 assert_select "span", :text => "Related to #8"
Chris@1494 822 assert_select "span", :text => "Blocks #11"
Chris@1494 823 end
Chris@1494 824 assert_select "tr#issue-2 td.relations" do
Chris@1494 825 assert_select "span", 1
Chris@1494 826 assert_select "span", :text => "Blocked by #12"
Chris@1494 827 end
Chris@1494 828 assert_select "tr#issue-3 td.relations" do
Chris@1494 829 assert_select "span", 0
Chris@1494 830 end
Chris@1494 831
Chris@1494 832 get :index, :set_filter => 1, :c => %w(relations), :format => 'csv'
Chris@1494 833 assert_response :success
Chris@1494 834 assert_equal 'text/csv; header=present', response.content_type
Chris@1494 835 lines = response.body.chomp.split("\n")
Chris@1494 836 assert_include '1,"Related to #7, Related to #8, Blocks #11"', lines
Chris@1494 837 assert_include '2,Blocked by #12', lines
Chris@1494 838 assert_include '3,""', lines
Chris@1494 839
Chris@1494 840 get :index, :set_filter => 1, :c => %w(subject relations), :format => 'pdf'
Chris@1494 841 assert_response :success
Chris@1494 842 assert_equal 'application/pdf', response.content_type
Chris@1494 843 end
Chris@1494 844
Chris@1494 845 def test_index_with_description_column
Chris@1494 846 get :index, :set_filter => 1, :c => %w(subject description)
Chris@1494 847
Chris@1494 848 assert_select 'table.issues thead th', 3 # columns: chekbox + id + subject
Chris@1494 849 assert_select 'td.description[colspan=3]', :text => 'Unable to print recipes'
Chris@1494 850
Chris@1494 851 get :index, :set_filter => 1, :c => %w(subject description), :format => 'pdf'
Chris@1494 852 assert_response :success
Chris@1494 853 assert_equal 'application/pdf', response.content_type
Chris@1494 854 end
Chris@1494 855
Chris@1494 856 def test_index_send_html_if_query_is_invalid
Chris@1494 857 get :index, :f => ['start_date'], :op => {:start_date => '='}
Chris@1494 858 assert_equal 'text/html', @response.content_type
Chris@1494 859 assert_template 'index'
Chris@1494 860 end
Chris@1494 861
Chris@1494 862 def test_index_send_nothing_if_query_is_invalid
Chris@1494 863 get :index, :f => ['start_date'], :op => {:start_date => '='}, :format => 'csv'
Chris@1494 864 assert_equal 'text/csv', @response.content_type
Chris@1494 865 assert @response.body.blank?
Chris@1494 866 end
Chris@1494 867
Chris@1494 868 def test_show_by_anonymous
Chris@1494 869 get :show, :id => 1
Chris@1494 870 assert_response :success
Chris@1494 871 assert_template 'show'
Chris@1494 872 assert_equal Issue.find(1), assigns(:issue)
Chris@1494 873 assert_select 'div.issue div.description', :text => /Unable to print recipes/
Chris@1494 874 # anonymous role is allowed to add a note
Chris@1494 875 assert_select 'form#issue-form' do
Chris@1494 876 assert_select 'fieldset' do
Chris@1494 877 assert_select 'legend', :text => 'Notes'
Chris@1494 878 assert_select 'textarea[name=?]', 'issue[notes]'
Chris@1494 879 end
Chris@1494 880 end
Chris@1494 881 assert_select 'title', :text => "Bug #1: Can&#x27;t print recipes - eCookbook - Redmine"
Chris@1494 882 end
Chris@1494 883
Chris@1494 884 def test_show_by_manager
Chris@1494 885 @request.session[:user_id] = 2
Chris@1494 886 get :show, :id => 1
Chris@1494 887 assert_response :success
Chris@1494 888 assert_select 'a', :text => /Quote/
Chris@1494 889 assert_select 'form#issue-form' do
Chris@1494 890 assert_select 'fieldset' do
Chris@1494 891 assert_select 'legend', :text => 'Change properties'
Chris@1494 892 assert_select 'input[name=?]', 'issue[subject]'
Chris@1494 893 end
Chris@1494 894 assert_select 'fieldset' do
Chris@1494 895 assert_select 'legend', :text => 'Log time'
Chris@1494 896 assert_select 'input[name=?]', 'time_entry[hours]'
Chris@1494 897 end
Chris@1494 898 assert_select 'fieldset' do
Chris@1494 899 assert_select 'legend', :text => 'Notes'
Chris@1494 900 assert_select 'textarea[name=?]', 'issue[notes]'
Chris@1494 901 end
Chris@1494 902 end
Chris@1494 903 end
Chris@1494 904
Chris@1494 905 def test_show_should_display_update_form
Chris@1494 906 @request.session[:user_id] = 2
Chris@1494 907 get :show, :id => 1
Chris@1494 908 assert_response :success
Chris@1494 909
Chris@1494 910 assert_select 'form#issue-form' do
Chris@1494 911 assert_select 'input[name=?]', 'issue[is_private]'
Chris@1494 912 assert_select 'select[name=?]', 'issue[project_id]'
Chris@1494 913 assert_select 'select[name=?]', 'issue[tracker_id]'
Chris@1494 914 assert_select 'input[name=?]', 'issue[subject]'
Chris@1494 915 assert_select 'textarea[name=?]', 'issue[description]'
Chris@1494 916 assert_select 'select[name=?]', 'issue[status_id]'
Chris@1494 917 assert_select 'select[name=?]', 'issue[priority_id]'
Chris@1494 918 assert_select 'select[name=?]', 'issue[assigned_to_id]'
Chris@1494 919 assert_select 'select[name=?]', 'issue[category_id]'
Chris@1494 920 assert_select 'select[name=?]', 'issue[fixed_version_id]'
Chris@1494 921 assert_select 'input[name=?]', 'issue[parent_issue_id]'
Chris@1494 922 assert_select 'input[name=?]', 'issue[start_date]'
Chris@1494 923 assert_select 'input[name=?]', 'issue[due_date]'
Chris@1494 924 assert_select 'select[name=?]', 'issue[done_ratio]'
Chris@1494 925 assert_select 'input[name=?]', 'issue[custom_field_values][2]'
Chris@1494 926 assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0
Chris@1494 927 assert_select 'textarea[name=?]', 'issue[notes]'
Chris@1494 928 end
Chris@1494 929 end
Chris@1494 930
Chris@1494 931 def test_show_should_display_update_form_with_minimal_permissions
Chris@1494 932 Role.find(1).update_attribute :permissions, [:view_issues, :add_issue_notes]
Chris@1494 933 WorkflowTransition.delete_all :role_id => 1
Chris@1494 934
Chris@1494 935 @request.session[:user_id] = 2
Chris@1494 936 get :show, :id => 1
Chris@1494 937 assert_response :success
Chris@1494 938
Chris@1494 939 assert_select 'form#issue-form' do
Chris@1494 940 assert_select 'input[name=?]', 'issue[is_private]', 0
Chris@1494 941 assert_select 'select[name=?]', 'issue[project_id]', 0
Chris@1494 942 assert_select 'select[name=?]', 'issue[tracker_id]', 0
Chris@1494 943 assert_select 'input[name=?]', 'issue[subject]', 0
Chris@1494 944 assert_select 'textarea[name=?]', 'issue[description]', 0
Chris@1494 945 assert_select 'select[name=?]', 'issue[status_id]', 0
Chris@1494 946 assert_select 'select[name=?]', 'issue[priority_id]', 0
Chris@1494 947 assert_select 'select[name=?]', 'issue[assigned_to_id]', 0
Chris@1494 948 assert_select 'select[name=?]', 'issue[category_id]', 0
Chris@1494 949 assert_select 'select[name=?]', 'issue[fixed_version_id]', 0
Chris@1494 950 assert_select 'input[name=?]', 'issue[parent_issue_id]', 0
Chris@1494 951 assert_select 'input[name=?]', 'issue[start_date]', 0
Chris@1494 952 assert_select 'input[name=?]', 'issue[due_date]', 0
Chris@1494 953 assert_select 'select[name=?]', 'issue[done_ratio]', 0
Chris@1494 954 assert_select 'input[name=?]', 'issue[custom_field_values][2]', 0
Chris@1494 955 assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0
Chris@1494 956 assert_select 'textarea[name=?]', 'issue[notes]'
Chris@1494 957 end
Chris@1494 958 end
Chris@1494 959
Chris@1494 960 def test_show_should_display_update_form_with_workflow_permissions
Chris@1494 961 Role.find(1).update_attribute :permissions, [:view_issues, :add_issue_notes]
Chris@1494 962
Chris@1494 963 @request.session[:user_id] = 2
Chris@1494 964 get :show, :id => 1
Chris@1494 965 assert_response :success
Chris@1494 966
Chris@1494 967 assert_select 'form#issue-form' do
Chris@1494 968 assert_select 'input[name=?]', 'issue[is_private]', 0
Chris@1494 969 assert_select 'select[name=?]', 'issue[project_id]', 0
Chris@1494 970 assert_select 'select[name=?]', 'issue[tracker_id]', 0
Chris@1494 971 assert_select 'input[name=?]', 'issue[subject]', 0
Chris@1494 972 assert_select 'textarea[name=?]', 'issue[description]', 0
Chris@1494 973 assert_select 'select[name=?]', 'issue[status_id]'
Chris@1494 974 assert_select 'select[name=?]', 'issue[priority_id]', 0
Chris@1494 975 assert_select 'select[name=?]', 'issue[assigned_to_id]'
Chris@1494 976 assert_select 'select[name=?]', 'issue[category_id]', 0
Chris@1494 977 assert_select 'select[name=?]', 'issue[fixed_version_id]'
Chris@1494 978 assert_select 'input[name=?]', 'issue[parent_issue_id]', 0
Chris@1494 979 assert_select 'input[name=?]', 'issue[start_date]', 0
Chris@1494 980 assert_select 'input[name=?]', 'issue[due_date]', 0
Chris@1494 981 assert_select 'select[name=?]', 'issue[done_ratio]'
Chris@1494 982 assert_select 'input[name=?]', 'issue[custom_field_values][2]', 0
Chris@1494 983 assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0
Chris@1494 984 assert_select 'textarea[name=?]', 'issue[notes]'
Chris@1494 985 end
Chris@1494 986 end
Chris@1494 987
Chris@1494 988 def test_show_should_not_display_update_form_without_permissions
Chris@1494 989 Role.find(1).update_attribute :permissions, [:view_issues]
Chris@1494 990
Chris@1494 991 @request.session[:user_id] = 2
Chris@1494 992 get :show, :id => 1
Chris@1494 993 assert_response :success
Chris@1494 994
Chris@1494 995 assert_select 'form#issue-form', 0
Chris@1494 996 end
Chris@1494 997
Chris@1494 998 def test_update_form_should_not_display_inactive_enumerations
Chris@1494 999 assert !IssuePriority.find(15).active?
Chris@1494 1000
Chris@1494 1001 @request.session[:user_id] = 2
Chris@1494 1002 get :show, :id => 1
Chris@1494 1003 assert_response :success
Chris@1494 1004
Chris@1494 1005 assert_select 'form#issue-form' do
Chris@1494 1006 assert_select 'select[name=?]', 'issue[priority_id]' do
Chris@1494 1007 assert_select 'option[value=4]'
Chris@1494 1008 assert_select 'option[value=15]', 0
Chris@1494 1009 end
Chris@1494 1010 end
Chris@1494 1011 end
Chris@1494 1012
Chris@1494 1013 def test_update_form_should_allow_attachment_upload
Chris@1494 1014 @request.session[:user_id] = 2
Chris@1494 1015 get :show, :id => 1
Chris@1494 1016
Chris@1494 1017 assert_select 'form#issue-form[method=post][enctype=multipart/form-data]' do
Chris@1494 1018 assert_select 'input[type=file][name=?]', 'attachments[dummy][file]'
Chris@1494 1019 end
Chris@1494 1020 end
Chris@1494 1021
Chris@1494 1022 def test_show_should_deny_anonymous_access_without_permission
Chris@1494 1023 Role.anonymous.remove_permission!(:view_issues)
Chris@1494 1024 get :show, :id => 1
Chris@1494 1025 assert_response :redirect
Chris@1494 1026 end
Chris@1494 1027
Chris@1494 1028 def test_show_should_deny_anonymous_access_to_private_issue
Chris@1494 1029 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@1494 1030 get :show, :id => 1
Chris@1494 1031 assert_response :redirect
Chris@1494 1032 end
Chris@1494 1033
Chris@1494 1034 def test_show_should_deny_non_member_access_without_permission
Chris@1494 1035 Role.non_member.remove_permission!(:view_issues)
Chris@1494 1036 @request.session[:user_id] = 9
Chris@1494 1037 get :show, :id => 1
Chris@1494 1038 assert_response 403
Chris@1494 1039 end
Chris@1494 1040
Chris@1494 1041 def test_show_should_deny_non_member_access_to_private_issue
Chris@1494 1042 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@1494 1043 @request.session[:user_id] = 9
Chris@1494 1044 get :show, :id => 1
Chris@1494 1045 assert_response 403
Chris@1494 1046 end
Chris@1494 1047
Chris@1494 1048 def test_show_should_deny_member_access_without_permission
Chris@1494 1049 Role.find(1).remove_permission!(:view_issues)
Chris@1494 1050 @request.session[:user_id] = 2
Chris@1494 1051 get :show, :id => 1
Chris@1494 1052 assert_response 403
Chris@1494 1053 end
Chris@1494 1054
Chris@1494 1055 def test_show_should_deny_member_access_to_private_issue_without_permission
Chris@1494 1056 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@1494 1057 @request.session[:user_id] = 3
Chris@1494 1058 get :show, :id => 1
Chris@1494 1059 assert_response 403
Chris@1494 1060 end
Chris@1494 1061
Chris@1494 1062 def test_show_should_allow_author_access_to_private_issue
Chris@1494 1063 Issue.update_all(["is_private = ?, author_id = 3", true], "id = 1")
Chris@1494 1064 @request.session[:user_id] = 3
Chris@1494 1065 get :show, :id => 1
Chris@1494 1066 assert_response :success
Chris@1494 1067 end
Chris@1494 1068
Chris@1494 1069 def test_show_should_allow_assignee_access_to_private_issue
Chris@1494 1070 Issue.update_all(["is_private = ?, assigned_to_id = 3", true], "id = 1")
Chris@1494 1071 @request.session[:user_id] = 3
Chris@1494 1072 get :show, :id => 1
Chris@1494 1073 assert_response :success
Chris@1494 1074 end
Chris@1494 1075
Chris@1494 1076 def test_show_should_allow_member_access_to_private_issue_with_permission
Chris@1494 1077 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@1494 1078 User.find(3).roles_for_project(Project.find(1)).first.update_attribute :issues_visibility, 'all'
Chris@1494 1079 @request.session[:user_id] = 3
Chris@1494 1080 get :show, :id => 1
Chris@1494 1081 assert_response :success
Chris@1494 1082 end
Chris@1494 1083
Chris@1494 1084 def test_show_should_not_disclose_relations_to_invisible_issues
Chris@1494 1085 Setting.cross_project_issue_relations = '1'
Chris@1494 1086 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
Chris@1494 1087 # Relation to a private project issue
Chris@1494 1088 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
Chris@1494 1089
Chris@1494 1090 get :show, :id => 1
Chris@1494 1091 assert_response :success
Chris@1494 1092
Chris@1494 1093 assert_select 'div#relations' do
Chris@1494 1094 assert_select 'a', :text => /#2$/
Chris@1494 1095 assert_select 'a', :text => /#4$/, :count => 0
Chris@1494 1096 end
Chris@1494 1097 end
Chris@1494 1098
Chris@1494 1099 def test_show_should_list_subtasks
Chris@1494 1100 Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :parent_issue_id => 1, :subject => 'Child Issue')
Chris@1494 1101
Chris@1494 1102 get :show, :id => 1
Chris@1494 1103 assert_response :success
Chris@1494 1104
Chris@1494 1105 assert_select 'div#issue_tree' do
Chris@1494 1106 assert_select 'td.subject', :text => /Child Issue/
Chris@1494 1107 end
Chris@1494 1108 end
Chris@1494 1109
Chris@1494 1110 def test_show_should_list_parents
Chris@1494 1111 issue = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :parent_issue_id => 1, :subject => 'Child Issue')
Chris@1494 1112
Chris@1494 1113 get :show, :id => issue.id
Chris@1494 1114 assert_response :success
Chris@1494 1115
Chris@1494 1116 assert_select 'div.subject' do
Chris@1494 1117 assert_select 'h3', 'Child Issue'
Chris@1494 1118 assert_select 'a[href=/issues/1]'
Chris@1494 1119 end
Chris@1494 1120 end
Chris@1494 1121
Chris@1494 1122 def test_show_should_not_display_prev_next_links_without_query_in_session
Chris@1494 1123 get :show, :id => 1
Chris@1494 1124 assert_response :success
Chris@1494 1125 assert_nil assigns(:prev_issue_id)
Chris@1494 1126 assert_nil assigns(:next_issue_id)
Chris@1494 1127
Chris@1494 1128 assert_select 'div.next-prev-links', 0
Chris@1494 1129 end
Chris@1494 1130
Chris@1494 1131 def test_show_should_display_prev_next_links_with_query_in_session
Chris@1494 1132 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => nil}
Chris@1494 1133 @request.session['issues_index_sort'] = 'id'
Chris@1494 1134
Chris@1494 1135 with_settings :display_subprojects_issues => '0' do
Chris@1494 1136 get :show, :id => 3
Chris@1494 1137 end
Chris@1494 1138
Chris@1494 1139 assert_response :success
Chris@1494 1140 # Previous and next issues for all projects
Chris@1494 1141 assert_equal 2, assigns(:prev_issue_id)
Chris@1494 1142 assert_equal 5, assigns(:next_issue_id)
Chris@1494 1143
Chris@1494 1144 count = Issue.open.visible.count
Chris@1494 1145
Chris@1494 1146 assert_select 'div.next-prev-links' do
Chris@1494 1147 assert_select 'a[href=/issues/2]', :text => /Previous/
Chris@1494 1148 assert_select 'a[href=/issues/5]', :text => /Next/
Chris@1494 1149 assert_select 'span.position', :text => "3 of #{count}"
Chris@1494 1150 end
Chris@1494 1151 end
Chris@1494 1152
Chris@1494 1153 def test_show_should_display_prev_next_links_with_saved_query_in_session
Chris@1494 1154 query = IssueQuery.create!(:name => 'test', :visibility => IssueQuery::VISIBILITY_PUBLIC, :user_id => 1,
Chris@1494 1155 :filters => {'status_id' => {:values => ['5'], :operator => '='}},
Chris@1494 1156 :sort_criteria => [['id', 'asc']])
Chris@1494 1157 @request.session[:query] = {:id => query.id, :project_id => nil}
Chris@1494 1158
Chris@1494 1159 get :show, :id => 11
Chris@1494 1160
Chris@1494 1161 assert_response :success
Chris@1494 1162 assert_equal query, assigns(:query)
Chris@1494 1163 # Previous and next issues for all projects
Chris@1494 1164 assert_equal 8, assigns(:prev_issue_id)
Chris@1494 1165 assert_equal 12, assigns(:next_issue_id)
Chris@1494 1166
Chris@1494 1167 assert_select 'div.next-prev-links' do
Chris@1494 1168 assert_select 'a[href=/issues/8]', :text => /Previous/
Chris@1494 1169 assert_select 'a[href=/issues/12]', :text => /Next/
Chris@1494 1170 end
Chris@1494 1171 end
Chris@1494 1172
Chris@1494 1173 def test_show_should_display_prev_next_links_with_query_and_sort_on_association
Chris@1494 1174 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => nil}
Chris@1494 1175
Chris@1494 1176 %w(project tracker status priority author assigned_to category fixed_version).each do |assoc_sort|
Chris@1494 1177 @request.session['issues_index_sort'] = assoc_sort
Chris@1494 1178
Chris@1494 1179 get :show, :id => 3
Chris@1494 1180 assert_response :success, "Wrong response status for #{assoc_sort} sort"
Chris@1494 1181
Chris@1494 1182 assert_select 'div.next-prev-links' do
Chris@1494 1183 assert_select 'a', :text => /(Previous|Next)/
Chris@1494 1184 end
Chris@1494 1185 end
Chris@1494 1186 end
Chris@1494 1187
Chris@1494 1188 def test_show_should_display_prev_next_links_with_project_query_in_session
Chris@1494 1189 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => 1}
Chris@1494 1190 @request.session['issues_index_sort'] = 'id'
Chris@1494 1191
Chris@1494 1192 with_settings :display_subprojects_issues => '0' do
Chris@1494 1193 get :show, :id => 3
Chris@1494 1194 end
Chris@1494 1195
Chris@1494 1196 assert_response :success
Chris@1494 1197 # Previous and next issues inside project
Chris@1494 1198 assert_equal 2, assigns(:prev_issue_id)
Chris@1494 1199 assert_equal 7, assigns(:next_issue_id)
Chris@1494 1200
Chris@1494 1201 assert_select 'div.next-prev-links' do
Chris@1494 1202 assert_select 'a[href=/issues/2]', :text => /Previous/
Chris@1494 1203 assert_select 'a[href=/issues/7]', :text => /Next/
Chris@1494 1204 end
Chris@1494 1205 end
Chris@1494 1206
Chris@1494 1207 def test_show_should_not_display_prev_link_for_first_issue
Chris@1494 1208 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'o'}}, :project_id => 1}
Chris@1494 1209 @request.session['issues_index_sort'] = 'id'
Chris@1494 1210
Chris@1494 1211 with_settings :display_subprojects_issues => '0' do
Chris@1494 1212 get :show, :id => 1
Chris@1494 1213 end
Chris@1494 1214
Chris@1494 1215 assert_response :success
Chris@1494 1216 assert_nil assigns(:prev_issue_id)
Chris@1494 1217 assert_equal 2, assigns(:next_issue_id)
Chris@1494 1218
Chris@1494 1219 assert_select 'div.next-prev-links' do
Chris@1494 1220 assert_select 'a', :text => /Previous/, :count => 0
Chris@1494 1221 assert_select 'a[href=/issues/2]', :text => /Next/
Chris@1494 1222 end
Chris@1494 1223 end
Chris@1494 1224
Chris@1494 1225 def test_show_should_not_display_prev_next_links_for_issue_not_in_query_results
Chris@1494 1226 @request.session[:query] = {:filters => {'status_id' => {:values => [''], :operator => 'c'}}, :project_id => 1}
Chris@1494 1227 @request.session['issues_index_sort'] = 'id'
Chris@1494 1228
Chris@1494 1229 get :show, :id => 1
Chris@1494 1230
Chris@1494 1231 assert_response :success
Chris@1494 1232 assert_nil assigns(:prev_issue_id)
Chris@1494 1233 assert_nil assigns(:next_issue_id)
Chris@1494 1234
Chris@1494 1235 assert_select 'a', :text => /Previous/, :count => 0
Chris@1494 1236 assert_select 'a', :text => /Next/, :count => 0
Chris@1494 1237 end
Chris@1494 1238
Chris@1494 1239 def test_show_show_should_display_prev_next_links_with_query_sort_by_user_custom_field
Chris@1494 1240 cf = IssueCustomField.create!(:name => 'User', :is_for_all => true, :tracker_ids => [1,2,3], :field_format => 'user')
Chris@1494 1241 CustomValue.create!(:custom_field => cf, :customized => Issue.find(1), :value => '2')
Chris@1494 1242 CustomValue.create!(:custom_field => cf, :customized => Issue.find(2), :value => '3')
Chris@1494 1243 CustomValue.create!(:custom_field => cf, :customized => Issue.find(3), :value => '3')
Chris@1494 1244 CustomValue.create!(:custom_field => cf, :customized => Issue.find(5), :value => '')
Chris@1494 1245
Chris@1494 1246 query = IssueQuery.create!(:name => 'test', :visibility => IssueQuery::VISIBILITY_PUBLIC, :user_id => 1, :filters => {},
Chris@1494 1247 :sort_criteria => [["cf_#{cf.id}", 'asc'], ['id', 'asc']])
Chris@1494 1248 @request.session[:query] = {:id => query.id, :project_id => nil}
Chris@1494 1249
Chris@1494 1250 get :show, :id => 3
Chris@1494 1251 assert_response :success
Chris@1494 1252
Chris@1494 1253 assert_equal 2, assigns(:prev_issue_id)
Chris@1494 1254 assert_equal 1, assigns(:next_issue_id)
Chris@1494 1255
Chris@1494 1256 assert_select 'div.next-prev-links' do
Chris@1494 1257 assert_select 'a[href=/issues/2]', :text => /Previous/
Chris@1494 1258 assert_select 'a[href=/issues/1]', :text => /Next/
Chris@1494 1259 end
Chris@1494 1260 end
Chris@1494 1261
Chris@1494 1262 def test_show_should_display_link_to_the_assignee
Chris@1494 1263 get :show, :id => 2
Chris@1494 1264 assert_response :success
Chris@1494 1265 assert_select '.assigned-to' do
Chris@1494 1266 assert_select 'a[href=/users/3]'
Chris@1494 1267 end
Chris@1494 1268 end
Chris@1494 1269
Chris@1494 1270 def test_show_should_display_visible_changesets_from_other_projects
Chris@1494 1271 project = Project.find(2)
Chris@1494 1272 issue = project.issues.first
Chris@1494 1273 issue.changeset_ids = [102]
Chris@1494 1274 issue.save!
Chris@1494 1275 # changesets from other projects should be displayed even if repository
Chris@1494 1276 # is disabled on issue's project
Chris@1494 1277 project.disable_module! :repository
Chris@1494 1278
Chris@1494 1279 @request.session[:user_id] = 2
Chris@1494 1280 get :show, :id => issue.id
Chris@1494 1281
Chris@1494 1282 assert_select 'a[href=?]', '/projects/ecookbook/repository/revisions/3'
Chris@1494 1283 end
Chris@1494 1284
Chris@1494 1285 def test_show_should_display_watchers
Chris@1494 1286 @request.session[:user_id] = 2
Chris@1494 1287 Issue.find(1).add_watcher User.find(2)
Chris@1494 1288
Chris@1494 1289 get :show, :id => 1
Chris@1494 1290 assert_select 'div#watchers ul' do
Chris@1494 1291 assert_select 'li' do
Chris@1494 1292 assert_select 'a[href=/users/2]'
Chris@1494 1293 assert_select 'a img[alt=Delete]'
Chris@1494 1294 end
Chris@1494 1295 end
Chris@1494 1296 end
Chris@1494 1297
Chris@1494 1298 def test_show_should_display_watchers_with_gravatars
Chris@1494 1299 @request.session[:user_id] = 2
Chris@1494 1300 Issue.find(1).add_watcher User.find(2)
Chris@1494 1301
Chris@1494 1302 with_settings :gravatar_enabled => '1' do
Chris@1494 1303 get :show, :id => 1
Chris@1494 1304 end
Chris@1494 1305
Chris@1494 1306 assert_select 'div#watchers ul' do
Chris@1494 1307 assert_select 'li' do
Chris@1494 1308 assert_select 'img.gravatar'
Chris@1494 1309 assert_select 'a[href=/users/2]'
Chris@1494 1310 assert_select 'a img[alt=Delete]'
Chris@1494 1311 end
Chris@1494 1312 end
Chris@1494 1313 end
Chris@1494 1314
Chris@1494 1315 def test_show_with_thumbnails_enabled_should_display_thumbnails
Chris@1494 1316 @request.session[:user_id] = 2
Chris@1494 1317
Chris@1494 1318 with_settings :thumbnails_enabled => '1' do
Chris@1494 1319 get :show, :id => 14
Chris@1494 1320 assert_response :success
Chris@1494 1321 end
Chris@1494 1322
Chris@1494 1323 assert_select 'div.thumbnails' do
Chris@1494 1324 assert_select 'a[href=/attachments/16/testfile.png]' do
Chris@1494 1325 assert_select 'img[src=/attachments/thumbnail/16]'
Chris@1494 1326 end
Chris@1494 1327 end
Chris@1494 1328 end
Chris@1494 1329
Chris@1494 1330 def test_show_with_thumbnails_disabled_should_not_display_thumbnails
Chris@1494 1331 @request.session[:user_id] = 2
Chris@1494 1332
Chris@1494 1333 with_settings :thumbnails_enabled => '0' do
Chris@1494 1334 get :show, :id => 14
Chris@1494 1335 assert_response :success
Chris@1494 1336 end
Chris@1494 1337
Chris@1494 1338 assert_select 'div.thumbnails', 0
Chris@1494 1339 end
Chris@1494 1340
Chris@1494 1341 def test_show_with_multi_custom_field
Chris@1494 1342 field = CustomField.find(1)
Chris@1494 1343 field.update_attribute :multiple, true
Chris@1494 1344 issue = Issue.find(1)
Chris@1494 1345 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
Chris@1494 1346 issue.save!
Chris@1494 1347
Chris@1494 1348 get :show, :id => 1
Chris@1494 1349 assert_response :success
Chris@1494 1350
Chris@1494 1351 assert_select 'td', :text => 'MySQL, Oracle'
Chris@1494 1352 end
Chris@1494 1353
Chris@1494 1354 def test_show_with_multi_user_custom_field
Chris@1494 1355 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
Chris@1494 1356 :tracker_ids => [1], :is_for_all => true)
Chris@1494 1357 issue = Issue.find(1)
Chris@1494 1358 issue.custom_field_values = {field.id => ['2', '3']}
Chris@1494 1359 issue.save!
Chris@1494 1360
Chris@1494 1361 get :show, :id => 1
Chris@1494 1362 assert_response :success
Chris@1494 1363
Chris@1494 1364 # TODO: should display links
Chris@1494 1365 assert_select 'td', :text => 'Dave Lopper, John Smith'
Chris@1494 1366 end
Chris@1494 1367
Chris@1494 1368 def test_show_should_display_private_notes_with_permission_only
Chris@1494 1369 journal = Journal.create!(:journalized => Issue.find(2), :notes => 'Privates notes', :private_notes => true, :user_id => 1)
Chris@1494 1370 @request.session[:user_id] = 2
Chris@1494 1371
Chris@1494 1372 get :show, :id => 2
Chris@1494 1373 assert_response :success
Chris@1494 1374 assert_include journal, assigns(:journals)
Chris@1494 1375
Chris@1494 1376 Role.find(1).remove_permission! :view_private_notes
Chris@1494 1377 get :show, :id => 2
Chris@1494 1378 assert_response :success
Chris@1494 1379 assert_not_include journal, assigns(:journals)
Chris@1494 1380 end
Chris@1494 1381
Chris@1494 1382 def test_show_atom
Chris@1494 1383 get :show, :id => 2, :format => 'atom'
Chris@1494 1384 assert_response :success
Chris@1494 1385 assert_template 'journals/index'
Chris@1494 1386 # Inline image
Chris@1494 1387 assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10'))
Chris@1494 1388 end
Chris@1494 1389
Chris@1494 1390 def test_show_export_to_pdf
Chris@1494 1391 get :show, :id => 3, :format => 'pdf'
Chris@1494 1392 assert_response :success
Chris@1494 1393 assert_equal 'application/pdf', @response.content_type
Chris@1494 1394 assert @response.body.starts_with?('%PDF')
Chris@1494 1395 assert_not_nil assigns(:issue)
Chris@1494 1396 end
Chris@1494 1397
Chris@1494 1398 def test_show_export_to_pdf_with_ancestors
Chris@1494 1399 issue = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1)
Chris@1494 1400
Chris@1494 1401 get :show, :id => issue.id, :format => 'pdf'
Chris@1494 1402 assert_response :success
Chris@1494 1403 assert_equal 'application/pdf', @response.content_type
Chris@1494 1404 assert @response.body.starts_with?('%PDF')
Chris@1494 1405 end
Chris@1494 1406
Chris@1494 1407 def test_show_export_to_pdf_with_descendants
Chris@1494 1408 c1 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1)
Chris@1494 1409 c2 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => 1)
Chris@1494 1410 c3 = Issue.generate!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => c1.id)
Chris@1494 1411
Chris@1494 1412 get :show, :id => 1, :format => 'pdf'
Chris@1494 1413 assert_response :success
Chris@1494 1414 assert_equal 'application/pdf', @response.content_type
Chris@1494 1415 assert @response.body.starts_with?('%PDF')
Chris@1494 1416 end
Chris@1494 1417
Chris@1494 1418 def test_show_export_to_pdf_with_journals
Chris@1494 1419 get :show, :id => 1, :format => 'pdf'
Chris@1494 1420 assert_response :success
Chris@1494 1421 assert_equal 'application/pdf', @response.content_type
Chris@1494 1422 assert @response.body.starts_with?('%PDF')
Chris@1494 1423 end
Chris@1494 1424
Chris@1494 1425 def test_show_export_to_pdf_with_changesets
Chris@1494 1426 Issue.find(3).changesets = Changeset.find_all_by_id(100, 101, 102)
Chris@1494 1427
Chris@1494 1428 get :show, :id => 3, :format => 'pdf'
Chris@1494 1429 assert_response :success
Chris@1494 1430 assert_equal 'application/pdf', @response.content_type
Chris@1494 1431 assert @response.body.starts_with?('%PDF')
Chris@1494 1432 end
Chris@1494 1433
Chris@1494 1434 def test_show_invalid_should_respond_with_404
Chris@1494 1435 get :show, :id => 999
Chris@1494 1436 assert_response 404
Chris@1494 1437 end
Chris@1494 1438
Chris@1494 1439 def test_get_new
Chris@1494 1440 @request.session[:user_id] = 2
Chris@1494 1441 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1442 assert_response :success
Chris@1494 1443 assert_template 'new'
Chris@1494 1444
Chris@1494 1445 assert_select 'form#issue-form' do
Chris@1494 1446 assert_select 'input[name=?]', 'issue[is_private]'
Chris@1494 1447 assert_select 'select[name=?]', 'issue[project_id]', 0
Chris@1494 1448 assert_select 'select[name=?]', 'issue[tracker_id]'
Chris@1494 1449 assert_select 'input[name=?]', 'issue[subject]'
Chris@1494 1450 assert_select 'textarea[name=?]', 'issue[description]'
Chris@1494 1451 assert_select 'select[name=?]', 'issue[status_id]'
Chris@1494 1452 assert_select 'select[name=?]', 'issue[priority_id]'
Chris@1494 1453 assert_select 'select[name=?]', 'issue[assigned_to_id]'
Chris@1494 1454 assert_select 'select[name=?]', 'issue[category_id]'
Chris@1494 1455 assert_select 'select[name=?]', 'issue[fixed_version_id]'
Chris@1494 1456 assert_select 'input[name=?]', 'issue[parent_issue_id]'
Chris@1494 1457 assert_select 'input[name=?]', 'issue[start_date]'
Chris@1494 1458 assert_select 'input[name=?]', 'issue[due_date]'
Chris@1494 1459 assert_select 'select[name=?]', 'issue[done_ratio]'
Chris@1494 1460 assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Default string'
Chris@1494 1461 assert_select 'input[name=?]', 'issue[watcher_user_ids][]'
Chris@1494 1462 end
Chris@1494 1463
Chris@1494 1464 # Be sure we don't display inactive IssuePriorities
Chris@1494 1465 assert ! IssuePriority.find(15).active?
Chris@1494 1466 assert_select 'select[name=?]', 'issue[priority_id]' do
Chris@1494 1467 assert_select 'option[value=15]', 0
Chris@1494 1468 end
Chris@1494 1469 end
Chris@1494 1470
Chris@1494 1471 def test_get_new_with_minimal_permissions
Chris@1494 1472 Role.find(1).update_attribute :permissions, [:add_issues]
Chris@1494 1473 WorkflowTransition.delete_all :role_id => 1
Chris@1494 1474
Chris@1494 1475 @request.session[:user_id] = 2
Chris@1494 1476 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1477 assert_response :success
Chris@1494 1478 assert_template 'new'
Chris@1494 1479
Chris@1494 1480 assert_select 'form#issue-form' do
Chris@1494 1481 assert_select 'input[name=?]', 'issue[is_private]', 0
Chris@1494 1482 assert_select 'select[name=?]', 'issue[project_id]', 0
Chris@1494 1483 assert_select 'select[name=?]', 'issue[tracker_id]'
Chris@1494 1484 assert_select 'input[name=?]', 'issue[subject]'
Chris@1494 1485 assert_select 'textarea[name=?]', 'issue[description]'
Chris@1494 1486 assert_select 'select[name=?]', 'issue[status_id]'
Chris@1494 1487 assert_select 'select[name=?]', 'issue[priority_id]'
Chris@1494 1488 assert_select 'select[name=?]', 'issue[assigned_to_id]'
Chris@1494 1489 assert_select 'select[name=?]', 'issue[category_id]'
Chris@1494 1490 assert_select 'select[name=?]', 'issue[fixed_version_id]'
Chris@1494 1491 assert_select 'input[name=?]', 'issue[parent_issue_id]', 0
Chris@1494 1492 assert_select 'input[name=?]', 'issue[start_date]'
Chris@1494 1493 assert_select 'input[name=?]', 'issue[due_date]'
Chris@1494 1494 assert_select 'select[name=?]', 'issue[done_ratio]'
Chris@1494 1495 assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Default string'
Chris@1494 1496 assert_select 'input[name=?]', 'issue[watcher_user_ids][]', 0
Chris@1494 1497 end
Chris@1494 1498 end
Chris@1494 1499
Chris@1494 1500 def test_get_new_with_list_custom_field
Chris@1494 1501 @request.session[:user_id] = 2
Chris@1494 1502 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1503 assert_response :success
Chris@1494 1504 assert_template 'new'
Chris@1494 1505
Chris@1494 1506 assert_select 'select.list_cf[name=?]', 'issue[custom_field_values][1]' do
Chris@1494 1507 assert_select 'option', 4
Chris@1494 1508 assert_select 'option[value=MySQL]', :text => 'MySQL'
Chris@1494 1509 end
Chris@1494 1510 end
Chris@1494 1511
Chris@1494 1512 def test_get_new_with_multi_custom_field
Chris@1494 1513 field = IssueCustomField.find(1)
Chris@1494 1514 field.update_attribute :multiple, true
Chris@1494 1515
Chris@1494 1516 @request.session[:user_id] = 2
Chris@1494 1517 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1518 assert_response :success
Chris@1494 1519 assert_template 'new'
Chris@1494 1520
Chris@1494 1521 assert_select 'select[name=?][multiple=multiple]', 'issue[custom_field_values][1][]' do
Chris@1494 1522 assert_select 'option', 3
Chris@1494 1523 assert_select 'option[value=MySQL]', :text => 'MySQL'
Chris@1494 1524 end
Chris@1494 1525 assert_select 'input[name=?][type=hidden][value=?]', 'issue[custom_field_values][1][]', ''
Chris@1494 1526 end
Chris@1494 1527
Chris@1494 1528 def test_get_new_with_multi_user_custom_field
Chris@1494 1529 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
Chris@1494 1530 :tracker_ids => [1], :is_for_all => true)
Chris@1494 1531
Chris@1494 1532 @request.session[:user_id] = 2
Chris@1494 1533 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1534 assert_response :success
Chris@1494 1535 assert_template 'new'
Chris@1494 1536
Chris@1494 1537 assert_select 'select[name=?][multiple=multiple]', "issue[custom_field_values][#{field.id}][]" do
Chris@1494 1538 assert_select 'option', Project.find(1).users.count
Chris@1494 1539 assert_select 'option[value=2]', :text => 'John Smith'
Chris@1494 1540 end
Chris@1494 1541 assert_select 'input[name=?][type=hidden][value=?]', "issue[custom_field_values][#{field.id}][]", ''
Chris@1494 1542 end
Chris@1494 1543
Chris@1494 1544 def test_get_new_with_date_custom_field
Chris@1494 1545 field = IssueCustomField.create!(:name => 'Date', :field_format => 'date', :tracker_ids => [1], :is_for_all => true)
Chris@1494 1546
Chris@1494 1547 @request.session[:user_id] = 2
Chris@1494 1548 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1549 assert_response :success
Chris@1494 1550
Chris@1494 1551 assert_select 'input[name=?]', "issue[custom_field_values][#{field.id}]"
Chris@1494 1552 end
Chris@1494 1553
Chris@1494 1554 def test_get_new_with_text_custom_field
Chris@1494 1555 field = IssueCustomField.create!(:name => 'Text', :field_format => 'text', :tracker_ids => [1], :is_for_all => true)
Chris@1494 1556
Chris@1494 1557 @request.session[:user_id] = 2
Chris@1494 1558 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1559 assert_response :success
Chris@1494 1560
Chris@1494 1561 assert_select 'textarea[name=?]', "issue[custom_field_values][#{field.id}]"
Chris@1494 1562 end
Chris@1494 1563
Chris@1494 1564 def test_get_new_without_default_start_date_is_creation_date
Chris@1494 1565 with_settings :default_issue_start_date_to_creation_date => 0 do
Chris@1494 1566 @request.session[:user_id] = 2
Chris@1494 1567 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1568 assert_response :success
Chris@1494 1569 assert_template 'new'
Chris@1494 1570 assert_select 'input[name=?]', 'issue[start_date]'
Chris@1494 1571 assert_select 'input[name=?][value]', 'issue[start_date]', 0
Chris@1494 1572 end
Chris@1494 1573 end
Chris@1494 1574
Chris@1494 1575 def test_get_new_with_default_start_date_is_creation_date
Chris@1494 1576 with_settings :default_issue_start_date_to_creation_date => 1 do
Chris@1494 1577 @request.session[:user_id] = 2
Chris@1494 1578 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1579 assert_response :success
Chris@1494 1580 assert_template 'new'
Chris@1494 1581 assert_select 'input[name=?][value=?]', 'issue[start_date]',
Chris@1494 1582 Date.today.to_s
Chris@1494 1583 end
Chris@1494 1584 end
Chris@1494 1585
Chris@1494 1586 def test_get_new_form_should_allow_attachment_upload
Chris@1494 1587 @request.session[:user_id] = 2
Chris@1494 1588 get :new, :project_id => 1, :tracker_id => 1
Chris@1494 1589
Chris@1494 1590 assert_select 'form[id=issue-form][method=post][enctype=multipart/form-data]' do
Chris@1494 1591 assert_select 'input[name=?][type=file]', 'attachments[dummy][file]'
Chris@1494 1592 end
Chris@1494 1593 end
Chris@1494 1594
Chris@1494 1595 def test_get_new_should_prefill_the_form_from_params
Chris@1494 1596 @request.session[:user_id] = 2
Chris@1494 1597 get :new, :project_id => 1,
Chris@1494 1598 :issue => {:tracker_id => 3, :description => 'Prefilled', :custom_field_values => {'2' => 'Custom field value'}}
Chris@1494 1599
Chris@1494 1600 issue = assigns(:issue)
Chris@1494 1601 assert_equal 3, issue.tracker_id
Chris@1494 1602 assert_equal 'Prefilled', issue.description
Chris@1494 1603 assert_equal 'Custom field value', issue.custom_field_value(2)
Chris@1494 1604
Chris@1494 1605 assert_select 'select[name=?]', 'issue[tracker_id]' do
Chris@1494 1606 assert_select 'option[value=3][selected=selected]'
Chris@1494 1607 end
Chris@1494 1608 assert_select 'textarea[name=?]', 'issue[description]', :text => /Prefilled/
Chris@1494 1609 assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Custom field value'
Chris@1494 1610 end
Chris@1494 1611
Chris@1494 1612 def test_get_new_should_mark_required_fields
Chris@1494 1613 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1614 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1615 WorkflowPermission.delete_all
Chris@1494 1616 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'required')
Chris@1494 1617 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'required')
Chris@1494 1618 @request.session[:user_id] = 2
Chris@1494 1619
Chris@1494 1620 get :new, :project_id => 1
Chris@1494 1621 assert_response :success
Chris@1494 1622 assert_template 'new'
Chris@1494 1623
Chris@1494 1624 assert_select 'label[for=issue_start_date]' do
Chris@1494 1625 assert_select 'span[class=required]', 0
Chris@1494 1626 end
Chris@1494 1627 assert_select 'label[for=issue_due_date]' do
Chris@1494 1628 assert_select 'span[class=required]'
Chris@1494 1629 end
Chris@1494 1630 assert_select 'label[for=?]', "issue_custom_field_values_#{cf1.id}" do
Chris@1494 1631 assert_select 'span[class=required]', 0
Chris@1494 1632 end
Chris@1494 1633 assert_select 'label[for=?]', "issue_custom_field_values_#{cf2.id}" do
Chris@1494 1634 assert_select 'span[class=required]'
Chris@1494 1635 end
Chris@1494 1636 end
Chris@1494 1637
Chris@1494 1638 def test_get_new_should_not_display_readonly_fields
Chris@1494 1639 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1640 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1641 WorkflowPermission.delete_all
Chris@1494 1642 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => 'due_date', :rule => 'readonly')
Chris@1494 1643 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'readonly')
Chris@1494 1644 @request.session[:user_id] = 2
Chris@1494 1645
Chris@1494 1646 get :new, :project_id => 1
Chris@1494 1647 assert_response :success
Chris@1494 1648 assert_template 'new'
Chris@1494 1649
Chris@1494 1650 assert_select 'input[name=?]', 'issue[start_date]'
Chris@1494 1651 assert_select 'input[name=?]', 'issue[due_date]', 0
Chris@1494 1652 assert_select 'input[name=?]', "issue[custom_field_values][#{cf1.id}]"
Chris@1494 1653 assert_select 'input[name=?]', "issue[custom_field_values][#{cf2.id}]", 0
Chris@1494 1654 end
Chris@1494 1655
Chris@1494 1656 def test_get_new_without_tracker_id
Chris@1494 1657 @request.session[:user_id] = 2
Chris@1494 1658 get :new, :project_id => 1
Chris@1494 1659 assert_response :success
Chris@1494 1660 assert_template 'new'
Chris@1494 1661
Chris@1494 1662 issue = assigns(:issue)
Chris@1494 1663 assert_not_nil issue
Chris@1494 1664 assert_equal Project.find(1).trackers.first, issue.tracker
Chris@1494 1665 end
Chris@1494 1666
Chris@1494 1667 def test_get_new_with_no_default_status_should_display_an_error
Chris@1494 1668 @request.session[:user_id] = 2
Chris@1494 1669 IssueStatus.delete_all
Chris@1494 1670
Chris@1494 1671 get :new, :project_id => 1
Chris@1494 1672 assert_response 500
Chris@1494 1673 assert_error_tag :content => /No default issue/
Chris@1494 1674 end
Chris@1494 1675
Chris@1494 1676 def test_get_new_with_no_tracker_should_display_an_error
Chris@1494 1677 @request.session[:user_id] = 2
Chris@1494 1678 Tracker.delete_all
Chris@1494 1679
Chris@1494 1680 get :new, :project_id => 1
Chris@1494 1681 assert_response 500
Chris@1494 1682 assert_error_tag :content => /No tracker/
Chris@1494 1683 end
Chris@1494 1684
Chris@1494 1685 def test_update_form_for_new_issue
Chris@1494 1686 @request.session[:user_id] = 2
Chris@1494 1687 xhr :post, :update_form, :project_id => 1,
Chris@1494 1688 :issue => {:tracker_id => 2,
Chris@1494 1689 :subject => 'This is the test_new issue',
Chris@1494 1690 :description => 'This is the description',
Chris@1494 1691 :priority_id => 5}
Chris@1494 1692 assert_response :success
Chris@1494 1693 assert_template 'update_form'
Chris@1494 1694 assert_template 'form'
Chris@1494 1695 assert_equal 'text/javascript', response.content_type
Chris@1494 1696
Chris@1494 1697 issue = assigns(:issue)
Chris@1494 1698 assert_kind_of Issue, issue
Chris@1494 1699 assert_equal 1, issue.project_id
Chris@1494 1700 assert_equal 2, issue.tracker_id
Chris@1494 1701 assert_equal 'This is the test_new issue', issue.subject
Chris@1494 1702 end
Chris@1494 1703
Chris@1494 1704 def test_update_form_for_new_issue_should_propose_transitions_based_on_initial_status
Chris@1494 1705 @request.session[:user_id] = 2
Chris@1494 1706 WorkflowTransition.delete_all
Chris@1494 1707 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 2)
Chris@1494 1708 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1, :new_status_id => 5)
Chris@1494 1709 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 5, :new_status_id => 4)
Chris@1494 1710
Chris@1494 1711 xhr :post, :update_form, :project_id => 1,
Chris@1494 1712 :issue => {:tracker_id => 1,
Chris@1494 1713 :status_id => 5,
Chris@1494 1714 :subject => 'This is an issue'}
Chris@1494 1715
Chris@1494 1716 assert_equal 5, assigns(:issue).status_id
Chris@1494 1717 assert_equal [1,2,5], assigns(:allowed_statuses).map(&:id).sort
Chris@1494 1718 end
Chris@1494 1719
Chris@1494 1720 def test_post_create
Chris@1494 1721 @request.session[:user_id] = 2
Chris@1494 1722 assert_difference 'Issue.count' do
Chris@1494 1723 post :create, :project_id => 1,
Chris@1494 1724 :issue => {:tracker_id => 3,
Chris@1494 1725 :status_id => 2,
Chris@1494 1726 :subject => 'This is the test_new issue',
Chris@1494 1727 :description => 'This is the description',
Chris@1494 1728 :priority_id => 5,
Chris@1494 1729 :start_date => '2010-11-07',
Chris@1494 1730 :estimated_hours => '',
Chris@1494 1731 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@1494 1732 end
Chris@1494 1733 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@1494 1734
Chris@1494 1735 issue = Issue.find_by_subject('This is the test_new issue')
Chris@1494 1736 assert_not_nil issue
Chris@1494 1737 assert_equal 2, issue.author_id
Chris@1494 1738 assert_equal 3, issue.tracker_id
Chris@1494 1739 assert_equal 2, issue.status_id
Chris@1494 1740 assert_equal Date.parse('2010-11-07'), issue.start_date
Chris@1494 1741 assert_nil issue.estimated_hours
Chris@1494 1742 v = issue.custom_values.where(:custom_field_id => 2).first
Chris@1494 1743 assert_not_nil v
Chris@1494 1744 assert_equal 'Value for field 2', v.value
Chris@1494 1745 end
Chris@1494 1746
Chris@1494 1747 def test_post_new_with_group_assignment
Chris@1494 1748 group = Group.find(11)
Chris@1494 1749 project = Project.find(1)
Chris@1494 1750 project.members << Member.new(:principal => group, :roles => [Role.givable.first])
Chris@1494 1751
Chris@1494 1752 with_settings :issue_group_assignment => '1' do
Chris@1494 1753 @request.session[:user_id] = 2
Chris@1494 1754 assert_difference 'Issue.count' do
Chris@1494 1755 post :create, :project_id => project.id,
Chris@1494 1756 :issue => {:tracker_id => 3,
Chris@1494 1757 :status_id => 1,
Chris@1494 1758 :subject => 'This is the test_new_with_group_assignment issue',
Chris@1494 1759 :assigned_to_id => group.id}
Chris@1494 1760 end
Chris@1494 1761 end
Chris@1494 1762 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@1494 1763
Chris@1494 1764 issue = Issue.find_by_subject('This is the test_new_with_group_assignment issue')
Chris@1494 1765 assert_not_nil issue
Chris@1494 1766 assert_equal group, issue.assigned_to
Chris@1494 1767 end
Chris@1494 1768
Chris@1494 1769 def test_post_create_without_start_date_and_default_start_date_is_not_creation_date
Chris@1494 1770 with_settings :default_issue_start_date_to_creation_date => 0 do
Chris@1494 1771 @request.session[:user_id] = 2
Chris@1494 1772 assert_difference 'Issue.count' do
Chris@1494 1773 post :create, :project_id => 1,
Chris@1494 1774 :issue => {:tracker_id => 3,
Chris@1494 1775 :status_id => 2,
Chris@1494 1776 :subject => 'This is the test_new issue',
Chris@1494 1777 :description => 'This is the description',
Chris@1494 1778 :priority_id => 5,
Chris@1494 1779 :estimated_hours => '',
Chris@1494 1780 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@1494 1781 end
Chris@1494 1782 assert_redirected_to :controller => 'issues', :action => 'show',
Chris@1494 1783 :id => Issue.last.id
Chris@1494 1784 issue = Issue.find_by_subject('This is the test_new issue')
Chris@1494 1785 assert_not_nil issue
Chris@1494 1786 assert_nil issue.start_date
Chris@1494 1787 end
Chris@1494 1788 end
Chris@1494 1789
Chris@1494 1790 def test_post_create_without_start_date_and_default_start_date_is_creation_date
Chris@1494 1791 with_settings :default_issue_start_date_to_creation_date => 1 do
Chris@1494 1792 @request.session[:user_id] = 2
Chris@1494 1793 assert_difference 'Issue.count' do
Chris@1494 1794 post :create, :project_id => 1,
Chris@1494 1795 :issue => {:tracker_id => 3,
Chris@1494 1796 :status_id => 2,
Chris@1494 1797 :subject => 'This is the test_new issue',
Chris@1494 1798 :description => 'This is the description',
Chris@1494 1799 :priority_id => 5,
Chris@1494 1800 :estimated_hours => '',
Chris@1494 1801 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@1494 1802 end
Chris@1494 1803 assert_redirected_to :controller => 'issues', :action => 'show',
Chris@1494 1804 :id => Issue.last.id
Chris@1494 1805 issue = Issue.find_by_subject('This is the test_new issue')
Chris@1494 1806 assert_not_nil issue
Chris@1494 1807 assert_equal Date.today, issue.start_date
Chris@1494 1808 end
Chris@1494 1809 end
Chris@1494 1810
Chris@1494 1811 def test_post_create_and_continue
Chris@1494 1812 @request.session[:user_id] = 2
Chris@1494 1813 assert_difference 'Issue.count' do
Chris@1494 1814 post :create, :project_id => 1,
Chris@1494 1815 :issue => {:tracker_id => 3, :subject => 'This is first issue', :priority_id => 5},
Chris@1494 1816 :continue => ''
Chris@1494 1817 end
Chris@1494 1818
Chris@1494 1819 issue = Issue.first(:order => 'id DESC')
Chris@1494 1820 assert_redirected_to :controller => 'issues', :action => 'new', :project_id => 'ecookbook', :issue => {:tracker_id => 3}
Chris@1494 1821 assert_not_nil flash[:notice], "flash was not set"
Chris@1494 1822 assert_include %|<a href="/issues/#{issue.id}" title="This is first issue">##{issue.id}</a>|, flash[:notice], "issue link not found in the flash message"
Chris@1494 1823 end
Chris@1494 1824
Chris@1494 1825 def test_post_create_without_custom_fields_param
Chris@1494 1826 @request.session[:user_id] = 2
Chris@1494 1827 assert_difference 'Issue.count' do
Chris@1494 1828 post :create, :project_id => 1,
Chris@1494 1829 :issue => {:tracker_id => 1,
Chris@1494 1830 :subject => 'This is the test_new issue',
Chris@1494 1831 :description => 'This is the description',
Chris@1494 1832 :priority_id => 5}
Chris@1494 1833 end
Chris@1494 1834 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@1494 1835 end
Chris@1494 1836
Chris@1494 1837 def test_post_create_with_multi_custom_field
Chris@1494 1838 field = IssueCustomField.find_by_name('Database')
Chris@1494 1839 field.update_attribute(:multiple, true)
Chris@1494 1840
Chris@1494 1841 @request.session[:user_id] = 2
Chris@1494 1842 assert_difference 'Issue.count' do
Chris@1494 1843 post :create, :project_id => 1,
Chris@1494 1844 :issue => {:tracker_id => 1,
Chris@1494 1845 :subject => 'This is the test_new issue',
Chris@1494 1846 :description => 'This is the description',
Chris@1494 1847 :priority_id => 5,
Chris@1494 1848 :custom_field_values => {'1' => ['', 'MySQL', 'Oracle']}}
Chris@1494 1849 end
Chris@1494 1850 assert_response 302
Chris@1494 1851 issue = Issue.first(:order => 'id DESC')
Chris@1494 1852 assert_equal ['MySQL', 'Oracle'], issue.custom_field_value(1).sort
Chris@1494 1853 end
Chris@1494 1854
Chris@1494 1855 def test_post_create_with_empty_multi_custom_field
Chris@1494 1856 field = IssueCustomField.find_by_name('Database')
Chris@1494 1857 field.update_attribute(:multiple, true)
Chris@1494 1858
Chris@1494 1859 @request.session[:user_id] = 2
Chris@1494 1860 assert_difference 'Issue.count' do
Chris@1494 1861 post :create, :project_id => 1,
Chris@1494 1862 :issue => {:tracker_id => 1,
Chris@1494 1863 :subject => 'This is the test_new issue',
Chris@1494 1864 :description => 'This is the description',
Chris@1494 1865 :priority_id => 5,
Chris@1494 1866 :custom_field_values => {'1' => ['']}}
Chris@1494 1867 end
Chris@1494 1868 assert_response 302
Chris@1494 1869 issue = Issue.first(:order => 'id DESC')
Chris@1494 1870 assert_equal [''], issue.custom_field_value(1).sort
Chris@1494 1871 end
Chris@1494 1872
Chris@1494 1873 def test_post_create_with_multi_user_custom_field
Chris@1494 1874 field = IssueCustomField.create!(:name => 'Multi user', :field_format => 'user', :multiple => true,
Chris@1494 1875 :tracker_ids => [1], :is_for_all => true)
Chris@1494 1876
Chris@1494 1877 @request.session[:user_id] = 2
Chris@1494 1878 assert_difference 'Issue.count' do
Chris@1494 1879 post :create, :project_id => 1,
Chris@1494 1880 :issue => {:tracker_id => 1,
Chris@1494 1881 :subject => 'This is the test_new issue',
Chris@1494 1882 :description => 'This is the description',
Chris@1494 1883 :priority_id => 5,
Chris@1494 1884 :custom_field_values => {field.id.to_s => ['', '2', '3']}}
Chris@1494 1885 end
Chris@1494 1886 assert_response 302
Chris@1494 1887 issue = Issue.first(:order => 'id DESC')
Chris@1494 1888 assert_equal ['2', '3'], issue.custom_field_value(field).sort
Chris@1494 1889 end
Chris@1494 1890
Chris@1494 1891 def test_post_create_with_required_custom_field_and_without_custom_fields_param
Chris@1494 1892 field = IssueCustomField.find_by_name('Database')
Chris@1494 1893 field.update_attribute(:is_required, true)
Chris@1494 1894
Chris@1494 1895 @request.session[:user_id] = 2
Chris@1494 1896 assert_no_difference 'Issue.count' do
Chris@1494 1897 post :create, :project_id => 1,
Chris@1494 1898 :issue => {:tracker_id => 1,
Chris@1494 1899 :subject => 'This is the test_new issue',
Chris@1494 1900 :description => 'This is the description',
Chris@1494 1901 :priority_id => 5}
Chris@1494 1902 end
Chris@1494 1903 assert_response :success
Chris@1494 1904 assert_template 'new'
Chris@1494 1905 issue = assigns(:issue)
Chris@1494 1906 assert_not_nil issue
Chris@1494 1907 assert_error_tag :content => /Database can&#x27;t be blank/
Chris@1494 1908 end
Chris@1494 1909
Chris@1494 1910 def test_create_should_validate_required_fields
Chris@1494 1911 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1912 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1913 WorkflowPermission.delete_all
Chris@1494 1914 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => 'due_date', :rule => 'required')
Chris@1494 1915 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'required')
Chris@1494 1916 @request.session[:user_id] = 2
Chris@1494 1917
Chris@1494 1918 assert_no_difference 'Issue.count' do
Chris@1494 1919 post :create, :project_id => 1, :issue => {
Chris@1494 1920 :tracker_id => 2,
Chris@1494 1921 :status_id => 1,
Chris@1494 1922 :subject => 'Test',
Chris@1494 1923 :start_date => '',
Chris@1494 1924 :due_date => '',
Chris@1494 1925 :custom_field_values => {cf1.id.to_s => '', cf2.id.to_s => ''}
Chris@1494 1926 }
Chris@1494 1927 assert_response :success
Chris@1494 1928 assert_template 'new'
Chris@1494 1929 end
Chris@1494 1930
Chris@1494 1931 assert_error_tag :content => /Due date can&#x27;t be blank/i
Chris@1494 1932 assert_error_tag :content => /Bar can&#x27;t be blank/i
Chris@1494 1933 end
Chris@1494 1934
Chris@1494 1935 def test_create_should_ignore_readonly_fields
Chris@1494 1936 cf1 = IssueCustomField.create!(:name => 'Foo', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1937 cf2 = IssueCustomField.create!(:name => 'Bar', :field_format => 'string', :is_for_all => true, :tracker_ids => [1, 2])
Chris@1494 1938 WorkflowPermission.delete_all
Chris@1494 1939 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => 'due_date', :rule => 'readonly')
Chris@1494 1940 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2, :role_id => 1, :field_name => cf2.id.to_s, :rule => 'readonly')
Chris@1494 1941 @request.session[:user_id] = 2
Chris@1494 1942
Chris@1494 1943 assert_difference 'Issue.count' do
Chris@1494 1944 post :create, :project_id => 1, :issue => {
Chris@1494 1945 :tracker_id => 2,
Chris@1494 1946 :status_id => 1,
Chris@1494 1947 :subject => 'Test',
Chris@1494 1948 :start_date => '2012-07-14',
Chris@1494 1949 :due_date => '2012-07-16',
Chris@1494 1950 :custom_field_values => {cf1.id.to_s => 'value1', cf2.id.to_s => 'value2'}
Chris@1494 1951 }
Chris@1494 1952 assert_response 302
Chris@1494 1953 end
Chris@1494 1954
Chris@1494 1955 issue = Issue.first(:order => 'id DESC')
Chris@1494 1956 assert_equal Date.parse('2012-07-14'), issue.start_date
Chris@1494 1957 assert_nil issue.due_date
Chris@1494 1958 assert_equal 'value1', issue.custom_field_value(cf1)
Chris@1494 1959 assert_nil issue.custom_field_value(cf2)
Chris@1494 1960 end
Chris@1494 1961
Chris@1494 1962 def test_post_create_with_watchers
Chris@1494 1963 @request.session[:user_id] = 2
Chris@1494 1964 ActionMailer::Base.deliveries.clear
Chris@1494 1965
Chris@1494 1966 assert_difference 'Watcher.count', 2 do
Chris@1494 1967 post :create, :project_id => 1,
Chris@1494 1968 :issue => {:tracker_id => 1,
Chris@1494 1969 :subject => 'This is a new issue with watchers',
Chris@1494 1970 :description => 'This is the description',
Chris@1494 1971 :priority_id => 5,
Chris@1494 1972 :watcher_user_ids => ['2', '3']}
Chris@1494 1973 end
Chris@1494 1974 issue = Issue.find_by_subject('This is a new issue with watchers')
Chris@1494 1975 assert_not_nil issue
Chris@1494 1976 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
Chris@1494 1977
Chris@1494 1978 # Watchers added
Chris@1494 1979 assert_equal [2, 3], issue.watcher_user_ids.sort
Chris@1494 1980 assert issue.watched_by?(User.find(3))
Chris@1494 1981 # Watchers notified
Chris@1494 1982 mail = ActionMailer::Base.deliveries.last
Chris@1494 1983 assert_not_nil mail
Chris@1494 1984 assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
Chris@1494 1985 end
Chris@1494 1986
Chris@1494 1987 def test_post_create_subissue
Chris@1494 1988 @request.session[:user_id] = 2
Chris@1494 1989
Chris@1494 1990 assert_difference 'Issue.count' do
Chris@1494 1991 post :create, :project_id => 1,
Chris@1494 1992 :issue => {:tracker_id => 1,
Chris@1494 1993 :subject => 'This is a child issue',
Chris@1494 1994 :parent_issue_id => '2'}
Chris@1494 1995 assert_response 302
Chris@1494 1996 end
Chris@1494 1997 issue = Issue.order('id DESC').first
Chris@1494 1998 assert_equal Issue.find(2), issue.parent
Chris@1494 1999 end
Chris@1494 2000
Chris@1494 2001 def test_post_create_subissue_with_sharp_parent_id
Chris@1494 2002 @request.session[:user_id] = 2
Chris@1494 2003
Chris@1494 2004 assert_difference 'Issue.count' do
Chris@1494 2005 post :create, :project_id => 1,
Chris@1494 2006 :issue => {:tracker_id => 1,
Chris@1494 2007 :subject => 'This is a child issue',
Chris@1494 2008 :parent_issue_id => '#2'}
Chris@1494 2009 assert_response 302
Chris@1494 2010 end
Chris@1494 2011 issue = Issue.order('id DESC').first
Chris@1494 2012 assert_equal Issue.find(2), issue.parent
Chris@1494 2013 end
Chris@1494 2014
Chris@1494 2015 def test_post_create_subissue_with_non_visible_parent_id_should_not_validate
Chris@1494 2016 @request.session[:user_id] = 2
Chris@1494 2017
Chris@1494 2018 assert_no_difference 'Issue.count' do
Chris@1494 2019 post :create, :project_id => 1,
Chris@1494 2020 :issue => {:tracker_id => 1,
Chris@1494 2021 :subject => 'This is a child issue',
Chris@1494 2022 :parent_issue_id => '4'}
Chris@1494 2023
Chris@1494 2024 assert_response :success
Chris@1494 2025 assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', '4'
Chris@1494 2026 assert_error_tag :content => /Parent task is invalid/i
Chris@1494 2027 end
Chris@1494 2028 end
Chris@1494 2029
Chris@1494 2030 def test_post_create_subissue_with_non_numeric_parent_id_should_not_validate
Chris@1494 2031 @request.session[:user_id] = 2
Chris@1494 2032
Chris@1494 2033 assert_no_difference 'Issue.count' do
Chris@1494 2034 post :create, :project_id => 1,
Chris@1494 2035 :issue => {:tracker_id => 1,
Chris@1494 2036 :subject => 'This is a child issue',
Chris@1494 2037 :parent_issue_id => '01ABC'}
Chris@1494 2038
Chris@1494 2039 assert_response :success
Chris@1494 2040 assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', '01ABC'
Chris@1494 2041 assert_error_tag :content => /Parent task is invalid/i
Chris@1494 2042 end
Chris@1494 2043 end
Chris@1494 2044
Chris@1494 2045 def test_post_create_private
Chris@1494 2046 @request.session[:user_id] = 2
Chris@1494 2047
Chris@1494 2048 assert_difference 'Issue.count' do
Chris@1494 2049 post :create, :project_id => 1,
Chris@1494 2050 :issue => {:tracker_id => 1,
Chris@1494 2051 :subject => 'This is a private issue',
Chris@1494 2052 :is_private => '1'}
Chris@1494 2053 end
Chris@1494 2054 issue = Issue.first(:order => 'id DESC')
Chris@1494 2055 assert issue.is_private?
Chris@1494 2056 end
Chris@1494 2057
Chris@1494 2058 def test_post_create_private_with_set_own_issues_private_permission
Chris@1494 2059 role = Role.find(1)
Chris@1494 2060 role.remove_permission! :set_issues_private
Chris@1494 2061 role.add_permission! :set_own_issues_private
Chris@1494 2062
Chris@1494 2063 @request.session[:user_id] = 2
Chris@1494 2064
Chris@1494 2065 assert_difference 'Issue.count' do
Chris@1494 2066 post :create, :project_id => 1,
Chris@1494 2067 :issue => {:tracker_id => 1,
Chris@1494 2068 :subject => 'This is a private issue',
Chris@1494 2069 :is_private => '1'}
Chris@1494 2070 end
Chris@1494 2071 issue = Issue.first(:order => 'id DESC')
Chris@1494 2072 assert issue.is_private?
Chris@1494 2073 end
Chris@1494 2074
Chris@1494 2075 def test_post_create_should_send_a_notification
Chris@1494 2076 ActionMailer::Base.deliveries.clear
Chris@1494 2077 @request.session[:user_id] = 2
Chris@1494 2078 assert_difference 'Issue.count' do
Chris@1494 2079 post :create, :project_id => 1,
Chris@1494 2080 :issue => {:tracker_id => 3,
Chris@1494 2081 :subject => 'This is the test_new issue',
Chris@1494 2082 :description => 'This is the description',
Chris@1494 2083 :priority_id => 5,
Chris@1494 2084 :estimated_hours => '',
Chris@1494 2085 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@1494 2086 end
Chris@1494 2087 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@1494 2088
Chris@1494 2089 assert_equal 1, ActionMailer::Base.deliveries.size
Chris@1494 2090 end
Chris@1494 2091
Chris@1494 2092 def test_post_create_should_preserve_fields_values_on_validation_failure
Chris@1494 2093 @request.session[:user_id] = 2
Chris@1494 2094 post :create, :project_id => 1,
Chris@1494 2095 :issue => {:tracker_id => 1,
Chris@1494 2096 # empty subject
Chris@1494 2097 :subject => '',
Chris@1494 2098 :description => 'This is a description',
Chris@1494 2099 :priority_id => 6,
Chris@1494 2100 :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
Chris@1494 2101 assert_response :success
Chris@1494 2102 assert_template 'new'
Chris@1494 2103
Chris@1494 2104 assert_select 'textarea[name=?]', 'issue[description]', :text => 'This is a description'
Chris@1494 2105 assert_select 'select[name=?]', 'issue[priority_id]' do
Chris@1494 2106 assert_select 'option[value=6][selected=selected]', :text => 'High'
Chris@1494 2107 end
Chris@1494 2108 # Custom fields
Chris@1494 2109 assert_select 'select[name=?]', 'issue[custom_field_values][1]' do
Chris@1494 2110 assert_select 'option[value=Oracle][selected=selected]', :text => 'Oracle'
Chris@1494 2111 end
Chris@1494 2112 assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', 'Value for field 2'
Chris@1494 2113 end
Chris@1494 2114
Chris@1494 2115 def test_post_create_with_failure_should_preserve_watchers
Chris@1494 2116 assert !User.find(8).member_of?(Project.find(1))
Chris@1494 2117
Chris@1494 2118 @request.session[:user_id] = 2
Chris@1494 2119 post :create, :project_id => 1,
Chris@1494 2120 :issue => {:tracker_id => 1,
Chris@1494 2121 :watcher_user_ids => ['3', '8']}
Chris@1494 2122 assert_response :success
Chris@1494 2123 assert_template 'new'
Chris@1494 2124
Chris@1494 2125 assert_select 'input[name=?][value=2]:not(checked)', 'issue[watcher_user_ids][]'
Chris@1494 2126 assert_select 'input[name=?][value=3][checked=checked]', 'issue[watcher_user_ids][]'
Chris@1494 2127 assert_select 'input[name=?][value=8][checked=checked]', 'issue[watcher_user_ids][]'
Chris@1494 2128 end
Chris@1494 2129
Chris@1494 2130 def test_post_create_should_ignore_non_safe_attributes
Chris@1494 2131 @request.session[:user_id] = 2
Chris@1494 2132 assert_nothing_raised do
Chris@1494 2133 post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" }
Chris@1494 2134 end
Chris@1494 2135 end
Chris@1494 2136
Chris@1494 2137 def test_post_create_with_attachment
Chris@1494 2138 set_tmp_attachments_directory
Chris@1494 2139 @request.session[:user_id] = 2
Chris@1494 2140
Chris@1494 2141 assert_difference 'Issue.count' do
Chris@1494 2142 assert_difference 'Attachment.count' do
Chris@1494 2143 post :create, :project_id => 1,
Chris@1494 2144 :issue => { :tracker_id => '1', :subject => 'With attachment' },
Chris@1494 2145 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@1494 2146 end
Chris@1494 2147 end
Chris@1494 2148
Chris@1494 2149 issue = Issue.first(:order => 'id DESC')
Chris@1494 2150 attachment = Attachment.first(:order => 'id DESC')
Chris@1494 2151
Chris@1494 2152 assert_equal issue, attachment.container
Chris@1494 2153 assert_equal 2, attachment.author_id
Chris@1494 2154 assert_equal 'testfile.txt', attachment.filename
Chris@1494 2155 assert_equal 'text/plain', attachment.content_type
Chris@1494 2156 assert_equal 'test file', attachment.description
Chris@1494 2157 assert_equal 59, attachment.filesize
Chris@1494 2158 assert File.exists?(attachment.diskfile)
Chris@1494 2159 assert_equal 59, File.size(attachment.diskfile)
Chris@1494 2160 end
Chris@1494 2161
Chris@1494 2162 def test_post_create_with_attachment_should_notify_with_attachments
Chris@1494 2163 ActionMailer::Base.deliveries.clear
Chris@1494 2164 set_tmp_attachments_directory
Chris@1494 2165 @request.session[:user_id] = 2
Chris@1494 2166
Chris@1494 2167 with_settings :host_name => 'mydomain.foo', :protocol => 'http' do
Chris@1494 2168 assert_difference 'Issue.count' do
Chris@1494 2169 post :create, :project_id => 1,
Chris@1494 2170 :issue => { :tracker_id => '1', :subject => 'With attachment' },
Chris@1494 2171 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@1494 2172 end
Chris@1494 2173 end
Chris@1494 2174
Chris@1494 2175 assert_not_nil ActionMailer::Base.deliveries.last
Chris@1494 2176 assert_select_email do
Chris@1494 2177 assert_select 'a[href^=?]', 'http://mydomain.foo/attachments/download', 'testfile.txt'
Chris@1494 2178 end
Chris@1494 2179 end
Chris@1494 2180
Chris@1494 2181 def test_post_create_with_failure_should_save_attachments
Chris@1494 2182 set_tmp_attachments_directory
Chris@1494 2183 @request.session[:user_id] = 2
Chris@1494 2184
Chris@1494 2185 assert_no_difference 'Issue.count' do
Chris@1494 2186 assert_difference 'Attachment.count' do
Chris@1494 2187 post :create, :project_id => 1,
Chris@1494 2188 :issue => { :tracker_id => '1', :subject => '' },
Chris@1494 2189 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@1494 2190 assert_response :success
Chris@1494 2191 assert_template 'new'
Chris@1494 2192 end
Chris@1494 2193 end
Chris@1494 2194
Chris@1494 2195 attachment = Attachment.first(:order => 'id DESC')
Chris@1494 2196 assert_equal 'testfile.txt', attachment.filename
Chris@1494 2197 assert File.exists?(attachment.diskfile)
Chris@1494 2198 assert_nil attachment.container
Chris@1494 2199
Chris@1494 2200 assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token
Chris@1494 2201 assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt'
Chris@1494 2202 end
Chris@1494 2203
Chris@1494 2204 def test_post_create_with_failure_should_keep_saved_attachments
Chris@1494 2205 set_tmp_attachments_directory
Chris@1494 2206 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
Chris@1494 2207 @request.session[:user_id] = 2
Chris@1494 2208
Chris@1494 2209 assert_no_difference 'Issue.count' do
Chris@1494 2210 assert_no_difference 'Attachment.count' do
Chris@1494 2211 post :create, :project_id => 1,
Chris@1494 2212 :issue => { :tracker_id => '1', :subject => '' },
Chris@1494 2213 :attachments => {'p0' => {'token' => attachment.token}}
Chris@1494 2214 assert_response :success
Chris@1494 2215 assert_template 'new'
Chris@1494 2216 end
Chris@1494 2217 end
Chris@1494 2218
Chris@1494 2219 assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token
Chris@1494 2220 assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt'
Chris@1494 2221 end
Chris@1494 2222
Chris@1494 2223 def test_post_create_should_attach_saved_attachments
Chris@1494 2224 set_tmp_attachments_directory
Chris@1494 2225 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
Chris@1494 2226 @request.session[:user_id] = 2
Chris@1494 2227
Chris@1494 2228 assert_difference 'Issue.count' do
Chris@1494 2229 assert_no_difference 'Attachment.count' do
Chris@1494 2230 post :create, :project_id => 1,
Chris@1494 2231 :issue => { :tracker_id => '1', :subject => 'Saved attachments' },
Chris@1494 2232 :attachments => {'p0' => {'token' => attachment.token}}
Chris@1494 2233 assert_response 302
Chris@1494 2234 end
Chris@1494 2235 end
Chris@1494 2236
Chris@1494 2237 issue = Issue.first(:order => 'id DESC')
Chris@1494 2238 assert_equal 1, issue.attachments.count
Chris@1494 2239
Chris@1494 2240 attachment.reload
Chris@1494 2241 assert_equal issue, attachment.container
Chris@1494 2242 end
Chris@1494 2243
Chris@1494 2244 context "without workflow privilege" do
Chris@1494 2245 setup do
Chris@1494 2246 WorkflowTransition.delete_all(["role_id = ?", Role.anonymous.id])
Chris@1494 2247 Role.anonymous.add_permission! :add_issues, :add_issue_notes
Chris@1494 2248 end
Chris@1494 2249
Chris@1494 2250 context "#new" do
Chris@1494 2251 should "propose default status only" do
Chris@1494 2252 get :new, :project_id => 1
Chris@1494 2253 assert_response :success
Chris@1494 2254 assert_template 'new'
Chris@1494 2255 assert_select 'select[name=?]', 'issue[status_id]' do
Chris@1494 2256 assert_select 'option', 1
Chris@1494 2257 assert_select 'option[value=?]', IssueStatus.default.id.to_s
Chris@1494 2258 end
Chris@1494 2259 end
Chris@1494 2260
Chris@1494 2261 should "accept default status" do
Chris@1494 2262 assert_difference 'Issue.count' do
Chris@1494 2263 post :create, :project_id => 1,
Chris@1494 2264 :issue => {:tracker_id => 1,
Chris@1494 2265 :subject => 'This is an issue',
Chris@1494 2266 :status_id => 1}
Chris@1494 2267 end
Chris@1494 2268 issue = Issue.last(:order => 'id')
Chris@1494 2269 assert_equal IssueStatus.default, issue.status
Chris@1494 2270 end
Chris@1494 2271
Chris@1494 2272 should "ignore unauthorized status" do
Chris@1494 2273 assert_difference 'Issue.count' do
Chris@1494 2274 post :create, :project_id => 1,
Chris@1494 2275 :issue => {:tracker_id => 1,
Chris@1494 2276 :subject => 'This is an issue',
Chris@1494 2277 :status_id => 3}
Chris@1494 2278 end
Chris@1494 2279 issue = Issue.last(:order => 'id')
Chris@1494 2280 assert_equal IssueStatus.default, issue.status
Chris@1494 2281 end
Chris@1494 2282 end
Chris@1494 2283
Chris@1494 2284 context "#update" do
Chris@1494 2285 should "ignore status change" do
Chris@1494 2286 assert_difference 'Journal.count' do
Chris@1494 2287 put :update, :id => 1, :issue => {:status_id => 3, :notes => 'just trying'}
Chris@1494 2288 end
Chris@1494 2289 assert_equal 1, Issue.find(1).status_id
Chris@1494 2290 end
Chris@1494 2291
Chris@1494 2292 should "ignore attributes changes" do
Chris@1494 2293 assert_difference 'Journal.count' do
Chris@1494 2294 put :update, :id => 1, :issue => {:subject => 'changed', :assigned_to_id => 2, :notes => 'just trying'}
Chris@1494 2295 end
Chris@1494 2296 issue = Issue.find(1)
Chris@1494 2297 assert_equal "Can't print recipes", issue.subject
Chris@1494 2298 assert_nil issue.assigned_to
Chris@1494 2299 end
Chris@1494 2300 end
Chris@1494 2301 end
Chris@1494 2302
Chris@1494 2303 context "with workflow privilege" do
Chris@1494 2304 setup do
Chris@1494 2305 WorkflowTransition.delete_all(["role_id = ?", Role.anonymous.id])
Chris@1494 2306 WorkflowTransition.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3)
Chris@1494 2307 WorkflowTransition.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 4)
Chris@1494 2308 Role.anonymous.add_permission! :add_issues, :add_issue_notes
Chris@1494 2309 end
Chris@1494 2310
Chris@1494 2311 context "#update" do
Chris@1494 2312 should "accept authorized status" do
Chris@1494 2313 assert_difference 'Journal.count' do
Chris@1494 2314 put :update, :id => 1, :issue => {:status_id => 3, :notes => 'just trying'}
Chris@1494 2315 end
Chris@1494 2316 assert_equal 3, Issue.find(1).status_id
Chris@1494 2317 end
Chris@1494 2318
Chris@1494 2319 should "ignore unauthorized status" do
Chris@1494 2320 assert_difference 'Journal.count' do
Chris@1494 2321 put :update, :id => 1, :issue => {:status_id => 2, :notes => 'just trying'}
Chris@1494 2322 end
Chris@1494 2323 assert_equal 1, Issue.find(1).status_id
Chris@1494 2324 end
Chris@1494 2325
Chris@1494 2326 should "accept authorized attributes changes" do
Chris@1494 2327 assert_difference 'Journal.count' do
Chris@1494 2328 put :update, :id => 1, :issue => {:assigned_to_id => 2, :notes => 'just trying'}
Chris@1494 2329 end
Chris@1494 2330 issue = Issue.find(1)
Chris@1494 2331 assert_equal 2, issue.assigned_to_id
Chris@1494 2332 end
Chris@1494 2333
Chris@1494 2334 should "ignore unauthorized attributes changes" do
Chris@1494 2335 assert_difference 'Journal.count' do
Chris@1494 2336 put :update, :id => 1, :issue => {:subject => 'changed', :notes => 'just trying'}
Chris@1494 2337 end
Chris@1494 2338 issue = Issue.find(1)
Chris@1494 2339 assert_equal "Can't print recipes", issue.subject
Chris@1494 2340 end
Chris@1494 2341 end
Chris@1494 2342
Chris@1494 2343 context "and :edit_issues permission" do
Chris@1494 2344 setup do
Chris@1494 2345 Role.anonymous.add_permission! :add_issues, :edit_issues
Chris@1494 2346 end
Chris@1494 2347
Chris@1494 2348 should "accept authorized status" do
Chris@1494 2349 assert_difference 'Journal.count' do
Chris@1494 2350 put :update, :id => 1, :issue => {:status_id => 3, :notes => 'just trying'}
Chris@1494 2351 end
Chris@1494 2352 assert_equal 3, Issue.find(1).status_id
Chris@1494 2353 end
Chris@1494 2354
Chris@1494 2355 should "ignore unauthorized status" do
Chris@1494 2356 assert_difference 'Journal.count' do
Chris@1494 2357 put :update, :id => 1, :issue => {:status_id => 2, :notes => 'just trying'}
Chris@1494 2358 end
Chris@1494 2359 assert_equal 1, Issue.find(1).status_id
Chris@1494 2360 end
Chris@1494 2361
Chris@1494 2362 should "accept authorized attributes changes" do
Chris@1494 2363 assert_difference 'Journal.count' do
Chris@1494 2364 put :update, :id => 1, :issue => {:subject => 'changed', :assigned_to_id => 2, :notes => 'just trying'}
Chris@1494 2365 end
Chris@1494 2366 issue = Issue.find(1)
Chris@1494 2367 assert_equal "changed", issue.subject
Chris@1494 2368 assert_equal 2, issue.assigned_to_id
Chris@1494 2369 end
Chris@1494 2370 end
Chris@1494 2371 end
Chris@1494 2372
Chris@1494 2373 def test_new_as_copy
Chris@1494 2374 @request.session[:user_id] = 2
Chris@1494 2375 get :new, :project_id => 1, :copy_from => 1
Chris@1494 2376
Chris@1494 2377 assert_response :success
Chris@1494 2378 assert_template 'new'
Chris@1494 2379
Chris@1494 2380 assert_not_nil assigns(:issue)
Chris@1494 2381 orig = Issue.find(1)
Chris@1494 2382 assert_equal 1, assigns(:issue).project_id
Chris@1494 2383 assert_equal orig.subject, assigns(:issue).subject
Chris@1494 2384 assert assigns(:issue).copy?
Chris@1494 2385
Chris@1494 2386 assert_select 'form[id=issue-form][action=/projects/ecookbook/issues]' do
Chris@1494 2387 assert_select 'select[name=?]', 'issue[project_id]' do
Chris@1494 2388 assert_select 'option[value=1][selected=selected]', :text => 'eCookbook'
Chris@1494 2389 assert_select 'option[value=2]:not([selected])', :text => 'OnlineStore'
Chris@1494 2390 end
Chris@1494 2391 assert_select 'input[name=copy_from][value=1]'
Chris@1494 2392 end
Chris@1494 2393
Chris@1494 2394 # "New issue" menu item should not link to copy
Chris@1494 2395 assert_select '#main-menu a.new-issue[href=/projects/ecookbook/issues/new]'
Chris@1494 2396 end
Chris@1494 2397
Chris@1494 2398 def test_new_as_copy_with_attachments_should_show_copy_attachments_checkbox
Chris@1494 2399 @request.session[:user_id] = 2
Chris@1494 2400 issue = Issue.find(3)
Chris@1494 2401 assert issue.attachments.count > 0
Chris@1494 2402 get :new, :project_id => 1, :copy_from => 3
Chris@1494 2403
Chris@1494 2404 assert_select 'input[name=copy_attachments][type=checkbox][checked=checked][value=1]'
Chris@1494 2405 end
Chris@1494 2406
Chris@1494 2407 def test_new_as_copy_without_attachments_should_not_show_copy_attachments_checkbox
Chris@1494 2408 @request.session[:user_id] = 2
Chris@1494 2409 issue = Issue.find(3)
Chris@1494 2410 issue.attachments.delete_all
Chris@1494 2411 get :new, :project_id => 1, :copy_from => 3
Chris@1494 2412
Chris@1494 2413 assert_select 'input[name=copy_attachments]', 0
Chris@1494 2414 end
Chris@1494 2415
Chris@1494 2416 def test_new_as_copy_with_subtasks_should_show_copy_subtasks_checkbox
Chris@1494 2417 @request.session[:user_id] = 2
Chris@1494 2418 issue = Issue.generate_with_descendants!
Chris@1494 2419 get :new, :project_id => 1, :copy_from => issue.id
Chris@1494 2420
Chris@1494 2421 assert_select 'input[type=checkbox][name=copy_subtasks][checked=checked][value=1]'
Chris@1494 2422 end
Chris@1494 2423
Chris@1494 2424 def test_new_as_copy_with_invalid_issue_should_respond_with_404
Chris@1494 2425 @request.session[:user_id] = 2
Chris@1494 2426 get :new, :project_id => 1, :copy_from => 99999
Chris@1494 2427 assert_response 404
Chris@1494 2428 end
Chris@1494 2429
Chris@1494 2430 def test_create_as_copy_on_different_project
Chris@1494 2431 @request.session[:user_id] = 2
Chris@1494 2432 assert_difference 'Issue.count' do
Chris@1494 2433 post :create, :project_id => 1, :copy_from => 1,
Chris@1494 2434 :issue => {:project_id => '2', :tracker_id => '3', :status_id => '1', :subject => 'Copy'}
Chris@1494 2435
Chris@1494 2436 assert_not_nil assigns(:issue)
Chris@1494 2437 assert assigns(:issue).copy?
Chris@1494 2438 end
Chris@1494 2439 issue = Issue.first(:order => 'id DESC')
Chris@1494 2440 assert_redirected_to "/issues/#{issue.id}"
Chris@1494 2441
Chris@1494 2442 assert_equal 2, issue.project_id
Chris@1494 2443 assert_equal 3, issue.tracker_id
Chris@1494 2444 assert_equal 'Copy', issue.subject
Chris@1494 2445 end
Chris@1494 2446
Chris@1494 2447 def test_create_as_copy_should_copy_attachments
Chris@1494 2448 @request.session[:user_id] = 2
Chris@1494 2449 issue = Issue.find(3)
Chris@1494 2450 count = issue.attachments.count
Chris@1494 2451 assert count > 0
Chris@1494 2452 assert_difference 'Issue.count' do
Chris@1494 2453 assert_difference 'Attachment.count', count do
Chris@1494 2454 assert_difference 'Journal.count', 2 do
Chris@1494 2455 post :create, :project_id => 1, :copy_from => 3,
Chris@1494 2456 :issue => {:project_id => '1', :tracker_id => '3',
Chris@1494 2457 :status_id => '1', :subject => 'Copy with attachments'},
Chris@1494 2458 :copy_attachments => '1'
Chris@1494 2459 end
Chris@1494 2460 end
Chris@1494 2461 end
Chris@1494 2462 copy = Issue.first(:order => 'id DESC')
Chris@1494 2463 assert_equal count, copy.attachments.count
Chris@1494 2464 assert_equal issue.attachments.map(&:filename).sort, copy.attachments.map(&:filename).sort
Chris@1494 2465 end
Chris@1494 2466
Chris@1494 2467 def test_create_as_copy_without_copy_attachments_option_should_not_copy_attachments
Chris@1494 2468 @request.session[:user_id] = 2
Chris@1494 2469 issue = Issue.find(3)
Chris@1494 2470 count = issue.attachments.count
Chris@1494 2471 assert count > 0
Chris@1494 2472 assert_difference 'Issue.count' do
Chris@1494 2473 assert_no_difference 'Attachment.count' do
Chris@1494 2474 assert_difference 'Journal.count', 2 do
Chris@1494 2475 post :create, :project_id => 1, :copy_from => 3,
Chris@1494 2476 :issue => {:project_id => '1', :tracker_id => '3',
Chris@1494 2477 :status_id => '1', :subject => 'Copy with attachments'}
Chris@1494 2478 end
Chris@1494 2479 end
Chris@1494 2480 end
Chris@1494 2481 copy = Issue.first(:order => 'id DESC')
Chris@1494 2482 assert_equal 0, copy.attachments.count
Chris@1494 2483 end
Chris@1494 2484
Chris@1494 2485 def test_create_as_copy_with_attachments_should_add_new_files
Chris@1494 2486 @request.session[:user_id] = 2
Chris@1494 2487 issue = Issue.find(3)
Chris@1494 2488 count = issue.attachments.count
Chris@1494 2489 assert count > 0
Chris@1494 2490 assert_difference 'Issue.count' do
Chris@1494 2491 assert_difference 'Attachment.count', count + 1 do
Chris@1494 2492 assert_difference 'Journal.count', 2 do
Chris@1494 2493 post :create, :project_id => 1, :copy_from => 3,
Chris@1494 2494 :issue => {:project_id => '1', :tracker_id => '3',
Chris@1494 2495 :status_id => '1', :subject => 'Copy with attachments'},
Chris@1494 2496 :copy_attachments => '1',
Chris@1494 2497 :attachments => {'1' =>
Chris@1494 2498 {'file' => uploaded_test_file('testfile.txt', 'text/plain'),
Chris@1494 2499 'description' => 'test file'}}
Chris@1494 2500 end
Chris@1494 2501 end
Chris@1494 2502 end
Chris@1494 2503 copy = Issue.first(:order => 'id DESC')
Chris@1494 2504 assert_equal count + 1, copy.attachments.count
Chris@1494 2505 end
Chris@1494 2506
Chris@1494 2507 def test_create_as_copy_should_add_relation_with_copied_issue
Chris@1494 2508 @request.session[:user_id] = 2
Chris@1494 2509 assert_difference 'Issue.count' do
Chris@1494 2510 assert_difference 'IssueRelation.count' do
Chris@1494 2511 post :create, :project_id => 1, :copy_from => 1,
Chris@1494 2512 :issue => {:project_id => '1', :tracker_id => '3',
Chris@1494 2513 :status_id => '1', :subject => 'Copy'}
Chris@1494 2514 end
Chris@1494 2515 end
Chris@1494 2516 copy = Issue.first(:order => 'id DESC')
Chris@1494 2517 assert_equal 1, copy.relations.size
Chris@1494 2518 end
Chris@1494 2519
Chris@1494 2520 def test_create_as_copy_should_copy_subtasks
Chris@1494 2521 @request.session[:user_id] = 2
Chris@1494 2522 issue = Issue.generate_with_descendants!
Chris@1494 2523 count = issue.descendants.count
Chris@1494 2524 assert_difference 'Issue.count', count + 1 do
Chris@1494 2525 assert_difference 'Journal.count', (count + 1) * 2 do
Chris@1494 2526 post :create, :project_id => 1, :copy_from => issue.id,
Chris@1494 2527 :issue => {:project_id => '1', :tracker_id => '3',
Chris@1494 2528 :status_id => '1', :subject => 'Copy with subtasks'},
Chris@1494 2529 :copy_subtasks => '1'
Chris@1494 2530 end
Chris@1494 2531 end
Chris@1494 2532 copy = Issue.where(:parent_id => nil).first(:order => 'id DESC')
Chris@1494 2533 assert_equal count, copy.descendants.count
Chris@1494 2534 assert_equal issue.descendants.map(&:subject).sort, copy.descendants.map(&:subject).sort
Chris@1494 2535 end
Chris@1494 2536
Chris@1494 2537 def test_create_as_copy_without_copy_subtasks_option_should_not_copy_subtasks
Chris@1494 2538 @request.session[:user_id] = 2
Chris@1494 2539 issue = Issue.generate_with_descendants!
Chris@1494 2540 assert_difference 'Issue.count', 1 do
Chris@1494 2541 assert_difference 'Journal.count', 2 do
Chris@1494 2542 post :create, :project_id => 1, :copy_from => 3,
Chris@1494 2543 :issue => {:project_id => '1', :tracker_id => '3',
Chris@1494 2544 :status_id => '1', :subject => 'Copy with subtasks'}
Chris@1494 2545 end
Chris@1494 2546 end
Chris@1494 2547 copy = Issue.where(:parent_id => nil).first(:order => 'id DESC')
Chris@1494 2548 assert_equal 0, copy.descendants.count
Chris@1494 2549 end
Chris@1494 2550
Chris@1494 2551 def test_create_as_copy_with_failure
Chris@1494 2552 @request.session[:user_id] = 2
Chris@1494 2553 post :create, :project_id => 1, :copy_from => 1,
Chris@1494 2554 :issue => {:project_id => '2', :tracker_id => '3', :status_id => '1', :subject => ''}
Chris@1494 2555
Chris@1494 2556 assert_response :success
Chris@1494 2557 assert_template 'new'
Chris@1494 2558
Chris@1494 2559 assert_not_nil assigns(:issue)
Chris@1494 2560 assert assigns(:issue).copy?
Chris@1494 2561
Chris@1494 2562 assert_select 'form#issue-form[action=/projects/ecookbook/issues]' do
Chris@1494 2563 assert_select 'select[name=?]', 'issue[project_id]' do
Chris@1494 2564 assert_select 'option[value=1]:not([selected])', :text => 'eCookbook'
Chris@1494 2565 assert_select 'option[value=2][selected=selected]', :text => 'OnlineStore'
Chris@1494 2566 end
Chris@1494 2567 assert_select 'input[name=copy_from][value=1]'
Chris@1494 2568 end
Chris@1494 2569 end
Chris@1494 2570
Chris@1494 2571 def test_create_as_copy_on_project_without_permission_should_ignore_target_project
Chris@1494 2572 @request.session[:user_id] = 2
Chris@1494 2573 assert !User.find(2).member_of?(Project.find(4))
Chris@1494 2574
Chris@1494 2575 assert_difference 'Issue.count' do
Chris@1494 2576 post :create, :project_id => 1, :copy_from => 1,
Chris@1494 2577 :issue => {:project_id => '4', :tracker_id => '3', :status_id => '1', :subject => 'Copy'}
Chris@1494 2578 end
Chris@1494 2579 issue = Issue.first(:order => 'id DESC')
Chris@1494 2580 assert_equal 1, issue.project_id
Chris@1494 2581 end
Chris@1494 2582
Chris@1494 2583 def test_get_edit
Chris@1494 2584 @request.session[:user_id] = 2
Chris@1494 2585 get :edit, :id => 1
Chris@1494 2586 assert_response :success
Chris@1494 2587 assert_template 'edit'
Chris@1494 2588 assert_not_nil assigns(:issue)
Chris@1494 2589 assert_equal Issue.find(1), assigns(:issue)
Chris@1494 2590
Chris@1494 2591 # Be sure we don't display inactive IssuePriorities
Chris@1494 2592 assert ! IssuePriority.find(15).active?
Chris@1494 2593 assert_select 'select[name=?]', 'issue[priority_id]' do
Chris@1494 2594 assert_select 'option[value=15]', 0
Chris@1494 2595 end
Chris@1494 2596 end
Chris@1494 2597
Chris@1494 2598 def test_get_edit_should_display_the_time_entry_form_with_log_time_permission
Chris@1494 2599 @request.session[:user_id] = 2
Chris@1494 2600 Role.find_by_name('Manager').update_attribute :permissions, [:view_issues, :edit_issues, :log_time]
Chris@1494 2601
Chris@1494 2602 get :edit, :id => 1
Chris@1494 2603 assert_select 'input[name=?]', 'time_entry[hours]'
Chris@1494 2604 end
Chris@1494 2605
Chris@1494 2606 def test_get_edit_should_not_display_the_time_entry_form_without_log_time_permission
Chris@1494 2607 @request.session[:user_id] = 2
Chris@1494 2608 Role.find_by_name('Manager').remove_permission! :log_time
Chris@1494 2609
Chris@1494 2610 get :edit, :id => 1
Chris@1494 2611 assert_select 'input[name=?]', 'time_entry[hours]', 0
Chris@1494 2612 end
Chris@1494 2613
Chris@1494 2614 def test_get_edit_with_params
Chris@1494 2615 @request.session[:user_id] = 2
Chris@1494 2616 get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 },
Chris@1494 2617 :time_entry => { :hours => '2.5', :comments => 'test_get_edit_with_params', :activity_id => 10 }
Chris@1494 2618 assert_response :success
Chris@1494 2619 assert_template 'edit'
Chris@1494 2620
Chris@1494 2621 issue = assigns(:issue)
Chris@1494 2622 assert_not_nil issue
Chris@1494 2623
Chris@1494 2624 assert_equal 5, issue.status_id
Chris@1494 2625 assert_select 'select[name=?]', 'issue[status_id]' do
Chris@1494 2626 assert_select 'option[value=5][selected=selected]', :text => 'Closed'
Chris@1494 2627 end
Chris@1494 2628
Chris@1494 2629 assert_equal 7, issue.priority_id
Chris@1494 2630 assert_select 'select[name=?]', 'issue[priority_id]' do
Chris@1494 2631 assert_select 'option[value=7][selected=selected]', :text => 'Urgent'
Chris@1494 2632 end
Chris@1494 2633
Chris@1494 2634 assert_select 'input[name=?][value=2.5]', 'time_entry[hours]'
Chris@1494 2635 assert_select 'select[name=?]', 'time_entry[activity_id]' do
Chris@1494 2636 assert_select 'option[value=10][selected=selected]', :text => 'Development'
Chris@1494 2637 end
Chris@1494 2638 assert_select 'input[name=?][value=test_get_edit_with_params]', 'time_entry[comments]'
Chris@1494 2639 end
Chris@1494 2640
Chris@1494 2641 def test_get_edit_with_multi_custom_field
Chris@1494 2642 field = CustomField.find(1)
Chris@1494 2643 field.update_attribute :multiple, true
Chris@1494 2644 issue = Issue.find(1)
Chris@1494 2645 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
Chris@1494 2646 issue.save!
Chris@1494 2647
Chris@1494 2648 @request.session[:user_id] = 2
Chris@1494 2649 get :edit, :id => 1
Chris@1494 2650 assert_response :success
Chris@1494 2651 assert_template 'edit'
Chris@1494 2652
Chris@1494 2653 assert_select 'select[name=?][multiple=multiple]', 'issue[custom_field_values][1][]' do
Chris@1494 2654 assert_select 'option', 3
Chris@1494 2655 assert_select 'option[value=MySQL][selected=selected]'
Chris@1494 2656 assert_select 'option[value=Oracle][selected=selected]'
Chris@1494 2657 assert_select 'option[value=PostgreSQL]:not([selected])'
Chris@1494 2658 end
Chris@1494 2659 end
Chris@1494 2660
Chris@1494 2661 def test_update_form_for_existing_issue
Chris@1494 2662 @request.session[:user_id] = 2
Chris@1494 2663 xhr :put, :update_form, :project_id => 1,
Chris@1494 2664 :id => 1,
Chris@1494 2665 :issue => {:tracker_id => 2,
Chris@1494 2666 :subject => 'This is the test_new issue',
Chris@1494 2667 :description => 'This is the description',
Chris@1494 2668 :priority_id => 5}
Chris@1494 2669 assert_response :success
Chris@1494 2670 assert_equal 'text/javascript', response.content_type
Chris@1494 2671 assert_template 'update_form'
Chris@1494 2672 assert_template 'form'
Chris@1494 2673
Chris@1494 2674 issue = assigns(:issue)
Chris@1494 2675 assert_kind_of Issue, issue
Chris@1494 2676 assert_equal 1, issue.id
Chris@1494 2677 assert_equal 1, issue.project_id
Chris@1494 2678 assert_equal 2, issue.tracker_id
Chris@1494 2679 assert_equal 'This is the test_new issue', issue.subject
Chris@1494 2680 end
Chris@1494 2681
Chris@1494 2682 def test_update_form_for_existing_issue_should_keep_issue_author
Chris@1494 2683 @request.session[:user_id] = 3
Chris@1494 2684 xhr :put, :update_form, :project_id => 1, :id => 1, :issue => {:subject => 'Changed'}
Chris@1494 2685 assert_response :success
Chris@1494 2686 assert_equal 'text/javascript', response.content_type
Chris@1494 2687
Chris@1494 2688 issue = assigns(:issue)
Chris@1494 2689 assert_equal User.find(2), issue.author
Chris@1494 2690 assert_equal 2, issue.author_id
Chris@1494 2691 assert_not_equal User.current, issue.author
Chris@1494 2692 end
Chris@1494 2693
Chris@1494 2694 def test_update_form_for_existing_issue_should_propose_transitions_based_on_initial_status
Chris@1494 2695 @request.session[:user_id] = 2
Chris@1494 2696 WorkflowTransition.delete_all
Chris@1494 2697 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 1)
Chris@1494 2698 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 5)
Chris@1494 2699 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 5, :new_status_id => 4)
Chris@1494 2700
Chris@1494 2701 xhr :put, :update_form, :project_id => 1,
Chris@1494 2702 :id => 2,
Chris@1494 2703 :issue => {:tracker_id => 2,
Chris@1494 2704 :status_id => 5,
Chris@1494 2705 :subject => 'This is an issue'}
Chris@1494 2706
Chris@1494 2707 assert_equal 5, assigns(:issue).status_id
Chris@1494 2708 assert_equal [1,2,5], assigns(:allowed_statuses).map(&:id).sort
Chris@1494 2709 end
Chris@1494 2710
Chris@1494 2711 def test_update_form_for_existing_issue_with_project_change
Chris@1494 2712 @request.session[:user_id] = 2
Chris@1494 2713 xhr :put, :update_form, :project_id => 1,
Chris@1494 2714 :id => 1,
Chris@1494 2715 :issue => {:project_id => 2,
Chris@1494 2716 :tracker_id => 2,
Chris@1494 2717 :subject => 'This is the test_new issue',
Chris@1494 2718 :description => 'This is the description',
Chris@1494 2719 :priority_id => 5}
Chris@1494 2720 assert_response :success
Chris@1494 2721 assert_template 'form'
Chris@1494 2722
Chris@1494 2723 issue = assigns(:issue)
Chris@1494 2724 assert_kind_of Issue, issue
Chris@1494 2725 assert_equal 1, issue.id
Chris@1494 2726 assert_equal 2, issue.project_id
Chris@1494 2727 assert_equal 2, issue.tracker_id
Chris@1494 2728 assert_equal 'This is the test_new issue', issue.subject
Chris@1494 2729 end
Chris@1494 2730
Chris@1494 2731 def test_update_form_should_propose_default_status_for_existing_issue
Chris@1494 2732 @request.session[:user_id] = 2
Chris@1494 2733 WorkflowTransition.delete_all
Chris@1494 2734 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2, :old_status_id => 2, :new_status_id => 3)
Chris@1494 2735
Chris@1494 2736 xhr :put, :update_form, :project_id => 1, :id => 2
Chris@1494 2737 assert_response :success
Chris@1494 2738 assert_equal [2,3], assigns(:allowed_statuses).map(&:id).sort
Chris@1494 2739 end
Chris@1494 2740
Chris@1494 2741 def test_put_update_without_custom_fields_param
Chris@1494 2742 @request.session[:user_id] = 2
Chris@1494 2743 ActionMailer::Base.deliveries.clear
Chris@1494 2744
Chris@1494 2745 issue = Issue.find(1)
Chris@1494 2746 assert_equal '125', issue.custom_value_for(2).value
Chris@1494 2747 old_subject = issue.subject
Chris@1494 2748 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
Chris@1494 2749
Chris@1494 2750 assert_difference('Journal.count') do
Chris@1494 2751 assert_difference('JournalDetail.count', 2) do
Chris@1494 2752 put :update, :id => 1, :issue => {:subject => new_subject,
Chris@1494 2753 :priority_id => '6',
Chris@1494 2754 :category_id => '1' # no change
Chris@1494 2755 }
Chris@1494 2756 end
Chris@1494 2757 end
Chris@1494 2758 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2759 issue.reload
Chris@1494 2760 assert_equal new_subject, issue.subject
Chris@1494 2761 # Make sure custom fields were not cleared
Chris@1494 2762 assert_equal '125', issue.custom_value_for(2).value
Chris@1494 2763
Chris@1494 2764 mail = ActionMailer::Base.deliveries.last
Chris@1494 2765 assert_not_nil mail
Chris@1494 2766 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
Chris@1494 2767 assert_mail_body_match "Subject changed from #{old_subject} to #{new_subject}", mail
Chris@1494 2768 end
Chris@1494 2769
Chris@1494 2770 def test_put_update_with_project_change
Chris@1494 2771 @request.session[:user_id] = 2
Chris@1494 2772 ActionMailer::Base.deliveries.clear
Chris@1494 2773
Chris@1494 2774 assert_difference('Journal.count') do
Chris@1494 2775 assert_difference('JournalDetail.count', 3) do
Chris@1494 2776 put :update, :id => 1, :issue => {:project_id => '2',
Chris@1494 2777 :tracker_id => '1', # no change
Chris@1494 2778 :priority_id => '6',
Chris@1494 2779 :category_id => '3'
Chris@1494 2780 }
Chris@1494 2781 end
Chris@1494 2782 end
Chris@1494 2783 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2784 issue = Issue.find(1)
Chris@1494 2785 assert_equal 2, issue.project_id
Chris@1494 2786 assert_equal 1, issue.tracker_id
Chris@1494 2787 assert_equal 6, issue.priority_id
Chris@1494 2788 assert_equal 3, issue.category_id
Chris@1494 2789
Chris@1494 2790 mail = ActionMailer::Base.deliveries.last
Chris@1494 2791 assert_not_nil mail
Chris@1494 2792 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
Chris@1494 2793 assert_mail_body_match "Project changed from eCookbook to OnlineStore", mail
Chris@1494 2794 end
Chris@1494 2795
Chris@1494 2796 def test_put_update_with_tracker_change
Chris@1494 2797 @request.session[:user_id] = 2
Chris@1494 2798 ActionMailer::Base.deliveries.clear
Chris@1494 2799
Chris@1494 2800 assert_difference('Journal.count') do
Chris@1494 2801 assert_difference('JournalDetail.count', 2) do
Chris@1494 2802 put :update, :id => 1, :issue => {:project_id => '1',
Chris@1494 2803 :tracker_id => '2',
Chris@1494 2804 :priority_id => '6'
Chris@1494 2805 }
Chris@1494 2806 end
Chris@1494 2807 end
Chris@1494 2808 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2809 issue = Issue.find(1)
Chris@1494 2810 assert_equal 1, issue.project_id
Chris@1494 2811 assert_equal 2, issue.tracker_id
Chris@1494 2812 assert_equal 6, issue.priority_id
Chris@1494 2813 assert_equal 1, issue.category_id
Chris@1494 2814
Chris@1494 2815 mail = ActionMailer::Base.deliveries.last
Chris@1494 2816 assert_not_nil mail
Chris@1494 2817 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
Chris@1494 2818 assert_mail_body_match "Tracker changed from Bug to Feature request", mail
Chris@1494 2819 end
Chris@1494 2820
Chris@1494 2821 def test_put_update_with_custom_field_change
Chris@1494 2822 @request.session[:user_id] = 2
Chris@1494 2823 issue = Issue.find(1)
Chris@1494 2824 assert_equal '125', issue.custom_value_for(2).value
Chris@1494 2825
Chris@1494 2826 assert_difference('Journal.count') do
Chris@1494 2827 assert_difference('JournalDetail.count', 3) do
Chris@1494 2828 put :update, :id => 1, :issue => {:subject => 'Custom field change',
Chris@1494 2829 :priority_id => '6',
Chris@1494 2830 :category_id => '1', # no change
Chris@1494 2831 :custom_field_values => { '2' => 'New custom value' }
Chris@1494 2832 }
Chris@1494 2833 end
Chris@1494 2834 end
Chris@1494 2835 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2836 issue.reload
Chris@1494 2837 assert_equal 'New custom value', issue.custom_value_for(2).value
Chris@1494 2838
Chris@1494 2839 mail = ActionMailer::Base.deliveries.last
Chris@1494 2840 assert_not_nil mail
Chris@1494 2841 assert_mail_body_match "Searchable field changed from 125 to New custom value", mail
Chris@1494 2842 end
Chris@1494 2843
Chris@1494 2844 def test_put_update_with_multi_custom_field_change
Chris@1494 2845 field = CustomField.find(1)
Chris@1494 2846 field.update_attribute :multiple, true
Chris@1494 2847 issue = Issue.find(1)
Chris@1494 2848 issue.custom_field_values = {1 => ['MySQL', 'Oracle']}
Chris@1494 2849 issue.save!
Chris@1494 2850
Chris@1494 2851 @request.session[:user_id] = 2
Chris@1494 2852 assert_difference('Journal.count') do
Chris@1494 2853 assert_difference('JournalDetail.count', 3) do
Chris@1494 2854 put :update, :id => 1,
Chris@1494 2855 :issue => {
Chris@1494 2856 :subject => 'Custom field change',
Chris@1494 2857 :custom_field_values => { '1' => ['', 'Oracle', 'PostgreSQL'] }
Chris@1494 2858 }
Chris@1494 2859 end
Chris@1494 2860 end
Chris@1494 2861 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2862 assert_equal ['Oracle', 'PostgreSQL'], Issue.find(1).custom_field_value(1).sort
Chris@1494 2863 end
Chris@1494 2864
Chris@1494 2865 def test_put_update_with_status_and_assignee_change
Chris@1494 2866 issue = Issue.find(1)
Chris@1494 2867 assert_equal 1, issue.status_id
Chris@1494 2868 @request.session[:user_id] = 2
Chris@1494 2869 assert_difference('TimeEntry.count', 0) do
Chris@1494 2870 put :update,
Chris@1494 2871 :id => 1,
Chris@1494 2872 :issue => { :status_id => 2, :assigned_to_id => 3, :notes => 'Assigned to dlopper' },
Chris@1494 2873 :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
Chris@1494 2874 end
Chris@1494 2875 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2876 issue.reload
Chris@1494 2877 assert_equal 2, issue.status_id
Chris@1494 2878 j = Journal.order('id DESC').first
Chris@1494 2879 assert_equal 'Assigned to dlopper', j.notes
Chris@1494 2880 assert_equal 2, j.details.size
Chris@1494 2881
Chris@1494 2882 mail = ActionMailer::Base.deliveries.last
Chris@1494 2883 assert_mail_body_match "Status changed from New to Assigned", mail
Chris@1494 2884 # subject should contain the new status
Chris@1494 2885 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
Chris@1494 2886 end
Chris@1494 2887
Chris@1494 2888 def test_put_update_with_note_only
Chris@1494 2889 notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
Chris@1494 2890 # anonymous user
Chris@1494 2891 put :update,
Chris@1494 2892 :id => 1,
Chris@1494 2893 :issue => { :notes => notes }
Chris@1494 2894 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2895 j = Journal.order('id DESC').first
Chris@1494 2896 assert_equal notes, j.notes
Chris@1494 2897 assert_equal 0, j.details.size
Chris@1494 2898 assert_equal User.anonymous, j.user
Chris@1494 2899
Chris@1494 2900 mail = ActionMailer::Base.deliveries.last
Chris@1494 2901 assert_mail_body_match notes, mail
Chris@1494 2902 end
Chris@1494 2903
Chris@1494 2904 def test_put_update_with_private_note_only
Chris@1494 2905 notes = 'Private note'
Chris@1494 2906 @request.session[:user_id] = 2
Chris@1494 2907
Chris@1494 2908 assert_difference 'Journal.count' do
Chris@1494 2909 put :update, :id => 1, :issue => {:notes => notes, :private_notes => '1'}
Chris@1494 2910 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2911 end
Chris@1494 2912
Chris@1494 2913 j = Journal.order('id DESC').first
Chris@1494 2914 assert_equal notes, j.notes
Chris@1494 2915 assert_equal true, j.private_notes
Chris@1494 2916 end
Chris@1494 2917
Chris@1494 2918 def test_put_update_with_private_note_and_changes
Chris@1494 2919 notes = 'Private note'
Chris@1494 2920 @request.session[:user_id] = 2
Chris@1494 2921
Chris@1494 2922 assert_difference 'Journal.count', 2 do
Chris@1494 2923 put :update, :id => 1, :issue => {:subject => 'New subject', :notes => notes, :private_notes => '1'}
Chris@1494 2924 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2925 end
Chris@1494 2926
Chris@1494 2927 j = Journal.order('id DESC').first
Chris@1494 2928 assert_equal notes, j.notes
Chris@1494 2929 assert_equal true, j.private_notes
Chris@1494 2930 assert_equal 0, j.details.count
Chris@1494 2931
Chris@1494 2932 j = Journal.order('id DESC').offset(1).first
Chris@1494 2933 assert_nil j.notes
Chris@1494 2934 assert_equal false, j.private_notes
Chris@1494 2935 assert_equal 1, j.details.count
Chris@1494 2936 end
Chris@1494 2937
Chris@1494 2938 def test_put_update_with_note_and_spent_time
Chris@1494 2939 @request.session[:user_id] = 2
Chris@1494 2940 spent_hours_before = Issue.find(1).spent_hours
Chris@1494 2941 assert_difference('TimeEntry.count') do
Chris@1494 2942 put :update,
Chris@1494 2943 :id => 1,
Chris@1494 2944 :issue => { :notes => '2.5 hours added' },
Chris@1494 2945 :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id }
Chris@1494 2946 end
Chris@1494 2947 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2948
Chris@1494 2949 issue = Issue.find(1)
Chris@1494 2950
Chris@1494 2951 j = Journal.order('id DESC').first
Chris@1494 2952 assert_equal '2.5 hours added', j.notes
Chris@1494 2953 assert_equal 0, j.details.size
Chris@1494 2954
Chris@1494 2955 t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time')
Chris@1494 2956 assert_not_nil t
Chris@1494 2957 assert_equal 2.5, t.hours
Chris@1494 2958 assert_equal spent_hours_before + 2.5, issue.spent_hours
Chris@1494 2959 end
Chris@1494 2960
Chris@1494 2961 def test_put_update_should_preserve_parent_issue_even_if_not_visible
Chris@1494 2962 parent = Issue.generate!(:project_id => 1, :is_private => true)
Chris@1494 2963 issue = Issue.generate!(:parent_issue_id => parent.id)
Chris@1494 2964 assert !parent.visible?(User.find(3))
Chris@1494 2965 @request.session[:user_id] = 3
Chris@1494 2966
Chris@1494 2967 get :edit, :id => issue.id
Chris@1494 2968 assert_select 'input[name=?][value=?]', 'issue[parent_issue_id]', parent.id.to_s
Chris@1494 2969
Chris@1494 2970 put :update, :id => issue.id, :issue => {:subject => 'New subject', :parent_issue_id => parent.id.to_s}
Chris@1494 2971 assert_response 302
Chris@1494 2972 assert_equal parent, issue.parent
Chris@1494 2973 end
Chris@1494 2974
Chris@1494 2975 def test_put_update_with_attachment_only
Chris@1494 2976 set_tmp_attachments_directory
Chris@1494 2977
Chris@1494 2978 # Delete all fixtured journals, a race condition can occur causing the wrong
Chris@1494 2979 # journal to get fetched in the next find.
Chris@1494 2980 Journal.delete_all
Chris@1494 2981
Chris@1494 2982 # anonymous user
Chris@1494 2983 assert_difference 'Attachment.count' do
Chris@1494 2984 put :update, :id => 1,
Chris@1494 2985 :issue => {:notes => ''},
Chris@1494 2986 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@1494 2987 end
Chris@1494 2988
Chris@1494 2989 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 2990 j = Issue.find(1).journals.reorder('id DESC').first
Chris@1494 2991 assert j.notes.blank?
Chris@1494 2992 assert_equal 1, j.details.size
Chris@1494 2993 assert_equal 'testfile.txt', j.details.first.value
Chris@1494 2994 assert_equal User.anonymous, j.user
Chris@1494 2995
Chris@1494 2996 attachment = Attachment.first(:order => 'id DESC')
Chris@1494 2997 assert_equal Issue.find(1), attachment.container
Chris@1494 2998 assert_equal User.anonymous, attachment.author
Chris@1494 2999 assert_equal 'testfile.txt', attachment.filename
Chris@1494 3000 assert_equal 'text/plain', attachment.content_type
Chris@1494 3001 assert_equal 'test file', attachment.description
Chris@1494 3002 assert_equal 59, attachment.filesize
Chris@1494 3003 assert File.exists?(attachment.diskfile)
Chris@1494 3004 assert_equal 59, File.size(attachment.diskfile)
Chris@1494 3005
Chris@1494 3006 mail = ActionMailer::Base.deliveries.last
Chris@1494 3007 assert_mail_body_match 'testfile.txt', mail
Chris@1494 3008 end
Chris@1494 3009
Chris@1494 3010 def test_put_update_with_failure_should_save_attachments
Chris@1494 3011 set_tmp_attachments_directory
Chris@1494 3012 @request.session[:user_id] = 2
Chris@1494 3013
Chris@1494 3014 assert_no_difference 'Journal.count' do
Chris@1494 3015 assert_difference 'Attachment.count' do
Chris@1494 3016 put :update, :id => 1,
Chris@1494 3017 :issue => { :subject => '' },
Chris@1494 3018 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@1494 3019 assert_response :success
Chris@1494 3020 assert_template 'edit'
Chris@1494 3021 end
Chris@1494 3022 end
Chris@1494 3023
Chris@1494 3024 attachment = Attachment.first(:order => 'id DESC')
Chris@1494 3025 assert_equal 'testfile.txt', attachment.filename
Chris@1494 3026 assert File.exists?(attachment.diskfile)
Chris@1494 3027 assert_nil attachment.container
Chris@1494 3028
Chris@1494 3029 assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token
Chris@1494 3030 assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt'
Chris@1494 3031 end
Chris@1494 3032
Chris@1494 3033 def test_put_update_with_failure_should_keep_saved_attachments
Chris@1494 3034 set_tmp_attachments_directory
Chris@1494 3035 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
Chris@1494 3036 @request.session[:user_id] = 2
Chris@1494 3037
Chris@1494 3038 assert_no_difference 'Journal.count' do
Chris@1494 3039 assert_no_difference 'Attachment.count' do
Chris@1494 3040 put :update, :id => 1,
Chris@1494 3041 :issue => { :subject => '' },
Chris@1494 3042 :attachments => {'p0' => {'token' => attachment.token}}
Chris@1494 3043 assert_response :success
Chris@1494 3044 assert_template 'edit'
Chris@1494 3045 end
Chris@1494 3046 end
Chris@1494 3047
Chris@1494 3048 assert_select 'input[name=?][value=?]', 'attachments[p0][token]', attachment.token
Chris@1494 3049 assert_select 'input[name=?][value=?]', 'attachments[p0][filename]', 'testfile.txt'
Chris@1494 3050 end
Chris@1494 3051
Chris@1494 3052 def test_put_update_should_attach_saved_attachments
Chris@1494 3053 set_tmp_attachments_directory
Chris@1494 3054 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
Chris@1494 3055 @request.session[:user_id] = 2
Chris@1494 3056
Chris@1494 3057 assert_difference 'Journal.count' do
Chris@1494 3058 assert_difference 'JournalDetail.count' do
Chris@1494 3059 assert_no_difference 'Attachment.count' do
Chris@1494 3060 put :update, :id => 1,
Chris@1494 3061 :issue => {:notes => 'Attachment added'},
Chris@1494 3062 :attachments => {'p0' => {'token' => attachment.token}}
Chris@1494 3063 assert_redirected_to '/issues/1'
Chris@1494 3064 end
Chris@1494 3065 end
Chris@1494 3066 end
Chris@1494 3067
Chris@1494 3068 attachment.reload
Chris@1494 3069 assert_equal Issue.find(1), attachment.container
Chris@1494 3070
Chris@1494 3071 journal = Journal.first(:order => 'id DESC')
Chris@1494 3072 assert_equal 1, journal.details.size
Chris@1494 3073 assert_equal 'testfile.txt', journal.details.first.value
Chris@1494 3074 end
Chris@1494 3075
Chris@1494 3076 def test_put_update_with_attachment_that_fails_to_save
Chris@1494 3077 set_tmp_attachments_directory
Chris@1494 3078
Chris@1494 3079 # Delete all fixtured journals, a race condition can occur causing the wrong
Chris@1494 3080 # journal to get fetched in the next find.
Chris@1494 3081 Journal.delete_all
Chris@1494 3082
Chris@1494 3083 # Mock out the unsaved attachment
Chris@1494 3084 Attachment.any_instance.stubs(:create).returns(Attachment.new)
Chris@1494 3085
Chris@1494 3086 # anonymous user
Chris@1494 3087 put :update,
Chris@1494 3088 :id => 1,
Chris@1494 3089 :issue => {:notes => ''},
Chris@1494 3090 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
Chris@1494 3091 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 3092 assert_equal '1 file(s) could not be saved.', flash[:warning]
Chris@1494 3093 end
Chris@1494 3094
Chris@1494 3095 def test_put_update_with_no_change
Chris@1494 3096 issue = Issue.find(1)
Chris@1494 3097 issue.journals.clear
Chris@1494 3098 ActionMailer::Base.deliveries.clear
Chris@1494 3099
Chris@1494 3100 put :update,
Chris@1494 3101 :id => 1,
Chris@1494 3102 :issue => {:notes => ''}
Chris@1494 3103 assert_redirected_to :action => 'show', :id => '1'
Chris@1494 3104
Chris@1494 3105 issue.reload
Chris@1494 3106 assert issue.journals.empty?
Chris@1494 3107 # No email should be sent
Chris@1494 3108 assert ActionMailer::Base.deliveries.empty?
Chris@1494 3109 end
Chris@1494 3110
Chris@1494 3111 def test_put_update_should_send_a_notification
Chris@1494 3112 @request.session[:user_id] = 2
Chris@1494 3113 ActionMailer::Base.deliveries.clear
Chris@1494 3114 issue = Issue.find(1)
Chris@1494 3115 old_subject = issue.subject
Chris@1494 3116 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
Chris@1494 3117
Chris@1494 3118 put :update, :id => 1, :issue => {:subject => new_subject,
Chris@1494 3119 :priority_id => '6',
Chris@1494 3120 :category_id => '1' # no change
Chris@1494 3121 }
Chris@1494 3122 assert_equal 1, ActionMailer::Base.deliveries.size
Chris@1494 3123 end
Chris@1494 3124
Chris@1494 3125 def test_put_update_with_invalid_spent_time_hours_only
Chris@1494 3126 @request.session[:user_id] = 2
Chris@1494 3127 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
Chris@1494 3128
Chris@1494 3129 assert_no_difference('Journal.count') do
Chris@1494 3130 put :update,
Chris@1494 3131 :id => 1,
Chris@1494 3132 :issue => {:notes => notes},
Chris@1494 3133 :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
Chris@1494 3134 end
Chris@1494 3135 assert_response :success
Chris@1494 3136 assert_template 'edit'
Chris@1494 3137
Chris@1494 3138 assert_error_tag :descendant => {:content => /Activity can&#x27;t be blank/}
Chris@1494 3139 assert_select 'textarea[name=?]', 'issue[notes]', :text => notes
Chris@1494 3140 assert_select 'input[name=?][value=?]', 'time_entry[hours]', '2z'
Chris@1494 3141 end
Chris@1494 3142
Chris@1494 3143 def test_put_update_with_invalid_spent_time_comments_only
Chris@1494 3144 @request.session[:user_id] = 2
Chris@1494 3145 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
Chris@1494 3146
Chris@1494 3147 assert_no_difference('Journal.count') do
Chris@1494 3148 put :update,
Chris@1494 3149 :id => 1,
Chris@1494 3150 :issue => {:notes => notes},
Chris@1494 3151 :time_entry => {"comments"=>"this is my comment", "activity_id"=>"", "hours"=>""}
Chris@1494 3152 end
Chris@1494 3153 assert_response :success
Chris@1494 3154 assert_template 'edit'
Chris@1494 3155
Chris@1494 3156 assert_error_tag :descendant => {:content => /Activity can&#x27;t be blank/}
Chris@1494 3157 assert_error_tag :descendant => {:content => /Hours can&#x27;t be blank/}
Chris@1494 3158 assert_select 'textarea[name=?]', 'issue[notes]', :text => notes
Chris@1494 3159 assert_select 'input[name=?][value=?]', 'time_entry[comments]', 'this is my comment'
Chris@1494 3160 end
Chris@1494 3161
Chris@1494 3162 def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject
Chris@1494 3163 issue = Issue.find(2)
Chris@1494 3164 @request.session[:user_id] = 2
Chris@1494 3165
Chris@1494 3166 put :update,
Chris@1494 3167 :id => issue.id,
Chris@1494 3168 :issue => {
Chris@1494 3169 :fixed_version_id => 4
Chris@1494 3170 }
Chris@1494 3171
Chris@1494 3172 assert_response :redirect
Chris@1494 3173 issue.reload
Chris@1494 3174 assert_equal 4, issue.fixed_version_id
Chris@1494 3175 assert_not_equal issue.project_id, issue.fixed_version.project_id
Chris@1494 3176 end
Chris@1494 3177
Chris@1494 3178 def test_put_update_should_redirect_back_using_the_back_url_parameter
Chris@1494 3179 issue = Issue.find(2)
Chris@1494 3180 @request.session[:user_id] = 2
Chris@1494 3181
Chris@1494 3182 put :update,
Chris@1494 3183 :id => issue.id,
Chris@1494 3184 :issue => {
Chris@1494 3185 :fixed_version_id => 4
Chris@1494 3186 },
Chris@1494 3187 :back_url => '/issues'
Chris@1494 3188
Chris@1494 3189 assert_response :redirect
Chris@1494 3190 assert_redirected_to '/issues'
Chris@1494 3191 end
Chris@1494 3192
Chris@1494 3193 def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
Chris@1494 3194 issue = Issue.find(2)
Chris@1494 3195 @request.session[:user_id] = 2
Chris@1494 3196
Chris@1494 3197 put :update,
Chris@1494 3198 :id => issue.id,
Chris@1494 3199 :issue => {
Chris@1494 3200 :fixed_version_id => 4
Chris@1494 3201 },
Chris@1494 3202 :back_url => 'http://google.com'
Chris@1494 3203
Chris@1494 3204 assert_response :redirect
Chris@1494 3205 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id
Chris@1494 3206 end
Chris@1494 3207
Chris@1494 3208 def test_get_bulk_edit
Chris@1494 3209 @request.session[:user_id] = 2
Chris@1494 3210 get :bulk_edit, :ids => [1, 2]
Chris@1494 3211 assert_response :success
Chris@1494 3212 assert_template 'bulk_edit'
Chris@1494 3213
Chris@1494 3214 assert_select 'ul#bulk-selection' do
Chris@1494 3215 assert_select 'li', 2
Chris@1494 3216 assert_select 'li a', :text => 'Bug #1'
Chris@1494 3217 end
Chris@1494 3218
Chris@1494 3219 assert_select 'form#bulk_edit_form[action=?]', '/issues/bulk_update' do
Chris@1494 3220 assert_select 'input[name=?]', 'ids[]', 2
Chris@1494 3221 assert_select 'input[name=?][value=1][type=hidden]', 'ids[]'
Chris@1494 3222
Chris@1494 3223 assert_select 'select[name=?]', 'issue[project_id]'
Chris@1494 3224 assert_select 'input[name=?]', 'issue[parent_issue_id]'
Chris@1494 3225
Chris@1494 3226 # Project specific custom field, date type
Chris@1494 3227 field = CustomField.find(9)
Chris@1494 3228 assert !field.is_for_all?
Chris@1494 3229 assert_equal 'date', field.field_format
Chris@1494 3230 assert_select 'input[name=?]', 'issue[custom_field_values][9]'
Chris@1494 3231
Chris@1494 3232 # System wide custom field
Chris@1494 3233 assert CustomField.find(1).is_for_all?
Chris@1494 3234 assert_select 'select[name=?]', 'issue[custom_field_values][1]'
Chris@1494 3235
Chris@1494 3236 # Be sure we don't display inactive IssuePriorities
Chris@1494 3237 assert ! IssuePriority.find(15).active?
Chris@1494 3238 assert_select 'select[name=?]', 'issue[priority_id]' do
Chris@1494 3239 assert_select 'option[value=15]', 0
Chris@1494 3240 end
Chris@1494 3241 end
Chris@1494 3242 end
Chris@1494 3243
Chris@1494 3244 def test_get_bulk_edit_on_different_projects
Chris@1494 3245 @request.session[:user_id] = 2
Chris@1494 3246 get :bulk_edit, :ids => [1, 2, 6]
Chris@1494 3247 assert_response :success
Chris@1494 3248 assert_template 'bulk_edit'
Chris@1494 3249
Chris@1494 3250 # Can not set issues from different projects as children of an issue
Chris@1494 3251 assert_select 'input[name=?]', 'issue[parent_issue_id]', 0
Chris@1494 3252
Chris@1494 3253 # Project specific custom field, date type
Chris@1494 3254 field = CustomField.find(9)
Chris@1494 3255 assert !field.is_for_all?
Chris@1494 3256 assert !field.project_ids.include?(Issue.find(6).project_id)
Chris@1494 3257 assert_select 'input[name=?]', 'issue[custom_field_values][9]', 0
Chris@1494 3258 end
Chris@1494 3259
Chris@1494 3260 def test_get_bulk_edit_with_user_custom_field
Chris@1494 3261 field = IssueCustomField.create!(:name => 'Tester', :field_format => 'user', :is_for_all => true)
Chris@1494 3262
Chris@1494 3263 @request.session[:user_id] = 2
Chris@1494 3264 get :bulk_edit, :ids => [1, 2]
Chris@1494 3265 assert_response :success
Chris@1494 3266 assert_template 'bulk_edit'
Chris@1494 3267
Chris@1494 3268 assert_select 'select.user_cf[name=?]', "issue[custom_field_values][#{field.id}]" do
Chris@1494 3269 assert_select 'option', Project.find(1).users.count + 2 # "no change" + "none" options
Chris@1494 3270 end
Chris@1494 3271 end
Chris@1494 3272
Chris@1494 3273 def test_get_bulk_edit_with_version_custom_field
Chris@1494 3274 field = IssueCustomField.create!(:name => 'Affected version', :field_format => 'version', :is_for_all => true)
Chris@1494 3275
Chris@1494 3276 @request.session[:user_id] = 2
Chris@1494 3277 get :bulk_edit, :ids => [1, 2]
Chris@1494 3278 assert_response :success
Chris@1494 3279 assert_template 'bulk_edit'
Chris@1494 3280
Chris@1494 3281 assert_select 'select.version_cf[name=?]', "issue[custom_field_values][#{field.id}]" do
Chris@1494 3282 assert_select 'option', Project.find(1).shared_versions.count + 2 # "no change" + "none" options
Chris@1494 3283 end
Chris@1494 3284 end
Chris@1494 3285
Chris@1494 3286 def test_get_bulk_edit_with_multi_custom_field
Chris@1494 3287 field = CustomField.find(1)
Chris@1494 3288 field.update_attribute :multiple, true
Chris@1494 3289
Chris@1494 3290 @request.session[:user_id] = 2
Chris@1494 3291 get :bulk_edit, :ids => [1, 2]
Chris@1494 3292 assert_response :success
Chris@1494 3293 assert_template 'bulk_edit'
Chris@1494 3294
Chris@1494 3295 assert_select 'select[name=?]', 'issue[custom_field_values][1][]' do
Chris@1494 3296 assert_select 'option', field.possible_values.size + 1 # "none" options
Chris@1494 3297 end
Chris@1494 3298 end
Chris@1494 3299
Chris@1494 3300 def test_bulk_edit_should_propose_to_clear_text_custom_fields
Chris@1494 3301 @request.session[:user_id] = 2
Chris@1494 3302 get :bulk_edit, :ids => [1, 3]
Chris@1494 3303 assert_select 'input[name=?][value=?]', 'issue[custom_field_values][2]', '__none__'
Chris@1494 3304 end
Chris@1494 3305
Chris@1494 3306 def test_bulk_edit_should_only_propose_statuses_allowed_for_all_issues
Chris@1494 3307 WorkflowTransition.delete_all
Chris@1494 3308 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1,
Chris@1494 3309 :old_status_id => 1, :new_status_id => 1)
Chris@1494 3310 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1,
Chris@1494 3311 :old_status_id => 1, :new_status_id => 3)
Chris@1494 3312 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1,
Chris@1494 3313 :old_status_id => 1, :new_status_id => 4)
Chris@1494 3314 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2,
Chris@1494 3315 :old_status_id => 2, :new_status_id => 1)
Chris@1494 3316 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2,
Chris@1494 3317 :old_status_id => 2, :new_status_id => 3)
Chris@1494 3318 WorkflowTransition.create!(:role_id => 1, :tracker_id => 2,
Chris@1494 3319 :old_status_id => 2, :new_status_id => 5)
Chris@1494 3320 @request.session[:user_id] = 2
Chris@1494 3321 get :bulk_edit, :ids => [1, 2]
Chris@1494 3322
Chris@1494 3323 assert_response :success
Chris@1494 3324 statuses = assigns(:available_statuses)
Chris@1494 3325 assert_not_nil statuses
Chris@1494 3326 assert_equal [1, 3], statuses.map(&:id).sort
Chris@1494 3327
Chris@1494 3328 assert_select 'select[name=?]', 'issue[status_id]' do
Chris@1494 3329 assert_select 'option', 3 # 2 statuses + "no change" option
Chris@1494 3330 end
Chris@1494 3331 end
Chris@1494 3332
Chris@1494 3333 def test_bulk_edit_should_propose_target_project_open_shared_versions
Chris@1494 3334 @request.session[:user_id] = 2
Chris@1494 3335 post :bulk_edit, :ids => [1, 2, 6], :issue => {:project_id => 1}
Chris@1494 3336 assert_response :success
Chris@1494 3337 assert_template 'bulk_edit'
Chris@1494 3338 assert_equal Project.find(1).shared_versions.open.all.sort, assigns(:versions).sort
Chris@1494 3339
Chris@1494 3340 assert_select 'select[name=?]', 'issue[fixed_version_id]' do
Chris@1494 3341 assert_select 'option', :text => '2.0'
Chris@1494 3342 end
Chris@1494 3343 end
Chris@1494 3344
Chris@1494 3345 def test_bulk_edit_should_propose_target_project_categories
Chris@1494 3346 @request.session[:user_id] = 2
Chris@1494 3347 post :bulk_edit, :ids => [1, 2, 6], :issue => {:project_id => 1}
Chris@1494 3348 assert_response :success
Chris@1494 3349 assert_template 'bulk_edit'
Chris@1494 3350 assert_equal Project.find(1).issue_categories.sort, assigns(:categories).sort
Chris@1494 3351
Chris@1494 3352 assert_select 'select[name=?]', 'issue[category_id]' do
Chris@1494 3353 assert_select 'option', :text => 'Recipes'
Chris@1494 3354 end
Chris@1494 3355 end
Chris@1494 3356
Chris@1494 3357 def test_bulk_update
Chris@1494 3358 @request.session[:user_id] = 2
Chris@1494 3359 # update issues priority
Chris@1494 3360 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
Chris@1494 3361 :issue => {:priority_id => 7,
Chris@1494 3362 :assigned_to_id => '',
Chris@1494 3363 :custom_field_values => {'2' => ''}}
Chris@1494 3364
Chris@1494 3365 assert_response 302
Chris@1494 3366 # check that the issues were updated
Chris@1494 3367 assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
Chris@1494 3368
Chris@1494 3369 issue = Issue.find(1)
Chris@1494 3370 journal = issue.journals.reorder('created_on DESC').first
Chris@1494 3371 assert_equal '125', issue.custom_value_for(2).value
Chris@1494 3372 assert_equal 'Bulk editing', journal.notes
Chris@1494 3373 assert_equal 1, journal.details.size
Chris@1494 3374 end
Chris@1494 3375
Chris@1494 3376 def test_bulk_update_with_group_assignee
Chris@1494 3377 group = Group.find(11)
Chris@1494 3378 project = Project.find(1)
Chris@1494 3379 project.members << Member.new(:principal => group, :roles => [Role.givable.first])
Chris@1494 3380
Chris@1494 3381 @request.session[:user_id] = 2
Chris@1494 3382 # update issues assignee
Chris@1494 3383 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
Chris@1494 3384 :issue => {:priority_id => '',
Chris@1494 3385 :assigned_to_id => group.id,
Chris@1494 3386 :custom_field_values => {'2' => ''}}
Chris@1494 3387
Chris@1494 3388 assert_response 302
Chris@1494 3389 assert_equal [group, group], Issue.find_all_by_id([1, 2]).collect {|i| i.assigned_to}
Chris@1494 3390 end
Chris@1494 3391
Chris@1494 3392 def test_bulk_update_on_different_projects
Chris@1494 3393 @request.session[:user_id] = 2
Chris@1494 3394 # update issues priority
Chris@1494 3395 post :bulk_update, :ids => [1, 2, 6], :notes => 'Bulk editing',
Chris@1494 3396 :issue => {:priority_id => 7,
Chris@1494 3397 :assigned_to_id => '',
Chris@1494 3398 :custom_field_values => {'2' => ''}}
Chris@1494 3399
Chris@1494 3400 assert_response 302
Chris@1494 3401 # check that the issues were updated
Chris@1494 3402 assert_equal [7, 7, 7], Issue.find([1,2,6]).map(&:priority_id)
Chris@1494 3403
Chris@1494 3404 issue = Issue.find(1)
Chris@1494 3405 journal = issue.journals.reorder('created_on DESC').first
Chris@1494 3406 assert_equal '125', issue.custom_value_for(2).value
Chris@1494 3407 assert_equal 'Bulk editing', journal.notes
Chris@1494 3408 assert_equal 1, journal.details.size
Chris@1494 3409 end
Chris@1494 3410
Chris@1494 3411 def test_bulk_update_on_different_projects_without_rights
Chris@1494 3412 @request.session[:user_id] = 3
Chris@1494 3413 user = User.find(3)
Chris@1494 3414 action = { :controller => "issues", :action => "bulk_update" }
Chris@1494 3415 assert user.allowed_to?(action, Issue.find(1).project)
Chris@1494 3416 assert ! user.allowed_to?(action, Issue.find(6).project)
Chris@1494 3417 post :bulk_update, :ids => [1, 6], :notes => 'Bulk should fail',
Chris@1494 3418 :issue => {:priority_id => 7,
Chris@1494 3419 :assigned_to_id => '',
Chris@1494 3420 :custom_field_values => {'2' => ''}}
Chris@1494 3421 assert_response 403
Chris@1494 3422 assert_not_equal "Bulk should fail", Journal.last.notes
Chris@1494 3423 end
Chris@1494 3424
Chris@1494 3425 def test_bullk_update_should_send_a_notification
Chris@1494 3426 @request.session[:user_id] = 2
Chris@1494 3427 ActionMailer::Base.deliveries.clear
Chris@1494 3428 post(:bulk_update,
Chris@1494 3429 {
Chris@1494 3430 :ids => [1, 2],
Chris@1494 3431 :notes => 'Bulk editing',
Chris@1494 3432 :issue => {
Chris@1494 3433 :priority_id => 7,
Chris@1494 3434 :assigned_to_id => '',
Chris@1494 3435 :custom_field_values => {'2' => ''}
Chris@1494 3436 }
Chris@1494 3437 })
Chris@1494 3438
Chris@1494 3439 assert_response 302
Chris@1494 3440 assert_equal 2, ActionMailer::Base.deliveries.size
Chris@1494 3441 end
Chris@1494 3442
Chris@1494 3443 def test_bulk_update_project
Chris@1494 3444 @request.session[:user_id] = 2
Chris@1494 3445 post :bulk_update, :ids => [1, 2], :issue => {:project_id => '2'}
Chris@1494 3446 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
Chris@1494 3447 # Issues moved to project 2
Chris@1494 3448 assert_equal 2, Issue.find(1).project_id
Chris@1494 3449 assert_equal 2, Issue.find(2).project_id
Chris@1494 3450 # No tracker change
Chris@1494 3451 assert_equal 1, Issue.find(1).tracker_id
Chris@1494 3452 assert_equal 2, Issue.find(2).tracker_id
Chris@1494 3453 end
Chris@1494 3454
Chris@1494 3455 def test_bulk_update_project_on_single_issue_should_follow_when_needed
Chris@1494 3456 @request.session[:user_id] = 2
Chris@1494 3457 post :bulk_update, :id => 1, :issue => {:project_id => '2'}, :follow => '1'
Chris@1494 3458 assert_redirected_to '/issues/1'
Chris@1494 3459 end
Chris@1494 3460
Chris@1494 3461 def test_bulk_update_project_on_multiple_issues_should_follow_when_needed
Chris@1494 3462 @request.session[:user_id] = 2
Chris@1494 3463 post :bulk_update, :id => [1, 2], :issue => {:project_id => '2'}, :follow => '1'
Chris@1494 3464 assert_redirected_to '/projects/onlinestore/issues'
Chris@1494 3465 end
Chris@1494 3466
Chris@1494 3467 def test_bulk_update_tracker
Chris@1494 3468 @request.session[:user_id] = 2
Chris@1494 3469 post :bulk_update, :ids => [1, 2], :issue => {:tracker_id => '2'}
Chris@1494 3470 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
Chris@1494 3471 assert_equal 2, Issue.find(1).tracker_id
Chris@1494 3472 assert_equal 2, Issue.find(2).tracker_id
Chris@1494 3473 end
Chris@1494 3474
Chris@1494 3475 def test_bulk_update_status
Chris@1494 3476 @request.session[:user_id] = 2
Chris@1494 3477 # update issues priority
Chris@1494 3478 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing status',
Chris@1494 3479 :issue => {:priority_id => '',
Chris@1494 3480 :assigned_to_id => '',
Chris@1494 3481 :status_id => '5'}
Chris@1494 3482
Chris@1494 3483 assert_response 302
Chris@1494 3484 issue = Issue.find(1)
Chris@1494 3485 assert issue.closed?
Chris@1494 3486 end
Chris@1494 3487
Chris@1494 3488 def test_bulk_update_priority
Chris@1494 3489 @request.session[:user_id] = 2
Chris@1494 3490 post :bulk_update, :ids => [1, 2], :issue => {:priority_id => 6}
Chris@1494 3491
Chris@1494 3492 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
Chris@1494 3493 assert_equal 6, Issue.find(1).priority_id
Chris@1494 3494 assert_equal 6, Issue.find(2).priority_id
Chris@1494 3495 end
Chris@1494 3496
Chris@1494 3497 def test_bulk_update_with_notes
Chris@1494 3498 @request.session[:user_id] = 2
Chris@1494 3499 post :bulk_update, :ids => [1, 2], :notes => 'Moving two issues'
Chris@1494 3500
Chris@1494 3501 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => 'ecookbook'
Chris@1494 3502 assert_equal 'Moving two issues', Issue.find(1).journals.sort_by(&:id).last.notes
Chris@1494 3503 assert_equal 'Moving two issues', Issue.find(2).journals.sort_by(&:id).last.notes
Chris@1494 3504 end
Chris@1494 3505
Chris@1494 3506 def test_bulk_update_parent_id
Chris@1494 3507 IssueRelation.delete_all
Chris@1494 3508 @request.session[:user_id] = 2
Chris@1494 3509 post :bulk_update, :ids => [1, 3],
Chris@1494 3510 :notes => 'Bulk editing parent',
Chris@1494 3511 :issue => {:priority_id => '', :assigned_to_id => '',
Chris@1494 3512 :status_id => '', :parent_issue_id => '2'}
Chris@1494 3513 assert_response 302
Chris@1494 3514 parent = Issue.find(2)
Chris@1494 3515 assert_equal parent.id, Issue.find(1).parent_id
Chris@1494 3516 assert_equal parent.id, Issue.find(3).parent_id
Chris@1494 3517 assert_equal [1, 3], parent.children.collect(&:id).sort
Chris@1494 3518 end
Chris@1494 3519
Chris@1494 3520 def test_bulk_update_custom_field
Chris@1494 3521 @request.session[:user_id] = 2
Chris@1494 3522 # update issues priority
Chris@1494 3523 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing custom field',
Chris@1494 3524 :issue => {:priority_id => '',
Chris@1494 3525 :assigned_to_id => '',
Chris@1494 3526 :custom_field_values => {'2' => '777'}}
Chris@1494 3527
Chris@1494 3528 assert_response 302
Chris@1494 3529
Chris@1494 3530 issue = Issue.find(1)
Chris@1494 3531 journal = issue.journals.reorder('created_on DESC').first
Chris@1494 3532 assert_equal '777', issue.custom_value_for(2).value
Chris@1494 3533 assert_equal 1, journal.details.size
Chris@1494 3534 assert_equal '125', journal.details.first.old_value
Chris@1494 3535 assert_equal '777', journal.details.first.value
Chris@1494 3536 end
Chris@1494 3537
Chris@1494 3538 def test_bulk_update_custom_field_to_blank
Chris@1494 3539 @request.session[:user_id] = 2
Chris@1494 3540 post :bulk_update, :ids => [1, 3], :notes => 'Bulk editing custom field',
Chris@1494 3541 :issue => {:priority_id => '',
Chris@1494 3542 :assigned_to_id => '',
Chris@1494 3543 :custom_field_values => {'1' => '__none__'}}
Chris@1494 3544 assert_response 302
Chris@1494 3545 assert_equal '', Issue.find(1).custom_field_value(1)
Chris@1494 3546 assert_equal '', Issue.find(3).custom_field_value(1)
Chris@1494 3547 end
Chris@1494 3548
Chris@1494 3549 def test_bulk_update_multi_custom_field
Chris@1494 3550 field = CustomField.find(1)
Chris@1494 3551 field.update_attribute :multiple, true
Chris@1494 3552
Chris@1494 3553 @request.session[:user_id] = 2
Chris@1494 3554 post :bulk_update, :ids => [1, 2, 3], :notes => 'Bulk editing multi custom field',
Chris@1494 3555 :issue => {:priority_id => '',
Chris@1494 3556 :assigned_to_id => '',
Chris@1494 3557 :custom_field_values => {'1' => ['MySQL', 'Oracle']}}
Chris@1494 3558
Chris@1494 3559 assert_response 302
Chris@1494 3560
Chris@1494 3561 assert_equal ['MySQL', 'Oracle'], Issue.find(1).custom_field_value(1).sort
Chris@1494 3562 assert_equal ['MySQL', 'Oracle'], Issue.find(3).custom_field_value(1).sort
Chris@1494 3563 # the custom field is not associated with the issue tracker
Chris@1494 3564 assert_nil Issue.find(2).custom_field_value(1)
Chris@1494 3565 end
Chris@1494 3566
Chris@1494 3567 def test_bulk_update_multi_custom_field_to_blank
Chris@1494 3568 field = CustomField.find(1)
Chris@1494 3569 field.update_attribute :multiple, true
Chris@1494 3570
Chris@1494 3571 @request.session[:user_id] = 2
Chris@1494 3572 post :bulk_update, :ids => [1, 3], :notes => 'Bulk editing multi custom field',
Chris@1494 3573 :issue => {:priority_id => '',
Chris@1494 3574 :assigned_to_id => '',
Chris@1494 3575 :custom_field_values => {'1' => ['__none__']}}
Chris@1494 3576 assert_response 302
Chris@1494 3577 assert_equal [''], Issue.find(1).custom_field_value(1)
Chris@1494 3578 assert_equal [''], Issue.find(3).custom_field_value(1)
Chris@1494 3579 end
Chris@1494 3580
Chris@1494 3581 def test_bulk_update_unassign
Chris@1494 3582 assert_not_nil Issue.find(2).assigned_to
Chris@1494 3583 @request.session[:user_id] = 2
Chris@1494 3584 # unassign issues
Chris@1494 3585 post :bulk_update, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'}
Chris@1494 3586 assert_response 302
Chris@1494 3587 # check that the issues were updated
Chris@1494 3588 assert_nil Issue.find(2).assigned_to
Chris@1494 3589 end
Chris@1494 3590
Chris@1494 3591 def test_post_bulk_update_should_allow_fixed_version_to_be_set_to_a_subproject
Chris@1494 3592 @request.session[:user_id] = 2
Chris@1494 3593
Chris@1494 3594 post :bulk_update, :ids => [1,2], :issue => {:fixed_version_id => 4}
Chris@1494 3595
Chris@1494 3596 assert_response :redirect
Chris@1494 3597 issues = Issue.find([1,2])
Chris@1494 3598 issues.each do |issue|
Chris@1494 3599 assert_equal 4, issue.fixed_version_id
Chris@1494 3600 assert_not_equal issue.project_id, issue.fixed_version.project_id
Chris@1494 3601 end
Chris@1494 3602 end
Chris@1494 3603
Chris@1494 3604 def test_post_bulk_update_should_redirect_back_using_the_back_url_parameter
Chris@1494 3605 @request.session[:user_id] = 2
Chris@1494 3606 post :bulk_update, :ids => [1,2], :back_url => '/issues'
Chris@1494 3607
Chris@1494 3608 assert_response :redirect
Chris@1494 3609 assert_redirected_to '/issues'
Chris@1494 3610 end
Chris@1494 3611
Chris@1494 3612 def test_post_bulk_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
Chris@1494 3613 @request.session[:user_id] = 2
Chris@1494 3614 post :bulk_update, :ids => [1,2], :back_url => 'http://google.com'
Chris@1494 3615
Chris@1494 3616 assert_response :redirect
Chris@1494 3617 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier
Chris@1494 3618 end
Chris@1494 3619
Chris@1494 3620 def test_bulk_update_with_all_failures_should_show_errors
Chris@1494 3621 @request.session[:user_id] = 2
Chris@1494 3622 post :bulk_update, :ids => [1, 2], :issue => {:start_date => 'foo'}
Chris@1494 3623
Chris@1494 3624 assert_response :success
Chris@1494 3625 assert_template 'bulk_edit'
Chris@1494 3626 assert_select '#errorExplanation span', :text => 'Failed to save 2 issue(s) on 2 selected: #1, #2.'
Chris@1494 3627 assert_select '#errorExplanation ul li', :text => 'Start date is not a valid date: #1, #2'
Chris@1494 3628
Chris@1494 3629 assert_equal [1, 2], assigns[:issues].map(&:id)
Chris@1494 3630 end
Chris@1494 3631
Chris@1494 3632 def test_bulk_update_with_some_failures_should_show_errors
Chris@1494 3633 issue1 = Issue.generate!(:start_date => '2013-05-12')
Chris@1494 3634 issue2 = Issue.generate!(:start_date => '2013-05-15')
Chris@1494 3635 issue3 = Issue.generate!
Chris@1494 3636 @request.session[:user_id] = 2
Chris@1494 3637 post :bulk_update, :ids => [issue1.id, issue2.id, issue3.id],
Chris@1494 3638 :issue => {:due_date => '2013-05-01'}
Chris@1494 3639 assert_response :success
Chris@1494 3640 assert_template 'bulk_edit'
Chris@1494 3641 assert_select '#errorExplanation span',
Chris@1494 3642 :text => "Failed to save 2 issue(s) on 3 selected: ##{issue1.id}, ##{issue2.id}."
Chris@1494 3643 assert_select '#errorExplanation ul li',
Chris@1494 3644 :text => "Due date must be greater than start date: ##{issue1.id}, ##{issue2.id}"
Chris@1494 3645 assert_equal [issue1.id, issue2.id], assigns[:issues].map(&:id)
Chris@1494 3646 end
Chris@1494 3647
Chris@1494 3648 def test_bulk_update_with_failure_should_preserved_form_values
Chris@1494 3649 @request.session[:user_id] = 2
Chris@1494 3650 post :bulk_update, :ids => [1, 2], :issue => {:tracker_id => '2', :start_date => 'foo'}
Chris@1494 3651
Chris@1494 3652 assert_response :success
Chris@1494 3653 assert_template 'bulk_edit'
Chris@1494 3654 assert_select 'select[name=?]', 'issue[tracker_id]' do
Chris@1494 3655 assert_select 'option[value=2][selected=selected]'
Chris@1494 3656 end
Chris@1494 3657 assert_select 'input[name=?][value=?]', 'issue[start_date]', 'foo'
Chris@1494 3658 end
Chris@1494 3659
Chris@1494 3660 def test_get_bulk_copy
Chris@1494 3661 @request.session[:user_id] = 2
Chris@1494 3662 get :bulk_edit, :ids => [1, 2, 3], :copy => '1'
Chris@1494 3663 assert_response :success
Chris@1494 3664 assert_template 'bulk_edit'
Chris@1494 3665
Chris@1494 3666 issues = assigns(:issues)
Chris@1494 3667 assert_not_nil issues
Chris@1494 3668 assert_equal [1, 2, 3], issues.map(&:id).sort
Chris@1494 3669
Chris@1494 3670 assert_select 'input[name=copy_attachments]'
Chris@1494 3671 end
Chris@1494 3672
Chris@1494 3673 def test_bulk_copy_to_another_project
Chris@1494 3674 @request.session[:user_id] = 2
Chris@1494 3675 assert_difference 'Issue.count', 2 do
Chris@1494 3676 assert_no_difference 'Project.find(1).issues.count' do
Chris@1494 3677 post :bulk_update, :ids => [1, 2], :issue => {:project_id => '2'}, :copy => '1'
Chris@1494 3678 end
Chris@1494 3679 end
Chris@1494 3680 assert_redirected_to '/projects/ecookbook/issues'
Chris@1494 3681
Chris@1494 3682 copies = Issue.all(:order => 'id DESC', :limit => issues.size)
Chris@1494 3683 copies.each do |copy|
Chris@1494 3684 assert_equal 2, copy.project_id
Chris@1494 3685 end
Chris@1494 3686 end
Chris@1494 3687
Chris@1494 3688 def test_bulk_copy_should_allow_not_changing_the_issue_attributes
Chris@1494 3689 @request.session[:user_id] = 2
Chris@1494 3690 issues = [
Chris@1494 3691 Issue.create!(:project_id => 1, :tracker_id => 1, :status_id => 1,
Chris@1494 3692 :priority_id => 2, :subject => 'issue 1', :author_id => 1,
Chris@1494 3693 :assigned_to_id => nil),
Chris@1494 3694 Issue.create!(:project_id => 2, :tracker_id => 3, :status_id => 2,
Chris@1494 3695 :priority_id => 1, :subject => 'issue 2', :author_id => 2,
Chris@1494 3696 :assigned_to_id => 3)
Chris@1494 3697 ]
Chris@1494 3698 assert_difference 'Issue.count', issues.size do
Chris@1494 3699 post :bulk_update, :ids => issues.map(&:id), :copy => '1',
Chris@1494 3700 :issue => {
Chris@1494 3701 :project_id => '', :tracker_id => '', :assigned_to_id => '',
Chris@1494 3702 :status_id => '', :start_date => '', :due_date => ''
Chris@1494 3703 }
Chris@1494 3704 end
Chris@1494 3705
Chris@1494 3706 copies = Issue.all(:order => 'id DESC', :limit => issues.size)
Chris@1494 3707 issues.each do |orig|
Chris@1494 3708 copy = copies.detect {|c| c.subject == orig.subject}
Chris@1494 3709 assert_not_nil copy
Chris@1494 3710 assert_equal orig.project_id, copy.project_id
Chris@1494 3711 assert_equal orig.tracker_id, copy.tracker_id
Chris@1494 3712 assert_equal orig.status_id, copy.status_id
Chris@1494 3713 assert_equal orig.assigned_to_id, copy.assigned_to_id
Chris@1494 3714 assert_equal orig.priority_id, copy.priority_id
Chris@1494 3715 end
Chris@1494 3716 end
Chris@1494 3717
Chris@1494 3718 def test_bulk_copy_should_allow_changing_the_issue_attributes
Chris@1494 3719 # Fixes random test failure with Mysql
Chris@1494 3720 # where Issue.where(:project_id => 2).limit(2).order('id desc')
Chris@1494 3721 # doesn't return the expected results
Chris@1494 3722 Issue.delete_all("project_id=2")
Chris@1494 3723
Chris@1494 3724 @request.session[:user_id] = 2
Chris@1494 3725 assert_difference 'Issue.count', 2 do
Chris@1494 3726 assert_no_difference 'Project.find(1).issues.count' do
Chris@1494 3727 post :bulk_update, :ids => [1, 2], :copy => '1',
Chris@1494 3728 :issue => {
Chris@1494 3729 :project_id => '2', :tracker_id => '', :assigned_to_id => '4',
Chris@1494 3730 :status_id => '1', :start_date => '2009-12-01', :due_date => '2009-12-31'
Chris@1494 3731 }
Chris@1494 3732 end
Chris@1494 3733 end
Chris@1494 3734
Chris@1494 3735 copied_issues = Issue.where(:project_id => 2).limit(2).order('id desc').to_a
Chris@1494 3736 assert_equal 2, copied_issues.size
Chris@1494 3737 copied_issues.each do |issue|
Chris@1494 3738 assert_equal 2, issue.project_id, "Project is incorrect"
Chris@1494 3739 assert_equal 4, issue.assigned_to_id, "Assigned to is incorrect"
Chris@1494 3740 assert_equal 1, issue.status_id, "Status is incorrect"
Chris@1494 3741 assert_equal '2009-12-01', issue.start_date.to_s, "Start date is incorrect"
Chris@1494 3742 assert_equal '2009-12-31', issue.due_date.to_s, "Due date is incorrect"
Chris@1494 3743 end
Chris@1494 3744 end
Chris@1494 3745
Chris@1494 3746 def test_bulk_copy_should_allow_adding_a_note
Chris@1494 3747 @request.session[:user_id] = 2
Chris@1494 3748 assert_difference 'Issue.count', 1 do
Chris@1494 3749 post :bulk_update, :ids => [1], :copy => '1',
Chris@1494 3750 :notes => 'Copying one issue',
Chris@1494 3751 :issue => {
Chris@1494 3752 :project_id => '', :tracker_id => '', :assigned_to_id => '4',
Chris@1494 3753 :status_id => '3', :start_date => '2009-12-01', :due_date => '2009-12-31'
Chris@1494 3754 }
Chris@1494 3755 end
Chris@1494 3756 issue = Issue.first(:order => 'id DESC')
Chris@1494 3757 assert_equal 1, issue.journals.size
Chris@1494 3758 journal = issue.journals.first
Chris@1494 3759 assert_equal 1, journal.details.size
Chris@1494 3760 assert_equal 'Copying one issue', journal.notes
Chris@1494 3761 end
Chris@1494 3762
Chris@1494 3763 def test_bulk_copy_should_allow_not_copying_the_attachments
Chris@1494 3764 attachment_count = Issue.find(3).attachments.size
Chris@1494 3765 assert attachment_count > 0
Chris@1494 3766 @request.session[:user_id] = 2
Chris@1494 3767
Chris@1494 3768 assert_difference 'Issue.count', 1 do
Chris@1494 3769 assert_no_difference 'Attachment.count' do
Chris@1494 3770 post :bulk_update, :ids => [3], :copy => '1',
Chris@1494 3771 :issue => {
Chris@1494 3772 :project_id => ''
Chris@1494 3773 }
Chris@1494 3774 end
Chris@1494 3775 end
Chris@1494 3776 end
Chris@1494 3777
Chris@1494 3778 def test_bulk_copy_should_allow_copying_the_attachments
Chris@1494 3779 attachment_count = Issue.find(3).attachments.size
Chris@1494 3780 assert attachment_count > 0
Chris@1494 3781 @request.session[:user_id] = 2
Chris@1494 3782
Chris@1494 3783 assert_difference 'Issue.count', 1 do
Chris@1494 3784 assert_difference 'Attachment.count', attachment_count do
Chris@1494 3785 post :bulk_update, :ids => [3], :copy => '1', :copy_attachments => '1',
Chris@1494 3786 :issue => {
Chris@1494 3787 :project_id => ''
Chris@1494 3788 }
Chris@1494 3789 end
Chris@1494 3790 end
Chris@1494 3791 end
Chris@1494 3792
Chris@1494 3793 def test_bulk_copy_should_add_relations_with_copied_issues
Chris@1494 3794 @request.session[:user_id] = 2
Chris@1494 3795
Chris@1494 3796 assert_difference 'Issue.count', 2 do
Chris@1494 3797 assert_difference 'IssueRelation.count', 2 do
Chris@1494 3798 post :bulk_update, :ids => [1, 3], :copy => '1',
Chris@1494 3799 :issue => {
Chris@1494 3800 :project_id => '1'
Chris@1494 3801 }
Chris@1494 3802 end
Chris@1494 3803 end
Chris@1494 3804 end
Chris@1494 3805
Chris@1494 3806 def test_bulk_copy_should_allow_not_copying_the_subtasks
Chris@1494 3807 issue = Issue.generate_with_descendants!
Chris@1494 3808 @request.session[:user_id] = 2
Chris@1494 3809
Chris@1494 3810 assert_difference 'Issue.count', 1 do
Chris@1494 3811 post :bulk_update, :ids => [issue.id], :copy => '1',
Chris@1494 3812 :issue => {
Chris@1494 3813 :project_id => ''
Chris@1494 3814 }
Chris@1494 3815 end
Chris@1494 3816 end
Chris@1494 3817
Chris@1494 3818 def test_bulk_copy_should_allow_copying_the_subtasks
Chris@1494 3819 issue = Issue.generate_with_descendants!
Chris@1494 3820 count = issue.descendants.count
Chris@1494 3821 @request.session[:user_id] = 2
Chris@1494 3822
Chris@1494 3823 assert_difference 'Issue.count', count+1 do
Chris@1494 3824 post :bulk_update, :ids => [issue.id], :copy => '1', :copy_subtasks => '1',
Chris@1494 3825 :issue => {
Chris@1494 3826 :project_id => ''
Chris@1494 3827 }
Chris@1494 3828 end
Chris@1494 3829 copy = Issue.where(:parent_id => nil).order("id DESC").first
Chris@1494 3830 assert_equal count, copy.descendants.count
Chris@1494 3831 end
Chris@1494 3832
Chris@1494 3833 def test_bulk_copy_should_not_copy_selected_subtasks_twice
Chris@1494 3834 issue = Issue.generate_with_descendants!
Chris@1494 3835 count = issue.descendants.count
Chris@1494 3836 @request.session[:user_id] = 2
Chris@1494 3837
Chris@1494 3838 assert_difference 'Issue.count', count+1 do
Chris@1494 3839 post :bulk_update, :ids => issue.self_and_descendants.map(&:id), :copy => '1', :copy_subtasks => '1',
Chris@1494 3840 :issue => {
Chris@1494 3841 :project_id => ''
Chris@1494 3842 }
Chris@1494 3843 end
Chris@1494 3844 copy = Issue.where(:parent_id => nil).order("id DESC").first
Chris@1494 3845 assert_equal count, copy.descendants.count
Chris@1494 3846 end
Chris@1494 3847
Chris@1494 3848 def test_bulk_copy_to_another_project_should_follow_when_needed
Chris@1494 3849 @request.session[:user_id] = 2
Chris@1494 3850 post :bulk_update, :ids => [1], :copy => '1', :issue => {:project_id => 2}, :follow => '1'
Chris@1494 3851 issue = Issue.first(:order => 'id DESC')
Chris@1494 3852 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
Chris@1494 3853 end
Chris@1494 3854
Chris@1494 3855 def test_bulk_copy_with_all_failures_should_display_errors
Chris@1494 3856 @request.session[:user_id] = 2
Chris@1494 3857 post :bulk_update, :ids => [1, 2], :copy => '1', :issue => {:start_date => 'foo'}
Chris@1494 3858
Chris@1494 3859 assert_response :success
Chris@1494 3860 end
Chris@1494 3861
Chris@1494 3862 def test_destroy_issue_with_no_time_entries
Chris@1494 3863 assert_nil TimeEntry.find_by_issue_id(2)
Chris@1494 3864 @request.session[:user_id] = 2
Chris@1494 3865
Chris@1494 3866 assert_difference 'Issue.count', -1 do
Chris@1494 3867 delete :destroy, :id => 2
Chris@1494 3868 end
Chris@1494 3869 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@1494 3870 assert_nil Issue.find_by_id(2)
Chris@1494 3871 end
Chris@1494 3872
Chris@1494 3873 def test_destroy_issues_with_time_entries
Chris@1494 3874 @request.session[:user_id] = 2
Chris@1494 3875
Chris@1494 3876 assert_no_difference 'Issue.count' do
Chris@1494 3877 delete :destroy, :ids => [1, 3]
Chris@1494 3878 end
Chris@1494 3879 assert_response :success
Chris@1494 3880 assert_template 'destroy'
Chris@1494 3881 assert_not_nil assigns(:hours)
Chris@1494 3882 assert Issue.find_by_id(1) && Issue.find_by_id(3)
Chris@1494 3883
Chris@1494 3884 assert_select 'form' do
Chris@1494 3885 assert_select 'input[name=_method][value=delete]'
Chris@1494 3886 end
Chris@1494 3887 end
Chris@1494 3888
Chris@1494 3889 def test_destroy_issues_and_destroy_time_entries
Chris@1494 3890 @request.session[:user_id] = 2
Chris@1494 3891
Chris@1494 3892 assert_difference 'Issue.count', -2 do
Chris@1494 3893 assert_difference 'TimeEntry.count', -3 do
Chris@1494 3894 delete :destroy, :ids => [1, 3], :todo => 'destroy'
Chris@1494 3895 end
Chris@1494 3896 end
Chris@1494 3897 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@1494 3898 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
Chris@1494 3899 assert_nil TimeEntry.find_by_id([1, 2])
Chris@1494 3900 end
Chris@1494 3901
Chris@1494 3902 def test_destroy_issues_and_assign_time_entries_to_project
Chris@1494 3903 @request.session[:user_id] = 2
Chris@1494 3904
Chris@1494 3905 assert_difference 'Issue.count', -2 do
Chris@1494 3906 assert_no_difference 'TimeEntry.count' do
Chris@1494 3907 delete :destroy, :ids => [1, 3], :todo => 'nullify'
Chris@1494 3908 end
Chris@1494 3909 end
Chris@1494 3910 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@1494 3911 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
Chris@1494 3912 assert_nil TimeEntry.find(1).issue_id
Chris@1494 3913 assert_nil TimeEntry.find(2).issue_id
Chris@1494 3914 end
Chris@1494 3915
Chris@1494 3916 def test_destroy_issues_and_reassign_time_entries_to_another_issue
Chris@1494 3917 @request.session[:user_id] = 2
Chris@1494 3918
Chris@1494 3919 assert_difference 'Issue.count', -2 do
Chris@1494 3920 assert_no_difference 'TimeEntry.count' do
Chris@1494 3921 delete :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
Chris@1494 3922 end
Chris@1494 3923 end
Chris@1494 3924 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@1494 3925 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
Chris@1494 3926 assert_equal 2, TimeEntry.find(1).issue_id
Chris@1494 3927 assert_equal 2, TimeEntry.find(2).issue_id
Chris@1494 3928 end
Chris@1494 3929
Chris@1494 3930 def test_destroy_issues_from_different_projects
Chris@1494 3931 @request.session[:user_id] = 2
Chris@1494 3932
Chris@1494 3933 assert_difference 'Issue.count', -3 do
Chris@1494 3934 delete :destroy, :ids => [1, 2, 6], :todo => 'destroy'
Chris@1494 3935 end
Chris@1494 3936 assert_redirected_to :controller => 'issues', :action => 'index'
Chris@1494 3937 assert !(Issue.find_by_id(1) || Issue.find_by_id(2) || Issue.find_by_id(6))
Chris@1494 3938 end
Chris@1494 3939
Chris@1494 3940 def test_destroy_parent_and_child_issues
Chris@1494 3941 parent = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'Parent Issue')
Chris@1494 3942 child = Issue.create!(:project_id => 1, :author_id => 1, :tracker_id => 1, :subject => 'Child Issue', :parent_issue_id => parent.id)
Chris@1494 3943 assert child.is_descendant_of?(parent.reload)
Chris@1494 3944
Chris@1494 3945 @request.session[:user_id] = 2
Chris@1494 3946 assert_difference 'Issue.count', -2 do
Chris@1494 3947 delete :destroy, :ids => [parent.id, child.id], :todo => 'destroy'
Chris@1494 3948 end
Chris@1494 3949 assert_response 302
Chris@1494 3950 end
Chris@1494 3951
Chris@1494 3952 def test_destroy_invalid_should_respond_with_404
Chris@1494 3953 @request.session[:user_id] = 2
Chris@1494 3954 assert_no_difference 'Issue.count' do
Chris@1494 3955 delete :destroy, :id => 999
Chris@1494 3956 end
Chris@1494 3957 assert_response 404
Chris@1494 3958 end
Chris@1494 3959
Chris@1494 3960 def test_default_search_scope
Chris@1494 3961 get :index
Chris@1494 3962
Chris@1494 3963 assert_select 'div#quick-search form' do
Chris@1494 3964 assert_select 'input[name=issues][value=1][type=hidden]'
Chris@1494 3965 end
Chris@1494 3966 end
Chris@1494 3967 end