annotate .svn/pristine/4b/4b6ad4ee97c77f142f2ea07d73404f963b296d82.svn-base @ 1433:cfa80f738847 bibliography_testing

Fixed assertion error message.
author luisf <luis.figueira@eecs.qmul.ac.uk>
date Tue, 08 Oct 2013 17:29:47 +0100
parents 038ba2d95de8
children
rev   line source
Chris@1296 1 # Redmine - project management software
Chris@1296 2 # Copyright (C) 2006-2012 Jean-Philippe Lang
Chris@1296 3 #
Chris@1296 4 # This program is free software; you can redistribute it and/or
Chris@1296 5 # modify it under the terms of the GNU General Public License
Chris@1296 6 # as published by the Free Software Foundation; either version 2
Chris@1296 7 # of the License, or (at your option) any later version.
Chris@1296 8 #
Chris@1296 9 # This program is distributed in the hope that it will be useful,
Chris@1296 10 # but WITHOUT ANY WARRANTY; without even the implied warranty of
Chris@1296 11 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
Chris@1296 12 # GNU General Public License for more details.
Chris@1296 13 #
Chris@1296 14 # You should have received a copy of the GNU General Public License
Chris@1296 15 # along with this program; if not, write to the Free Software
Chris@1296 16 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
Chris@1296 17
Chris@1296 18 require File.expand_path('../../test_helper', __FILE__)
Chris@1296 19
Chris@1296 20 class IssueTest < ActiveSupport::TestCase
Chris@1296 21 fixtures :projects, :users, :members, :member_roles, :roles,
Chris@1296 22 :groups_users,
Chris@1296 23 :trackers, :projects_trackers,
Chris@1296 24 :enabled_modules,
Chris@1296 25 :versions,
Chris@1296 26 :issue_statuses, :issue_categories, :issue_relations, :workflows,
Chris@1296 27 :enumerations,
Chris@1296 28 :issues, :journals, :journal_details,
Chris@1296 29 :custom_fields, :custom_fields_projects, :custom_fields_trackers, :custom_values,
Chris@1296 30 :time_entries
Chris@1296 31
Chris@1296 32 include Redmine::I18n
Chris@1296 33
Chris@1296 34 def teardown
Chris@1296 35 User.current = nil
Chris@1296 36 end
Chris@1296 37
Chris@1296 38 def test_create
Chris@1296 39 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@1296 40 :status_id => 1, :priority => IssuePriority.all.first,
Chris@1296 41 :subject => 'test_create',
Chris@1296 42 :description => 'IssueTest#test_create', :estimated_hours => '1:30')
Chris@1296 43 assert issue.save
Chris@1296 44 issue.reload
Chris@1296 45 assert_equal 1.5, issue.estimated_hours
Chris@1296 46 end
Chris@1296 47
Chris@1296 48 def test_create_minimal
Chris@1296 49 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@1296 50 :status_id => 1, :priority => IssuePriority.all.first,
Chris@1296 51 :subject => 'test_create')
Chris@1296 52 assert issue.save
Chris@1296 53 assert issue.description.nil?
Chris@1296 54 assert_nil issue.estimated_hours
Chris@1296 55 end
Chris@1296 56
Chris@1296 57 def test_start_date_format_should_be_validated
Chris@1296 58 set_language_if_valid 'en'
Chris@1296 59 ['2012', 'ABC', '2012-15-20'].each do |invalid_date|
Chris@1296 60 issue = Issue.new(:start_date => invalid_date)
Chris@1296 61 assert !issue.valid?
Chris@1296 62 assert_include 'Start date is not a valid date', issue.errors.full_messages, "No error found for invalid date #{invalid_date}"
Chris@1296 63 end
Chris@1296 64 end
Chris@1296 65
Chris@1296 66 def test_due_date_format_should_be_validated
Chris@1296 67 set_language_if_valid 'en'
Chris@1296 68 ['2012', 'ABC', '2012-15-20'].each do |invalid_date|
Chris@1296 69 issue = Issue.new(:due_date => invalid_date)
Chris@1296 70 assert !issue.valid?
Chris@1296 71 assert_include 'Due date is not a valid date', issue.errors.full_messages, "No error found for invalid date #{invalid_date}"
Chris@1296 72 end
Chris@1296 73 end
Chris@1296 74
Chris@1296 75 def test_due_date_lesser_than_start_date_should_not_validate
Chris@1296 76 set_language_if_valid 'en'
Chris@1296 77 issue = Issue.new(:start_date => '2012-10-06', :due_date => '2012-10-02')
Chris@1296 78 assert !issue.valid?
Chris@1296 79 assert_include 'Due date must be greater than start date', issue.errors.full_messages
Chris@1296 80 end
Chris@1296 81
Chris@1296 82 def test_create_with_required_custom_field
Chris@1296 83 set_language_if_valid 'en'
Chris@1296 84 field = IssueCustomField.find_by_name('Database')
Chris@1296 85 field.update_attribute(:is_required, true)
Chris@1296 86
Chris@1296 87 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 88 :status_id => 1, :subject => 'test_create',
Chris@1296 89 :description => 'IssueTest#test_create_with_required_custom_field')
Chris@1296 90 assert issue.available_custom_fields.include?(field)
Chris@1296 91 # No value for the custom field
Chris@1296 92 assert !issue.save
Chris@1296 93 assert_equal ["Database can't be blank"], issue.errors.full_messages
Chris@1296 94 # Blank value
Chris@1296 95 issue.custom_field_values = { field.id => '' }
Chris@1296 96 assert !issue.save
Chris@1296 97 assert_equal ["Database can't be blank"], issue.errors.full_messages
Chris@1296 98 # Invalid value
Chris@1296 99 issue.custom_field_values = { field.id => 'SQLServer' }
Chris@1296 100 assert !issue.save
Chris@1296 101 assert_equal ["Database is not included in the list"], issue.errors.full_messages
Chris@1296 102 # Valid value
Chris@1296 103 issue.custom_field_values = { field.id => 'PostgreSQL' }
Chris@1296 104 assert issue.save
Chris@1296 105 issue.reload
Chris@1296 106 assert_equal 'PostgreSQL', issue.custom_value_for(field).value
Chris@1296 107 end
Chris@1296 108
Chris@1296 109 def test_create_with_group_assignment
Chris@1296 110 with_settings :issue_group_assignment => '1' do
Chris@1296 111 assert Issue.new(:project_id => 2, :tracker_id => 1, :author_id => 1,
Chris@1296 112 :subject => 'Group assignment',
Chris@1296 113 :assigned_to_id => 11).save
Chris@1296 114 issue = Issue.first(:order => 'id DESC')
Chris@1296 115 assert_kind_of Group, issue.assigned_to
Chris@1296 116 assert_equal Group.find(11), issue.assigned_to
Chris@1296 117 end
Chris@1296 118 end
Chris@1296 119
Chris@1296 120 def test_create_with_parent_issue_id
Chris@1296 121 issue = Issue.new(:project_id => 1, :tracker_id => 1,
Chris@1296 122 :author_id => 1, :subject => 'Group assignment',
Chris@1296 123 :parent_issue_id => 1)
Chris@1296 124 assert_save issue
Chris@1296 125 assert_equal 1, issue.parent_issue_id
Chris@1296 126 assert_equal Issue.find(1), issue.parent
Chris@1296 127 end
Chris@1296 128
Chris@1296 129 def test_create_with_sharp_parent_issue_id
Chris@1296 130 issue = Issue.new(:project_id => 1, :tracker_id => 1,
Chris@1296 131 :author_id => 1, :subject => 'Group assignment',
Chris@1296 132 :parent_issue_id => "#1")
Chris@1296 133 assert_save issue
Chris@1296 134 assert_equal 1, issue.parent_issue_id
Chris@1296 135 assert_equal Issue.find(1), issue.parent
Chris@1296 136 end
Chris@1296 137
Chris@1296 138 def test_create_with_invalid_parent_issue_id
Chris@1296 139 set_language_if_valid 'en'
Chris@1296 140 issue = Issue.new(:project_id => 1, :tracker_id => 1,
Chris@1296 141 :author_id => 1, :subject => 'Group assignment',
Chris@1296 142 :parent_issue_id => '01ABC')
Chris@1296 143 assert !issue.save
Chris@1296 144 assert_equal '01ABC', issue.parent_issue_id
Chris@1296 145 assert_include 'Parent task is invalid', issue.errors.full_messages
Chris@1296 146 end
Chris@1296 147
Chris@1296 148 def test_create_with_invalid_sharp_parent_issue_id
Chris@1296 149 set_language_if_valid 'en'
Chris@1296 150 issue = Issue.new(:project_id => 1, :tracker_id => 1,
Chris@1296 151 :author_id => 1, :subject => 'Group assignment',
Chris@1296 152 :parent_issue_id => '#01ABC')
Chris@1296 153 assert !issue.save
Chris@1296 154 assert_equal '#01ABC', issue.parent_issue_id
Chris@1296 155 assert_include 'Parent task is invalid', issue.errors.full_messages
Chris@1296 156 end
Chris@1296 157
Chris@1296 158 def assert_visibility_match(user, issues)
Chris@1296 159 assert_equal issues.collect(&:id).sort, Issue.all.select {|issue| issue.visible?(user)}.collect(&:id).sort
Chris@1296 160 end
Chris@1296 161
Chris@1296 162 def test_visible_scope_for_anonymous
Chris@1296 163 # Anonymous user should see issues of public projects only
Chris@1296 164 issues = Issue.visible(User.anonymous).all
Chris@1296 165 assert issues.any?
Chris@1296 166 assert_nil issues.detect {|issue| !issue.project.is_public?}
Chris@1296 167 assert_nil issues.detect {|issue| issue.is_private?}
Chris@1296 168 assert_visibility_match User.anonymous, issues
Chris@1296 169 end
Chris@1296 170
Chris@1296 171 def test_visible_scope_for_anonymous_without_view_issues_permissions
Chris@1296 172 # Anonymous user should not see issues without permission
Chris@1296 173 Role.anonymous.remove_permission!(:view_issues)
Chris@1296 174 issues = Issue.visible(User.anonymous).all
Chris@1296 175 assert issues.empty?
Chris@1296 176 assert_visibility_match User.anonymous, issues
Chris@1296 177 end
Chris@1296 178
Chris@1296 179 def test_anonymous_should_not_see_private_issues_with_issues_visibility_set_to_default
Chris@1296 180 assert Role.anonymous.update_attribute(:issues_visibility, 'default')
Chris@1296 181 issue = Issue.generate!(:author => User.anonymous, :assigned_to => User.anonymous, :is_private => true)
Chris@1296 182 assert_nil Issue.where(:id => issue.id).visible(User.anonymous).first
Chris@1296 183 assert !issue.visible?(User.anonymous)
Chris@1296 184 end
Chris@1296 185
Chris@1296 186 def test_anonymous_should_not_see_private_issues_with_issues_visibility_set_to_own
Chris@1296 187 assert Role.anonymous.update_attribute(:issues_visibility, 'own')
Chris@1296 188 issue = Issue.generate!(:author => User.anonymous, :assigned_to => User.anonymous, :is_private => true)
Chris@1296 189 assert_nil Issue.where(:id => issue.id).visible(User.anonymous).first
Chris@1296 190 assert !issue.visible?(User.anonymous)
Chris@1296 191 end
Chris@1296 192
Chris@1296 193 def test_visible_scope_for_non_member
Chris@1296 194 user = User.find(9)
Chris@1296 195 assert user.projects.empty?
Chris@1296 196 # Non member user should see issues of public projects only
Chris@1296 197 issues = Issue.visible(user).all
Chris@1296 198 assert issues.any?
Chris@1296 199 assert_nil issues.detect {|issue| !issue.project.is_public?}
Chris@1296 200 assert_nil issues.detect {|issue| issue.is_private?}
Chris@1296 201 assert_visibility_match user, issues
Chris@1296 202 end
Chris@1296 203
Chris@1296 204 def test_visible_scope_for_non_member_with_own_issues_visibility
Chris@1296 205 Role.non_member.update_attribute :issues_visibility, 'own'
Chris@1296 206 Issue.create!(:project_id => 1, :tracker_id => 1, :author_id => 9, :subject => 'Issue by non member')
Chris@1296 207 user = User.find(9)
Chris@1296 208
Chris@1296 209 issues = Issue.visible(user).all
Chris@1296 210 assert issues.any?
Chris@1296 211 assert_nil issues.detect {|issue| issue.author != user}
Chris@1296 212 assert_visibility_match user, issues
Chris@1296 213 end
Chris@1296 214
Chris@1296 215 def test_visible_scope_for_non_member_without_view_issues_permissions
Chris@1296 216 # Non member user should not see issues without permission
Chris@1296 217 Role.non_member.remove_permission!(:view_issues)
Chris@1296 218 user = User.find(9)
Chris@1296 219 assert user.projects.empty?
Chris@1296 220 issues = Issue.visible(user).all
Chris@1296 221 assert issues.empty?
Chris@1296 222 assert_visibility_match user, issues
Chris@1296 223 end
Chris@1296 224
Chris@1296 225 def test_visible_scope_for_member
Chris@1296 226 user = User.find(9)
Chris@1296 227 # User should see issues of projects for which he has view_issues permissions only
Chris@1296 228 Role.non_member.remove_permission!(:view_issues)
Chris@1296 229 Member.create!(:principal => user, :project_id => 3, :role_ids => [2])
Chris@1296 230 issues = Issue.visible(user).all
Chris@1296 231 assert issues.any?
Chris@1296 232 assert_nil issues.detect {|issue| issue.project_id != 3}
Chris@1296 233 assert_nil issues.detect {|issue| issue.is_private?}
Chris@1296 234 assert_visibility_match user, issues
Chris@1296 235 end
Chris@1296 236
Chris@1296 237 def test_visible_scope_for_member_with_groups_should_return_assigned_issues
Chris@1296 238 user = User.find(8)
Chris@1296 239 assert user.groups.any?
Chris@1296 240 Member.create!(:principal => user.groups.first, :project_id => 1, :role_ids => [2])
Chris@1296 241 Role.non_member.remove_permission!(:view_issues)
Chris@1296 242
Chris@1296 243 issue = Issue.create(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@1296 244 :status_id => 1, :priority => IssuePriority.all.first,
Chris@1296 245 :subject => 'Assignment test',
Chris@1296 246 :assigned_to => user.groups.first,
Chris@1296 247 :is_private => true)
Chris@1296 248
Chris@1296 249 Role.find(2).update_attribute :issues_visibility, 'default'
Chris@1296 250 issues = Issue.visible(User.find(8)).all
Chris@1296 251 assert issues.any?
Chris@1296 252 assert issues.include?(issue)
Chris@1296 253
Chris@1296 254 Role.find(2).update_attribute :issues_visibility, 'own'
Chris@1296 255 issues = Issue.visible(User.find(8)).all
Chris@1296 256 assert issues.any?
Chris@1296 257 assert issues.include?(issue)
Chris@1296 258 end
Chris@1296 259
Chris@1296 260 def test_visible_scope_for_admin
Chris@1296 261 user = User.find(1)
Chris@1296 262 user.members.each(&:destroy)
Chris@1296 263 assert user.projects.empty?
Chris@1296 264 issues = Issue.visible(user).all
Chris@1296 265 assert issues.any?
Chris@1296 266 # Admin should see issues on private projects that he does not belong to
Chris@1296 267 assert issues.detect {|issue| !issue.project.is_public?}
Chris@1296 268 # Admin should see private issues of other users
Chris@1296 269 assert issues.detect {|issue| issue.is_private? && issue.author != user}
Chris@1296 270 assert_visibility_match user, issues
Chris@1296 271 end
Chris@1296 272
Chris@1296 273 def test_visible_scope_with_project
Chris@1296 274 project = Project.find(1)
Chris@1296 275 issues = Issue.visible(User.find(2), :project => project).all
Chris@1296 276 projects = issues.collect(&:project).uniq
Chris@1296 277 assert_equal 1, projects.size
Chris@1296 278 assert_equal project, projects.first
Chris@1296 279 end
Chris@1296 280
Chris@1296 281 def test_visible_scope_with_project_and_subprojects
Chris@1296 282 project = Project.find(1)
Chris@1296 283 issues = Issue.visible(User.find(2), :project => project, :with_subprojects => true).all
Chris@1296 284 projects = issues.collect(&:project).uniq
Chris@1296 285 assert projects.size > 1
Chris@1296 286 assert_equal [], projects.select {|p| !p.is_or_is_descendant_of?(project)}
Chris@1296 287 end
Chris@1296 288
Chris@1296 289 def test_visible_and_nested_set_scopes
Chris@1296 290 assert_equal 0, Issue.find(1).descendants.visible.all.size
Chris@1296 291 end
Chris@1296 292
Chris@1296 293 def test_open_scope
Chris@1296 294 issues = Issue.open.all
Chris@1296 295 assert_nil issues.detect(&:closed?)
Chris@1296 296 end
Chris@1296 297
Chris@1296 298 def test_open_scope_with_arg
Chris@1296 299 issues = Issue.open(false).all
Chris@1296 300 assert_equal issues, issues.select(&:closed?)
Chris@1296 301 end
Chris@1296 302
Chris@1296 303 def test_errors_full_messages_should_include_custom_fields_errors
Chris@1296 304 field = IssueCustomField.find_by_name('Database')
Chris@1296 305
Chris@1296 306 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 307 :status_id => 1, :subject => 'test_create',
Chris@1296 308 :description => 'IssueTest#test_create_with_required_custom_field')
Chris@1296 309 assert issue.available_custom_fields.include?(field)
Chris@1296 310 # Invalid value
Chris@1296 311 issue.custom_field_values = { field.id => 'SQLServer' }
Chris@1296 312
Chris@1296 313 assert !issue.valid?
Chris@1296 314 assert_equal 1, issue.errors.full_messages.size
Chris@1296 315 assert_equal "Database #{I18n.translate('activerecord.errors.messages.inclusion')}",
Chris@1296 316 issue.errors.full_messages.first
Chris@1296 317 end
Chris@1296 318
Chris@1296 319 def test_update_issue_with_required_custom_field
Chris@1296 320 field = IssueCustomField.find_by_name('Database')
Chris@1296 321 field.update_attribute(:is_required, true)
Chris@1296 322
Chris@1296 323 issue = Issue.find(1)
Chris@1296 324 assert_nil issue.custom_value_for(field)
Chris@1296 325 assert issue.available_custom_fields.include?(field)
Chris@1296 326 # No change to custom values, issue can be saved
Chris@1296 327 assert issue.save
Chris@1296 328 # Blank value
Chris@1296 329 issue.custom_field_values = { field.id => '' }
Chris@1296 330 assert !issue.save
Chris@1296 331 # Valid value
Chris@1296 332 issue.custom_field_values = { field.id => 'PostgreSQL' }
Chris@1296 333 assert issue.save
Chris@1296 334 issue.reload
Chris@1296 335 assert_equal 'PostgreSQL', issue.custom_value_for(field).value
Chris@1296 336 end
Chris@1296 337
Chris@1296 338 def test_should_not_update_attributes_if_custom_fields_validation_fails
Chris@1296 339 issue = Issue.find(1)
Chris@1296 340 field = IssueCustomField.find_by_name('Database')
Chris@1296 341 assert issue.available_custom_fields.include?(field)
Chris@1296 342
Chris@1296 343 issue.custom_field_values = { field.id => 'Invalid' }
Chris@1296 344 issue.subject = 'Should be not be saved'
Chris@1296 345 assert !issue.save
Chris@1296 346
Chris@1296 347 issue.reload
Chris@1296 348 assert_equal "Can't print recipes", issue.subject
Chris@1296 349 end
Chris@1296 350
Chris@1296 351 def test_should_not_recreate_custom_values_objects_on_update
Chris@1296 352 field = IssueCustomField.find_by_name('Database')
Chris@1296 353
Chris@1296 354 issue = Issue.find(1)
Chris@1296 355 issue.custom_field_values = { field.id => 'PostgreSQL' }
Chris@1296 356 assert issue.save
Chris@1296 357 custom_value = issue.custom_value_for(field)
Chris@1296 358 issue.reload
Chris@1296 359 issue.custom_field_values = { field.id => 'MySQL' }
Chris@1296 360 assert issue.save
Chris@1296 361 issue.reload
Chris@1296 362 assert_equal custom_value.id, issue.custom_value_for(field).id
Chris@1296 363 end
Chris@1296 364
Chris@1296 365 def test_should_not_update_custom_fields_on_changing_tracker_with_different_custom_fields
Chris@1296 366 issue = Issue.create!(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 367 :status_id => 1, :subject => 'Test',
Chris@1296 368 :custom_field_values => {'2' => 'Test'})
Chris@1296 369 assert !Tracker.find(2).custom_field_ids.include?(2)
Chris@1296 370
Chris@1296 371 issue = Issue.find(issue.id)
Chris@1296 372 issue.attributes = {:tracker_id => 2, :custom_field_values => {'1' => ''}}
Chris@1296 373
Chris@1296 374 issue = Issue.find(issue.id)
Chris@1296 375 custom_value = issue.custom_value_for(2)
Chris@1296 376 assert_not_nil custom_value
Chris@1296 377 assert_equal 'Test', custom_value.value
Chris@1296 378 end
Chris@1296 379
Chris@1296 380 def test_assigning_tracker_id_should_reload_custom_fields_values
Chris@1296 381 issue = Issue.new(:project => Project.find(1))
Chris@1296 382 assert issue.custom_field_values.empty?
Chris@1296 383 issue.tracker_id = 1
Chris@1296 384 assert issue.custom_field_values.any?
Chris@1296 385 end
Chris@1296 386
Chris@1296 387 def test_assigning_attributes_should_assign_project_and_tracker_first
Chris@1296 388 seq = sequence('seq')
Chris@1296 389 issue = Issue.new
Chris@1296 390 issue.expects(:project_id=).in_sequence(seq)
Chris@1296 391 issue.expects(:tracker_id=).in_sequence(seq)
Chris@1296 392 issue.expects(:subject=).in_sequence(seq)
Chris@1296 393 issue.attributes = {:tracker_id => 2, :project_id => 1, :subject => 'Test'}
Chris@1296 394 end
Chris@1296 395
Chris@1296 396 def test_assigning_tracker_and_custom_fields_should_assign_custom_fields
Chris@1296 397 attributes = ActiveSupport::OrderedHash.new
Chris@1296 398 attributes['custom_field_values'] = { '1' => 'MySQL' }
Chris@1296 399 attributes['tracker_id'] = '1'
Chris@1296 400 issue = Issue.new(:project => Project.find(1))
Chris@1296 401 issue.attributes = attributes
Chris@1296 402 assert_equal 'MySQL', issue.custom_field_value(1)
Chris@1296 403 end
Chris@1296 404
Chris@1296 405 def test_should_update_issue_with_disabled_tracker
Chris@1296 406 p = Project.find(1)
Chris@1296 407 issue = Issue.find(1)
Chris@1296 408
Chris@1296 409 p.trackers.delete(issue.tracker)
Chris@1296 410 assert !p.trackers.include?(issue.tracker)
Chris@1296 411
Chris@1296 412 issue.reload
Chris@1296 413 issue.subject = 'New subject'
Chris@1296 414 assert issue.save
Chris@1296 415 end
Chris@1296 416
Chris@1296 417 def test_should_not_set_a_disabled_tracker
Chris@1296 418 p = Project.find(1)
Chris@1296 419 p.trackers.delete(Tracker.find(2))
Chris@1296 420
Chris@1296 421 issue = Issue.find(1)
Chris@1296 422 issue.tracker_id = 2
Chris@1296 423 issue.subject = 'New subject'
Chris@1296 424 assert !issue.save
Chris@1296 425 assert_not_nil issue.errors[:tracker_id]
Chris@1296 426 end
Chris@1296 427
Chris@1296 428 def test_category_based_assignment
Chris@1296 429 issue = Issue.create(:project_id => 1, :tracker_id => 1, :author_id => 3,
Chris@1296 430 :status_id => 1, :priority => IssuePriority.all.first,
Chris@1296 431 :subject => 'Assignment test',
Chris@1296 432 :description => 'Assignment test', :category_id => 1)
Chris@1296 433 assert_equal IssueCategory.find(1).assigned_to, issue.assigned_to
Chris@1296 434 end
Chris@1296 435
Chris@1296 436 def test_new_statuses_allowed_to
Chris@1296 437 WorkflowTransition.delete_all
Chris@1296 438 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1,
Chris@1296 439 :old_status_id => 1, :new_status_id => 2,
Chris@1296 440 :author => false, :assignee => false)
Chris@1296 441 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1,
Chris@1296 442 :old_status_id => 1, :new_status_id => 3,
Chris@1296 443 :author => true, :assignee => false)
Chris@1296 444 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1, :old_status_id => 1,
Chris@1296 445 :new_status_id => 4, :author => false,
Chris@1296 446 :assignee => true)
Chris@1296 447 WorkflowTransition.create!(:role_id => 1, :tracker_id => 1,
Chris@1296 448 :old_status_id => 1, :new_status_id => 5,
Chris@1296 449 :author => true, :assignee => true)
Chris@1296 450 status = IssueStatus.find(1)
Chris@1296 451 role = Role.find(1)
Chris@1296 452 tracker = Tracker.find(1)
Chris@1296 453 user = User.find(2)
Chris@1296 454
Chris@1296 455 issue = Issue.generate!(:tracker => tracker, :status => status,
Chris@1296 456 :project_id => 1, :author_id => 1)
Chris@1296 457 assert_equal [1, 2], issue.new_statuses_allowed_to(user).map(&:id)
Chris@1296 458
Chris@1296 459 issue = Issue.generate!(:tracker => tracker, :status => status,
Chris@1296 460 :project_id => 1, :author => user)
Chris@1296 461 assert_equal [1, 2, 3, 5], issue.new_statuses_allowed_to(user).map(&:id)
Chris@1296 462
Chris@1296 463 issue = Issue.generate!(:tracker => tracker, :status => status,
Chris@1296 464 :project_id => 1, :author_id => 1,
Chris@1296 465 :assigned_to => user)
Chris@1296 466 assert_equal [1, 2, 4, 5], issue.new_statuses_allowed_to(user).map(&:id)
Chris@1296 467
Chris@1296 468 issue = Issue.generate!(:tracker => tracker, :status => status,
Chris@1296 469 :project_id => 1, :author => user,
Chris@1296 470 :assigned_to => user)
Chris@1296 471 assert_equal [1, 2, 3, 4, 5], issue.new_statuses_allowed_to(user).map(&:id)
Chris@1296 472 end
Chris@1296 473
Chris@1296 474 def test_new_statuses_allowed_to_should_return_all_transitions_for_admin
Chris@1296 475 admin = User.find(1)
Chris@1296 476 issue = Issue.find(1)
Chris@1296 477 assert !admin.member_of?(issue.project)
Chris@1296 478 expected_statuses = [issue.status] +
Chris@1296 479 WorkflowTransition.find_all_by_old_status_id(
Chris@1296 480 issue.status_id).map(&:new_status).uniq.sort
Chris@1296 481 assert_equal expected_statuses, issue.new_statuses_allowed_to(admin)
Chris@1296 482 end
Chris@1296 483
Chris@1296 484 def test_new_statuses_allowed_to_should_return_default_and_current_status_when_copying
Chris@1296 485 issue = Issue.find(1).copy
Chris@1296 486 assert_equal [1], issue.new_statuses_allowed_to(User.find(2)).map(&:id)
Chris@1296 487
Chris@1296 488 issue = Issue.find(2).copy
Chris@1296 489 assert_equal [1, 2], issue.new_statuses_allowed_to(User.find(2)).map(&:id)
Chris@1296 490 end
Chris@1296 491
Chris@1296 492 def test_safe_attributes_names_should_not_include_disabled_field
Chris@1296 493 tracker = Tracker.new(:core_fields => %w(assigned_to_id fixed_version_id))
Chris@1296 494
Chris@1296 495 issue = Issue.new(:tracker => tracker)
Chris@1296 496 assert_include 'tracker_id', issue.safe_attribute_names
Chris@1296 497 assert_include 'status_id', issue.safe_attribute_names
Chris@1296 498 assert_include 'subject', issue.safe_attribute_names
Chris@1296 499 assert_include 'description', issue.safe_attribute_names
Chris@1296 500 assert_include 'custom_field_values', issue.safe_attribute_names
Chris@1296 501 assert_include 'custom_fields', issue.safe_attribute_names
Chris@1296 502 assert_include 'lock_version', issue.safe_attribute_names
Chris@1296 503
Chris@1296 504 tracker.core_fields.each do |field|
Chris@1296 505 assert_include field, issue.safe_attribute_names
Chris@1296 506 end
Chris@1296 507
Chris@1296 508 tracker.disabled_core_fields.each do |field|
Chris@1296 509 assert_not_include field, issue.safe_attribute_names
Chris@1296 510 end
Chris@1296 511 end
Chris@1296 512
Chris@1296 513 def test_safe_attributes_should_ignore_disabled_fields
Chris@1296 514 tracker = Tracker.find(1)
Chris@1296 515 tracker.core_fields = %w(assigned_to_id due_date)
Chris@1296 516 tracker.save!
Chris@1296 517
Chris@1296 518 issue = Issue.new(:tracker => tracker)
Chris@1296 519 issue.safe_attributes = {'start_date' => '2012-07-14', 'due_date' => '2012-07-14'}
Chris@1296 520 assert_nil issue.start_date
Chris@1296 521 assert_equal Date.parse('2012-07-14'), issue.due_date
Chris@1296 522 end
Chris@1296 523
Chris@1296 524 def test_safe_attributes_should_accept_target_tracker_enabled_fields
Chris@1296 525 source = Tracker.find(1)
Chris@1296 526 source.core_fields = []
Chris@1296 527 source.save!
Chris@1296 528 target = Tracker.find(2)
Chris@1296 529 target.core_fields = %w(assigned_to_id due_date)
Chris@1296 530 target.save!
Chris@1296 531
Chris@1296 532 issue = Issue.new(:tracker => source)
Chris@1296 533 issue.safe_attributes = {'tracker_id' => 2, 'due_date' => '2012-07-14'}
Chris@1296 534 assert_equal target, issue.tracker
Chris@1296 535 assert_equal Date.parse('2012-07-14'), issue.due_date
Chris@1296 536 end
Chris@1296 537
Chris@1296 538 def test_safe_attributes_should_not_include_readonly_fields
Chris@1296 539 WorkflowPermission.delete_all
Chris@1296 540 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 541 :role_id => 1, :field_name => 'due_date',
Chris@1296 542 :rule => 'readonly')
Chris@1296 543 user = User.find(2)
Chris@1296 544
Chris@1296 545 issue = Issue.new(:project_id => 1, :tracker_id => 1)
Chris@1296 546 assert_equal %w(due_date), issue.read_only_attribute_names(user)
Chris@1296 547 assert_not_include 'due_date', issue.safe_attribute_names(user)
Chris@1296 548
Chris@1296 549 issue.send :safe_attributes=, {'start_date' => '2012-07-14', 'due_date' => '2012-07-14'}, user
Chris@1296 550 assert_equal Date.parse('2012-07-14'), issue.start_date
Chris@1296 551 assert_nil issue.due_date
Chris@1296 552 end
Chris@1296 553
Chris@1296 554 def test_safe_attributes_should_not_include_readonly_custom_fields
Chris@1296 555 cf1 = IssueCustomField.create!(:name => 'Writable field',
Chris@1296 556 :field_format => 'string',
Chris@1296 557 :is_for_all => true, :tracker_ids => [1])
Chris@1296 558 cf2 = IssueCustomField.create!(:name => 'Readonly field',
Chris@1296 559 :field_format => 'string',
Chris@1296 560 :is_for_all => true, :tracker_ids => [1])
Chris@1296 561 WorkflowPermission.delete_all
Chris@1296 562 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 563 :role_id => 1, :field_name => cf2.id.to_s,
Chris@1296 564 :rule => 'readonly')
Chris@1296 565 user = User.find(2)
Chris@1296 566 issue = Issue.new(:project_id => 1, :tracker_id => 1)
Chris@1296 567 assert_equal [cf2.id.to_s], issue.read_only_attribute_names(user)
Chris@1296 568 assert_not_include cf2.id.to_s, issue.safe_attribute_names(user)
Chris@1296 569
Chris@1296 570 issue.send :safe_attributes=, {'custom_field_values' => {
Chris@1296 571 cf1.id.to_s => 'value1', cf2.id.to_s => 'value2'
Chris@1296 572 }}, user
Chris@1296 573 assert_equal 'value1', issue.custom_field_value(cf1)
Chris@1296 574 assert_nil issue.custom_field_value(cf2)
Chris@1296 575
Chris@1296 576 issue.send :safe_attributes=, {'custom_fields' => [
Chris@1296 577 {'id' => cf1.id.to_s, 'value' => 'valuea'},
Chris@1296 578 {'id' => cf2.id.to_s, 'value' => 'valueb'}
Chris@1296 579 ]}, user
Chris@1296 580 assert_equal 'valuea', issue.custom_field_value(cf1)
Chris@1296 581 assert_nil issue.custom_field_value(cf2)
Chris@1296 582 end
Chris@1296 583
Chris@1296 584 def test_editable_custom_field_values_should_return_non_readonly_custom_values
Chris@1296 585 cf1 = IssueCustomField.create!(:name => 'Writable field', :field_format => 'string',
Chris@1296 586 :is_for_all => true, :tracker_ids => [1, 2])
Chris@1296 587 cf2 = IssueCustomField.create!(:name => 'Readonly field', :field_format => 'string',
Chris@1296 588 :is_for_all => true, :tracker_ids => [1, 2])
Chris@1296 589 WorkflowPermission.delete_all
Chris@1296 590 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, :role_id => 1,
Chris@1296 591 :field_name => cf2.id.to_s, :rule => 'readonly')
Chris@1296 592 user = User.find(2)
Chris@1296 593
Chris@1296 594 issue = Issue.new(:project_id => 1, :tracker_id => 1)
Chris@1296 595 values = issue.editable_custom_field_values(user)
Chris@1296 596 assert values.detect {|value| value.custom_field == cf1}
Chris@1296 597 assert_nil values.detect {|value| value.custom_field == cf2}
Chris@1296 598
Chris@1296 599 issue.tracker_id = 2
Chris@1296 600 values = issue.editable_custom_field_values(user)
Chris@1296 601 assert values.detect {|value| value.custom_field == cf1}
Chris@1296 602 assert values.detect {|value| value.custom_field == cf2}
Chris@1296 603 end
Chris@1296 604
Chris@1296 605 def test_safe_attributes_should_accept_target_tracker_writable_fields
Chris@1296 606 WorkflowPermission.delete_all
Chris@1296 607 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 608 :role_id => 1, :field_name => 'due_date',
Chris@1296 609 :rule => 'readonly')
Chris@1296 610 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2,
Chris@1296 611 :role_id => 1, :field_name => 'start_date',
Chris@1296 612 :rule => 'readonly')
Chris@1296 613 user = User.find(2)
Chris@1296 614
Chris@1296 615 issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1)
Chris@1296 616
Chris@1296 617 issue.send :safe_attributes=, {'start_date' => '2012-07-12',
Chris@1296 618 'due_date' => '2012-07-14'}, user
Chris@1296 619 assert_equal Date.parse('2012-07-12'), issue.start_date
Chris@1296 620 assert_nil issue.due_date
Chris@1296 621
Chris@1296 622 issue.send :safe_attributes=, {'start_date' => '2012-07-15',
Chris@1296 623 'due_date' => '2012-07-16',
Chris@1296 624 'tracker_id' => 2}, user
Chris@1296 625 assert_equal Date.parse('2012-07-12'), issue.start_date
Chris@1296 626 assert_equal Date.parse('2012-07-16'), issue.due_date
Chris@1296 627 end
Chris@1296 628
Chris@1296 629 def test_safe_attributes_should_accept_target_status_writable_fields
Chris@1296 630 WorkflowPermission.delete_all
Chris@1296 631 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 632 :role_id => 1, :field_name => 'due_date',
Chris@1296 633 :rule => 'readonly')
Chris@1296 634 WorkflowPermission.create!(:old_status_id => 2, :tracker_id => 1,
Chris@1296 635 :role_id => 1, :field_name => 'start_date',
Chris@1296 636 :rule => 'readonly')
Chris@1296 637 user = User.find(2)
Chris@1296 638
Chris@1296 639 issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1)
Chris@1296 640
Chris@1296 641 issue.send :safe_attributes=, {'start_date' => '2012-07-12',
Chris@1296 642 'due_date' => '2012-07-14'},
Chris@1296 643 user
Chris@1296 644 assert_equal Date.parse('2012-07-12'), issue.start_date
Chris@1296 645 assert_nil issue.due_date
Chris@1296 646
Chris@1296 647 issue.send :safe_attributes=, {'start_date' => '2012-07-15',
Chris@1296 648 'due_date' => '2012-07-16',
Chris@1296 649 'status_id' => 2},
Chris@1296 650 user
Chris@1296 651 assert_equal Date.parse('2012-07-12'), issue.start_date
Chris@1296 652 assert_equal Date.parse('2012-07-16'), issue.due_date
Chris@1296 653 end
Chris@1296 654
Chris@1296 655 def test_required_attributes_should_be_validated
Chris@1296 656 cf = IssueCustomField.create!(:name => 'Foo', :field_format => 'string',
Chris@1296 657 :is_for_all => true, :tracker_ids => [1, 2])
Chris@1296 658
Chris@1296 659 WorkflowPermission.delete_all
Chris@1296 660 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 661 :role_id => 1, :field_name => 'due_date',
Chris@1296 662 :rule => 'required')
Chris@1296 663 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 664 :role_id => 1, :field_name => 'category_id',
Chris@1296 665 :rule => 'required')
Chris@1296 666 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 667 :role_id => 1, :field_name => cf.id.to_s,
Chris@1296 668 :rule => 'required')
Chris@1296 669
Chris@1296 670 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2,
Chris@1296 671 :role_id => 1, :field_name => 'start_date',
Chris@1296 672 :rule => 'required')
Chris@1296 673 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 2,
Chris@1296 674 :role_id => 1, :field_name => cf.id.to_s,
Chris@1296 675 :rule => 'required')
Chris@1296 676 user = User.find(2)
Chris@1296 677
Chris@1296 678 issue = Issue.new(:project_id => 1, :tracker_id => 1,
Chris@1296 679 :status_id => 1, :subject => 'Required fields',
Chris@1296 680 :author => user)
Chris@1296 681 assert_equal [cf.id.to_s, "category_id", "due_date"],
Chris@1296 682 issue.required_attribute_names(user).sort
Chris@1296 683 assert !issue.save, "Issue was saved"
Chris@1296 684 assert_equal ["Category can't be blank", "Due date can't be blank", "Foo can't be blank"],
Chris@1296 685 issue.errors.full_messages.sort
Chris@1296 686
Chris@1296 687 issue.tracker_id = 2
Chris@1296 688 assert_equal [cf.id.to_s, "start_date"], issue.required_attribute_names(user).sort
Chris@1296 689 assert !issue.save, "Issue was saved"
Chris@1296 690 assert_equal ["Foo can't be blank", "Start date can't be blank"],
Chris@1296 691 issue.errors.full_messages.sort
Chris@1296 692
Chris@1296 693 issue.start_date = Date.today
Chris@1296 694 issue.custom_field_values = {cf.id.to_s => 'bar'}
Chris@1296 695 assert issue.save
Chris@1296 696 end
Chris@1296 697
Chris@1296 698 def test_required_attribute_names_for_multiple_roles_should_intersect_rules
Chris@1296 699 WorkflowPermission.delete_all
Chris@1296 700 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 701 :role_id => 1, :field_name => 'due_date',
Chris@1296 702 :rule => 'required')
Chris@1296 703 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 704 :role_id => 1, :field_name => 'start_date',
Chris@1296 705 :rule => 'required')
Chris@1296 706 user = User.find(2)
Chris@1296 707 member = Member.find(1)
Chris@1296 708 issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1)
Chris@1296 709
Chris@1296 710 assert_equal %w(due_date start_date), issue.required_attribute_names(user).sort
Chris@1296 711
Chris@1296 712 member.role_ids = [1, 2]
Chris@1296 713 member.save!
Chris@1296 714 assert_equal [], issue.required_attribute_names(user.reload)
Chris@1296 715
Chris@1296 716 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 717 :role_id => 2, :field_name => 'due_date',
Chris@1296 718 :rule => 'required')
Chris@1296 719 assert_equal %w(due_date), issue.required_attribute_names(user)
Chris@1296 720
Chris@1296 721 member.role_ids = [1, 2, 3]
Chris@1296 722 member.save!
Chris@1296 723 assert_equal [], issue.required_attribute_names(user.reload)
Chris@1296 724
Chris@1296 725 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 726 :role_id => 2, :field_name => 'due_date',
Chris@1296 727 :rule => 'readonly')
Chris@1296 728 # required + readonly => required
Chris@1296 729 assert_equal %w(due_date), issue.required_attribute_names(user)
Chris@1296 730 end
Chris@1296 731
Chris@1296 732 def test_read_only_attribute_names_for_multiple_roles_should_intersect_rules
Chris@1296 733 WorkflowPermission.delete_all
Chris@1296 734 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 735 :role_id => 1, :field_name => 'due_date',
Chris@1296 736 :rule => 'readonly')
Chris@1296 737 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 738 :role_id => 1, :field_name => 'start_date',
Chris@1296 739 :rule => 'readonly')
Chris@1296 740 user = User.find(2)
Chris@1296 741 member = Member.find(1)
Chris@1296 742 issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1)
Chris@1296 743
Chris@1296 744 assert_equal %w(due_date start_date), issue.read_only_attribute_names(user).sort
Chris@1296 745
Chris@1296 746 member.role_ids = [1, 2]
Chris@1296 747 member.save!
Chris@1296 748 assert_equal [], issue.read_only_attribute_names(user.reload)
Chris@1296 749
Chris@1296 750 WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1,
Chris@1296 751 :role_id => 2, :field_name => 'due_date',
Chris@1296 752 :rule => 'readonly')
Chris@1296 753 assert_equal %w(due_date), issue.read_only_attribute_names(user)
Chris@1296 754 end
Chris@1296 755
Chris@1296 756 def test_copy
Chris@1296 757 issue = Issue.new.copy_from(1)
Chris@1296 758 assert issue.copy?
Chris@1296 759 assert issue.save
Chris@1296 760 issue.reload
Chris@1296 761 orig = Issue.find(1)
Chris@1296 762 assert_equal orig.subject, issue.subject
Chris@1296 763 assert_equal orig.tracker, issue.tracker
Chris@1296 764 assert_equal "125", issue.custom_value_for(2).value
Chris@1296 765 end
Chris@1296 766
Chris@1296 767 def test_copy_should_copy_status
Chris@1296 768 orig = Issue.find(8)
Chris@1296 769 assert orig.status != IssueStatus.default
Chris@1296 770
Chris@1296 771 issue = Issue.new.copy_from(orig)
Chris@1296 772 assert issue.save
Chris@1296 773 issue.reload
Chris@1296 774 assert_equal orig.status, issue.status
Chris@1296 775 end
Chris@1296 776
Chris@1296 777 def test_copy_should_add_relation_with_copied_issue
Chris@1296 778 copied = Issue.find(1)
Chris@1296 779 issue = Issue.new.copy_from(copied)
Chris@1296 780 assert issue.save
Chris@1296 781 issue.reload
Chris@1296 782
Chris@1296 783 assert_equal 1, issue.relations.size
Chris@1296 784 relation = issue.relations.first
Chris@1296 785 assert_equal 'copied_to', relation.relation_type
Chris@1296 786 assert_equal copied, relation.issue_from
Chris@1296 787 assert_equal issue, relation.issue_to
Chris@1296 788 end
Chris@1296 789
Chris@1296 790 def test_copy_should_copy_subtasks
Chris@1296 791 issue = Issue.generate_with_descendants!
Chris@1296 792
Chris@1296 793 copy = issue.reload.copy
Chris@1296 794 copy.author = User.find(7)
Chris@1296 795 assert_difference 'Issue.count', 1+issue.descendants.count do
Chris@1296 796 assert copy.save
Chris@1296 797 end
Chris@1296 798 copy.reload
Chris@1296 799 assert_equal %w(Child1 Child2), copy.children.map(&:subject).sort
Chris@1296 800 child_copy = copy.children.detect {|c| c.subject == 'Child1'}
Chris@1296 801 assert_equal %w(Child11), child_copy.children.map(&:subject).sort
Chris@1296 802 assert_equal copy.author, child_copy.author
Chris@1296 803 end
Chris@1296 804
Chris@1296 805 def test_copy_should_copy_subtasks_to_target_project
Chris@1296 806 issue = Issue.generate_with_descendants!
Chris@1296 807
Chris@1296 808 copy = issue.copy(:project_id => 3)
Chris@1296 809 assert_difference 'Issue.count', 1+issue.descendants.count do
Chris@1296 810 assert copy.save
Chris@1296 811 end
Chris@1296 812 assert_equal [3], copy.reload.descendants.map(&:project_id).uniq
Chris@1296 813 end
Chris@1296 814
Chris@1296 815 def test_copy_should_not_copy_subtasks_twice_when_saving_twice
Chris@1296 816 issue = Issue.generate_with_descendants!
Chris@1296 817
Chris@1296 818 copy = issue.reload.copy
Chris@1296 819 assert_difference 'Issue.count', 1+issue.descendants.count do
Chris@1296 820 assert copy.save
Chris@1296 821 assert copy.save
Chris@1296 822 end
Chris@1296 823 end
Chris@1296 824
Chris@1296 825 def test_should_not_call_after_project_change_on_creation
Chris@1296 826 issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1,
Chris@1296 827 :subject => 'Test', :author_id => 1)
Chris@1296 828 issue.expects(:after_project_change).never
Chris@1296 829 issue.save!
Chris@1296 830 end
Chris@1296 831
Chris@1296 832 def test_should_not_call_after_project_change_on_update
Chris@1296 833 issue = Issue.find(1)
Chris@1296 834 issue.project = Project.find(1)
Chris@1296 835 issue.subject = 'No project change'
Chris@1296 836 issue.expects(:after_project_change).never
Chris@1296 837 issue.save!
Chris@1296 838 end
Chris@1296 839
Chris@1296 840 def test_should_call_after_project_change_on_project_change
Chris@1296 841 issue = Issue.find(1)
Chris@1296 842 issue.project = Project.find(2)
Chris@1296 843 issue.expects(:after_project_change).once
Chris@1296 844 issue.save!
Chris@1296 845 end
Chris@1296 846
Chris@1296 847 def test_adding_journal_should_update_timestamp
Chris@1296 848 issue = Issue.find(1)
Chris@1296 849 updated_on_was = issue.updated_on
Chris@1296 850
Chris@1296 851 issue.init_journal(User.first, "Adding notes")
Chris@1296 852 assert_difference 'Journal.count' do
Chris@1296 853 assert issue.save
Chris@1296 854 end
Chris@1296 855 issue.reload
Chris@1296 856
Chris@1296 857 assert_not_equal updated_on_was, issue.updated_on
Chris@1296 858 end
Chris@1296 859
Chris@1296 860 def test_should_close_duplicates
Chris@1296 861 # Create 3 issues
Chris@1296 862 issue1 = Issue.generate!
Chris@1296 863 issue2 = Issue.generate!
Chris@1296 864 issue3 = Issue.generate!
Chris@1296 865
Chris@1296 866 # 2 is a dupe of 1
Chris@1296 867 IssueRelation.create!(:issue_from => issue2, :issue_to => issue1,
Chris@1296 868 :relation_type => IssueRelation::TYPE_DUPLICATES)
Chris@1296 869 # And 3 is a dupe of 2
Chris@1296 870 IssueRelation.create!(:issue_from => issue3, :issue_to => issue2,
Chris@1296 871 :relation_type => IssueRelation::TYPE_DUPLICATES)
Chris@1296 872 # And 3 is a dupe of 1 (circular duplicates)
Chris@1296 873 IssueRelation.create!(:issue_from => issue3, :issue_to => issue1,
Chris@1296 874 :relation_type => IssueRelation::TYPE_DUPLICATES)
Chris@1296 875
Chris@1296 876 assert issue1.reload.duplicates.include?(issue2)
Chris@1296 877
Chris@1296 878 # Closing issue 1
Chris@1296 879 issue1.init_journal(User.find(:first), "Closing issue1")
Chris@1296 880 issue1.status = IssueStatus.find :first, :conditions => {:is_closed => true}
Chris@1296 881 assert issue1.save
Chris@1296 882 # 2 and 3 should be also closed
Chris@1296 883 assert issue2.reload.closed?
Chris@1296 884 assert issue3.reload.closed?
Chris@1296 885 end
Chris@1296 886
Chris@1296 887 def test_should_not_close_duplicated_issue
Chris@1296 888 issue1 = Issue.generate!
Chris@1296 889 issue2 = Issue.generate!
Chris@1296 890
Chris@1296 891 # 2 is a dupe of 1
Chris@1296 892 IssueRelation.create(:issue_from => issue2, :issue_to => issue1,
Chris@1296 893 :relation_type => IssueRelation::TYPE_DUPLICATES)
Chris@1296 894 # 2 is a dup of 1 but 1 is not a duplicate of 2
Chris@1296 895 assert !issue2.reload.duplicates.include?(issue1)
Chris@1296 896
Chris@1296 897 # Closing issue 2
Chris@1296 898 issue2.init_journal(User.find(:first), "Closing issue2")
Chris@1296 899 issue2.status = IssueStatus.find :first, :conditions => {:is_closed => true}
Chris@1296 900 assert issue2.save
Chris@1296 901 # 1 should not be also closed
Chris@1296 902 assert !issue1.reload.closed?
Chris@1296 903 end
Chris@1296 904
Chris@1296 905 def test_assignable_versions
Chris@1296 906 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 907 :status_id => 1, :fixed_version_id => 1,
Chris@1296 908 :subject => 'New issue')
Chris@1296 909 assert_equal ['open'], issue.assignable_versions.collect(&:status).uniq
Chris@1296 910 end
Chris@1296 911
Chris@1296 912 def test_should_not_be_able_to_assign_a_new_issue_to_a_closed_version
Chris@1296 913 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 914 :status_id => 1, :fixed_version_id => 1,
Chris@1296 915 :subject => 'New issue')
Chris@1296 916 assert !issue.save
Chris@1296 917 assert_not_nil issue.errors[:fixed_version_id]
Chris@1296 918 end
Chris@1296 919
Chris@1296 920 def test_should_not_be_able_to_assign_a_new_issue_to_a_locked_version
Chris@1296 921 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 922 :status_id => 1, :fixed_version_id => 2,
Chris@1296 923 :subject => 'New issue')
Chris@1296 924 assert !issue.save
Chris@1296 925 assert_not_nil issue.errors[:fixed_version_id]
Chris@1296 926 end
Chris@1296 927
Chris@1296 928 def test_should_be_able_to_assign_a_new_issue_to_an_open_version
Chris@1296 929 issue = Issue.new(:project_id => 1, :tracker_id => 1, :author_id => 1,
Chris@1296 930 :status_id => 1, :fixed_version_id => 3,
Chris@1296 931 :subject => 'New issue')
Chris@1296 932 assert issue.save
Chris@1296 933 end
Chris@1296 934
Chris@1296 935 def test_should_be_able_to_update_an_issue_assigned_to_a_closed_version
Chris@1296 936 issue = Issue.find(11)
Chris@1296 937 assert_equal 'closed', issue.fixed_version.status
Chris@1296 938 issue.subject = 'Subject changed'
Chris@1296 939 assert issue.save
Chris@1296 940 end
Chris@1296 941
Chris@1296 942 def test_should_not_be_able_to_reopen_an_issue_assigned_to_a_closed_version
Chris@1296 943 issue = Issue.find(11)
Chris@1296 944 issue.status_id = 1
Chris@1296 945 assert !issue.save
Chris@1296 946 assert_not_nil issue.errors[:base]
Chris@1296 947 end
Chris@1296 948
Chris@1296 949 def test_should_be_able_to_reopen_and_reassign_an_issue_assigned_to_a_closed_version
Chris@1296 950 issue = Issue.find(11)
Chris@1296 951 issue.status_id = 1
Chris@1296 952 issue.fixed_version_id = 3
Chris@1296 953 assert issue.save
Chris@1296 954 end
Chris@1296 955
Chris@1296 956 def test_should_be_able_to_reopen_an_issue_assigned_to_a_locked_version
Chris@1296 957 issue = Issue.find(12)
Chris@1296 958 assert_equal 'locked', issue.fixed_version.status
Chris@1296 959 issue.status_id = 1
Chris@1296 960 assert issue.save
Chris@1296 961 end
Chris@1296 962
Chris@1296 963 def test_should_not_be_able_to_keep_unshared_version_when_changing_project
Chris@1296 964 issue = Issue.find(2)
Chris@1296 965 assert_equal 2, issue.fixed_version_id
Chris@1296 966 issue.project_id = 3
Chris@1296 967 assert_nil issue.fixed_version_id
Chris@1296 968 issue.fixed_version_id = 2
Chris@1296 969 assert !issue.save
Chris@1296 970 assert_include 'Target version is not included in the list', issue.errors.full_messages
Chris@1296 971 end
Chris@1296 972
Chris@1296 973 def test_should_keep_shared_version_when_changing_project
Chris@1296 974 Version.find(2).update_attribute :sharing, 'tree'
Chris@1296 975
Chris@1296 976 issue = Issue.find(2)
Chris@1296 977 assert_equal 2, issue.fixed_version_id
Chris@1296 978 issue.project_id = 3
Chris@1296 979 assert_equal 2, issue.fixed_version_id
Chris@1296 980 assert issue.save
Chris@1296 981 end
Chris@1296 982
Chris@1296 983 def test_allowed_target_projects_on_move_should_include_projects_with_issue_tracking_enabled
Chris@1296 984 assert_include Project.find(2), Issue.allowed_target_projects_on_move(User.find(2))
Chris@1296 985 end
Chris@1296 986
Chris@1296 987 def test_allowed_target_projects_on_move_should_not_include_projects_with_issue_tracking_disabled
Chris@1296 988 Project.find(2).disable_module! :issue_tracking
Chris@1296 989 assert_not_include Project.find(2), Issue.allowed_target_projects_on_move(User.find(2))
Chris@1296 990 end
Chris@1296 991
Chris@1296 992 def test_move_to_another_project_with_same_category
Chris@1296 993 issue = Issue.find(1)
Chris@1296 994 issue.project = Project.find(2)
Chris@1296 995 assert issue.save
Chris@1296 996 issue.reload
Chris@1296 997 assert_equal 2, issue.project_id
Chris@1296 998 # Category changes
Chris@1296 999 assert_equal 4, issue.category_id
Chris@1296 1000 # Make sure time entries were move to the target project
Chris@1296 1001 assert_equal 2, issue.time_entries.first.project_id
Chris@1296 1002 end
Chris@1296 1003
Chris@1296 1004 def test_move_to_another_project_without_same_category
Chris@1296 1005 issue = Issue.find(2)
Chris@1296 1006 issue.project = Project.find(2)
Chris@1296 1007 assert issue.save
Chris@1296 1008 issue.reload
Chris@1296 1009 assert_equal 2, issue.project_id
Chris@1296 1010 # Category cleared
Chris@1296 1011 assert_nil issue.category_id
Chris@1296 1012 end
Chris@1296 1013
Chris@1296 1014 def test_move_to_another_project_should_clear_fixed_version_when_not_shared
Chris@1296 1015 issue = Issue.find(1)
Chris@1296 1016 issue.update_attribute(:fixed_version_id, 1)
Chris@1296 1017 issue.project = Project.find(2)
Chris@1296 1018 assert issue.save
Chris@1296 1019 issue.reload
Chris@1296 1020 assert_equal 2, issue.project_id
Chris@1296 1021 # Cleared fixed_version
Chris@1296 1022 assert_equal nil, issue.fixed_version
Chris@1296 1023 end
Chris@1296 1024
Chris@1296 1025 def test_move_to_another_project_should_keep_fixed_version_when_shared_with_the_target_project
Chris@1296 1026 issue = Issue.find(1)
Chris@1296 1027 issue.update_attribute(:fixed_version_id, 4)
Chris@1296 1028 issue.project = Project.find(5)
Chris@1296 1029 assert issue.save
Chris@1296 1030 issue.reload
Chris@1296 1031 assert_equal 5, issue.project_id
Chris@1296 1032 # Keep fixed_version
Chris@1296 1033 assert_equal 4, issue.fixed_version_id
Chris@1296 1034 end
Chris@1296 1035
Chris@1296 1036 def test_move_to_another_project_should_clear_fixed_version_when_not_shared_with_the_target_project
Chris@1296 1037 issue = Issue.find(1)
Chris@1296 1038 issue.update_attribute(:fixed_version_id, 1)
Chris@1296 1039 issue.project = Project.find(5)
Chris@1296 1040 assert issue.save
Chris@1296 1041 issue.reload
Chris@1296 1042 assert_equal 5, issue.project_id
Chris@1296 1043 # Cleared fixed_version
Chris@1296 1044 assert_equal nil, issue.fixed_version
Chris@1296 1045 end
Chris@1296 1046
Chris@1296 1047 def test_move_to_another_project_should_keep_fixed_version_when_shared_systemwide
Chris@1296 1048 issue = Issue.find(1)
Chris@1296 1049 issue.update_attribute(:fixed_version_id, 7)
Chris@1296 1050 issue.project = Project.find(2)
Chris@1296 1051 assert issue.save
Chris@1296 1052 issue.reload
Chris@1296 1053 assert_equal 2, issue.project_id
Chris@1296 1054 # Keep fixed_version
Chris@1296 1055 assert_equal 7, issue.fixed_version_id
Chris@1296 1056 end
Chris@1296 1057
Chris@1296 1058 def test_move_to_another_project_should_keep_parent_if_valid
Chris@1296 1059 issue = Issue.find(1)
Chris@1296 1060 issue.update_attribute(:parent_issue_id, 2)
Chris@1296 1061 issue.project = Project.find(3)
Chris@1296 1062 assert issue.save
Chris@1296 1063 issue.reload
Chris@1296 1064 assert_equal 2, issue.parent_id
Chris@1296 1065 end
Chris@1296 1066
Chris@1296 1067 def test_move_to_another_project_should_clear_parent_if_not_valid
Chris@1296 1068 issue = Issue.find(1)
Chris@1296 1069 issue.update_attribute(:parent_issue_id, 2)
Chris@1296 1070 issue.project = Project.find(2)
Chris@1296 1071 assert issue.save
Chris@1296 1072 issue.reload
Chris@1296 1073 assert_nil issue.parent_id
Chris@1296 1074 end
Chris@1296 1075
Chris@1296 1076 def test_move_to_another_project_with_disabled_tracker
Chris@1296 1077 issue = Issue.find(1)
Chris@1296 1078 target = Project.find(2)
Chris@1296 1079 target.tracker_ids = [3]
Chris@1296 1080 target.save
Chris@1296 1081 issue.project = target
Chris@1296 1082 assert issue.save
Chris@1296 1083 issue.reload
Chris@1296 1084 assert_equal 2, issue.project_id
Chris@1296 1085 assert_equal 3, issue.tracker_id
Chris@1296 1086 end
Chris@1296 1087
Chris@1296 1088 def test_copy_to_the_same_project
Chris@1296 1089 issue = Issue.find(1)
Chris@1296 1090 copy = issue.copy
Chris@1296 1091 assert_difference 'Issue.count' do
Chris@1296 1092 copy.save!
Chris@1296 1093 end
Chris@1296 1094 assert_kind_of Issue, copy
Chris@1296 1095 assert_equal issue.project, copy.project
Chris@1296 1096 assert_equal "125", copy.custom_value_for(2).value
Chris@1296 1097 end
Chris@1296 1098
Chris@1296 1099 def test_copy_to_another_project_and_tracker
Chris@1296 1100 issue = Issue.find(1)
Chris@1296 1101 copy = issue.copy(:project_id => 3, :tracker_id => 2)
Chris@1296 1102 assert_difference 'Issue.count' do
Chris@1296 1103 copy.save!
Chris@1296 1104 end
Chris@1296 1105 copy.reload
Chris@1296 1106 assert_kind_of Issue, copy
Chris@1296 1107 assert_equal Project.find(3), copy.project
Chris@1296 1108 assert_equal Tracker.find(2), copy.tracker
Chris@1296 1109 # Custom field #2 is not associated with target tracker
Chris@1296 1110 assert_nil copy.custom_value_for(2)
Chris@1296 1111 end
Chris@1296 1112
Chris@1296 1113 context "#copy" do
Chris@1296 1114 setup do
Chris@1296 1115 @issue = Issue.find(1)
Chris@1296 1116 end
Chris@1296 1117
Chris@1296 1118 should "not create a journal" do
Chris@1296 1119 copy = @issue.copy(:project_id => 3, :tracker_id => 2, :assigned_to_id => 3)
Chris@1296 1120 copy.save!
Chris@1296 1121 assert_equal 0, copy.reload.journals.size
Chris@1296 1122 end
Chris@1296 1123
Chris@1296 1124 should "allow assigned_to changes" do
Chris@1296 1125 copy = @issue.copy(:project_id => 3, :tracker_id => 2, :assigned_to_id => 3)
Chris@1296 1126 assert_equal 3, copy.assigned_to_id
Chris@1296 1127 end
Chris@1296 1128
Chris@1296 1129 should "allow status changes" do
Chris@1296 1130 copy = @issue.copy(:project_id => 3, :tracker_id => 2, :status_id => 2)
Chris@1296 1131 assert_equal 2, copy.status_id
Chris@1296 1132 end
Chris@1296 1133
Chris@1296 1134 should "allow start date changes" do
Chris@1296 1135 date = Date.today
Chris@1296 1136 copy = @issue.copy(:project_id => 3, :tracker_id => 2, :start_date => date)
Chris@1296 1137 assert_equal date, copy.start_date
Chris@1296 1138 end
Chris@1296 1139
Chris@1296 1140 should "allow due date changes" do
Chris@1296 1141 date = Date.today
Chris@1296 1142 copy = @issue.copy(:project_id => 3, :tracker_id => 2, :due_date => date)
Chris@1296 1143 assert_equal date, copy.due_date
Chris@1296 1144 end
Chris@1296 1145
Chris@1296 1146 should "set current user as author" do
Chris@1296 1147 User.current = User.find(9)
Chris@1296 1148 copy = @issue.copy(:project_id => 3, :tracker_id => 2)
Chris@1296 1149 assert_equal User.current, copy.author
Chris@1296 1150 end
Chris@1296 1151
Chris@1296 1152 should "create a journal with notes" do
Chris@1296 1153 date = Date.today
Chris@1296 1154 notes = "Notes added when copying"
Chris@1296 1155 copy = @issue.copy(:project_id => 3, :tracker_id => 2, :start_date => date)
Chris@1296 1156 copy.init_journal(User.current, notes)
Chris@1296 1157 copy.save!
Chris@1296 1158
Chris@1296 1159 assert_equal 1, copy.journals.size
Chris@1296 1160 journal = copy.journals.first
Chris@1296 1161 assert_equal 0, journal.details.size
Chris@1296 1162 assert_equal notes, journal.notes
Chris@1296 1163 end
Chris@1296 1164 end
Chris@1296 1165
Chris@1296 1166 def test_valid_parent_project
Chris@1296 1167 issue = Issue.find(1)
Chris@1296 1168 issue_in_same_project = Issue.find(2)
Chris@1296 1169 issue_in_child_project = Issue.find(5)
Chris@1296 1170 issue_in_grandchild_project = Issue.generate!(:project_id => 6, :tracker_id => 1)
Chris@1296 1171 issue_in_other_child_project = Issue.find(6)
Chris@1296 1172 issue_in_different_tree = Issue.find(4)
Chris@1296 1173
Chris@1296 1174 with_settings :cross_project_subtasks => '' do
Chris@1296 1175 assert_equal true, issue.valid_parent_project?(issue_in_same_project)
Chris@1296 1176 assert_equal false, issue.valid_parent_project?(issue_in_child_project)
Chris@1296 1177 assert_equal false, issue.valid_parent_project?(issue_in_grandchild_project)
Chris@1296 1178 assert_equal false, issue.valid_parent_project?(issue_in_different_tree)
Chris@1296 1179 end
Chris@1296 1180
Chris@1296 1181 with_settings :cross_project_subtasks => 'system' do
Chris@1296 1182 assert_equal true, issue.valid_parent_project?(issue_in_same_project)
Chris@1296 1183 assert_equal true, issue.valid_parent_project?(issue_in_child_project)
Chris@1296 1184 assert_equal true, issue.valid_parent_project?(issue_in_different_tree)
Chris@1296 1185 end
Chris@1296 1186
Chris@1296 1187 with_settings :cross_project_subtasks => 'tree' do
Chris@1296 1188 assert_equal true, issue.valid_parent_project?(issue_in_same_project)
Chris@1296 1189 assert_equal true, issue.valid_parent_project?(issue_in_child_project)
Chris@1296 1190 assert_equal true, issue.valid_parent_project?(issue_in_grandchild_project)
Chris@1296 1191 assert_equal false, issue.valid_parent_project?(issue_in_different_tree)
Chris@1296 1192
Chris@1296 1193 assert_equal true, issue_in_child_project.valid_parent_project?(issue_in_same_project)
Chris@1296 1194 assert_equal true, issue_in_child_project.valid_parent_project?(issue_in_other_child_project)
Chris@1296 1195 end
Chris@1296 1196
Chris@1296 1197 with_settings :cross_project_subtasks => 'descendants' do
Chris@1296 1198 assert_equal true, issue.valid_parent_project?(issue_in_same_project)
Chris@1296 1199 assert_equal false, issue.valid_parent_project?(issue_in_child_project)
Chris@1296 1200 assert_equal false, issue.valid_parent_project?(issue_in_grandchild_project)
Chris@1296 1201 assert_equal false, issue.valid_parent_project?(issue_in_different_tree)
Chris@1296 1202
Chris@1296 1203 assert_equal true, issue_in_child_project.valid_parent_project?(issue)
Chris@1296 1204 assert_equal false, issue_in_child_project.valid_parent_project?(issue_in_other_child_project)
Chris@1296 1205 end
Chris@1296 1206 end
Chris@1296 1207
Chris@1296 1208 def test_recipients_should_include_previous_assignee
Chris@1296 1209 user = User.find(3)
Chris@1296 1210 user.members.update_all ["mail_notification = ?", false]
Chris@1296 1211 user.update_attribute :mail_notification, 'only_assigned'
Chris@1296 1212
Chris@1296 1213 issue = Issue.find(2)
Chris@1296 1214 issue.assigned_to = nil
Chris@1296 1215 assert_include user.mail, issue.recipients
Chris@1296 1216 issue.save!
Chris@1296 1217 assert !issue.recipients.include?(user.mail)
Chris@1296 1218 end
Chris@1296 1219
Chris@1296 1220 def test_recipients_should_not_include_users_that_cannot_view_the_issue
Chris@1296 1221 issue = Issue.find(12)
Chris@1296 1222 assert issue.recipients.include?(issue.author.mail)
Chris@1296 1223 # copy the issue to a private project
Chris@1296 1224 copy = issue.copy(:project_id => 5, :tracker_id => 2)
Chris@1296 1225 # author is not a member of project anymore
Chris@1296 1226 assert !copy.recipients.include?(copy.author.mail)
Chris@1296 1227 end
Chris@1296 1228
Chris@1296 1229 def test_recipients_should_include_the_assigned_group_members
Chris@1296 1230 group_member = User.generate!
Chris@1296 1231 group = Group.generate!
Chris@1296 1232 group.users << group_member
Chris@1296 1233
Chris@1296 1234 issue = Issue.find(12)
Chris@1296 1235 issue.assigned_to = group
Chris@1296 1236 assert issue.recipients.include?(group_member.mail)
Chris@1296 1237 end
Chris@1296 1238
Chris@1296 1239 def test_watcher_recipients_should_not_include_users_that_cannot_view_the_issue
Chris@1296 1240 user = User.find(3)
Chris@1296 1241 issue = Issue.find(9)
Chris@1296 1242 Watcher.create!(:user => user, :watchable => issue)
Chris@1296 1243 assert issue.watched_by?(user)
Chris@1296 1244 assert !issue.watcher_recipients.include?(user.mail)
Chris@1296 1245 end
Chris@1296 1246
Chris@1296 1247 def test_issue_destroy
Chris@1296 1248 Issue.find(1).destroy
Chris@1296 1249 assert_nil Issue.find_by_id(1)
Chris@1296 1250 assert_nil TimeEntry.find_by_issue_id(1)
Chris@1296 1251 end
Chris@1296 1252
Chris@1296 1253 def test_destroying_a_deleted_issue_should_not_raise_an_error
Chris@1296 1254 issue = Issue.find(1)
Chris@1296 1255 Issue.find(1).destroy
Chris@1296 1256
Chris@1296 1257 assert_nothing_raised do
Chris@1296 1258 assert_no_difference 'Issue.count' do
Chris@1296 1259 issue.destroy
Chris@1296 1260 end
Chris@1296 1261 assert issue.destroyed?
Chris@1296 1262 end
Chris@1296 1263 end
Chris@1296 1264
Chris@1296 1265 def test_destroying_a_stale_issue_should_not_raise_an_error
Chris@1296 1266 issue = Issue.find(1)
Chris@1296 1267 Issue.find(1).update_attribute :subject, "Updated"
Chris@1296 1268
Chris@1296 1269 assert_nothing_raised do
Chris@1296 1270 assert_difference 'Issue.count', -1 do
Chris@1296 1271 issue.destroy
Chris@1296 1272 end
Chris@1296 1273 assert issue.destroyed?
Chris@1296 1274 end
Chris@1296 1275 end
Chris@1296 1276
Chris@1296 1277 def test_blocked
Chris@1296 1278 blocked_issue = Issue.find(9)
Chris@1296 1279 blocking_issue = Issue.find(10)
Chris@1296 1280
Chris@1296 1281 assert blocked_issue.blocked?
Chris@1296 1282 assert !blocking_issue.blocked?
Chris@1296 1283 end
Chris@1296 1284
Chris@1296 1285 def test_blocked_issues_dont_allow_closed_statuses
Chris@1296 1286 blocked_issue = Issue.find(9)
Chris@1296 1287
Chris@1296 1288 allowed_statuses = blocked_issue.new_statuses_allowed_to(users(:users_002))
Chris@1296 1289 assert !allowed_statuses.empty?
Chris@1296 1290 closed_statuses = allowed_statuses.select {|st| st.is_closed?}
Chris@1296 1291 assert closed_statuses.empty?
Chris@1296 1292 end
Chris@1296 1293
Chris@1296 1294 def test_unblocked_issues_allow_closed_statuses
Chris@1296 1295 blocking_issue = Issue.find(10)
Chris@1296 1296
Chris@1296 1297 allowed_statuses = blocking_issue.new_statuses_allowed_to(users(:users_002))
Chris@1296 1298 assert !allowed_statuses.empty?
Chris@1296 1299 closed_statuses = allowed_statuses.select {|st| st.is_closed?}
Chris@1296 1300 assert !closed_statuses.empty?
Chris@1296 1301 end
Chris@1296 1302
Chris@1296 1303 def test_reschedule_an_issue_without_dates
Chris@1296 1304 with_settings :non_working_week_days => [] do
Chris@1296 1305 issue = Issue.new(:start_date => nil, :due_date => nil)
Chris@1296 1306 issue.reschedule_on '2012-10-09'.to_date
Chris@1296 1307 assert_equal '2012-10-09'.to_date, issue.start_date
Chris@1296 1308 assert_equal '2012-10-09'.to_date, issue.due_date
Chris@1296 1309 end
Chris@1296 1310
Chris@1296 1311 with_settings :non_working_week_days => %w(6 7) do
Chris@1296 1312 issue = Issue.new(:start_date => nil, :due_date => nil)
Chris@1296 1313 issue.reschedule_on '2012-10-09'.to_date
Chris@1296 1314 assert_equal '2012-10-09'.to_date, issue.start_date
Chris@1296 1315 assert_equal '2012-10-09'.to_date, issue.due_date
Chris@1296 1316
Chris@1296 1317 issue = Issue.new(:start_date => nil, :due_date => nil)
Chris@1296 1318 issue.reschedule_on '2012-10-13'.to_date
Chris@1296 1319 assert_equal '2012-10-15'.to_date, issue.start_date
Chris@1296 1320 assert_equal '2012-10-15'.to_date, issue.due_date
Chris@1296 1321 end
Chris@1296 1322 end
Chris@1296 1323
Chris@1296 1324 def test_reschedule_an_issue_with_start_date
Chris@1296 1325 with_settings :non_working_week_days => [] do
Chris@1296 1326 issue = Issue.new(:start_date => '2012-10-09', :due_date => nil)
Chris@1296 1327 issue.reschedule_on '2012-10-13'.to_date
Chris@1296 1328 assert_equal '2012-10-13'.to_date, issue.start_date
Chris@1296 1329 assert_equal '2012-10-13'.to_date, issue.due_date
Chris@1296 1330 end
Chris@1296 1331
Chris@1296 1332 with_settings :non_working_week_days => %w(6 7) do
Chris@1296 1333 issue = Issue.new(:start_date => '2012-10-09', :due_date => nil)
Chris@1296 1334 issue.reschedule_on '2012-10-11'.to_date
Chris@1296 1335 assert_equal '2012-10-11'.to_date, issue.start_date
Chris@1296 1336 assert_equal '2012-10-11'.to_date, issue.due_date
Chris@1296 1337
Chris@1296 1338 issue = Issue.new(:start_date => '2012-10-09', :due_date => nil)
Chris@1296 1339 issue.reschedule_on '2012-10-13'.to_date
Chris@1296 1340 assert_equal '2012-10-15'.to_date, issue.start_date
Chris@1296 1341 assert_equal '2012-10-15'.to_date, issue.due_date
Chris@1296 1342 end
Chris@1296 1343 end
Chris@1296 1344
Chris@1296 1345 def test_reschedule_an_issue_with_start_and_due_dates
Chris@1296 1346 with_settings :non_working_week_days => [] do
Chris@1296 1347 issue = Issue.new(:start_date => '2012-10-09', :due_date => '2012-10-15')
Chris@1296 1348 issue.reschedule_on '2012-10-13'.to_date
Chris@1296 1349 assert_equal '2012-10-13'.to_date, issue.start_date
Chris@1296 1350 assert_equal '2012-10-19'.to_date, issue.due_date
Chris@1296 1351 end
Chris@1296 1352
Chris@1296 1353 with_settings :non_working_week_days => %w(6 7) do
Chris@1296 1354 issue = Issue.new(:start_date => '2012-10-09', :due_date => '2012-10-19') # 8 working days
Chris@1296 1355 issue.reschedule_on '2012-10-11'.to_date
Chris@1296 1356 assert_equal '2012-10-11'.to_date, issue.start_date
Chris@1296 1357 assert_equal '2012-10-23'.to_date, issue.due_date
Chris@1296 1358
Chris@1296 1359 issue = Issue.new(:start_date => '2012-10-09', :due_date => '2012-10-19')
Chris@1296 1360 issue.reschedule_on '2012-10-13'.to_date
Chris@1296 1361 assert_equal '2012-10-15'.to_date, issue.start_date
Chris@1296 1362 assert_equal '2012-10-25'.to_date, issue.due_date
Chris@1296 1363 end
Chris@1296 1364 end
Chris@1296 1365
Chris@1296 1366 def test_rescheduling_an_issue_to_a_later_due_date_should_reschedule_following_issue
Chris@1296 1367 issue1 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17')
Chris@1296 1368 issue2 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17')
Chris@1296 1369 IssueRelation.create!(:issue_from => issue1, :issue_to => issue2,
Chris@1296 1370 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1371 assert_equal Date.parse('2012-10-18'), issue2.reload.start_date
Chris@1296 1372
Chris@1296 1373 issue1.due_date = '2012-10-23'
Chris@1296 1374 issue1.save!
Chris@1296 1375 issue2.reload
Chris@1296 1376 assert_equal Date.parse('2012-10-24'), issue2.start_date
Chris@1296 1377 assert_equal Date.parse('2012-10-26'), issue2.due_date
Chris@1296 1378 end
Chris@1296 1379
Chris@1296 1380 def test_rescheduling_an_issue_to_an_earlier_due_date_should_reschedule_following_issue
Chris@1296 1381 issue1 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17')
Chris@1296 1382 issue2 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17')
Chris@1296 1383 IssueRelation.create!(:issue_from => issue1, :issue_to => issue2,
Chris@1296 1384 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1385 assert_equal Date.parse('2012-10-18'), issue2.reload.start_date
Chris@1296 1386
Chris@1296 1387 issue1.start_date = '2012-09-17'
Chris@1296 1388 issue1.due_date = '2012-09-18'
Chris@1296 1389 issue1.save!
Chris@1296 1390 issue2.reload
Chris@1296 1391 assert_equal Date.parse('2012-09-19'), issue2.start_date
Chris@1296 1392 assert_equal Date.parse('2012-09-21'), issue2.due_date
Chris@1296 1393 end
Chris@1296 1394
Chris@1296 1395 def test_rescheduling_reschedule_following_issue_earlier_should_consider_other_preceding_issues
Chris@1296 1396 issue1 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17')
Chris@1296 1397 issue2 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17')
Chris@1296 1398 issue3 = Issue.generate!(:start_date => '2012-10-01', :due_date => '2012-10-02')
Chris@1296 1399 IssueRelation.create!(:issue_from => issue1, :issue_to => issue2,
Chris@1296 1400 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1401 IssueRelation.create!(:issue_from => issue3, :issue_to => issue2,
Chris@1296 1402 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1403 assert_equal Date.parse('2012-10-18'), issue2.reload.start_date
Chris@1296 1404
Chris@1296 1405 issue1.start_date = '2012-09-17'
Chris@1296 1406 issue1.due_date = '2012-09-18'
Chris@1296 1407 issue1.save!
Chris@1296 1408 issue2.reload
Chris@1296 1409 # Issue 2 must start after Issue 3
Chris@1296 1410 assert_equal Date.parse('2012-10-03'), issue2.start_date
Chris@1296 1411 assert_equal Date.parse('2012-10-05'), issue2.due_date
Chris@1296 1412 end
Chris@1296 1413
Chris@1296 1414 def test_rescheduling_a_stale_issue_should_not_raise_an_error
Chris@1296 1415 with_settings :non_working_week_days => [] do
Chris@1296 1416 stale = Issue.find(1)
Chris@1296 1417 issue = Issue.find(1)
Chris@1296 1418 issue.subject = "Updated"
Chris@1296 1419 issue.save!
Chris@1296 1420 date = 10.days.from_now.to_date
Chris@1296 1421 assert_nothing_raised do
Chris@1296 1422 stale.reschedule_on!(date)
Chris@1296 1423 end
Chris@1296 1424 assert_equal date, stale.reload.start_date
Chris@1296 1425 end
Chris@1296 1426 end
Chris@1296 1427
Chris@1296 1428 def test_overdue
Chris@1296 1429 assert Issue.new(:due_date => 1.day.ago.to_date).overdue?
Chris@1296 1430 assert !Issue.new(:due_date => Date.today).overdue?
Chris@1296 1431 assert !Issue.new(:due_date => 1.day.from_now.to_date).overdue?
Chris@1296 1432 assert !Issue.new(:due_date => nil).overdue?
Chris@1296 1433 assert !Issue.new(:due_date => 1.day.ago.to_date,
Chris@1296 1434 :status => IssueStatus.find(:first,
Chris@1296 1435 :conditions => {:is_closed => true})
Chris@1296 1436 ).overdue?
Chris@1296 1437 end
Chris@1296 1438
Chris@1296 1439 context "#behind_schedule?" do
Chris@1296 1440 should "be false if the issue has no start_date" do
Chris@1296 1441 assert !Issue.new(:start_date => nil,
Chris@1296 1442 :due_date => 1.day.from_now.to_date,
Chris@1296 1443 :done_ratio => 0).behind_schedule?
Chris@1296 1444 end
Chris@1296 1445
Chris@1296 1446 should "be false if the issue has no end_date" do
Chris@1296 1447 assert !Issue.new(:start_date => 1.day.from_now.to_date,
Chris@1296 1448 :due_date => nil,
Chris@1296 1449 :done_ratio => 0).behind_schedule?
Chris@1296 1450 end
Chris@1296 1451
Chris@1296 1452 should "be false if the issue has more done than it's calendar time" do
Chris@1296 1453 assert !Issue.new(:start_date => 50.days.ago.to_date,
Chris@1296 1454 :due_date => 50.days.from_now.to_date,
Chris@1296 1455 :done_ratio => 90).behind_schedule?
Chris@1296 1456 end
Chris@1296 1457
Chris@1296 1458 should "be true if the issue hasn't been started at all" do
Chris@1296 1459 assert Issue.new(:start_date => 1.day.ago.to_date,
Chris@1296 1460 :due_date => 1.day.from_now.to_date,
Chris@1296 1461 :done_ratio => 0).behind_schedule?
Chris@1296 1462 end
Chris@1296 1463
Chris@1296 1464 should "be true if the issue has used more calendar time than it's done ratio" do
Chris@1296 1465 assert Issue.new(:start_date => 100.days.ago.to_date,
Chris@1296 1466 :due_date => Date.today,
Chris@1296 1467 :done_ratio => 90).behind_schedule?
Chris@1296 1468 end
Chris@1296 1469 end
Chris@1296 1470
Chris@1296 1471 context "#assignable_users" do
Chris@1296 1472 should "be Users" do
Chris@1296 1473 assert_kind_of User, Issue.find(1).assignable_users.first
Chris@1296 1474 end
Chris@1296 1475
Chris@1296 1476 should "include the issue author" do
Chris@1296 1477 non_project_member = User.generate!
Chris@1296 1478 issue = Issue.generate!(:author => non_project_member)
Chris@1296 1479
Chris@1296 1480 assert issue.assignable_users.include?(non_project_member)
Chris@1296 1481 end
Chris@1296 1482
Chris@1296 1483 should "include the current assignee" do
Chris@1296 1484 user = User.generate!
Chris@1296 1485 issue = Issue.generate!(:assigned_to => user)
Chris@1296 1486 user.lock!
Chris@1296 1487
Chris@1296 1488 assert Issue.find(issue.id).assignable_users.include?(user)
Chris@1296 1489 end
Chris@1296 1490
Chris@1296 1491 should "not show the issue author twice" do
Chris@1296 1492 assignable_user_ids = Issue.find(1).assignable_users.collect(&:id)
Chris@1296 1493 assert_equal 2, assignable_user_ids.length
Chris@1296 1494
Chris@1296 1495 assignable_user_ids.each do |user_id|
Chris@1296 1496 assert_equal 1, assignable_user_ids.select {|i| i == user_id}.length,
Chris@1296 1497 "User #{user_id} appears more or less than once"
Chris@1296 1498 end
Chris@1296 1499 end
Chris@1296 1500
Chris@1296 1501 context "with issue_group_assignment" do
Chris@1296 1502 should "include groups" do
Chris@1296 1503 issue = Issue.new(:project => Project.find(2))
Chris@1296 1504
Chris@1296 1505 with_settings :issue_group_assignment => '1' do
Chris@1296 1506 assert_equal %w(Group User), issue.assignable_users.map {|a| a.class.name}.uniq.sort
Chris@1296 1507 assert issue.assignable_users.include?(Group.find(11))
Chris@1296 1508 end
Chris@1296 1509 end
Chris@1296 1510 end
Chris@1296 1511
Chris@1296 1512 context "without issue_group_assignment" do
Chris@1296 1513 should "not include groups" do
Chris@1296 1514 issue = Issue.new(:project => Project.find(2))
Chris@1296 1515
Chris@1296 1516 with_settings :issue_group_assignment => '0' do
Chris@1296 1517 assert_equal %w(User), issue.assignable_users.map {|a| a.class.name}.uniq.sort
Chris@1296 1518 assert !issue.assignable_users.include?(Group.find(11))
Chris@1296 1519 end
Chris@1296 1520 end
Chris@1296 1521 end
Chris@1296 1522 end
Chris@1296 1523
Chris@1296 1524 def test_create_should_send_email_notification
Chris@1296 1525 ActionMailer::Base.deliveries.clear
Chris@1296 1526 issue = Issue.new(:project_id => 1, :tracker_id => 1,
Chris@1296 1527 :author_id => 3, :status_id => 1,
Chris@1296 1528 :priority => IssuePriority.all.first,
Chris@1296 1529 :subject => 'test_create', :estimated_hours => '1:30')
Chris@1296 1530
Chris@1296 1531 assert issue.save
Chris@1296 1532 assert_equal 1, ActionMailer::Base.deliveries.size
Chris@1296 1533 end
Chris@1296 1534
Chris@1296 1535 def test_stale_issue_should_not_send_email_notification
Chris@1296 1536 ActionMailer::Base.deliveries.clear
Chris@1296 1537 issue = Issue.find(1)
Chris@1296 1538 stale = Issue.find(1)
Chris@1296 1539
Chris@1296 1540 issue.init_journal(User.find(1))
Chris@1296 1541 issue.subject = 'Subjet update'
Chris@1296 1542 assert issue.save
Chris@1296 1543 assert_equal 1, ActionMailer::Base.deliveries.size
Chris@1296 1544 ActionMailer::Base.deliveries.clear
Chris@1296 1545
Chris@1296 1546 stale.init_journal(User.find(1))
Chris@1296 1547 stale.subject = 'Another subjet update'
Chris@1296 1548 assert_raise ActiveRecord::StaleObjectError do
Chris@1296 1549 stale.save
Chris@1296 1550 end
Chris@1296 1551 assert ActionMailer::Base.deliveries.empty?
Chris@1296 1552 end
Chris@1296 1553
Chris@1296 1554 def test_journalized_description
Chris@1296 1555 IssueCustomField.delete_all
Chris@1296 1556
Chris@1296 1557 i = Issue.first
Chris@1296 1558 old_description = i.description
Chris@1296 1559 new_description = "This is the new description"
Chris@1296 1560
Chris@1296 1561 i.init_journal(User.find(2))
Chris@1296 1562 i.description = new_description
Chris@1296 1563 assert_difference 'Journal.count', 1 do
Chris@1296 1564 assert_difference 'JournalDetail.count', 1 do
Chris@1296 1565 i.save!
Chris@1296 1566 end
Chris@1296 1567 end
Chris@1296 1568
Chris@1296 1569 detail = JournalDetail.first(:order => 'id DESC')
Chris@1296 1570 assert_equal i, detail.journal.journalized
Chris@1296 1571 assert_equal 'attr', detail.property
Chris@1296 1572 assert_equal 'description', detail.prop_key
Chris@1296 1573 assert_equal old_description, detail.old_value
Chris@1296 1574 assert_equal new_description, detail.value
Chris@1296 1575 end
Chris@1296 1576
Chris@1296 1577 def test_blank_descriptions_should_not_be_journalized
Chris@1296 1578 IssueCustomField.delete_all
Chris@1296 1579 Issue.update_all("description = NULL", "id=1")
Chris@1296 1580
Chris@1296 1581 i = Issue.find(1)
Chris@1296 1582 i.init_journal(User.find(2))
Chris@1296 1583 i.subject = "blank description"
Chris@1296 1584 i.description = "\r\n"
Chris@1296 1585
Chris@1296 1586 assert_difference 'Journal.count', 1 do
Chris@1296 1587 assert_difference 'JournalDetail.count', 1 do
Chris@1296 1588 i.save!
Chris@1296 1589 end
Chris@1296 1590 end
Chris@1296 1591 end
Chris@1296 1592
Chris@1296 1593 def test_journalized_multi_custom_field
Chris@1296 1594 field = IssueCustomField.create!(:name => 'filter', :field_format => 'list',
Chris@1296 1595 :is_filter => true, :is_for_all => true,
Chris@1296 1596 :tracker_ids => [1],
Chris@1296 1597 :possible_values => ['value1', 'value2', 'value3'],
Chris@1296 1598 :multiple => true)
Chris@1296 1599
Chris@1296 1600 issue = Issue.create!(:project_id => 1, :tracker_id => 1,
Chris@1296 1601 :subject => 'Test', :author_id => 1)
Chris@1296 1602
Chris@1296 1603 assert_difference 'Journal.count' do
Chris@1296 1604 assert_difference 'JournalDetail.count' do
Chris@1296 1605 issue.init_journal(User.first)
Chris@1296 1606 issue.custom_field_values = {field.id => ['value1']}
Chris@1296 1607 issue.save!
Chris@1296 1608 end
Chris@1296 1609 assert_difference 'JournalDetail.count' do
Chris@1296 1610 issue.init_journal(User.first)
Chris@1296 1611 issue.custom_field_values = {field.id => ['value1', 'value2']}
Chris@1296 1612 issue.save!
Chris@1296 1613 end
Chris@1296 1614 assert_difference 'JournalDetail.count', 2 do
Chris@1296 1615 issue.init_journal(User.first)
Chris@1296 1616 issue.custom_field_values = {field.id => ['value3', 'value2']}
Chris@1296 1617 issue.save!
Chris@1296 1618 end
Chris@1296 1619 assert_difference 'JournalDetail.count', 2 do
Chris@1296 1620 issue.init_journal(User.first)
Chris@1296 1621 issue.custom_field_values = {field.id => nil}
Chris@1296 1622 issue.save!
Chris@1296 1623 end
Chris@1296 1624 end
Chris@1296 1625 end
Chris@1296 1626
Chris@1296 1627 def test_description_eol_should_be_normalized
Chris@1296 1628 i = Issue.new(:description => "CR \r LF \n CRLF \r\n")
Chris@1296 1629 assert_equal "CR \r\n LF \r\n CRLF \r\n", i.description
Chris@1296 1630 end
Chris@1296 1631
Chris@1296 1632 def test_saving_twice_should_not_duplicate_journal_details
Chris@1296 1633 i = Issue.find(:first)
Chris@1296 1634 i.init_journal(User.find(2), 'Some notes')
Chris@1296 1635 # initial changes
Chris@1296 1636 i.subject = 'New subject'
Chris@1296 1637 i.done_ratio = i.done_ratio + 10
Chris@1296 1638 assert_difference 'Journal.count' do
Chris@1296 1639 assert i.save
Chris@1296 1640 end
Chris@1296 1641 # 1 more change
Chris@1296 1642 i.priority = IssuePriority.find(:first, :conditions => ["id <> ?", i.priority_id])
Chris@1296 1643 assert_no_difference 'Journal.count' do
Chris@1296 1644 assert_difference 'JournalDetail.count', 1 do
Chris@1296 1645 i.save
Chris@1296 1646 end
Chris@1296 1647 end
Chris@1296 1648 # no more change
Chris@1296 1649 assert_no_difference 'Journal.count' do
Chris@1296 1650 assert_no_difference 'JournalDetail.count' do
Chris@1296 1651 i.save
Chris@1296 1652 end
Chris@1296 1653 end
Chris@1296 1654 end
Chris@1296 1655
Chris@1296 1656 def test_all_dependent_issues
Chris@1296 1657 IssueRelation.delete_all
Chris@1296 1658 assert IssueRelation.create!(:issue_from => Issue.find(1),
Chris@1296 1659 :issue_to => Issue.find(2),
Chris@1296 1660 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1661 assert IssueRelation.create!(:issue_from => Issue.find(2),
Chris@1296 1662 :issue_to => Issue.find(3),
Chris@1296 1663 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1664 assert IssueRelation.create!(:issue_from => Issue.find(3),
Chris@1296 1665 :issue_to => Issue.find(8),
Chris@1296 1666 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1667
Chris@1296 1668 assert_equal [2, 3, 8], Issue.find(1).all_dependent_issues.collect(&:id).sort
Chris@1296 1669 end
Chris@1296 1670
Chris@1296 1671 def test_all_dependent_issues_with_persistent_circular_dependency
Chris@1296 1672 IssueRelation.delete_all
Chris@1296 1673 assert IssueRelation.create!(:issue_from => Issue.find(1),
Chris@1296 1674 :issue_to => Issue.find(2),
Chris@1296 1675 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1676 assert IssueRelation.create!(:issue_from => Issue.find(2),
Chris@1296 1677 :issue_to => Issue.find(3),
Chris@1296 1678 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1679
Chris@1296 1680 r = IssueRelation.create!(:issue_from => Issue.find(3),
Chris@1296 1681 :issue_to => Issue.find(7),
Chris@1296 1682 :relation_type => IssueRelation::TYPE_PRECEDES)
Chris@1296 1683 IssueRelation.update_all("issue_to_id = 1", ["id = ?", r.id])
Chris@1296 1684
Chris@1296 1685 assert_equal [2, 3], Issue.find(1).all_dependent_issues.collect(&:id).sort
Chris@1296 1686 end
Chris@1296 1687
Chris@1296 1688 def test_all_dependent_issues_with_persistent_multiple_circular_dependencies
Chris@1296 1689 IssueRelation.delete_all
Chris@1296 1690 assert IssueRelation.create!(:issue_from => Issue.find(1),
Chris@1296 1691 :issue_to => Issue.find(2),
Chris@1296 1692 :relation_type => IssueRelation::TYPE_RELATES)
Chris@1296 1693 assert IssueRelation.create!(:issue_from => Issue.find(2),
Chris@1296 1694 :issue_to => Issue.find(3),
Chris@1296 1695 :relation_type => IssueRelation::TYPE_RELATES)
Chris@1296 1696 assert IssueRelation.create!(:issue_from => Issue.find(3),
Chris@1296 1697 :issue_to => Issue.find(8),
Chris@1296 1698 :relation_type => IssueRelation::TYPE_RELATES)
Chris@1296 1699
Chris@1296 1700 r = IssueRelation.create!(:issue_from => Issue.find(8),
Chris@1296 1701 :issue_to => Issue.find(7),
Chris@1296 1702 :relation_type => IssueRelation::TYPE_RELATES)
Chris@1296 1703 IssueRelation.update_all("issue_to_id = 2", ["id = ?", r.id])
Chris@1296 1704
Chris@1296 1705 r = IssueRelation.create!(:issue_from => Issue.find(3),
Chris@1296 1706 :issue_to => Issue.find(7),
Chris@1296 1707 :relation_type => IssueRelation::TYPE_RELATES)
Chris@1296 1708 IssueRelation.update_all("issue_to_id = 1", ["id = ?", r.id])
Chris@1296 1709
Chris@1296 1710 assert_equal [2, 3, 8], Issue.find(1).all_dependent_issues.collect(&:id).sort
Chris@1296 1711 end
Chris@1296 1712
Chris@1296 1713 context "#done_ratio" do
Chris@1296 1714 setup do
Chris@1296 1715 @issue = Issue.find(1)
Chris@1296 1716 @issue_status = IssueStatus.find(1)
Chris@1296 1717 @issue_status.update_attribute(:default_done_ratio, 50)
Chris@1296 1718 @issue2 = Issue.find(2)
Chris@1296 1719 @issue_status2 = IssueStatus.find(2)
Chris@1296 1720 @issue_status2.update_attribute(:default_done_ratio, 0)
Chris@1296 1721 end
Chris@1296 1722
Chris@1296 1723 teardown do
Chris@1296 1724 Setting.issue_done_ratio = 'issue_field'
Chris@1296 1725 end
Chris@1296 1726
Chris@1296 1727 context "with Setting.issue_done_ratio using the issue_field" do
Chris@1296 1728 setup do
Chris@1296 1729 Setting.issue_done_ratio = 'issue_field'
Chris@1296 1730 end
Chris@1296 1731
Chris@1296 1732 should "read the issue's field" do
Chris@1296 1733 assert_equal 0, @issue.done_ratio
Chris@1296 1734 assert_equal 30, @issue2.done_ratio
Chris@1296 1735 end
Chris@1296 1736 end
Chris@1296 1737
Chris@1296 1738 context "with Setting.issue_done_ratio using the issue_status" do
Chris@1296 1739 setup do
Chris@1296 1740 Setting.issue_done_ratio = 'issue_status'
Chris@1296 1741 end
Chris@1296 1742
Chris@1296 1743 should "read the Issue Status's default done ratio" do
Chris@1296 1744 assert_equal 50, @issue.done_ratio
Chris@1296 1745 assert_equal 0, @issue2.done_ratio
Chris@1296 1746 end
Chris@1296 1747 end
Chris@1296 1748 end
Chris@1296 1749
Chris@1296 1750 context "#update_done_ratio_from_issue_status" do
Chris@1296 1751 setup do
Chris@1296 1752 @issue = Issue.find(1)
Chris@1296 1753 @issue_status = IssueStatus.find(1)
Chris@1296 1754 @issue_status.update_attribute(:default_done_ratio, 50)
Chris@1296 1755 @issue2 = Issue.find(2)
Chris@1296 1756 @issue_status2 = IssueStatus.find(2)
Chris@1296 1757 @issue_status2.update_attribute(:default_done_ratio, 0)
Chris@1296 1758 end
Chris@1296 1759
Chris@1296 1760 context "with Setting.issue_done_ratio using the issue_field" do
Chris@1296 1761 setup do
Chris@1296 1762 Setting.issue_done_ratio = 'issue_field'
Chris@1296 1763 end
Chris@1296 1764
Chris@1296 1765 should "not change the issue" do
Chris@1296 1766 @issue.update_done_ratio_from_issue_status
Chris@1296 1767 @issue2.update_done_ratio_from_issue_status
Chris@1296 1768
Chris@1296 1769 assert_equal 0, @issue.read_attribute(:done_ratio)
Chris@1296 1770 assert_equal 30, @issue2.read_attribute(:done_ratio)
Chris@1296 1771 end
Chris@1296 1772 end
Chris@1296 1773
Chris@1296 1774 context "with Setting.issue_done_ratio using the issue_status" do
Chris@1296 1775 setup do
Chris@1296 1776 Setting.issue_done_ratio = 'issue_status'
Chris@1296 1777 end
Chris@1296 1778
Chris@1296 1779 should "change the issue's done ratio" do
Chris@1296 1780 @issue.update_done_ratio_from_issue_status
Chris@1296 1781 @issue2.update_done_ratio_from_issue_status
Chris@1296 1782
Chris@1296 1783 assert_equal 50, @issue.read_attribute(:done_ratio)
Chris@1296 1784 assert_equal 0, @issue2.read_attribute(:done_ratio)
Chris@1296 1785 end
Chris@1296 1786 end
Chris@1296 1787 end
Chris@1296 1788
Chris@1296 1789 test "#by_tracker" do
Chris@1296 1790 User.current = User.anonymous
Chris@1296 1791 groups = Issue.by_tracker(Project.find(1))
Chris@1296 1792 assert_equal 3, groups.size
Chris@1296 1793 assert_equal 7, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1794 end
Chris@1296 1795
Chris@1296 1796 test "#by_version" do
Chris@1296 1797 User.current = User.anonymous
Chris@1296 1798 groups = Issue.by_version(Project.find(1))
Chris@1296 1799 assert_equal 3, groups.size
Chris@1296 1800 assert_equal 3, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1801 end
Chris@1296 1802
Chris@1296 1803 test "#by_priority" do
Chris@1296 1804 User.current = User.anonymous
Chris@1296 1805 groups = Issue.by_priority(Project.find(1))
Chris@1296 1806 assert_equal 4, groups.size
Chris@1296 1807 assert_equal 7, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1808 end
Chris@1296 1809
Chris@1296 1810 test "#by_category" do
Chris@1296 1811 User.current = User.anonymous
Chris@1296 1812 groups = Issue.by_category(Project.find(1))
Chris@1296 1813 assert_equal 2, groups.size
Chris@1296 1814 assert_equal 3, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1815 end
Chris@1296 1816
Chris@1296 1817 test "#by_assigned_to" do
Chris@1296 1818 User.current = User.anonymous
Chris@1296 1819 groups = Issue.by_assigned_to(Project.find(1))
Chris@1296 1820 assert_equal 2, groups.size
Chris@1296 1821 assert_equal 2, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1822 end
Chris@1296 1823
Chris@1296 1824 test "#by_author" do
Chris@1296 1825 User.current = User.anonymous
Chris@1296 1826 groups = Issue.by_author(Project.find(1))
Chris@1296 1827 assert_equal 4, groups.size
Chris@1296 1828 assert_equal 7, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1829 end
Chris@1296 1830
Chris@1296 1831 test "#by_subproject" do
Chris@1296 1832 User.current = User.anonymous
Chris@1296 1833 groups = Issue.by_subproject(Project.find(1))
Chris@1296 1834 # Private descendant not visible
Chris@1296 1835 assert_equal 1, groups.size
Chris@1296 1836 assert_equal 2, groups.inject(0) {|sum, group| sum + group['total'].to_i}
Chris@1296 1837 end
Chris@1296 1838
Chris@1296 1839 def test_recently_updated_scope
Chris@1296 1840 #should return the last updated issue
Chris@1296 1841 assert_equal Issue.reorder("updated_on DESC").first, Issue.recently_updated.limit(1).first
Chris@1296 1842 end
Chris@1296 1843
Chris@1296 1844 def test_on_active_projects_scope
Chris@1296 1845 assert Project.find(2).archive
Chris@1296 1846
Chris@1296 1847 before = Issue.on_active_project.length
Chris@1296 1848 # test inclusion to results
Chris@1296 1849 issue = Issue.generate!(:tracker => Project.find(2).trackers.first)
Chris@1296 1850 assert_equal before + 1, Issue.on_active_project.length
Chris@1296 1851
Chris@1296 1852 # Move to an archived project
Chris@1296 1853 issue.project = Project.find(2)
Chris@1296 1854 assert issue.save
Chris@1296 1855 assert_equal before, Issue.on_active_project.length
Chris@1296 1856 end
Chris@1296 1857
Chris@1296 1858 context "Issue#recipients" do
Chris@1296 1859 setup do
Chris@1296 1860 @project = Project.find(1)
Chris@1296 1861 @author = User.generate!
Chris@1296 1862 @assignee = User.generate!
Chris@1296 1863 @issue = Issue.generate!(:project => @project, :assigned_to => @assignee, :author => @author)
Chris@1296 1864 end
Chris@1296 1865
Chris@1296 1866 should "include project recipients" do
Chris@1296 1867 assert @project.recipients.present?
Chris@1296 1868 @project.recipients.each do |project_recipient|
Chris@1296 1869 assert @issue.recipients.include?(project_recipient)
Chris@1296 1870 end
Chris@1296 1871 end
Chris@1296 1872
Chris@1296 1873 should "include the author if the author is active" do
Chris@1296 1874 assert @issue.author, "No author set for Issue"
Chris@1296 1875 assert @issue.recipients.include?(@issue.author.mail)
Chris@1296 1876 end
Chris@1296 1877
Chris@1296 1878 should "include the assigned to user if the assigned to user is active" do
Chris@1296 1879 assert @issue.assigned_to, "No assigned_to set for Issue"
Chris@1296 1880 assert @issue.recipients.include?(@issue.assigned_to.mail)
Chris@1296 1881 end
Chris@1296 1882
Chris@1296 1883 should "not include users who opt out of all email" do
Chris@1296 1884 @author.update_attribute(:mail_notification, :none)
Chris@1296 1885
Chris@1296 1886 assert !@issue.recipients.include?(@issue.author.mail)
Chris@1296 1887 end
Chris@1296 1888
Chris@1296 1889 should "not include the issue author if they are only notified of assigned issues" do
Chris@1296 1890 @author.update_attribute(:mail_notification, :only_assigned)
Chris@1296 1891
Chris@1296 1892 assert !@issue.recipients.include?(@issue.author.mail)
Chris@1296 1893 end
Chris@1296 1894
Chris@1296 1895 should "not include the assigned user if they are only notified of owned issues" do
Chris@1296 1896 @assignee.update_attribute(:mail_notification, :only_owner)
Chris@1296 1897
Chris@1296 1898 assert !@issue.recipients.include?(@issue.assigned_to.mail)
Chris@1296 1899 end
Chris@1296 1900 end
Chris@1296 1901
Chris@1296 1902 def test_last_journal_id_with_journals_should_return_the_journal_id
Chris@1296 1903 assert_equal 2, Issue.find(1).last_journal_id
Chris@1296 1904 end
Chris@1296 1905
Chris@1296 1906 def test_last_journal_id_without_journals_should_return_nil
Chris@1296 1907 assert_nil Issue.find(3).last_journal_id
Chris@1296 1908 end
Chris@1296 1909
Chris@1296 1910 def test_journals_after_should_return_journals_with_greater_id
Chris@1296 1911 assert_equal [Journal.find(2)], Issue.find(1).journals_after('1')
Chris@1296 1912 assert_equal [], Issue.find(1).journals_after('2')
Chris@1296 1913 end
Chris@1296 1914
Chris@1296 1915 def test_journals_after_with_blank_arg_should_return_all_journals
Chris@1296 1916 assert_equal [Journal.find(1), Journal.find(2)], Issue.find(1).journals_after('')
Chris@1296 1917 end
Chris@1296 1918
Chris@1296 1919 def test_css_classes_should_include_priority
Chris@1296 1920 issue = Issue.new(:priority => IssuePriority.find(8))
Chris@1296 1921 classes = issue.css_classes.split(' ')
Chris@1296 1922 assert_include 'priority-8', classes
Chris@1296 1923 assert_include 'priority-highest', classes
Chris@1296 1924 end
Chris@1296 1925
Chris@1296 1926 def test_save_attachments_with_hash_should_save_attachments_in_keys_order
Chris@1296 1927 set_tmp_attachments_directory
Chris@1296 1928 issue = Issue.generate!
Chris@1296 1929 issue.save_attachments({
Chris@1296 1930 'p0' => {'file' => mock_file_with_options(:original_filename => 'upload')},
Chris@1296 1931 '3' => {'file' => mock_file_with_options(:original_filename => 'bar')},
Chris@1296 1932 '1' => {'file' => mock_file_with_options(:original_filename => 'foo')}
Chris@1296 1933 })
Chris@1296 1934 issue.attach_saved_attachments
Chris@1296 1935
Chris@1296 1936 assert_equal 3, issue.reload.attachments.count
Chris@1296 1937 assert_equal %w(upload foo bar), issue.attachments.map(&:filename)
Chris@1296 1938 end
Chris@1296 1939 end