annotate .svn/pristine/36/36a71efadc331c3001a59098a85ebc847b7a529a.svn-base @ 1477:f2ad2199b49a bibplugin_integration

Close obsolete branch bibplugin_integration
author Chris Cannam
date Fri, 30 Nov 2012 14:41:31 +0000
parents cbb26bc654de
children
rev   line source
Chris@909 1 # Redmine - project management software
Chris@909 2 # Copyright (C) 2006-2011 Jean-Philippe Lang
Chris@909 3 #
Chris@909 4 # This program is free software; you can redistribute it and/or
Chris@909 5 # modify it under the terms of the GNU General Public License
Chris@909 6 # as published by the Free Software Foundation; either version 2
Chris@909 7 # of the License, or (at your option) any later version.
Chris@909 8 #
Chris@909 9 # This program is distributed in the hope that it will be useful,
Chris@909 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
Chris@909 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Chris@909 12 # GNU General Public License for more details.
Chris@909 13 #
Chris@909 14 # You should have received a copy of the GNU General Public License
Chris@909 15 # along with this program; if not, write to the Free Software
Chris@909 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
Chris@909 17
Chris@909 18 require File.expand_path('../../test_helper', __FILE__)
Chris@909 19 require 'issues_controller'
Chris@909 20
Chris@909 21 class IssuesControllerTest < ActionController::TestCase
Chris@909 22 fixtures :projects,
Chris@909 23 :users,
Chris@909 24 :roles,
Chris@909 25 :members,
Chris@909 26 :member_roles,
Chris@909 27 :issues,
Chris@909 28 :issue_statuses,
Chris@909 29 :versions,
Chris@909 30 :trackers,
Chris@909 31 :projects_trackers,
Chris@909 32 :issue_categories,
Chris@909 33 :enabled_modules,
Chris@909 34 :enumerations,
Chris@909 35 :attachments,
Chris@909 36 :workflows,
Chris@909 37 :custom_fields,
Chris@909 38 :custom_values,
Chris@909 39 :custom_fields_projects,
Chris@909 40 :custom_fields_trackers,
Chris@909 41 :time_entries,
Chris@909 42 :journals,
Chris@909 43 :journal_details,
Chris@909 44 :queries
Chris@909 45
Chris@909 46 include Redmine::I18n
Chris@909 47
Chris@909 48 def setup
Chris@909 49 @controller = IssuesController.new
Chris@909 50 @request = ActionController::TestRequest.new
Chris@909 51 @response = ActionController::TestResponse.new
Chris@909 52 User.current = nil
Chris@909 53 end
Chris@909 54
Chris@909 55 def test_index
Chris@909 56 Setting.default_language = 'en'
Chris@909 57
Chris@909 58 get :index
Chris@909 59 assert_response :success
Chris@909 60 assert_template 'index'
Chris@909 61 assert_not_nil assigns(:issues)
Chris@909 62 assert_nil assigns(:project)
Chris@909 63 assert_tag :tag => 'a', :content => /Can't print recipes/
Chris@909 64 assert_tag :tag => 'a', :content => /Subproject issue/
Chris@909 65 # private projects hidden
Chris@909 66 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
Chris@909 67 assert_no_tag :tag => 'a', :content => /Issue on project 2/
Chris@909 68 # project column
Chris@909 69 assert_tag :tag => 'th', :content => /Project/
Chris@909 70 end
Chris@909 71
Chris@909 72 def test_index_should_not_list_issues_when_module_disabled
Chris@909 73 EnabledModule.delete_all("name = 'issue_tracking' AND project_id = 1")
Chris@909 74 get :index
Chris@909 75 assert_response :success
Chris@909 76 assert_template 'index'
Chris@909 77 assert_not_nil assigns(:issues)
Chris@909 78 assert_nil assigns(:project)
Chris@909 79 assert_no_tag :tag => 'a', :content => /Can't print recipes/
Chris@909 80 assert_tag :tag => 'a', :content => /Subproject issue/
Chris@909 81 end
Chris@909 82
Chris@909 83 def test_index_should_list_visible_issues_only
Chris@909 84 get :index, :per_page => 100
Chris@909 85 assert_response :success
Chris@909 86 assert_not_nil assigns(:issues)
Chris@909 87 assert_nil assigns(:issues).detect {|issue| !issue.visible?}
Chris@909 88 end
Chris@909 89
Chris@909 90 def test_index_with_project
Chris@909 91 Setting.display_subprojects_issues = 0
Chris@909 92 get :index, :project_id => 1
Chris@909 93 assert_response :success
Chris@909 94 assert_template 'index'
Chris@909 95 assert_not_nil assigns(:issues)
Chris@909 96 assert_tag :tag => 'a', :content => /Can't print recipes/
Chris@909 97 assert_no_tag :tag => 'a', :content => /Subproject issue/
Chris@909 98 end
Chris@909 99
Chris@909 100 def test_index_with_project_and_subprojects
Chris@909 101 Setting.display_subprojects_issues = 1
Chris@909 102 get :index, :project_id => 1
Chris@909 103 assert_response :success
Chris@909 104 assert_template 'index'
Chris@909 105 assert_not_nil assigns(:issues)
Chris@909 106 assert_tag :tag => 'a', :content => /Can't print recipes/
Chris@909 107 assert_tag :tag => 'a', :content => /Subproject issue/
Chris@909 108 assert_no_tag :tag => 'a', :content => /Issue of a private subproject/
Chris@909 109 end
Chris@909 110
Chris@909 111 def test_index_with_project_and_subprojects_should_show_private_subprojects
Chris@909 112 @request.session[:user_id] = 2
Chris@909 113 Setting.display_subprojects_issues = 1
Chris@909 114 get :index, :project_id => 1
Chris@909 115 assert_response :success
Chris@909 116 assert_template 'index'
Chris@909 117 assert_not_nil assigns(:issues)
Chris@909 118 assert_tag :tag => 'a', :content => /Can't print recipes/
Chris@909 119 assert_tag :tag => 'a', :content => /Subproject issue/
Chris@909 120 assert_tag :tag => 'a', :content => /Issue of a private subproject/
Chris@909 121 end
Chris@909 122
Chris@909 123 def test_index_with_project_and_default_filter
Chris@909 124 get :index, :project_id => 1, :set_filter => 1
Chris@909 125 assert_response :success
Chris@909 126 assert_template 'index'
Chris@909 127 assert_not_nil assigns(:issues)
Chris@909 128
Chris@909 129 query = assigns(:query)
Chris@909 130 assert_not_nil query
Chris@909 131 # default filter
Chris@909 132 assert_equal({'status_id' => {:operator => 'o', :values => ['']}}, query.filters)
Chris@909 133 end
Chris@909 134
Chris@909 135 def test_index_with_project_and_filter
Chris@909 136 get :index, :project_id => 1, :set_filter => 1,
Chris@909 137 :f => ['tracker_id'],
Chris@909 138 :op => {'tracker_id' => '='},
Chris@909 139 :v => {'tracker_id' => ['1']}
Chris@909 140 assert_response :success
Chris@909 141 assert_template 'index'
Chris@909 142 assert_not_nil assigns(:issues)
Chris@909 143
Chris@909 144 query = assigns(:query)
Chris@909 145 assert_not_nil query
Chris@909 146 assert_equal({'tracker_id' => {:operator => '=', :values => ['1']}}, query.filters)
Chris@909 147 end
Chris@909 148
Chris@909 149 def test_index_with_short_filters
Chris@909 150
Chris@909 151 to_test = {
Chris@909 152 'status_id' => {
Chris@909 153 'o' => { :op => 'o', :values => [''] },
Chris@909 154 'c' => { :op => 'c', :values => [''] },
Chris@909 155 '7' => { :op => '=', :values => ['7'] },
Chris@909 156 '7|3|4' => { :op => '=', :values => ['7', '3', '4'] },
Chris@909 157 '=7' => { :op => '=', :values => ['7'] },
Chris@909 158 '!3' => { :op => '!', :values => ['3'] },
Chris@909 159 '!7|3|4' => { :op => '!', :values => ['7', '3', '4'] }},
Chris@909 160 'subject' => {
Chris@909 161 'This is a subject' => { :op => '=', :values => ['This is a subject'] },
Chris@909 162 'o' => { :op => '=', :values => ['o'] },
Chris@909 163 '~This is part of a subject' => { :op => '~', :values => ['This is part of a subject'] },
Chris@909 164 '!~This is part of a subject' => { :op => '!~', :values => ['This is part of a subject'] }},
Chris@909 165 'tracker_id' => {
Chris@909 166 '3' => { :op => '=', :values => ['3'] },
Chris@909 167 '=3' => { :op => '=', :values => ['3'] }},
Chris@909 168 'start_date' => {
Chris@909 169 '2011-10-12' => { :op => '=', :values => ['2011-10-12'] },
Chris@909 170 '=2011-10-12' => { :op => '=', :values => ['2011-10-12'] },
Chris@909 171 '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] },
Chris@909 172 '<=2011-10-12' => { :op => '<=', :values => ['2011-10-12'] },
Chris@909 173 '><2011-10-01|2011-10-30' => { :op => '><', :values => ['2011-10-01', '2011-10-30'] },
Chris@909 174 '<t+2' => { :op => '<t+', :values => ['2'] },
Chris@909 175 '>t+2' => { :op => '>t+', :values => ['2'] },
Chris@909 176 't+2' => { :op => 't+', :values => ['2'] },
Chris@909 177 't' => { :op => 't', :values => [''] },
Chris@909 178 'w' => { :op => 'w', :values => [''] },
Chris@909 179 '>t-2' => { :op => '>t-', :values => ['2'] },
Chris@909 180 '<t-2' => { :op => '<t-', :values => ['2'] },
Chris@909 181 't-2' => { :op => 't-', :values => ['2'] }},
Chris@909 182 'created_on' => {
Chris@909 183 '>=2011-10-12' => { :op => '>=', :values => ['2011-10-12'] },
Chris@909 184 '<t+2' => { :op => '=', :values => ['<t+2'] },
Chris@909 185 '>t+2' => { :op => '=', :values => ['>t+2'] },
Chris@909 186 't+2' => { :op => 't', :values => ['+2'] }},
Chris@909 187 'cf_1' => {
Chris@909 188 'c' => { :op => '=', :values => ['c'] },
Chris@909 189 '!c' => { :op => '!', :values => ['c'] },
Chris@909 190 '!*' => { :op => '!*', :values => [''] },
Chris@909 191 '*' => { :op => '*', :values => [''] }},
Chris@909 192 'estimated_hours' => {
Chris@909 193 '=13.4' => { :op => '=', :values => ['13.4'] },
Chris@909 194 '>=45' => { :op => '>=', :values => ['45'] },
Chris@909 195 '<=125' => { :op => '<=', :values => ['125'] },
Chris@909 196 '><10.5|20.5' => { :op => '><', :values => ['10.5', '20.5'] },
Chris@909 197 '!*' => { :op => '!*', :values => [''] },
Chris@909 198 '*' => { :op => '*', :values => [''] }}
Chris@909 199 }
Chris@909 200
Chris@909 201 default_filter = { 'status_id' => {:operator => 'o', :values => [''] }}
Chris@909 202
Chris@909 203 to_test.each do |field, expression_and_expected|
Chris@909 204 expression_and_expected.each do |filter_expression, expected|
Chris@909 205
Chris@909 206 get :index, :set_filter => 1, field => filter_expression
Chris@909 207
Chris@909 208 assert_response :success
Chris@909 209 assert_template 'index'
Chris@909 210 assert_not_nil assigns(:issues)
Chris@909 211
Chris@909 212 query = assigns(:query)
Chris@909 213 assert_not_nil query
Chris@909 214 assert query.has_filter?(field)
Chris@909 215 assert_equal(default_filter.merge({field => {:operator => expected[:op], :values => expected[:values]}}), query.filters)
Chris@909 216 end
Chris@909 217 end
Chris@909 218
Chris@909 219 end
Chris@909 220
Chris@909 221 def test_index_with_project_and_empty_filters
Chris@909 222 get :index, :project_id => 1, :set_filter => 1, :fields => ['']
Chris@909 223 assert_response :success
Chris@909 224 assert_template 'index'
Chris@909 225 assert_not_nil assigns(:issues)
Chris@909 226
Chris@909 227 query = assigns(:query)
Chris@909 228 assert_not_nil query
Chris@909 229 # no filter
Chris@909 230 assert_equal({}, query.filters)
Chris@909 231 end
Chris@909 232
Chris@909 233 def test_index_with_query
Chris@909 234 get :index, :project_id => 1, :query_id => 5
Chris@909 235 assert_response :success
Chris@909 236 assert_template 'index'
Chris@909 237 assert_not_nil assigns(:issues)
Chris@909 238 assert_nil assigns(:issue_count_by_group)
Chris@909 239 end
Chris@909 240
Chris@909 241 def test_index_with_query_grouped_by_tracker
Chris@909 242 get :index, :project_id => 1, :query_id => 6
Chris@909 243 assert_response :success
Chris@909 244 assert_template 'index'
Chris@909 245 assert_not_nil assigns(:issues)
Chris@909 246 assert_not_nil assigns(:issue_count_by_group)
Chris@909 247 end
Chris@909 248
Chris@909 249 def test_index_with_query_grouped_by_list_custom_field
Chris@909 250 get :index, :project_id => 1, :query_id => 9
Chris@909 251 assert_response :success
Chris@909 252 assert_template 'index'
Chris@909 253 assert_not_nil assigns(:issues)
Chris@909 254 assert_not_nil assigns(:issue_count_by_group)
Chris@909 255 end
Chris@909 256
Chris@909 257 def test_index_with_query_id_and_project_id_should_set_session_query
Chris@909 258 get :index, :project_id => 1, :query_id => 4
Chris@909 259 assert_response :success
Chris@909 260 assert_kind_of Hash, session[:query]
Chris@909 261 assert_equal 4, session[:query][:id]
Chris@909 262 assert_equal 1, session[:query][:project_id]
Chris@909 263 end
Chris@909 264
Chris@909 265 def test_index_with_cross_project_query_in_session_should_show_project_issues
Chris@909 266 q = Query.create!(:name => "test", :user_id => 2, :is_public => false, :project => nil)
Chris@909 267 @request.session[:query] = {:id => q.id, :project_id => 1}
Chris@909 268
Chris@909 269 with_settings :display_subprojects_issues => '0' do
Chris@909 270 get :index, :project_id => 1
Chris@909 271 end
Chris@909 272 assert_response :success
Chris@909 273 assert_not_nil assigns(:query)
Chris@909 274 assert_equal q.id, assigns(:query).id
Chris@909 275 assert_equal 1, assigns(:query).project_id
Chris@909 276 assert_equal [1], assigns(:issues).map(&:project_id).uniq
Chris@909 277 end
Chris@909 278
Chris@909 279 def test_private_query_should_not_be_available_to_other_users
Chris@909 280 q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil)
Chris@909 281 @request.session[:user_id] = 3
Chris@909 282
Chris@909 283 get :index, :query_id => q.id
Chris@909 284 assert_response 403
Chris@909 285 end
Chris@909 286
Chris@909 287 def test_private_query_should_be_available_to_its_user
Chris@909 288 q = Query.create!(:name => "private", :user => User.find(2), :is_public => false, :project => nil)
Chris@909 289 @request.session[:user_id] = 2
Chris@909 290
Chris@909 291 get :index, :query_id => q.id
Chris@909 292 assert_response :success
Chris@909 293 end
Chris@909 294
Chris@909 295 def test_public_query_should_be_available_to_other_users
Chris@909 296 q = Query.create!(:name => "private", :user => User.find(2), :is_public => true, :project => nil)
Chris@909 297 @request.session[:user_id] = 3
Chris@909 298
Chris@909 299 get :index, :query_id => q.id
Chris@909 300 assert_response :success
Chris@909 301 end
Chris@909 302
Chris@909 303 def test_index_csv
Chris@909 304 get :index, :format => 'csv'
Chris@909 305 assert_response :success
Chris@909 306 assert_not_nil assigns(:issues)
Chris@909 307 assert_equal 'text/csv', @response.content_type
Chris@909 308 assert @response.body.starts_with?("#,")
Chris@909 309 lines = @response.body.chomp.split("\n")
Chris@909 310 assert_equal assigns(:query).columns.size + 1, lines[0].split(',').size
Chris@909 311 end
Chris@909 312
Chris@909 313 def test_index_csv_with_project
Chris@909 314 get :index, :project_id => 1, :format => 'csv'
Chris@909 315 assert_response :success
Chris@909 316 assert_not_nil assigns(:issues)
Chris@909 317 assert_equal 'text/csv', @response.content_type
Chris@909 318 end
Chris@909 319
Chris@909 320 def test_index_csv_with_description
Chris@909 321 get :index, :format => 'csv', :description => '1'
Chris@909 322 assert_response :success
Chris@909 323 assert_not_nil assigns(:issues)
Chris@909 324 assert_equal 'text/csv', @response.content_type
Chris@909 325 assert @response.body.starts_with?("#,")
Chris@909 326 lines = @response.body.chomp.split("\n")
Chris@909 327 assert_equal assigns(:query).columns.size + 2, lines[0].split(',').size
Chris@909 328 end
Chris@909 329
Chris@909 330 def test_index_csv_with_all_columns
Chris@909 331 get :index, :format => 'csv', :columns => 'all'
Chris@909 332 assert_response :success
Chris@909 333 assert_not_nil assigns(:issues)
Chris@909 334 assert_equal 'text/csv', @response.content_type
Chris@909 335 assert @response.body.starts_with?("#,")
Chris@909 336 lines = @response.body.chomp.split("\n")
Chris@909 337 assert_equal assigns(:query).available_columns.size + 1, lines[0].split(',').size
Chris@909 338 end
Chris@909 339
Chris@909 340 def test_index_csv_big_5
Chris@909 341 with_settings :default_language => "zh-TW" do
Chris@909 342 str_utf8 = "\xe4\xb8\x80\xe6\x9c\x88"
Chris@909 343 str_big5 = "\xa4@\xa4\xeb"
Chris@909 344 if str_utf8.respond_to?(:force_encoding)
Chris@909 345 str_utf8.force_encoding('UTF-8')
Chris@909 346 str_big5.force_encoding('Big5')
Chris@909 347 end
Chris@909 348 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@909 349 :status_id => 1, :priority => IssuePriority.all.first,
Chris@909 350 :subject => str_utf8)
Chris@909 351 assert issue.save
Chris@909 352
Chris@909 353 get :index, :project_id => 1,
Chris@909 354 :f => ['subject'],
Chris@909 355 :op => '=', :values => [str_utf8],
Chris@909 356 :format => 'csv'
Chris@909 357 assert_equal 'text/csv', @response.content_type
Chris@909 358 lines = @response.body.chomp.split("\n")
Chris@909 359 s1 = "\xaa\xac\xbaA"
Chris@909 360 if str_utf8.respond_to?(:force_encoding)
Chris@909 361 s1.force_encoding('Big5')
Chris@909 362 end
Chris@909 363 assert lines[0].include?(s1)
Chris@909 364 assert lines[1].include?(str_big5)
Chris@909 365 end
Chris@909 366 end
Chris@909 367
Chris@909 368 def test_index_csv_cannot_convert_should_be_replaced_big_5
Chris@909 369 with_settings :default_language => "zh-TW" do
Chris@909 370 str_utf8 = "\xe4\xbb\xa5\xe5\x86\x85"
Chris@909 371 if str_utf8.respond_to?(:force_encoding)
Chris@909 372 str_utf8.force_encoding('UTF-8')
Chris@909 373 end
Chris@909 374 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@909 375 :status_id => 1, :priority => IssuePriority.all.first,
Chris@909 376 :subject => str_utf8)
Chris@909 377 assert issue.save
Chris@909 378
Chris@909 379 get :index, :project_id => 1,
Chris@909 380 :f => ['subject'],
Chris@909 381 :op => '=', :values => [str_utf8],
Chris@909 382 :c => ['status', 'subject'],
Chris@909 383 :format => 'csv',
Chris@909 384 :set_filter => 1
Chris@909 385 assert_equal 'text/csv', @response.content_type
Chris@909 386 lines = @response.body.chomp.split("\n")
Chris@909 387 s1 = "\xaa\xac\xbaA" # status
Chris@909 388 if str_utf8.respond_to?(:force_encoding)
Chris@909 389 s1.force_encoding('Big5')
Chris@909 390 end
Chris@909 391 assert lines[0].include?(s1)
Chris@909 392 s2 = lines[1].split(",")[2]
Chris@909 393 if s1.respond_to?(:force_encoding)
Chris@909 394 s3 = "\xa5H?" # subject
Chris@909 395 s3.force_encoding('Big5')
Chris@909 396 assert_equal s3, s2
Chris@909 397 elsif RUBY_PLATFORM == 'java'
Chris@909 398 assert_equal "??", s2
Chris@909 399 else
Chris@909 400 assert_equal "\xa5H???", s2
Chris@909 401 end
Chris@909 402 end
Chris@909 403 end
Chris@909 404
Chris@909 405 def test_index_csv_tw
Chris@909 406 with_settings :default_language => "zh-TW" do
Chris@909 407 str1 = "test_index_csv_tw"
Chris@909 408 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@909 409 :status_id => 1, :priority => IssuePriority.all.first,
Chris@909 410 :subject => str1, :estimated_hours => '1234.5')
Chris@909 411 assert issue.save
Chris@909 412 assert_equal 1234.5, issue.estimated_hours
Chris@909 413
Chris@909 414 get :index, :project_id => 1,
Chris@909 415 :f => ['subject'],
Chris@909 416 :op => '=', :values => [str1],
Chris@909 417 :c => ['estimated_hours', 'subject'],
Chris@909 418 :format => 'csv',
Chris@909 419 :set_filter => 1
Chris@909 420 assert_equal 'text/csv', @response.content_type
Chris@909 421 lines = @response.body.chomp.split("\n")
Chris@909 422 assert_equal "#{issue.id},1234.5,#{str1}", lines[1]
Chris@909 423
Chris@909 424 str_tw = "Traditional Chinese (\xe7\xb9\x81\xe9\xab\x94\xe4\xb8\xad\xe6\x96\x87)"
Chris@909 425 if str_tw.respond_to?(:force_encoding)
Chris@909 426 str_tw.force_encoding('UTF-8')
Chris@909 427 end
Chris@909 428 assert_equal str_tw, l(:general_lang_name)
Chris@909 429 assert_equal ',', l(:general_csv_separator)
Chris@909 430 assert_equal '.', l(:general_csv_decimal_separator)
Chris@909 431 end
Chris@909 432 end
Chris@909 433
Chris@909 434 def test_index_csv_fr
Chris@909 435 with_settings :default_language => "fr" do
Chris@909 436 str1 = "test_index_csv_fr"
Chris@909 437 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@909 438 :status_id => 1, :priority => IssuePriority.all.first,
Chris@909 439 :subject => str1, :estimated_hours => '1234.5')
Chris@909 440 assert issue.save
Chris@909 441 assert_equal 1234.5, issue.estimated_hours
Chris@909 442
Chris@909 443 get :index, :project_id => 1,
Chris@909 444 :f => ['subject'],
Chris@909 445 :op => '=', :values => [str1],
Chris@909 446 :c => ['estimated_hours', 'subject'],
Chris@909 447 :format => 'csv',
Chris@909 448 :set_filter => 1
Chris@909 449 assert_equal 'text/csv', @response.content_type
Chris@909 450 lines = @response.body.chomp.split("\n")
Chris@909 451 assert_equal "#{issue.id};1234,5;#{str1}", lines[1]
Chris@909 452
Chris@909 453 str_fr = "Fran\xc3\xa7ais"
Chris@909 454 if str_fr.respond_to?(:force_encoding)
Chris@909 455 str_fr.force_encoding('UTF-8')
Chris@909 456 end
Chris@909 457 assert_equal str_fr, l(:general_lang_name)
Chris@909 458 assert_equal ';', l(:general_csv_separator)
Chris@909 459 assert_equal ',', l(:general_csv_decimal_separator)
Chris@909 460 end
Chris@909 461 end
Chris@909 462
Chris@909 463 def test_index_pdf
Chris@909 464 ["en", "zh", "zh-TW", "ja", "ko"].each do |lang|
Chris@909 465 with_settings :default_language => lang do
Chris@909 466
Chris@909 467 get :index
Chris@909 468 assert_response :success
Chris@909 469 assert_template 'index'
Chris@909 470
Chris@909 471 if lang == "ja"
Chris@909 472 if RUBY_PLATFORM != 'java'
Chris@909 473 assert_equal "CP932", l(:general_pdf_encoding)
Chris@909 474 end
Chris@909 475 if RUBY_PLATFORM == 'java' && l(:general_pdf_encoding) == "CP932"
Chris@909 476 next
Chris@909 477 end
Chris@909 478 end
Chris@909 479
Chris@909 480 get :index, :format => 'pdf'
Chris@909 481 assert_response :success
Chris@909 482 assert_not_nil assigns(:issues)
Chris@909 483 assert_equal 'application/pdf', @response.content_type
Chris@909 484
Chris@909 485 get :index, :project_id => 1, :format => 'pdf'
Chris@909 486 assert_response :success
Chris@909 487 assert_not_nil assigns(:issues)
Chris@909 488 assert_equal 'application/pdf', @response.content_type
Chris@909 489
Chris@909 490 get :index, :project_id => 1, :query_id => 6, :format => 'pdf'
Chris@909 491 assert_response :success
Chris@909 492 assert_not_nil assigns(:issues)
Chris@909 493 assert_equal 'application/pdf', @response.content_type
Chris@909 494 end
Chris@909 495 end
Chris@909 496 end
Chris@909 497
Chris@909 498 def test_index_pdf_with_query_grouped_by_list_custom_field
Chris@909 499 get :index, :project_id => 1, :query_id => 9, :format => 'pdf'
Chris@909 500 assert_response :success
Chris@909 501 assert_not_nil assigns(:issues)
Chris@909 502 assert_not_nil assigns(:issue_count_by_group)
Chris@909 503 assert_equal 'application/pdf', @response.content_type
Chris@909 504 end
Chris@909 505
Chris@909 506 def test_index_sort
Chris@909 507 get :index, :sort => 'tracker,id:desc'
Chris@909 508 assert_response :success
Chris@909 509
Chris@909 510 sort_params = @request.session['issues_index_sort']
Chris@909 511 assert sort_params.is_a?(String)
Chris@909 512 assert_equal 'tracker,id:desc', sort_params
Chris@909 513
Chris@909 514 issues = assigns(:issues)
Chris@909 515 assert_not_nil issues
Chris@909 516 assert !issues.empty?
Chris@909 517 assert_equal issues.sort {|a,b| a.tracker == b.tracker ? b.id <=> a.id : a.tracker <=> b.tracker }.collect(&:id), issues.collect(&:id)
Chris@909 518 end
Chris@909 519
Chris@909 520 def test_index_sort_by_field_not_included_in_columns
Chris@909 521 Setting.issue_list_default_columns = %w(subject author)
Chris@909 522 get :index, :sort => 'tracker'
Chris@909 523 end
Chris@909 524
Chris@909 525 def test_index_sort_by_assigned_to
Chris@909 526 get :index, :sort => 'assigned_to'
Chris@909 527 assert_response :success
Chris@909 528 assignees = assigns(:issues).collect(&:assigned_to).compact
Chris@909 529 assert_equal assignees.sort, assignees
Chris@909 530 end
Chris@909 531
Chris@909 532 def test_index_sort_by_assigned_to_desc
Chris@909 533 get :index, :sort => 'assigned_to:desc'
Chris@909 534 assert_response :success
Chris@909 535 assignees = assigns(:issues).collect(&:assigned_to).compact
Chris@909 536 assert_equal assignees.sort.reverse, assignees
Chris@909 537 end
Chris@909 538
Chris@909 539 def test_index_group_by_assigned_to
Chris@909 540 get :index, :group_by => 'assigned_to', :sort => 'priority'
Chris@909 541 assert_response :success
Chris@909 542 end
Chris@909 543
Chris@909 544 def test_index_sort_by_author
Chris@909 545 get :index, :sort => 'author'
Chris@909 546 assert_response :success
Chris@909 547 authors = assigns(:issues).collect(&:author)
Chris@909 548 assert_equal authors.sort, authors
Chris@909 549 end
Chris@909 550
Chris@909 551 def test_index_sort_by_author_desc
Chris@909 552 get :index, :sort => 'author:desc'
Chris@909 553 assert_response :success
Chris@909 554 authors = assigns(:issues).collect(&:author)
Chris@909 555 assert_equal authors.sort.reverse, authors
Chris@909 556 end
Chris@909 557
Chris@909 558 def test_index_group_by_author
Chris@909 559 get :index, :group_by => 'author', :sort => 'priority'
Chris@909 560 assert_response :success
Chris@909 561 end
Chris@909 562
Chris@909 563 def test_index_with_columns
Chris@909 564 columns = ['tracker', 'subject', 'assigned_to']
Chris@909 565 get :index, :set_filter => 1, :c => columns
Chris@909 566 assert_response :success
Chris@909 567
Chris@909 568 # query should use specified columns
Chris@909 569 query = assigns(:query)
Chris@909 570 assert_kind_of Query, query
Chris@909 571 assert_equal columns, query.column_names.map(&:to_s)
Chris@909 572
Chris@909 573 # columns should be stored in session
Chris@909 574 assert_kind_of Hash, session[:query]
Chris@909 575 assert_kind_of Array, session[:query][:column_names]
Chris@909 576 assert_equal columns, session[:query][:column_names].map(&:to_s)
Chris@909 577
Chris@909 578 # ensure only these columns are kept in the selected columns list
Chris@909 579 assert_tag :tag => 'select', :attributes => { :id => 'selected_columns' },
Chris@909 580 :children => { :count => 3 }
Chris@909 581 assert_no_tag :tag => 'option', :attributes => { :value => 'project' },
Chris@909 582 :parent => { :tag => 'select', :attributes => { :id => "selected_columns" } }
Chris@909 583 end
Chris@909 584
Chris@909 585 def test_index_without_project_should_implicitly_add_project_column_to_default_columns
Chris@909 586 Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to']
Chris@909 587 get :index, :set_filter => 1
Chris@909 588
Chris@909 589 # query should use specified columns
Chris@909 590 query = assigns(:query)
Chris@909 591 assert_kind_of Query, query
Chris@909 592 assert_equal [:project, :tracker, :subject, :assigned_to], query.columns.map(&:name)
Chris@909 593 end
Chris@909 594
Chris@909 595 def test_index_without_project_and_explicit_default_columns_should_not_add_project_column
Chris@909 596 Setting.issue_list_default_columns = ['tracker', 'subject', 'assigned_to']
Chris@909 597 columns = ['tracker', 'subject', 'assigned_to']
Chris@909 598 get :index, :set_filter => 1, :c => columns
Chris@909 599
Chris@909 600 # query should use specified columns
Chris@909 601 query = assigns(:query)
Chris@909 602 assert_kind_of Query, query
Chris@909 603 assert_equal columns.map(&:to_sym), query.columns.map(&:name)
Chris@909 604 end
Chris@909 605
Chris@909 606 def test_index_with_custom_field_column
Chris@909 607 columns = %w(tracker subject cf_2)
Chris@909 608 get :index, :set_filter => 1, :c => columns
Chris@909 609 assert_response :success
Chris@909 610
Chris@909 611 # query should use specified columns
Chris@909 612 query = assigns(:query)
Chris@909 613 assert_kind_of Query, query
Chris@909 614 assert_equal columns, query.column_names.map(&:to_s)
Chris@909 615
Chris@909 616 assert_tag :td,
Chris@909 617 :attributes => {:class => 'cf_2 string'},
Chris@909 618 :ancestor => {:tag => 'table', :attributes => {:class => /issues/}}
Chris@909 619 end
Chris@909 620
Chris@909 621 def test_index_with_date_column
Chris@909 622 Issue.find(1).update_attribute :start_date, '1987-08-24'
Chris@909 623
Chris@909 624 with_settings :date_format => '%d/%m/%Y' do
Chris@909 625 get :index, :set_filter => 1, :c => %w(start_date)
Chris@909 626 assert_tag 'td', :attributes => {:class => /start_date/}, :content => '24/08/1987'
Chris@909 627 end
Chris@909 628 end
Chris@909 629
Chris@909 630 def test_index_with_done_ratio
Chris@909 631 Issue.find(1).update_attribute :done_ratio, 40
Chris@909 632
Chris@909 633 get :index, :set_filter => 1, :c => %w(done_ratio)
Chris@909 634 assert_tag 'td', :attributes => {:class => /done_ratio/},
Chris@909 635 :child => {:tag => 'table', :attributes => {:class => 'progress'},
Chris@909 636 :descendant => {:tag => 'td', :attributes => {:class => 'closed', :style => 'width: 40%;'}}
Chris@909 637 }
Chris@909 638 end
Chris@909 639
Chris@909 640 def test_index_with_fixed_version
Chris@909 641 get :index, :set_filter => 1, :c => %w(fixed_version)
Chris@909 642 assert_tag 'td', :attributes => {:class => /fixed_version/},
Chris@909 643 :child => {:tag => 'a', :content => '1.0', :attributes => {:href => '/versions/2'}}
Chris@909 644 end
Chris@909 645
Chris@909 646 def test_index_send_html_if_query_is_invalid
Chris@909 647 get :index, :f => ['start_date'], :op => {:start_date => '='}
Chris@909 648 assert_equal 'text/html', @response.content_type
Chris@909 649 assert_template 'index'
Chris@909 650 end
Chris@909 651
Chris@909 652 def test_index_send_nothing_if_query_is_invalid
Chris@909 653 get :index, :f => ['start_date'], :op => {:start_date => '='}, :format => 'csv'
Chris@909 654 assert_equal 'text/csv', @response.content_type
Chris@909 655 assert @response.body.blank?
Chris@909 656 end
Chris@909 657
Chris@909 658 def test_show_by_anonymous
Chris@909 659 get :show, :id => 1
Chris@909 660 assert_response :success
Chris@909 661 assert_template 'show'
Chris@909 662 assert_not_nil assigns(:issue)
Chris@909 663 assert_equal Issue.find(1), assigns(:issue)
Chris@909 664
Chris@909 665 # anonymous role is allowed to add a note
Chris@909 666 assert_tag :tag => 'form',
Chris@909 667 :descendant => { :tag => 'fieldset',
Chris@909 668 :child => { :tag => 'legend',
Chris@909 669 :content => /Notes/ } }
Chris@909 670 assert_tag :tag => 'title',
Chris@909 671 :content => "Bug #1: Can't print recipes - eCookbook - Redmine"
Chris@909 672 end
Chris@909 673
Chris@909 674 def test_show_by_manager
Chris@909 675 @request.session[:user_id] = 2
Chris@909 676 get :show, :id => 1
Chris@909 677 assert_response :success
Chris@909 678
Chris@909 679 assert_tag :tag => 'a',
Chris@909 680 :content => /Quote/
Chris@909 681
Chris@909 682 assert_tag :tag => 'form',
Chris@909 683 :descendant => { :tag => 'fieldset',
Chris@909 684 :child => { :tag => 'legend',
Chris@909 685 :content => /Change properties/ } },
Chris@909 686 :descendant => { :tag => 'fieldset',
Chris@909 687 :child => { :tag => 'legend',
Chris@909 688 :content => /Log time/ } },
Chris@909 689 :descendant => { :tag => 'fieldset',
Chris@909 690 :child => { :tag => 'legend',
Chris@909 691 :content => /Notes/ } }
Chris@909 692 end
Chris@909 693
Chris@909 694 def test_update_form_should_not_display_inactive_enumerations
Chris@909 695 @request.session[:user_id] = 2
Chris@909 696 get :show, :id => 1
Chris@909 697 assert_response :success
Chris@909 698
Chris@909 699 assert ! IssuePriority.find(15).active?
Chris@909 700 assert_no_tag :option, :attributes => {:value => '15'},
Chris@909 701 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
Chris@909 702 end
Chris@909 703
Chris@909 704 def test_update_form_should_allow_attachment_upload
Chris@909 705 @request.session[:user_id] = 2
Chris@909 706 get :show, :id => 1
Chris@909 707
Chris@909 708 assert_tag :tag => 'form',
Chris@909 709 :attributes => {:id => 'issue-form', :method => 'post', :enctype => 'multipart/form-data'},
Chris@909 710 :descendant => {
Chris@909 711 :tag => 'input',
Chris@909 712 :attributes => {:type => 'file', :name => 'attachments[1][file]'}
Chris@909 713 }
Chris@909 714 end
Chris@909 715
Chris@909 716 def test_show_should_deny_anonymous_access_without_permission
Chris@909 717 Role.anonymous.remove_permission!(:view_issues)
Chris@909 718 get :show, :id => 1
Chris@909 719 assert_response :redirect
Chris@909 720 end
Chris@909 721
Chris@909 722 def test_show_should_deny_anonymous_access_to_private_issue
Chris@909 723 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@909 724 get :show, :id => 1
Chris@909 725 assert_response :redirect
Chris@909 726 end
Chris@909 727
Chris@909 728 def test_show_should_deny_non_member_access_without_permission
Chris@909 729 Role.non_member.remove_permission!(:view_issues)
Chris@909 730 @request.session[:user_id] = 9
Chris@909 731 get :show, :id => 1
Chris@909 732 assert_response 403
Chris@909 733 end
Chris@909 734
Chris@909 735 def test_show_should_deny_non_member_access_to_private_issue
Chris@909 736 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@909 737 @request.session[:user_id] = 9
Chris@909 738 get :show, :id => 1
Chris@909 739 assert_response 403
Chris@909 740 end
Chris@909 741
Chris@909 742 def test_show_should_deny_member_access_without_permission
Chris@909 743 Role.find(1).remove_permission!(:view_issues)
Chris@909 744 @request.session[:user_id] = 2
Chris@909 745 get :show, :id => 1
Chris@909 746 assert_response 403
Chris@909 747 end
Chris@909 748
Chris@909 749 def test_show_should_deny_member_access_to_private_issue_without_permission
Chris@909 750 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@909 751 @request.session[:user_id] = 3
Chris@909 752 get :show, :id => 1
Chris@909 753 assert_response 403
Chris@909 754 end
Chris@909 755
Chris@909 756 def test_show_should_allow_author_access_to_private_issue
Chris@909 757 Issue.update_all(["is_private = ?, author_id = 3", true], "id = 1")
Chris@909 758 @request.session[:user_id] = 3
Chris@909 759 get :show, :id => 1
Chris@909 760 assert_response :success
Chris@909 761 end
Chris@909 762
Chris@909 763 def test_show_should_allow_assignee_access_to_private_issue
Chris@909 764 Issue.update_all(["is_private = ?, assigned_to_id = 3", true], "id = 1")
Chris@909 765 @request.session[:user_id] = 3
Chris@909 766 get :show, :id => 1
Chris@909 767 assert_response :success
Chris@909 768 end
Chris@909 769
Chris@909 770 def test_show_should_allow_member_access_to_private_issue_with_permission
Chris@909 771 Issue.update_all(["is_private = ?", true], "id = 1")
Chris@909 772 User.find(3).roles_for_project(Project.find(1)).first.update_attribute :issues_visibility, 'all'
Chris@909 773 @request.session[:user_id] = 3
Chris@909 774 get :show, :id => 1
Chris@909 775 assert_response :success
Chris@909 776 end
Chris@909 777
Chris@909 778 def test_show_should_not_disclose_relations_to_invisible_issues
Chris@909 779 Setting.cross_project_issue_relations = '1'
Chris@909 780 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(2), :relation_type => 'relates')
Chris@909 781 # Relation to a private project issue
Chris@909 782 IssueRelation.create!(:issue_from => Issue.find(1), :issue_to => Issue.find(4), :relation_type => 'relates')
Chris@909 783
Chris@909 784 get :show, :id => 1
Chris@909 785 assert_response :success
Chris@909 786
Chris@909 787 assert_tag :div, :attributes => { :id => 'relations' },
Chris@909 788 :descendant => { :tag => 'a', :content => /#2$/ }
Chris@909 789 assert_no_tag :div, :attributes => { :id => 'relations' },
Chris@909 790 :descendant => { :tag => 'a', :content => /#4$/ }
Chris@909 791 end
Chris@909 792
Chris@909 793 def test_show_atom
Chris@909 794 get :show, :id => 2, :format => 'atom'
Chris@909 795 assert_response :success
Chris@909 796 assert_template 'journals/index'
Chris@909 797 # Inline image
Chris@909 798 assert_select 'content', :text => Regexp.new(Regexp.quote('http://test.host/attachments/download/10'))
Chris@909 799 end
Chris@909 800
Chris@909 801 def test_show_export_to_pdf
Chris@909 802 get :show, :id => 3, :format => 'pdf'
Chris@909 803 assert_response :success
Chris@909 804 assert_equal 'application/pdf', @response.content_type
Chris@909 805 assert @response.body.starts_with?('%PDF')
Chris@909 806 assert_not_nil assigns(:issue)
Chris@909 807 end
Chris@909 808
Chris@909 809 def test_get_new
Chris@909 810 @request.session[:user_id] = 2
Chris@909 811 get :new, :project_id => 1, :tracker_id => 1
Chris@909 812 assert_response :success
Chris@909 813 assert_template 'new'
Chris@909 814
Chris@909 815 assert_tag :tag => 'input', :attributes => { :name => 'issue[custom_field_values][2]',
Chris@909 816 :value => 'Default string' }
Chris@909 817
Chris@909 818 # Be sure we don't display inactive IssuePriorities
Chris@909 819 assert ! IssuePriority.find(15).active?
Chris@909 820 assert_no_tag :option, :attributes => {:value => '15'},
Chris@909 821 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
Chris@909 822 end
Chris@909 823
Chris@909 824 def test_get_new_without_default_start_date_is_creation_date
Chris@909 825 Setting.default_issue_start_date_to_creation_date = 0
Chris@909 826
Chris@909 827 @request.session[:user_id] = 2
Chris@909 828 get :new, :project_id => 1, :tracker_id => 1
Chris@909 829 assert_response :success
Chris@909 830 assert_template 'new'
Chris@909 831
Chris@909 832 assert_tag :tag => 'input', :attributes => { :name => 'issue[start_date]',
Chris@909 833 :value => nil }
Chris@909 834 end
Chris@909 835
Chris@909 836 def test_get_new_with_default_start_date_is_creation_date
Chris@909 837 Setting.default_issue_start_date_to_creation_date = 1
Chris@909 838
Chris@909 839 @request.session[:user_id] = 2
Chris@909 840 get :new, :project_id => 1, :tracker_id => 1
Chris@909 841 assert_response :success
Chris@909 842 assert_template 'new'
Chris@909 843
Chris@909 844 assert_tag :tag => 'input', :attributes => { :name => 'issue[start_date]',
Chris@909 845 :value => Date.today.to_s }
Chris@909 846 end
Chris@909 847
Chris@909 848 def test_get_new_form_should_allow_attachment_upload
Chris@909 849 @request.session[:user_id] = 2
Chris@909 850 get :new, :project_id => 1, :tracker_id => 1
Chris@909 851
Chris@909 852 assert_tag :tag => 'form',
Chris@909 853 :attributes => {:id => 'issue-form', :method => 'post', :enctype => 'multipart/form-data'},
Chris@909 854 :descendant => {
Chris@909 855 :tag => 'input',
Chris@909 856 :attributes => {:type => 'file', :name => 'attachments[1][file]'}
Chris@909 857 }
Chris@909 858 end
Chris@909 859
Chris@909 860 def test_get_new_without_tracker_id
Chris@909 861 @request.session[:user_id] = 2
Chris@909 862 get :new, :project_id => 1
Chris@909 863 assert_response :success
Chris@909 864 assert_template 'new'
Chris@909 865
Chris@909 866 issue = assigns(:issue)
Chris@909 867 assert_not_nil issue
Chris@909 868 assert_equal Project.find(1).trackers.first, issue.tracker
Chris@909 869 end
Chris@909 870
Chris@909 871 def test_get_new_with_no_default_status_should_display_an_error
Chris@909 872 @request.session[:user_id] = 2
Chris@909 873 IssueStatus.delete_all
Chris@909 874
Chris@909 875 get :new, :project_id => 1
Chris@909 876 assert_response 500
Chris@909 877 assert_error_tag :content => /No default issue/
Chris@909 878 end
Chris@909 879
Chris@909 880 def test_get_new_with_no_tracker_should_display_an_error
Chris@909 881 @request.session[:user_id] = 2
Chris@909 882 Tracker.delete_all
Chris@909 883
Chris@909 884 get :new, :project_id => 1
Chris@909 885 assert_response 500
Chris@909 886 assert_error_tag :content => /No tracker/
Chris@909 887 end
Chris@909 888
Chris@909 889 def test_update_new_form
Chris@909 890 @request.session[:user_id] = 2
Chris@909 891 xhr :post, :new, :project_id => 1,
Chris@909 892 :issue => {:tracker_id => 2,
Chris@909 893 :subject => 'This is the test_new issue',
Chris@909 894 :description => 'This is the description',
Chris@909 895 :priority_id => 5}
Chris@909 896 assert_response :success
Chris@909 897 assert_template 'attributes'
Chris@909 898
Chris@909 899 issue = assigns(:issue)
Chris@909 900 assert_kind_of Issue, issue
Chris@909 901 assert_equal 1, issue.project_id
Chris@909 902 assert_equal 2, issue.tracker_id
Chris@909 903 assert_equal 'This is the test_new issue', issue.subject
Chris@909 904 end
Chris@909 905
Chris@909 906 def test_post_create
Chris@909 907 @request.session[:user_id] = 2
Chris@909 908 assert_difference 'Issue.count' do
Chris@909 909 post :create, :project_id => 1,
Chris@909 910 :issue => {:tracker_id => 3,
Chris@909 911 :status_id => 2,
Chris@909 912 :subject => 'This is the test_new issue',
Chris@909 913 :description => 'This is the description',
Chris@909 914 :priority_id => 5,
Chris@909 915 :start_date => '2010-11-07',
Chris@909 916 :estimated_hours => '',
Chris@909 917 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@909 918 end
Chris@909 919 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@909 920
Chris@909 921 issue = Issue.find_by_subject('This is the test_new issue')
Chris@909 922 assert_not_nil issue
Chris@909 923 assert_equal 2, issue.author_id
Chris@909 924 assert_equal 3, issue.tracker_id
Chris@909 925 assert_equal 2, issue.status_id
Chris@909 926 assert_equal Date.parse('2010-11-07'), issue.start_date
Chris@909 927 assert_nil issue.estimated_hours
Chris@909 928 v = issue.custom_values.find(:first, :conditions => {:custom_field_id => 2})
Chris@909 929 assert_not_nil v
Chris@909 930 assert_equal 'Value for field 2', v.value
Chris@909 931 end
Chris@909 932
Chris@909 933 def test_post_new_with_group_assignment
Chris@909 934 group = Group.find(11)
Chris@909 935 project = Project.find(1)
Chris@909 936 project.members << Member.new(:principal => group, :roles => [Role.first])
Chris@909 937
Chris@909 938 with_settings :issue_group_assignment => '1' do
Chris@909 939 @request.session[:user_id] = 2
Chris@909 940 assert_difference 'Issue.count' do
Chris@909 941 post :create, :project_id => project.id,
Chris@909 942 :issue => {:tracker_id => 3,
Chris@909 943 :status_id => 1,
Chris@909 944 :subject => 'This is the test_new_with_group_assignment issue',
Chris@909 945 :assigned_to_id => group.id}
Chris@909 946 end
Chris@909 947 end
Chris@909 948 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@909 949
Chris@909 950 issue = Issue.find_by_subject('This is the test_new_with_group_assignment issue')
Chris@909 951 assert_not_nil issue
Chris@909 952 assert_equal group, issue.assigned_to
Chris@909 953 end
Chris@909 954
Chris@909 955 def test_post_create_without_start_date_and_default_start_date_is_not_creation_date
Chris@909 956 Setting.default_issue_start_date_to_creation_date = 0
Chris@909 957
Chris@909 958 @request.session[:user_id] = 2
Chris@909 959 assert_difference 'Issue.count' do
Chris@909 960 post :create, :project_id => 1,
Chris@909 961 :issue => {:tracker_id => 3,
Chris@909 962 :status_id => 2,
Chris@909 963 :subject => 'This is the test_new issue',
Chris@909 964 :description => 'This is the description',
Chris@909 965 :priority_id => 5,
Chris@909 966 :estimated_hours => '',
Chris@909 967 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@909 968 end
Chris@909 969 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@909 970
Chris@909 971 issue = Issue.find_by_subject('This is the test_new issue')
Chris@909 972 assert_not_nil issue
Chris@909 973 assert_nil issue.start_date
Chris@909 974 end
Chris@909 975
Chris@909 976 def test_post_create_without_start_date_and_default_start_date_is_creation_date
Chris@909 977 Setting.default_issue_start_date_to_creation_date = 1
Chris@909 978
Chris@909 979 @request.session[:user_id] = 2
Chris@909 980 assert_difference 'Issue.count' do
Chris@909 981 post :create, :project_id => 1,
Chris@909 982 :issue => {:tracker_id => 3,
Chris@909 983 :status_id => 2,
Chris@909 984 :subject => 'This is the test_new issue',
Chris@909 985 :description => 'This is the description',
Chris@909 986 :priority_id => 5,
Chris@909 987 :estimated_hours => '',
Chris@909 988 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@909 989 end
Chris@909 990 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@909 991
Chris@909 992 issue = Issue.find_by_subject('This is the test_new issue')
Chris@909 993 assert_not_nil issue
Chris@909 994 assert_equal Date.today, issue.start_date
Chris@909 995 end
Chris@909 996
Chris@909 997 def test_post_create_and_continue
Chris@909 998 @request.session[:user_id] = 2
Chris@909 999 assert_difference 'Issue.count' do
Chris@909 1000 post :create, :project_id => 1,
Chris@909 1001 :issue => {:tracker_id => 3, :subject => 'This is first issue', :priority_id => 5},
Chris@909 1002 :continue => ''
Chris@909 1003 end
Chris@909 1004
Chris@909 1005 issue = Issue.first(:order => 'id DESC')
Chris@909 1006 assert_redirected_to :controller => 'issues', :action => 'new', :project_id => 'ecookbook', :issue => {:tracker_id => 3}
Chris@909 1007 assert_not_nil flash[:notice], "flash was not set"
Chris@909 1008 assert flash[:notice].include?("<a href='/issues/#{issue.id}'>##{issue.id}</a>"), "issue link not found in flash: #{flash[:notice]}"
Chris@909 1009 end
Chris@909 1010
Chris@909 1011 def test_post_create_without_custom_fields_param
Chris@909 1012 @request.session[:user_id] = 2
Chris@909 1013 assert_difference 'Issue.count' do
Chris@909 1014 post :create, :project_id => 1,
Chris@909 1015 :issue => {:tracker_id => 1,
Chris@909 1016 :subject => 'This is the test_new issue',
Chris@909 1017 :description => 'This is the description',
Chris@909 1018 :priority_id => 5}
Chris@909 1019 end
Chris@909 1020 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@909 1021 end
Chris@909 1022
Chris@909 1023 def test_post_create_with_required_custom_field_and_without_custom_fields_param
Chris@909 1024 field = IssueCustomField.find_by_name('Database')
Chris@909 1025 field.update_attribute(:is_required, true)
Chris@909 1026
Chris@909 1027 @request.session[:user_id] = 2
Chris@909 1028 post :create, :project_id => 1,
Chris@909 1029 :issue => {:tracker_id => 1,
Chris@909 1030 :subject => 'This is the test_new issue',
Chris@909 1031 :description => 'This is the description',
Chris@909 1032 :priority_id => 5}
Chris@909 1033 assert_response :success
Chris@909 1034 assert_template 'new'
Chris@909 1035 issue = assigns(:issue)
Chris@909 1036 assert_not_nil issue
Chris@909 1037 assert_equal I18n.translate('activerecord.errors.messages.invalid'), issue.errors.on(:custom_values)
Chris@909 1038 end
Chris@909 1039
Chris@909 1040 def test_post_create_with_watchers
Chris@909 1041 @request.session[:user_id] = 2
Chris@909 1042 ActionMailer::Base.deliveries.clear
Chris@909 1043
Chris@909 1044 assert_difference 'Watcher.count', 2 do
Chris@909 1045 post :create, :project_id => 1,
Chris@909 1046 :issue => {:tracker_id => 1,
Chris@909 1047 :subject => 'This is a new issue with watchers',
Chris@909 1048 :description => 'This is the description',
Chris@909 1049 :priority_id => 5,
Chris@909 1050 :watcher_user_ids => ['2', '3']}
Chris@909 1051 end
Chris@909 1052 issue = Issue.find_by_subject('This is a new issue with watchers')
Chris@909 1053 assert_not_nil issue
Chris@909 1054 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue
Chris@909 1055
Chris@909 1056 # Watchers added
Chris@909 1057 assert_equal [2, 3], issue.watcher_user_ids.sort
Chris@909 1058 assert issue.watched_by?(User.find(3))
Chris@909 1059 # Watchers notified
Chris@909 1060 mail = ActionMailer::Base.deliveries.last
Chris@909 1061 assert_kind_of TMail::Mail, mail
Chris@909 1062 assert [mail.bcc, mail.cc].flatten.include?(User.find(3).mail)
Chris@909 1063 end
Chris@909 1064
Chris@909 1065 def test_post_create_subissue
Chris@909 1066 @request.session[:user_id] = 2
Chris@909 1067
Chris@909 1068 assert_difference 'Issue.count' do
Chris@909 1069 post :create, :project_id => 1,
Chris@909 1070 :issue => {:tracker_id => 1,
Chris@909 1071 :subject => 'This is a child issue',
Chris@909 1072 :parent_issue_id => 2}
Chris@909 1073 end
Chris@909 1074 issue = Issue.find_by_subject('This is a child issue')
Chris@909 1075 assert_not_nil issue
Chris@909 1076 assert_equal Issue.find(2), issue.parent
Chris@909 1077 end
Chris@909 1078
Chris@909 1079 def test_post_create_subissue_with_non_numeric_parent_id
Chris@909 1080 @request.session[:user_id] = 2
Chris@909 1081
Chris@909 1082 assert_difference 'Issue.count' do
Chris@909 1083 post :create, :project_id => 1,
Chris@909 1084 :issue => {:tracker_id => 1,
Chris@909 1085 :subject => 'This is a child issue',
Chris@909 1086 :parent_issue_id => 'ABC'}
Chris@909 1087 end
Chris@909 1088 issue = Issue.find_by_subject('This is a child issue')
Chris@909 1089 assert_not_nil issue
Chris@909 1090 assert_nil issue.parent
Chris@909 1091 end
Chris@909 1092
Chris@909 1093 def test_post_create_private
Chris@909 1094 @request.session[:user_id] = 2
Chris@909 1095
Chris@909 1096 assert_difference 'Issue.count' do
Chris@909 1097 post :create, :project_id => 1,
Chris@909 1098 :issue => {:tracker_id => 1,
Chris@909 1099 :subject => 'This is a private issue',
Chris@909 1100 :is_private => '1'}
Chris@909 1101 end
Chris@909 1102 issue = Issue.first(:order => 'id DESC')
Chris@909 1103 assert issue.is_private?
Chris@909 1104 end
Chris@909 1105
Chris@909 1106 def test_post_create_private_with_set_own_issues_private_permission
Chris@909 1107 role = Role.find(1)
Chris@909 1108 role.remove_permission! :set_issues_private
Chris@909 1109 role.add_permission! :set_own_issues_private
Chris@909 1110
Chris@909 1111 @request.session[:user_id] = 2
Chris@909 1112
Chris@909 1113 assert_difference 'Issue.count' do
Chris@909 1114 post :create, :project_id => 1,
Chris@909 1115 :issue => {:tracker_id => 1,
Chris@909 1116 :subject => 'This is a private issue',
Chris@909 1117 :is_private => '1'}
Chris@909 1118 end
Chris@909 1119 issue = Issue.first(:order => 'id DESC')
Chris@909 1120 assert issue.is_private?
Chris@909 1121 end
Chris@909 1122
Chris@909 1123 def test_post_create_should_send_a_notification
Chris@909 1124 ActionMailer::Base.deliveries.clear
Chris@909 1125 @request.session[:user_id] = 2
Chris@909 1126 assert_difference 'Issue.count' do
Chris@909 1127 post :create, :project_id => 1,
Chris@909 1128 :issue => {:tracker_id => 3,
Chris@909 1129 :subject => 'This is the test_new issue',
Chris@909 1130 :description => 'This is the description',
Chris@909 1131 :priority_id => 5,
Chris@909 1132 :estimated_hours => '',
Chris@909 1133 :custom_field_values => {'2' => 'Value for field 2'}}
Chris@909 1134 end
Chris@909 1135 assert_redirected_to :controller => 'issues', :action => 'show', :id => Issue.last.id
Chris@909 1136
Chris@909 1137 assert_equal 1, ActionMailer::Base.deliveries.size
Chris@909 1138 end
Chris@909 1139
Chris@909 1140 def test_post_create_should_preserve_fields_values_on_validation_failure
Chris@909 1141 @request.session[:user_id] = 2
Chris@909 1142 post :create, :project_id => 1,
Chris@909 1143 :issue => {:tracker_id => 1,
Chris@909 1144 # empty subject
Chris@909 1145 :subject => '',
Chris@909 1146 :description => 'This is a description',
Chris@909 1147 :priority_id => 6,
Chris@909 1148 :custom_field_values => {'1' => 'Oracle', '2' => 'Value for field 2'}}
Chris@909 1149 assert_response :success
Chris@909 1150 assert_template 'new'
Chris@909 1151
Chris@909 1152 assert_tag :textarea, :attributes => { :name => 'issue[description]' },
Chris@909 1153 :content => 'This is a description'
Chris@909 1154 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
Chris@909 1155 :child => { :tag => 'option', :attributes => { :selected => 'selected',
Chris@909 1156 :value => '6' },
Chris@909 1157 :content => 'High' }
Chris@909 1158 # Custom fields
Chris@909 1159 assert_tag :select, :attributes => { :name => 'issue[custom_field_values][1]' },
Chris@909 1160 :child => { :tag => 'option', :attributes => { :selected => 'selected',
Chris@909 1161 :value => 'Oracle' },
Chris@909 1162 :content => 'Oracle' }
Chris@909 1163 assert_tag :input, :attributes => { :name => 'issue[custom_field_values][2]',
Chris@909 1164 :value => 'Value for field 2'}
Chris@909 1165 end
Chris@909 1166
Chris@909 1167 def test_post_create_should_ignore_non_safe_attributes
Chris@909 1168 @request.session[:user_id] = 2
Chris@909 1169 assert_nothing_raised do
Chris@909 1170 post :create, :project_id => 1, :issue => { :tracker => "A param can not be a Tracker" }
Chris@909 1171 end
Chris@909 1172 end
Chris@909 1173
Chris@909 1174 def test_post_create_with_attachment
Chris@909 1175 set_tmp_attachments_directory
Chris@909 1176 @request.session[:user_id] = 2
Chris@909 1177
Chris@909 1178 assert_difference 'Issue.count' do
Chris@909 1179 assert_difference 'Attachment.count' do
Chris@909 1180 post :create, :project_id => 1,
Chris@909 1181 :issue => { :tracker_id => '1', :subject => 'With attachment' },
Chris@909 1182 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@909 1183 end
Chris@909 1184 end
Chris@909 1185
Chris@909 1186 issue = Issue.first(:order => 'id DESC')
Chris@909 1187 attachment = Attachment.first(:order => 'id DESC')
Chris@909 1188
Chris@909 1189 assert_equal issue, attachment.container
Chris@909 1190 assert_equal 2, attachment.author_id
Chris@909 1191 assert_equal 'testfile.txt', attachment.filename
Chris@909 1192 assert_equal 'text/plain', attachment.content_type
Chris@909 1193 assert_equal 'test file', attachment.description
Chris@909 1194 assert_equal 59, attachment.filesize
Chris@909 1195 assert File.exists?(attachment.diskfile)
Chris@909 1196 assert_equal 59, File.size(attachment.diskfile)
Chris@909 1197 end
Chris@909 1198
Chris@909 1199 context "without workflow privilege" do
Chris@909 1200 setup do
Chris@909 1201 Workflow.delete_all(["role_id = ?", Role.anonymous.id])
Chris@909 1202 Role.anonymous.add_permission! :add_issues, :add_issue_notes
Chris@909 1203 end
Chris@909 1204
Chris@909 1205 context "#new" do
Chris@909 1206 should "propose default status only" do
Chris@909 1207 get :new, :project_id => 1
Chris@909 1208 assert_response :success
Chris@909 1209 assert_template 'new'
Chris@909 1210 assert_tag :tag => 'select',
Chris@909 1211 :attributes => {:name => 'issue[status_id]'},
Chris@909 1212 :children => {:count => 1},
Chris@909 1213 :child => {:tag => 'option', :attributes => {:value => IssueStatus.default.id.to_s}}
Chris@909 1214 end
Chris@909 1215
Chris@909 1216 should "accept default status" do
Chris@909 1217 assert_difference 'Issue.count' do
Chris@909 1218 post :create, :project_id => 1,
Chris@909 1219 :issue => {:tracker_id => 1,
Chris@909 1220 :subject => 'This is an issue',
Chris@909 1221 :status_id => 1}
Chris@909 1222 end
Chris@909 1223 issue = Issue.last(:order => 'id')
Chris@909 1224 assert_equal IssueStatus.default, issue.status
Chris@909 1225 end
Chris@909 1226
Chris@909 1227 should "ignore unauthorized status" do
Chris@909 1228 assert_difference 'Issue.count' do
Chris@909 1229 post :create, :project_id => 1,
Chris@909 1230 :issue => {:tracker_id => 1,
Chris@909 1231 :subject => 'This is an issue',
Chris@909 1232 :status_id => 3}
Chris@909 1233 end
Chris@909 1234 issue = Issue.last(:order => 'id')
Chris@909 1235 assert_equal IssueStatus.default, issue.status
Chris@909 1236 end
Chris@909 1237 end
Chris@909 1238
Chris@909 1239 context "#update" do
Chris@909 1240 should "ignore status change" do
Chris@909 1241 assert_difference 'Journal.count' do
Chris@909 1242 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
Chris@909 1243 end
Chris@909 1244 assert_equal 1, Issue.find(1).status_id
Chris@909 1245 end
Chris@909 1246
Chris@909 1247 should "ignore attributes changes" do
Chris@909 1248 assert_difference 'Journal.count' do
Chris@909 1249 put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2}
Chris@909 1250 end
Chris@909 1251 issue = Issue.find(1)
Chris@909 1252 assert_equal "Can't print recipes", issue.subject
Chris@909 1253 assert_nil issue.assigned_to
Chris@909 1254 end
Chris@909 1255 end
Chris@909 1256 end
Chris@909 1257
Chris@909 1258 context "with workflow privilege" do
Chris@909 1259 setup do
Chris@909 1260 Workflow.delete_all(["role_id = ?", Role.anonymous.id])
Chris@909 1261 Workflow.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 3)
Chris@909 1262 Workflow.create!(:role => Role.anonymous, :tracker_id => 1, :old_status_id => 1, :new_status_id => 4)
Chris@909 1263 Role.anonymous.add_permission! :add_issues, :add_issue_notes
Chris@909 1264 end
Chris@909 1265
Chris@909 1266 context "#update" do
Chris@909 1267 should "accept authorized status" do
Chris@909 1268 assert_difference 'Journal.count' do
Chris@909 1269 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
Chris@909 1270 end
Chris@909 1271 assert_equal 3, Issue.find(1).status_id
Chris@909 1272 end
Chris@909 1273
Chris@909 1274 should "ignore unauthorized status" do
Chris@909 1275 assert_difference 'Journal.count' do
Chris@909 1276 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2}
Chris@909 1277 end
Chris@909 1278 assert_equal 1, Issue.find(1).status_id
Chris@909 1279 end
Chris@909 1280
Chris@909 1281 should "accept authorized attributes changes" do
Chris@909 1282 assert_difference 'Journal.count' do
Chris@909 1283 put :update, :id => 1, :notes => 'just trying', :issue => {:assigned_to_id => 2}
Chris@909 1284 end
Chris@909 1285 issue = Issue.find(1)
Chris@909 1286 assert_equal 2, issue.assigned_to_id
Chris@909 1287 end
Chris@909 1288
Chris@909 1289 should "ignore unauthorized attributes changes" do
Chris@909 1290 assert_difference 'Journal.count' do
Chris@909 1291 put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed'}
Chris@909 1292 end
Chris@909 1293 issue = Issue.find(1)
Chris@909 1294 assert_equal "Can't print recipes", issue.subject
Chris@909 1295 end
Chris@909 1296 end
Chris@909 1297
Chris@909 1298 context "and :edit_issues permission" do
Chris@909 1299 setup do
Chris@909 1300 Role.anonymous.add_permission! :add_issues, :edit_issues
Chris@909 1301 end
Chris@909 1302
Chris@909 1303 should "accept authorized status" do
Chris@909 1304 assert_difference 'Journal.count' do
Chris@909 1305 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 3}
Chris@909 1306 end
Chris@909 1307 assert_equal 3, Issue.find(1).status_id
Chris@909 1308 end
Chris@909 1309
Chris@909 1310 should "ignore unauthorized status" do
Chris@909 1311 assert_difference 'Journal.count' do
Chris@909 1312 put :update, :id => 1, :notes => 'just trying', :issue => {:status_id => 2}
Chris@909 1313 end
Chris@909 1314 assert_equal 1, Issue.find(1).status_id
Chris@909 1315 end
Chris@909 1316
Chris@909 1317 should "accept authorized attributes changes" do
Chris@909 1318 assert_difference 'Journal.count' do
Chris@909 1319 put :update, :id => 1, :notes => 'just trying', :issue => {:subject => 'changed', :assigned_to_id => 2}
Chris@909 1320 end
Chris@909 1321 issue = Issue.find(1)
Chris@909 1322 assert_equal "changed", issue.subject
Chris@909 1323 assert_equal 2, issue.assigned_to_id
Chris@909 1324 end
Chris@909 1325 end
Chris@909 1326 end
Chris@909 1327
Chris@909 1328 def test_copy_issue
Chris@909 1329 @request.session[:user_id] = 2
Chris@909 1330 get :new, :project_id => 1, :copy_from => 1
Chris@909 1331 assert_template 'new'
Chris@909 1332 assert_not_nil assigns(:issue)
Chris@909 1333 orig = Issue.find(1)
Chris@909 1334 assert_equal orig.subject, assigns(:issue).subject
Chris@909 1335 end
Chris@909 1336
Chris@909 1337 def test_get_edit
Chris@909 1338 @request.session[:user_id] = 2
Chris@909 1339 get :edit, :id => 1
Chris@909 1340 assert_response :success
Chris@909 1341 assert_template 'edit'
Chris@909 1342 assert_not_nil assigns(:issue)
Chris@909 1343 assert_equal Issue.find(1), assigns(:issue)
Chris@909 1344
Chris@909 1345 # Be sure we don't display inactive IssuePriorities
Chris@909 1346 assert ! IssuePriority.find(15).active?
Chris@909 1347 assert_no_tag :option, :attributes => {:value => '15'},
Chris@909 1348 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
Chris@909 1349 end
Chris@909 1350
Chris@909 1351 def test_get_edit_should_display_the_time_entry_form_with_log_time_permission
Chris@909 1352 @request.session[:user_id] = 2
Chris@909 1353 Role.find_by_name('Manager').update_attribute :permissions, [:view_issues, :edit_issues, :log_time]
Chris@909 1354
Chris@909 1355 get :edit, :id => 1
Chris@909 1356 assert_tag 'input', :attributes => {:name => 'time_entry[hours]'}
Chris@909 1357 end
Chris@909 1358
Chris@909 1359 def test_get_edit_should_not_display_the_time_entry_form_without_log_time_permission
Chris@909 1360 @request.session[:user_id] = 2
Chris@909 1361 Role.find_by_name('Manager').remove_permission! :log_time
Chris@909 1362
Chris@909 1363 get :edit, :id => 1
Chris@909 1364 assert_no_tag 'input', :attributes => {:name => 'time_entry[hours]'}
Chris@909 1365 end
Chris@909 1366
Chris@909 1367 def test_get_edit_with_params
Chris@909 1368 @request.session[:user_id] = 2
Chris@909 1369 get :edit, :id => 1, :issue => { :status_id => 5, :priority_id => 7 },
Chris@909 1370 :time_entry => { :hours => '2.5', :comments => 'test_get_edit_with_params', :activity_id => TimeEntryActivity.first.id }
Chris@909 1371 assert_response :success
Chris@909 1372 assert_template 'edit'
Chris@909 1373
Chris@909 1374 issue = assigns(:issue)
Chris@909 1375 assert_not_nil issue
Chris@909 1376
Chris@909 1377 assert_equal 5, issue.status_id
Chris@909 1378 assert_tag :select, :attributes => { :name => 'issue[status_id]' },
Chris@909 1379 :child => { :tag => 'option',
Chris@909 1380 :content => 'Closed',
Chris@909 1381 :attributes => { :selected => 'selected' } }
Chris@909 1382
Chris@909 1383 assert_equal 7, issue.priority_id
Chris@909 1384 assert_tag :select, :attributes => { :name => 'issue[priority_id]' },
Chris@909 1385 :child => { :tag => 'option',
Chris@909 1386 :content => 'Urgent',
Chris@909 1387 :attributes => { :selected => 'selected' } }
Chris@909 1388
Chris@909 1389 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => '2.5' }
Chris@909 1390 assert_tag :select, :attributes => { :name => 'time_entry[activity_id]' },
Chris@909 1391 :child => { :tag => 'option',
Chris@909 1392 :attributes => { :selected => 'selected', :value => TimeEntryActivity.first.id } }
Chris@909 1393 assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => 'test_get_edit_with_params' }
Chris@909 1394 end
Chris@909 1395
Chris@909 1396 def test_update_edit_form
Chris@909 1397 @request.session[:user_id] = 2
Chris@909 1398 xhr :post, :new, :project_id => 1,
Chris@909 1399 :id => 1,
Chris@909 1400 :issue => {:tracker_id => 2,
Chris@909 1401 :subject => 'This is the test_new issue',
Chris@909 1402 :description => 'This is the description',
Chris@909 1403 :priority_id => 5}
Chris@909 1404 assert_response :success
Chris@909 1405 assert_template 'attributes'
Chris@909 1406
Chris@909 1407 issue = assigns(:issue)
Chris@909 1408 assert_kind_of Issue, issue
Chris@909 1409 assert_equal 1, issue.id
Chris@909 1410 assert_equal 1, issue.project_id
Chris@909 1411 assert_equal 2, issue.tracker_id
Chris@909 1412 assert_equal 'This is the test_new issue', issue.subject
Chris@909 1413 end
Chris@909 1414
Chris@909 1415 def test_update_using_invalid_http_verbs
Chris@909 1416 @request.session[:user_id] = 2
Chris@909 1417 subject = 'Updated by an invalid http verb'
Chris@909 1418
Chris@909 1419 get :update, :id => 1, :issue => {:subject => subject}
Chris@909 1420 assert_not_equal subject, Issue.find(1).subject
Chris@909 1421
Chris@909 1422 post :update, :id => 1, :issue => {:subject => subject}
Chris@909 1423 assert_not_equal subject, Issue.find(1).subject
Chris@909 1424
Chris@909 1425 delete :update, :id => 1, :issue => {:subject => subject}
Chris@909 1426 assert_not_equal subject, Issue.find(1).subject
Chris@909 1427 end
Chris@909 1428
Chris@909 1429 def test_put_update_without_custom_fields_param
Chris@909 1430 @request.session[:user_id] = 2
Chris@909 1431 ActionMailer::Base.deliveries.clear
Chris@909 1432
Chris@909 1433 issue = Issue.find(1)
Chris@909 1434 assert_equal '125', issue.custom_value_for(2).value
Chris@909 1435 old_subject = issue.subject
Chris@909 1436 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
Chris@909 1437
Chris@909 1438 assert_difference('Journal.count') do
Chris@909 1439 assert_difference('JournalDetail.count', 2) do
Chris@909 1440 put :update, :id => 1, :issue => {:subject => new_subject,
Chris@909 1441 :priority_id => '6',
Chris@909 1442 :category_id => '1' # no change
Chris@909 1443 }
Chris@909 1444 end
Chris@909 1445 end
Chris@909 1446 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1447 issue.reload
Chris@909 1448 assert_equal new_subject, issue.subject
Chris@909 1449 # Make sure custom fields were not cleared
Chris@909 1450 assert_equal '125', issue.custom_value_for(2).value
Chris@909 1451
Chris@909 1452 mail = ActionMailer::Base.deliveries.last
Chris@909 1453 assert_kind_of TMail::Mail, mail
Chris@909 1454 assert mail.subject.starts_with?("[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}]")
Chris@909 1455 assert mail.body.include?("Subject changed from #{old_subject} to #{new_subject}")
Chris@909 1456 end
Chris@909 1457
Chris@909 1458 def test_put_update_with_custom_field_change
Chris@909 1459 @request.session[:user_id] = 2
Chris@909 1460 issue = Issue.find(1)
Chris@909 1461 assert_equal '125', issue.custom_value_for(2).value
Chris@909 1462
Chris@909 1463 assert_difference('Journal.count') do
Chris@909 1464 assert_difference('JournalDetail.count', 3) do
Chris@909 1465 put :update, :id => 1, :issue => {:subject => 'Custom field change',
Chris@909 1466 :priority_id => '6',
Chris@909 1467 :category_id => '1', # no change
Chris@909 1468 :custom_field_values => { '2' => 'New custom value' }
Chris@909 1469 }
Chris@909 1470 end
Chris@909 1471 end
Chris@909 1472 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1473 issue.reload
Chris@909 1474 assert_equal 'New custom value', issue.custom_value_for(2).value
Chris@909 1475
Chris@909 1476 mail = ActionMailer::Base.deliveries.last
Chris@909 1477 assert_kind_of TMail::Mail, mail
Chris@909 1478 assert mail.body.include?("Searchable field changed from 125 to New custom value")
Chris@909 1479 end
Chris@909 1480
Chris@909 1481 def test_put_update_with_status_and_assignee_change
Chris@909 1482 issue = Issue.find(1)
Chris@909 1483 assert_equal 1, issue.status_id
Chris@909 1484 @request.session[:user_id] = 2
Chris@909 1485 assert_difference('TimeEntry.count', 0) do
Chris@909 1486 put :update,
Chris@909 1487 :id => 1,
Chris@909 1488 :issue => { :status_id => 2, :assigned_to_id => 3 },
Chris@909 1489 :notes => 'Assigned to dlopper',
Chris@909 1490 :time_entry => { :hours => '', :comments => '', :activity_id => TimeEntryActivity.first }
Chris@909 1491 end
Chris@909 1492 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1493 issue.reload
Chris@909 1494 assert_equal 2, issue.status_id
Chris@909 1495 j = Journal.find(:first, :order => 'id DESC')
Chris@909 1496 assert_equal 'Assigned to dlopper', j.notes
Chris@909 1497 assert_equal 2, j.details.size
Chris@909 1498
Chris@909 1499 mail = ActionMailer::Base.deliveries.last
Chris@909 1500 assert mail.body.include?("Status changed from New to Assigned")
Chris@909 1501 # subject should contain the new status
Chris@909 1502 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
Chris@909 1503 end
Chris@909 1504
Chris@909 1505 def test_put_update_with_note_only
Chris@909 1506 notes = 'Note added by IssuesControllerTest#test_update_with_note_only'
Chris@909 1507 # anonymous user
Chris@909 1508 put :update,
Chris@909 1509 :id => 1,
Chris@909 1510 :notes => notes
Chris@909 1511 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1512 j = Journal.find(:first, :order => 'id DESC')
Chris@909 1513 assert_equal notes, j.notes
Chris@909 1514 assert_equal 0, j.details.size
Chris@909 1515 assert_equal User.anonymous, j.user
Chris@909 1516
Chris@909 1517 mail = ActionMailer::Base.deliveries.last
Chris@909 1518 assert mail.body.include?(notes)
Chris@909 1519 end
Chris@909 1520
Chris@909 1521 def test_put_update_with_note_and_spent_time
Chris@909 1522 @request.session[:user_id] = 2
Chris@909 1523 spent_hours_before = Issue.find(1).spent_hours
Chris@909 1524 assert_difference('TimeEntry.count') do
Chris@909 1525 put :update,
Chris@909 1526 :id => 1,
Chris@909 1527 :notes => '2.5 hours added',
Chris@909 1528 :time_entry => { :hours => '2.5', :comments => 'test_put_update_with_note_and_spent_time', :activity_id => TimeEntryActivity.first.id }
Chris@909 1529 end
Chris@909 1530 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1531
Chris@909 1532 issue = Issue.find(1)
Chris@909 1533
Chris@909 1534 j = Journal.find(:first, :order => 'id DESC')
Chris@909 1535 assert_equal '2.5 hours added', j.notes
Chris@909 1536 assert_equal 0, j.details.size
Chris@909 1537
Chris@909 1538 t = issue.time_entries.find_by_comments('test_put_update_with_note_and_spent_time')
Chris@909 1539 assert_not_nil t
Chris@909 1540 assert_equal 2.5, t.hours
Chris@909 1541 assert_equal spent_hours_before + 2.5, issue.spent_hours
Chris@909 1542 end
Chris@909 1543
Chris@909 1544 def test_put_update_with_attachment_only
Chris@909 1545 set_tmp_attachments_directory
Chris@909 1546
Chris@909 1547 # Delete all fixtured journals, a race condition can occur causing the wrong
Chris@909 1548 # journal to get fetched in the next find.
Chris@909 1549 Journal.delete_all
Chris@909 1550
Chris@909 1551 # anonymous user
Chris@909 1552 assert_difference 'Attachment.count' do
Chris@909 1553 put :update, :id => 1,
Chris@909 1554 :notes => '',
Chris@909 1555 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain'), 'description' => 'test file'}}
Chris@909 1556 end
Chris@909 1557
Chris@909 1558 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1559 j = Issue.find(1).journals.find(:first, :order => 'id DESC')
Chris@909 1560 assert j.notes.blank?
Chris@909 1561 assert_equal 1, j.details.size
Chris@909 1562 assert_equal 'testfile.txt', j.details.first.value
Chris@909 1563 assert_equal User.anonymous, j.user
Chris@909 1564
Chris@909 1565 attachment = Attachment.first(:order => 'id DESC')
Chris@909 1566 assert_equal Issue.find(1), attachment.container
Chris@909 1567 assert_equal User.anonymous, attachment.author
Chris@909 1568 assert_equal 'testfile.txt', attachment.filename
Chris@909 1569 assert_equal 'text/plain', attachment.content_type
Chris@909 1570 assert_equal 'test file', attachment.description
Chris@909 1571 assert_equal 59, attachment.filesize
Chris@909 1572 assert File.exists?(attachment.diskfile)
Chris@909 1573 assert_equal 59, File.size(attachment.diskfile)
Chris@909 1574
Chris@909 1575 mail = ActionMailer::Base.deliveries.last
Chris@909 1576 assert mail.body.include?('testfile.txt')
Chris@909 1577 end
Chris@909 1578
Chris@909 1579 def test_put_update_with_attachment_that_fails_to_save
Chris@909 1580 set_tmp_attachments_directory
Chris@909 1581
Chris@909 1582 # Delete all fixtured journals, a race condition can occur causing the wrong
Chris@909 1583 # journal to get fetched in the next find.
Chris@909 1584 Journal.delete_all
Chris@909 1585
Chris@909 1586 # Mock out the unsaved attachment
Chris@909 1587 Attachment.any_instance.stubs(:create).returns(Attachment.new)
Chris@909 1588
Chris@909 1589 # anonymous user
Chris@909 1590 put :update,
Chris@909 1591 :id => 1,
Chris@909 1592 :notes => '',
Chris@909 1593 :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
Chris@909 1594 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1595 assert_equal '1 file(s) could not be saved.', flash[:warning]
Chris@909 1596
Chris@909 1597 end if Object.const_defined?(:Mocha)
Chris@909 1598
Chris@909 1599 def test_put_update_with_no_change
Chris@909 1600 issue = Issue.find(1)
Chris@909 1601 issue.journals.clear
Chris@909 1602 ActionMailer::Base.deliveries.clear
Chris@909 1603
Chris@909 1604 put :update,
Chris@909 1605 :id => 1,
Chris@909 1606 :notes => ''
Chris@909 1607 assert_redirected_to :action => 'show', :id => '1'
Chris@909 1608
Chris@909 1609 issue.reload
Chris@909 1610 assert issue.journals.empty?
Chris@909 1611 # No email should be sent
Chris@909 1612 assert ActionMailer::Base.deliveries.empty?
Chris@909 1613 end
Chris@909 1614
Chris@909 1615 def test_put_update_should_send_a_notification
Chris@909 1616 @request.session[:user_id] = 2
Chris@909 1617 ActionMailer::Base.deliveries.clear
Chris@909 1618 issue = Issue.find(1)
Chris@909 1619 old_subject = issue.subject
Chris@909 1620 new_subject = 'Subject modified by IssuesControllerTest#test_post_edit'
Chris@909 1621
Chris@909 1622 put :update, :id => 1, :issue => {:subject => new_subject,
Chris@909 1623 :priority_id => '6',
Chris@909 1624 :category_id => '1' # no change
Chris@909 1625 }
Chris@909 1626 assert_equal 1, ActionMailer::Base.deliveries.size
Chris@909 1627 end
Chris@909 1628
Chris@909 1629 def test_put_update_with_invalid_spent_time_hours_only
Chris@909 1630 @request.session[:user_id] = 2
Chris@909 1631 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
Chris@909 1632
Chris@909 1633 assert_no_difference('Journal.count') do
Chris@909 1634 put :update,
Chris@909 1635 :id => 1,
Chris@909 1636 :notes => notes,
Chris@909 1637 :time_entry => {"comments"=>"", "activity_id"=>"", "hours"=>"2z"}
Chris@909 1638 end
Chris@909 1639 assert_response :success
Chris@909 1640 assert_template 'edit'
Chris@909 1641
Chris@909 1642 assert_error_tag :descendant => {:content => /Activity can't be blank/}
Chris@909 1643 assert_tag :textarea, :attributes => { :name => 'notes' }, :content => notes
Chris@909 1644 assert_tag :input, :attributes => { :name => 'time_entry[hours]', :value => "2z" }
Chris@909 1645 end
Chris@909 1646
Chris@909 1647 def test_put_update_with_invalid_spent_time_comments_only
Chris@909 1648 @request.session[:user_id] = 2
Chris@909 1649 notes = 'Note added by IssuesControllerTest#test_post_edit_with_invalid_spent_time'
Chris@909 1650
Chris@909 1651 assert_no_difference('Journal.count') do
Chris@909 1652 put :update,
Chris@909 1653 :id => 1,
Chris@909 1654 :notes => notes,
Chris@909 1655 :time_entry => {"comments"=>"this is my comment", "activity_id"=>"", "hours"=>""}
Chris@909 1656 end
Chris@909 1657 assert_response :success
Chris@909 1658 assert_template 'edit'
Chris@909 1659
Chris@909 1660 assert_error_tag :descendant => {:content => /Activity can't be blank/}
Chris@909 1661 assert_error_tag :descendant => {:content => /Hours can't be blank/}
Chris@909 1662 assert_tag :textarea, :attributes => { :name => 'notes' }, :content => notes
Chris@909 1663 assert_tag :input, :attributes => { :name => 'time_entry[comments]', :value => "this is my comment" }
Chris@909 1664 end
Chris@909 1665
Chris@909 1666 def test_put_update_should_allow_fixed_version_to_be_set_to_a_subproject
Chris@909 1667 issue = Issue.find(2)
Chris@909 1668 @request.session[:user_id] = 2
Chris@909 1669
Chris@909 1670 put :update,
Chris@909 1671 :id => issue.id,
Chris@909 1672 :issue => {
Chris@909 1673 :fixed_version_id => 4
Chris@909 1674 }
Chris@909 1675
Chris@909 1676 assert_response :redirect
Chris@909 1677 issue.reload
Chris@909 1678 assert_equal 4, issue.fixed_version_id
Chris@909 1679 assert_not_equal issue.project_id, issue.fixed_version.project_id
Chris@909 1680 end
Chris@909 1681
Chris@909 1682 def test_put_update_should_redirect_back_using_the_back_url_parameter
Chris@909 1683 issue = Issue.find(2)
Chris@909 1684 @request.session[:user_id] = 2
Chris@909 1685
Chris@909 1686 put :update,
Chris@909 1687 :id => issue.id,
Chris@909 1688 :issue => {
Chris@909 1689 :fixed_version_id => 4
Chris@909 1690 },
Chris@909 1691 :back_url => '/issues'
Chris@909 1692
Chris@909 1693 assert_response :redirect
Chris@909 1694 assert_redirected_to '/issues'
Chris@909 1695 end
Chris@909 1696
Chris@909 1697 def test_put_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
Chris@909 1698 issue = Issue.find(2)
Chris@909 1699 @request.session[:user_id] = 2
Chris@909 1700
Chris@909 1701 put :update,
Chris@909 1702 :id => issue.id,
Chris@909 1703 :issue => {
Chris@909 1704 :fixed_version_id => 4
Chris@909 1705 },
Chris@909 1706 :back_url => 'http://google.com'
Chris@909 1707
Chris@909 1708 assert_response :redirect
Chris@909 1709 assert_redirected_to :controller => 'issues', :action => 'show', :id => issue.id
Chris@909 1710 end
Chris@909 1711
Chris@909 1712 def test_get_bulk_edit
Chris@909 1713 @request.session[:user_id] = 2
Chris@909 1714 get :bulk_edit, :ids => [1, 2]
Chris@909 1715 assert_response :success
Chris@909 1716 assert_template 'bulk_edit'
Chris@909 1717
Chris@909 1718 assert_tag :input, :attributes => {:name => 'issue[parent_issue_id]'}
Chris@909 1719
Chris@909 1720 # Project specific custom field, date type
Chris@909 1721 field = CustomField.find(9)
Chris@909 1722 assert !field.is_for_all?
Chris@909 1723 assert_equal 'date', field.field_format
Chris@909 1724 assert_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
Chris@909 1725
Chris@909 1726 # System wide custom field
Chris@909 1727 assert CustomField.find(1).is_for_all?
Chris@909 1728 assert_tag :select, :attributes => {:name => 'issue[custom_field_values][1]'}
Chris@909 1729
Chris@909 1730 # Be sure we don't display inactive IssuePriorities
Chris@909 1731 assert ! IssuePriority.find(15).active?
Chris@909 1732 assert_no_tag :option, :attributes => {:value => '15'},
Chris@909 1733 :parent => {:tag => 'select', :attributes => {:id => 'issue_priority_id'} }
Chris@909 1734 end
Chris@909 1735
Chris@909 1736 def test_get_bulk_edit_on_different_projects
Chris@909 1737 @request.session[:user_id] = 2
Chris@909 1738 get :bulk_edit, :ids => [1, 2, 6]
Chris@909 1739 assert_response :success
Chris@909 1740 assert_template 'bulk_edit'
Chris@909 1741
Chris@909 1742 # Can not set issues from different projects as children of an issue
Chris@909 1743 assert_no_tag :input, :attributes => {:name => 'issue[parent_issue_id]'}
Chris@909 1744
Chris@909 1745 # Project specific custom field, date type
Chris@909 1746 field = CustomField.find(9)
Chris@909 1747 assert !field.is_for_all?
Chris@909 1748 assert !field.project_ids.include?(Issue.find(6).project_id)
Chris@909 1749 assert_no_tag :input, :attributes => {:name => 'issue[custom_field_values][9]'}
Chris@909 1750 end
Chris@909 1751
Chris@909 1752 def test_get_bulk_edit_with_user_custom_field
Chris@909 1753 field = IssueCustomField.create!(:name => 'Tester', :field_format => 'user', :is_for_all => true)
Chris@909 1754
Chris@909 1755 @request.session[:user_id] = 2
Chris@909 1756 get :bulk_edit, :ids => [1, 2]
Chris@909 1757 assert_response :success
Chris@909 1758 assert_template 'bulk_edit'
Chris@909 1759
Chris@909 1760 assert_tag :select,
Chris@909 1761 :attributes => {:name => "issue[custom_field_values][#{field.id}]"},
Chris@909 1762 :children => {
Chris@909 1763 :only => {:tag => 'option'},
Chris@909 1764 :count => Project.find(1).users.count + 1
Chris@909 1765 }
Chris@909 1766 end
Chris@909 1767
Chris@909 1768 def test_get_bulk_edit_with_version_custom_field
Chris@909 1769 field = IssueCustomField.create!(:name => 'Affected version', :field_format => 'version', :is_for_all => true)
Chris@909 1770
Chris@909 1771 @request.session[:user_id] = 2
Chris@909 1772 get :bulk_edit, :ids => [1, 2]
Chris@909 1773 assert_response :success
Chris@909 1774 assert_template 'bulk_edit'
Chris@909 1775
Chris@909 1776 assert_tag :select,
Chris@909 1777 :attributes => {:name => "issue[custom_field_values][#{field.id}]"},
Chris@909 1778 :children => {
Chris@909 1779 :only => {:tag => 'option'},
Chris@909 1780 :count => Project.find(1).shared_versions.count + 1
Chris@909 1781 }
Chris@909 1782 end
Chris@909 1783
Chris@909 1784 def test_bulk_update
Chris@909 1785 @request.session[:user_id] = 2
Chris@909 1786 # update issues priority
Chris@909 1787 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
Chris@909 1788 :issue => {:priority_id => 7,
Chris@909 1789 :assigned_to_id => '',
Chris@909 1790 :custom_field_values => {'2' => ''}}
Chris@909 1791
Chris@909 1792 assert_response 302
Chris@909 1793 # check that the issues were updated
Chris@909 1794 assert_equal [7, 7], Issue.find_all_by_id([1, 2]).collect {|i| i.priority.id}
Chris@909 1795
Chris@909 1796 issue = Issue.find(1)
Chris@909 1797 journal = issue.journals.find(:first, :order => 'created_on DESC')
Chris@909 1798 assert_equal '125', issue.custom_value_for(2).value
Chris@909 1799 assert_equal 'Bulk editing', journal.notes
Chris@909 1800 assert_equal 1, journal.details.size
Chris@909 1801 end
Chris@909 1802
Chris@909 1803 def test_bulk_update_with_group_assignee
Chris@909 1804 group = Group.find(11)
Chris@909 1805 project = Project.find(1)
Chris@909 1806 project.members << Member.new(:principal => group, :roles => [Role.first])
Chris@909 1807
Chris@909 1808 @request.session[:user_id] = 2
Chris@909 1809 # update issues assignee
Chris@909 1810 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing',
Chris@909 1811 :issue => {:priority_id => '',
Chris@909 1812 :assigned_to_id => group.id,
Chris@909 1813 :custom_field_values => {'2' => ''}}
Chris@909 1814
Chris@909 1815 assert_response 302
Chris@909 1816 assert_equal [group, group], Issue.find_all_by_id([1, 2]).collect {|i| i.assigned_to}
Chris@909 1817 end
Chris@909 1818
Chris@909 1819 def test_bulk_update_on_different_projects
Chris@909 1820 @request.session[:user_id] = 2
Chris@909 1821 # update issues priority
Chris@909 1822 post :bulk_update, :ids => [1, 2, 6], :notes => 'Bulk editing',
Chris@909 1823 :issue => {:priority_id => 7,
Chris@909 1824 :assigned_to_id => '',
Chris@909 1825 :custom_field_values => {'2' => ''}}
Chris@909 1826
Chris@909 1827 assert_response 302
Chris@909 1828 # check that the issues were updated
Chris@909 1829 assert_equal [7, 7, 7], Issue.find([1,2,6]).map(&:priority_id)
Chris@909 1830
Chris@909 1831 issue = Issue.find(1)
Chris@909 1832 journal = issue.journals.find(:first, :order => 'created_on DESC')
Chris@909 1833 assert_equal '125', issue.custom_value_for(2).value
Chris@909 1834 assert_equal 'Bulk editing', journal.notes
Chris@909 1835 assert_equal 1, journal.details.size
Chris@909 1836 end
Chris@909 1837
Chris@909 1838 def test_bulk_update_on_different_projects_without_rights
Chris@909 1839 @request.session[:user_id] = 3
Chris@909 1840 user = User.find(3)
Chris@909 1841 action = { :controller => "issues", :action => "bulk_update" }
Chris@909 1842 assert user.allowed_to?(action, Issue.find(1).project)
Chris@909 1843 assert ! user.allowed_to?(action, Issue.find(6).project)
Chris@909 1844 post :bulk_update, :ids => [1, 6], :notes => 'Bulk should fail',
Chris@909 1845 :issue => {:priority_id => 7,
Chris@909 1846 :assigned_to_id => '',
Chris@909 1847 :custom_field_values => {'2' => ''}}
Chris@909 1848 assert_response 403
Chris@909 1849 assert_not_equal "Bulk should fail", Journal.last.notes
Chris@909 1850 end
Chris@909 1851
Chris@909 1852 def test_bullk_update_should_send_a_notification
Chris@909 1853 @request.session[:user_id] = 2
Chris@909 1854 ActionMailer::Base.deliveries.clear
Chris@909 1855 post(:bulk_update,
Chris@909 1856 {
Chris@909 1857 :ids => [1, 2],
Chris@909 1858 :notes => 'Bulk editing',
Chris@909 1859 :issue => {
Chris@909 1860 :priority_id => 7,
Chris@909 1861 :assigned_to_id => '',
Chris@909 1862 :custom_field_values => {'2' => ''}
Chris@909 1863 }
Chris@909 1864 })
Chris@909 1865
Chris@909 1866 assert_response 302
Chris@909 1867 assert_equal 2, ActionMailer::Base.deliveries.size
Chris@909 1868 end
Chris@909 1869
Chris@909 1870 def test_bulk_update_status
Chris@909 1871 @request.session[:user_id] = 2
Chris@909 1872 # update issues priority
Chris@909 1873 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing status',
Chris@909 1874 :issue => {:priority_id => '',
Chris@909 1875 :assigned_to_id => '',
Chris@909 1876 :status_id => '5'}
Chris@909 1877
Chris@909 1878 assert_response 302
Chris@909 1879 issue = Issue.find(1)
Chris@909 1880 assert issue.closed?
Chris@909 1881 end
Chris@909 1882
Chris@909 1883 def test_bulk_update_parent_id
Chris@909 1884 @request.session[:user_id] = 2
Chris@909 1885 post :bulk_update, :ids => [1, 3],
Chris@909 1886 :notes => 'Bulk editing parent',
Chris@909 1887 :issue => {:priority_id => '', :assigned_to_id => '', :status_id => '', :parent_issue_id => '2'}
Chris@909 1888
Chris@909 1889 assert_response 302
Chris@909 1890 parent = Issue.find(2)
Chris@909 1891 assert_equal parent.id, Issue.find(1).parent_id
Chris@909 1892 assert_equal parent.id, Issue.find(3).parent_id
Chris@909 1893 assert_equal [1, 3], parent.children.collect(&:id).sort
Chris@909 1894 end
Chris@909 1895
Chris@909 1896 def test_bulk_update_custom_field
Chris@909 1897 @request.session[:user_id] = 2
Chris@909 1898 # update issues priority
Chris@909 1899 post :bulk_update, :ids => [1, 2], :notes => 'Bulk editing custom field',
Chris@909 1900 :issue => {:priority_id => '',
Chris@909 1901 :assigned_to_id => '',
Chris@909 1902 :custom_field_values => {'2' => '777'}}
Chris@909 1903
Chris@909 1904 assert_response 302
Chris@909 1905
Chris@909 1906 issue = Issue.find(1)
Chris@909 1907 journal = issue.journals.find(:first, :order => 'created_on DESC')
Chris@909 1908 assert_equal '777', issue.custom_value_for(2).value
Chris@909 1909 assert_equal 1, journal.details.size
Chris@909 1910 assert_equal '125', journal.details.first.old_value
Chris@909 1911 assert_equal '777', journal.details.first.value
Chris@909 1912 end
Chris@909 1913
Chris@909 1914 def test_bulk_update_unassign
Chris@909 1915 assert_not_nil Issue.find(2).assigned_to
Chris@909 1916 @request.session[:user_id] = 2
Chris@909 1917 # unassign issues
Chris@909 1918 post :bulk_update, :ids => [1, 2], :notes => 'Bulk unassigning', :issue => {:assigned_to_id => 'none'}
Chris@909 1919 assert_response 302
Chris@909 1920 # check that the issues were updated
Chris@909 1921 assert_nil Issue.find(2).assigned_to
Chris@909 1922 end
Chris@909 1923
Chris@909 1924 def test_post_bulk_update_should_allow_fixed_version_to_be_set_to_a_subproject
Chris@909 1925 @request.session[:user_id] = 2
Chris@909 1926
Chris@909 1927 post :bulk_update, :ids => [1,2], :issue => {:fixed_version_id => 4}
Chris@909 1928
Chris@909 1929 assert_response :redirect
Chris@909 1930 issues = Issue.find([1,2])
Chris@909 1931 issues.each do |issue|
Chris@909 1932 assert_equal 4, issue.fixed_version_id
Chris@909 1933 assert_not_equal issue.project_id, issue.fixed_version.project_id
Chris@909 1934 end
Chris@909 1935 end
Chris@909 1936
Chris@909 1937 def test_post_bulk_update_should_redirect_back_using_the_back_url_parameter
Chris@909 1938 @request.session[:user_id] = 2
Chris@909 1939 post :bulk_update, :ids => [1,2], :back_url => '/issues'
Chris@909 1940
Chris@909 1941 assert_response :redirect
Chris@909 1942 assert_redirected_to '/issues'
Chris@909 1943 end
Chris@909 1944
Chris@909 1945 def test_post_bulk_update_should_not_redirect_back_using_the_back_url_parameter_off_the_host
Chris@909 1946 @request.session[:user_id] = 2
Chris@909 1947 post :bulk_update, :ids => [1,2], :back_url => 'http://google.com'
Chris@909 1948
Chris@909 1949 assert_response :redirect
Chris@909 1950 assert_redirected_to :controller => 'issues', :action => 'index', :project_id => Project.find(1).identifier
Chris@909 1951 end
Chris@909 1952
Chris@909 1953 def test_destroy_issue_with_no_time_entries
Chris@909 1954 assert_nil TimeEntry.find_by_issue_id(2)
Chris@909 1955 @request.session[:user_id] = 2
Chris@909 1956 post :destroy, :id => 2
Chris@909 1957 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@909 1958 assert_nil Issue.find_by_id(2)
Chris@909 1959 end
Chris@909 1960
Chris@909 1961 def test_destroy_issues_with_time_entries
Chris@909 1962 @request.session[:user_id] = 2
Chris@909 1963 post :destroy, :ids => [1, 3]
Chris@909 1964 assert_response :success
Chris@909 1965 assert_template 'destroy'
Chris@909 1966 assert_not_nil assigns(:hours)
Chris@909 1967 assert Issue.find_by_id(1) && Issue.find_by_id(3)
Chris@909 1968 end
Chris@909 1969
Chris@909 1970 def test_destroy_issues_and_destroy_time_entries
Chris@909 1971 @request.session[:user_id] = 2
Chris@909 1972 post :destroy, :ids => [1, 3], :todo => 'destroy'
Chris@909 1973 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@909 1974 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
Chris@909 1975 assert_nil TimeEntry.find_by_id([1, 2])
Chris@909 1976 end
Chris@909 1977
Chris@909 1978 def test_destroy_issues_and_assign_time_entries_to_project
Chris@909 1979 @request.session[:user_id] = 2
Chris@909 1980 post :destroy, :ids => [1, 3], :todo => 'nullify'
Chris@909 1981 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@909 1982 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
Chris@909 1983 assert_nil TimeEntry.find(1).issue_id
Chris@909 1984 assert_nil TimeEntry.find(2).issue_id
Chris@909 1985 end
Chris@909 1986
Chris@909 1987 def test_destroy_issues_and_reassign_time_entries_to_another_issue
Chris@909 1988 @request.session[:user_id] = 2
Chris@909 1989 post :destroy, :ids => [1, 3], :todo => 'reassign', :reassign_to_id => 2
Chris@909 1990 assert_redirected_to :action => 'index', :project_id => 'ecookbook'
Chris@909 1991 assert !(Issue.find_by_id(1) || Issue.find_by_id(3))
Chris@909 1992 assert_equal 2, TimeEntry.find(1).issue_id
Chris@909 1993 assert_equal 2, TimeEntry.find(2).issue_id
Chris@909 1994 end
Chris@909 1995
Chris@909 1996 def test_destroy_issues_from_different_projects
Chris@909 1997 @request.session[:user_id] = 2
Chris@909 1998 post :destroy, :ids => [1, 2, 6], :todo => 'destroy'
Chris@909 1999 assert_redirected_to :controller => 'issues', :action => 'index'
Chris@909 2000 assert !(Issue.find_by_id(1) || Issue.find_by_id(2) || Issue.find_by_id(6))
Chris@909 2001 end
Chris@909 2002
Chris@909 2003 def test_destroy_parent_and_child_issues
Chris@909 2004 parent = Issue.generate!(:project_id => 1, :tracker_id => 1)
Chris@909 2005 child = Issue.generate!(:project_id => 1, :tracker_id => 1, :parent_issue_id => parent.id)
Chris@909 2006 assert child.is_descendant_of?(parent.reload)
Chris@909 2007
Chris@909 2008 @request.session[:user_id] = 2
Chris@909 2009 assert_difference 'Issue.count', -2 do
Chris@909 2010 post :destroy, :ids => [parent.id, child.id], :todo => 'destroy'
Chris@909 2011 end
Chris@909 2012 assert_response 302
Chris@909 2013 end
Chris@909 2014
Chris@909 2015 def test_default_search_scope
Chris@909 2016 get :index
Chris@909 2017 assert_tag :div, :attributes => {:id => 'quick-search'},
Chris@909 2018 :child => {:tag => 'form',
Chris@909 2019 :child => {:tag => 'input', :attributes => {:name => 'issues', :type => 'hidden', :value => '1'}}}
Chris@909 2020 end
Chris@909 2021 end