issue_nested_set_test.rb
380 lines
| 15.1 KiB
| text/x-ruby
|
RubyLexer
|
r5685 | # Redmine - project management software | ||
# Copyright (C) 2006-2011 Jean-Philippe Lang | ||||
|
r3459 | # | ||
# This program is free software; you can redistribute it and/or | ||||
# modify it under the terms of the GNU General Public License | ||||
# as published by the Free Software Foundation; either version 2 | ||||
# of the License, or (at your option) any later version. | ||||
|
r5685 | # | ||
|
r3459 | # This program is distributed in the hope that it will be useful, | ||
# but WITHOUT ANY WARRANTY; without even the implied warranty of | ||||
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | ||||
# GNU General Public License for more details. | ||||
|
r5685 | # | ||
|
r3459 | # You should have received a copy of the GNU General Public License | ||
# along with this program; if not, write to the Free Software | ||||
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. | ||||
|
r4395 | require File.expand_path('../../test_helper', __FILE__) | ||
|
r3459 | |||
class IssueNestedSetTest < ActiveSupport::TestCase | ||||
fixtures :projects, :users, :members, :member_roles, :roles, | ||||
:trackers, :projects_trackers, | ||||
:versions, | ||||
|
r5685 | :issue_statuses, :issue_categories, :issue_relations, :workflows, | ||
|
r3459 | :enumerations, | ||
:issues, | ||||
:custom_fields, :custom_fields_projects, :custom_fields_trackers, :custom_values, | ||||
:time_entries | ||||
self.use_transactional_fixtures = false | ||||
|
r5685 | |||
|
r3459 | def test_create_root_issue | ||
issue1 = create_issue! | ||||
issue2 = create_issue! | ||||
issue1.reload | ||||
issue2.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [issue1.id, nil, 1, 2], [issue1.root_id, issue1.parent_id, issue1.lft, issue1.rgt] | ||
assert_equal [issue2.id, nil, 1, 2], [issue2.root_id, issue2.parent_id, issue2.lft, issue2.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_create_child_issue | ||
parent = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent.id) | ||||
parent.reload | ||||
child.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [parent.id, nil, 1, 4], [parent.root_id, parent.parent_id, parent.lft, parent.rgt] | ||
assert_equal [parent.id, parent.id, 2, 3], [child.root_id, child.parent_id, child.lft, child.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_creating_a_child_in_different_project_should_not_validate | ||
issue = create_issue! | ||||
child = Issue.new(:project_id => 2, :tracker_id => 1, :author_id => 1, :subject => 'child', :parent_issue_id => issue.id) | ||||
assert !child.save | ||||
assert_not_nil child.errors.on(:parent_issue_id) | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_move_a_root_to_child | ||
parent1 = create_issue! | ||||
parent2 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
|
r5685 | |||
|
r3459 | parent2.parent_issue_id = parent1.id | ||
parent2.save! | ||||
child.reload | ||||
parent1.reload | ||||
parent2.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [parent1.id, 1, 6], [parent1.root_id, parent1.lft, parent1.rgt] | ||
assert_equal [parent1.id, 4, 5], [parent2.root_id, parent2.lft, parent2.rgt] | ||||
assert_equal [parent1.id, 2, 3], [child.root_id, child.lft, child.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_move_a_child_to_root | ||
parent1 = create_issue! | ||||
parent2 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
|
r5685 | |||
|
r3459 | child.parent_issue_id = nil | ||
child.save! | ||||
child.reload | ||||
parent1.reload | ||||
parent2.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [parent1.id, 1, 2], [parent1.root_id, parent1.lft, parent1.rgt] | ||
assert_equal [parent2.id, 1, 2], [parent2.root_id, parent2.lft, parent2.rgt] | ||||
assert_equal [child.id, 1, 2], [child.root_id, child.lft, child.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_move_a_child_to_another_issue | ||
parent1 = create_issue! | ||||
parent2 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
|
r5685 | |||
|
r3459 | child.parent_issue_id = parent2.id | ||
child.save! | ||||
child.reload | ||||
parent1.reload | ||||
parent2.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [parent1.id, 1, 2], [parent1.root_id, parent1.lft, parent1.rgt] | ||
assert_equal [parent2.id, 1, 4], [parent2.root_id, parent2.lft, parent2.rgt] | ||||
assert_equal [parent2.id, 2, 3], [child.root_id, child.lft, child.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_move_a_child_with_descendants_to_another_issue | ||
parent1 = create_issue! | ||||
parent2 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
grandchild = create_issue!(:parent_issue_id => child.id) | ||||
|
r5685 | |||
|
r3459 | parent1.reload | ||
parent2.reload | ||||
child.reload | ||||
grandchild.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [parent1.id, 1, 6], [parent1.root_id, parent1.lft, parent1.rgt] | ||
assert_equal [parent2.id, 1, 2], [parent2.root_id, parent2.lft, parent2.rgt] | ||||
assert_equal [parent1.id, 2, 5], [child.root_id, child.lft, child.rgt] | ||||
assert_equal [parent1.id, 3, 4], [grandchild.root_id, grandchild.lft, grandchild.rgt] | ||||
|
r5685 | |||
|
r3459 | child.reload.parent_issue_id = parent2.id | ||
child.save! | ||||
child.reload | ||||
grandchild.reload | ||||
parent1.reload | ||||
parent2.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [parent1.id, 1, 2], [parent1.root_id, parent1.lft, parent1.rgt] | ||
assert_equal [parent2.id, 1, 6], [parent2.root_id, parent2.lft, parent2.rgt] | ||||
assert_equal [parent2.id, 2, 5], [child.root_id, child.lft, child.rgt] | ||||
assert_equal [parent2.id, 3, 4], [grandchild.root_id, grandchild.lft, grandchild.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_move_a_child_with_descendants_to_another_project | ||
parent1 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
grandchild = create_issue!(:parent_issue_id => child.id) | ||||
|
r5685 | |||
|
r3459 | assert child.reload.move_to_project(Project.find(2)) | ||
child.reload | ||||
grandchild.reload | ||||
parent1.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal [1, parent1.id, 1, 2], [parent1.project_id, parent1.root_id, parent1.lft, parent1.rgt] | ||
assert_equal [2, child.id, 1, 4], [child.project_id, child.root_id, child.lft, child.rgt] | ||||
assert_equal [2, child.id, 2, 3], [grandchild.project_id, grandchild.root_id, grandchild.lft, grandchild.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_invalid_move_to_another_project | ||
parent1 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
grandchild = create_issue!(:parent_issue_id => child.id, :tracker_id => 2) | ||||
Project.find(2).tracker_ids = [1] | ||||
|
r5685 | |||
|
r3459 | parent1.reload | ||
assert_equal [1, parent1.id, 1, 6], [parent1.project_id, parent1.root_id, parent1.lft, parent1.rgt] | ||||
|
r5685 | |||
|
r3459 | # child can not be moved to Project 2 because its child is on a disabled tracker | ||
assert_equal false, Issue.find(child.id).move_to_project(Project.find(2)) | ||||
child.reload | ||||
grandchild.reload | ||||
parent1.reload | ||||
|
r5685 | |||
|
r3459 | # no change | ||
assert_equal [1, parent1.id, 1, 6], [parent1.project_id, parent1.root_id, parent1.lft, parent1.rgt] | ||||
assert_equal [1, parent1.id, 2, 5], [child.project_id, child.root_id, child.lft, child.rgt] | ||||
assert_equal [1, parent1.id, 3, 4], [grandchild.project_id, grandchild.root_id, grandchild.lft, grandchild.rgt] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_moving_an_issue_to_a_descendant_should_not_validate | ||
parent1 = create_issue! | ||||
parent2 = create_issue! | ||||
child = create_issue!(:parent_issue_id => parent1.id) | ||||
grandchild = create_issue!(:parent_issue_id => child.id) | ||||
|
r5685 | |||
|
r3459 | child.reload | ||
child.parent_issue_id = grandchild.id | ||||
assert !child.save | ||||
assert_not_nil child.errors.on(:parent_issue_id) | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_moving_an_issue_should_keep_valid_relations_only | ||
issue1 = create_issue! | ||||
issue2 = create_issue! | ||||
issue3 = create_issue!(:parent_issue_id => issue2.id) | ||||
issue4 = create_issue! | ||||
r1 = IssueRelation.create!(:issue_from => issue1, :issue_to => issue2, :relation_type => IssueRelation::TYPE_PRECEDES) | ||||
r2 = IssueRelation.create!(:issue_from => issue1, :issue_to => issue3, :relation_type => IssueRelation::TYPE_PRECEDES) | ||||
r3 = IssueRelation.create!(:issue_from => issue2, :issue_to => issue4, :relation_type => IssueRelation::TYPE_PRECEDES) | ||||
issue2.reload | ||||
issue2.parent_issue_id = issue1.id | ||||
issue2.save! | ||||
assert !IssueRelation.exists?(r1.id) | ||||
assert !IssueRelation.exists?(r2.id) | ||||
assert IssueRelation.exists?(r3.id) | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_destroy_should_destroy_children | ||
issue1 = create_issue! | ||||
issue2 = create_issue! | ||||
issue3 = create_issue!(:parent_issue_id => issue2.id) | ||||
issue4 = create_issue!(:parent_issue_id => issue1.id) | ||||
|
r5685 | |||
|
r4615 | issue3.init_journal(User.find(2)) | ||
issue3.subject = 'child with journal' | ||||
issue3.save! | ||||
|
r5685 | |||
|
r4615 | assert_difference 'Issue.count', -2 do | ||
assert_difference 'Journal.count', -1 do | ||||
assert_difference 'JournalDetail.count', -1 do | ||||
Issue.find(issue2.id).destroy | ||||
end | ||||
end | ||||
end | ||||
|
r5685 | |||
|
r3459 | issue1.reload | ||
issue4.reload | ||||
assert !Issue.exists?(issue2.id) | ||||
assert !Issue.exists?(issue3.id) | ||||
assert_equal [issue1.id, 1, 4], [issue1.root_id, issue1.lft, issue1.rgt] | ||||
assert_equal [issue1.id, 2, 3], [issue4.root_id, issue4.lft, issue4.rgt] | ||||
end | ||||
|
r5685 | |||
|
r5165 | def test_destroy_parent_issue_updated_during_children_destroy | ||
parent = create_issue! | ||||
create_issue!(:start_date => Date.today, :parent_issue_id => parent.id) | ||||
create_issue!(:start_date => 2.days.from_now, :parent_issue_id => parent.id) | ||||
|
r5685 | |||
|
r5165 | assert_difference 'Issue.count', -3 do | ||
Issue.find(parent.id).destroy | ||||
end | ||||
end | ||||
|
r5685 | |||
|
r4615 | def test_destroy_child_issue_with_children | ||
root = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'root') | ||||
child = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => root.id) | ||||
leaf = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'leaf', :parent_issue_id => child.id) | ||||
leaf.init_journal(User.find(2)) | ||||
leaf.subject = 'leaf with journal' | ||||
leaf.save! | ||||
|
r5685 | |||
|
r4615 | assert_difference 'Issue.count', -2 do | ||
assert_difference 'Journal.count', -1 do | ||||
assert_difference 'JournalDetail.count', -1 do | ||||
Issue.find(child.id).destroy | ||||
end | ||||
end | ||||
end | ||||
|
r5685 | |||
|
r4615 | root = Issue.find(root.id) | ||
assert root.leaf?, "Root issue is not a leaf (lft: #{root.lft}, rgt: #{root.rgt})" | ||||
end | ||||
|
r6191 | |||
def test_destroy_issue_with_grand_child | ||||
parent = create_issue! | ||||
issue = create_issue!(:parent_issue_id => parent.id) | ||||
child = create_issue!(:parent_issue_id => issue.id) | ||||
grandchild1 = create_issue!(:parent_issue_id => child.id) | ||||
grandchild2 = create_issue!(:parent_issue_id => child.id) | ||||
assert_difference 'Issue.count', -4 do | ||||
Issue.find(issue.id).destroy | ||||
parent.reload | ||||
assert_equal [1, 2], [parent.lft, parent.rgt] | ||||
end | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_parent_priority_should_be_the_highest_child_priority | ||
parent = create_issue!(:priority => IssuePriority.find_by_name('Normal')) | ||||
# Create children | ||||
child1 = create_issue!(:priority => IssuePriority.find_by_name('High'), :parent_issue_id => parent.id) | ||||
assert_equal 'High', parent.reload.priority.name | ||||
child2 = create_issue!(:priority => IssuePriority.find_by_name('Immediate'), :parent_issue_id => child1.id) | ||||
assert_equal 'Immediate', child1.reload.priority.name | ||||
assert_equal 'Immediate', parent.reload.priority.name | ||||
child3 = create_issue!(:priority => IssuePriority.find_by_name('Low'), :parent_issue_id => parent.id) | ||||
assert_equal 'Immediate', parent.reload.priority.name | ||||
# Destroy a child | ||||
child1.destroy | ||||
assert_equal 'Low', parent.reload.priority.name | ||||
# Update a child | ||||
child3.reload.priority = IssuePriority.find_by_name('Normal') | ||||
child3.save! | ||||
assert_equal 'Normal', parent.reload.priority.name | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_parent_dates_should_be_lowest_start_and_highest_due_dates | ||
parent = create_issue! | ||||
create_issue!(:start_date => '2010-01-25', :due_date => '2010-02-15', :parent_issue_id => parent.id) | ||||
create_issue!( :due_date => '2010-02-13', :parent_issue_id => parent.id) | ||||
create_issue!(:start_date => '2010-02-01', :due_date => '2010-02-22', :parent_issue_id => parent.id) | ||||
parent.reload | ||||
assert_equal Date.parse('2010-01-25'), parent.start_date | ||||
assert_equal Date.parse('2010-02-22'), parent.due_date | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_parent_done_ratio_should_be_average_done_ratio_of_leaves | ||
parent = create_issue! | ||||
create_issue!(:done_ratio => 20, :parent_issue_id => parent.id) | ||||
assert_equal 20, parent.reload.done_ratio | ||||
create_issue!(:done_ratio => 70, :parent_issue_id => parent.id) | ||||
assert_equal 45, parent.reload.done_ratio | ||||
|
r5685 | |||
|
r3459 | child = create_issue!(:done_ratio => 0, :parent_issue_id => parent.id) | ||
assert_equal 30, parent.reload.done_ratio | ||||
|
r5685 | |||
|
r3459 | create_issue!(:done_ratio => 30, :parent_issue_id => child.id) | ||
assert_equal 30, child.reload.done_ratio | ||||
assert_equal 40, parent.reload.done_ratio | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_parent_done_ratio_should_be_weighted_by_estimated_times_if_any | ||
parent = create_issue! | ||||
create_issue!(:estimated_hours => 10, :done_ratio => 20, :parent_issue_id => parent.id) | ||||
assert_equal 20, parent.reload.done_ratio | ||||
create_issue!(:estimated_hours => 20, :done_ratio => 50, :parent_issue_id => parent.id) | ||||
assert_equal (50 * 20 + 20 * 10) / 30, parent.reload.done_ratio | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_parent_estimate_should_be_sum_of_leaves | ||
parent = create_issue! | ||||
create_issue!(:estimated_hours => nil, :parent_issue_id => parent.id) | ||||
assert_equal nil, parent.reload.estimated_hours | ||||
create_issue!(:estimated_hours => 5, :parent_issue_id => parent.id) | ||||
assert_equal 5, parent.reload.estimated_hours | ||||
create_issue!(:estimated_hours => 7, :parent_issue_id => parent.id) | ||||
assert_equal 12, parent.reload.estimated_hours | ||||
end | ||||
|
r3460 | |||
|
r3707 | def test_move_parent_updates_old_parent_attributes | ||
first_parent = create_issue! | ||||
second_parent = create_issue! | ||||
child = create_issue!(:estimated_hours => 5, :parent_issue_id => first_parent.id) | ||||
assert_equal 5, first_parent.reload.estimated_hours | ||||
child.update_attributes(:estimated_hours => 7, :parent_issue_id => second_parent.id) | ||||
assert_equal 7, second_parent.reload.estimated_hours | ||||
assert_nil first_parent.reload.estimated_hours | ||||
end | ||||
|
r3460 | |||
def test_reschuling_a_parent_should_reschedule_subtasks | ||||
parent = create_issue! | ||||
c1 = create_issue!(:start_date => '2010-05-12', :due_date => '2010-05-18', :parent_issue_id => parent.id) | ||||
c2 = create_issue!(:start_date => '2010-06-03', :due_date => '2010-06-10', :parent_issue_id => parent.id) | ||||
parent.reload | ||||
parent.reschedule_after(Date.parse('2010-06-02')) | ||||
c1.reload | ||||
assert_equal [Date.parse('2010-06-02'), Date.parse('2010-06-08')], [c1.start_date, c1.due_date] | ||||
c2.reload | ||||
assert_equal [Date.parse('2010-06-03'), Date.parse('2010-06-10')], [c2.start_date, c2.due_date] # no change | ||||
parent.reload | ||||
assert_equal [Date.parse('2010-06-02'), Date.parse('2010-06-10')], [parent.start_date, parent.due_date] | ||||
end | ||||
|
r5685 | |||
|
r3459 | def test_project_copy_should_copy_issue_tree | ||
p = Project.create!(:name => 'Tree copy', :identifier => 'tree-copy', :tracker_ids => [1, 2]) | ||||
i1 = create_issue!(:project_id => p.id, :subject => 'i1') | ||||
i2 = create_issue!(:project_id => p.id, :subject => 'i2', :parent_issue_id => i1.id) | ||||
i3 = create_issue!(:project_id => p.id, :subject => 'i3', :parent_issue_id => i1.id) | ||||
i4 = create_issue!(:project_id => p.id, :subject => 'i4', :parent_issue_id => i2.id) | ||||
i5 = create_issue!(:project_id => p.id, :subject => 'i5') | ||||
c = Project.new(:name => 'Copy', :identifier => 'copy', :tracker_ids => [1, 2]) | ||||
c.copy(p, :only => 'issues') | ||||
c.reload | ||||
|
r5685 | |||
|
r3459 | assert_equal 5, c.issues.count | ||
ic1, ic2, ic3, ic4, ic5 = c.issues.find(:all, :order => 'subject') | ||||
assert ic1.root? | ||||
assert_equal ic1, ic2.parent | ||||
assert_equal ic1, ic3.parent | ||||
assert_equal ic2, ic4.parent | ||||
assert ic5.root? | ||||
end | ||||
|
r5685 | |||
|
r3459 | # Helper that creates an issue with default attributes | ||
def create_issue!(attributes={}) | ||||
Issue.create!({:project_id => 1, :tracker_id => 1, :author_id => 1, :subject => 'test'}.merge(attributes)) | ||||
end | ||||
end | ||||