| @@ -892,7 +892,7 class Issue < ActiveRecord::Base | |||||
| 892 | @soonest_start = nil if reload |
|
892 | @soonest_start = nil if reload | |
| 893 | @soonest_start ||= ( |
|
893 | @soonest_start ||= ( | |
| 894 | relations_to(reload).collect{|relation| relation.successor_soonest_start} + |
|
894 | relations_to(reload).collect{|relation| relation.successor_soonest_start} + | |
| 895 | ancestors.collect(&:soonest_start) |
|
895 | [(@parent_issue || parent).try(:soonest_start)] | |
| 896 | ).compact.max |
|
896 | ).compact.max | |
| 897 | end |
|
897 | end | |
| 898 |
|
898 | |||
| @@ -1512,6 +1512,25 class IssueTest < ActiveSupport::TestCase | |||||
| 1512 | end |
|
1512 | end | |
| 1513 | end |
|
1513 | end | |
| 1514 |
|
1514 | |||
|
|
1515 | def test_child_issue_should_consider_parent_soonest_start_on_create | |||
|
|
1516 | set_language_if_valid 'en' | |||
|
|
1517 | issue1 = Issue.generate!(:start_date => '2012-10-15', :due_date => '2012-10-17') | |||
|
|
1518 | issue2 = Issue.generate!(:start_date => '2012-10-18', :due_date => '2012-10-20') | |||
|
|
1519 | IssueRelation.create!(:issue_from => issue1, :issue_to => issue2, | |||
|
|
1520 | :relation_type => IssueRelation::TYPE_PRECEDES) | |||
|
|
1521 | issue1.reload | |||
|
|
1522 | issue2.reload | |||
|
|
1523 | assert_equal Date.parse('2012-10-18'), issue2.start_date | |||
|
|
1524 | ||||
|
|
1525 | child = Issue.new(:parent_issue_id => issue2.id, :start_date => '2012-10-16', | |||
|
|
1526 | :project_id => 1, :tracker_id => 1, :status_id => 1, :subject => 'Child', :author_id => 1) | |||
|
|
1527 | assert !child.valid? | |||
|
|
1528 | assert_include 'Start date is invalid', child.errors.full_messages | |||
|
|
1529 | assert_equal Date.parse('2012-10-18'), child.soonest_start | |||
|
|
1530 | child.start_date = '2012-10-18' | |||
|
|
1531 | assert child.save | |||
|
|
1532 | end | |||
|
|
1533 | ||||
| 1515 | def test_overdue |
|
1534 | def test_overdue | |
| 1516 | assert Issue.new(:due_date => 1.day.ago.to_date).overdue? |
|
1535 | assert Issue.new(:due_date => 1.day.ago.to_date).overdue? | |
| 1517 | assert !Issue.new(:due_date => Date.today).overdue? |
|
1536 | assert !Issue.new(:due_date => Date.today).overdue? | |
General Comments 0
You need to be logged in to leave comments.
Login now