Chris@1494: # Redmine - project management software Chris@1494: # Copyright (C) 2006-2014 Jean-Philippe Lang Chris@1494: # Chris@1494: # This program is free software; you can redistribute it and/or Chris@1494: # modify it under the terms of the GNU General Public License Chris@1494: # as published by the Free Software Foundation; either version 2 Chris@1494: # of the License, or (at your option) any later version. Chris@1494: # Chris@1494: # This program is distributed in the hope that it will be useful, Chris@1494: # but WITHOUT ANY WARRANTY; without even the implied warranty of Chris@1494: # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the Chris@1494: # GNU General Public License for more details. Chris@1494: # Chris@1494: # You should have received a copy of the GNU General Public License Chris@1494: # along with this program; if not, write to the Free Software Chris@1494: # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. Chris@1494: Chris@1494: require File.expand_path('../../test_helper', __FILE__) Chris@1494: Chris@1494: class IssueNestedSetTest < ActiveSupport::TestCase Chris@1494: fixtures :projects, :users, :roles, Chris@1494: :trackers, :projects_trackers, Chris@1494: :issue_statuses, :issue_categories, :issue_relations, Chris@1494: :enumerations, Chris@1494: :issues Chris@1494: Chris@1494: def test_create_root_issue Chris@1494: issue1 = Issue.generate! Chris@1494: issue2 = Issue.generate! Chris@1494: issue1.reload Chris@1494: issue2.reload Chris@1494: Chris@1494: assert_equal [issue1.id, nil, 1, 2], [issue1.root_id, issue1.parent_id, issue1.lft, issue1.rgt] Chris@1494: assert_equal [issue2.id, nil, 1, 2], [issue2.root_id, issue2.parent_id, issue2.lft, issue2.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_create_child_issue Chris@1494: parent = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent.id) Chris@1494: parent.reload Chris@1494: child.reload Chris@1494: Chris@1494: assert_equal [parent.id, nil, 1, 4], [parent.root_id, parent.parent_id, parent.lft, parent.rgt] Chris@1494: assert_equal [parent.id, parent.id, 2, 3], [child.root_id, child.parent_id, child.lft, child.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_creating_a_child_in_a_subproject_should_validate Chris@1494: issue = Issue.generate! Chris@1494: child = Issue.new(:project_id => 3, :tracker_id => 2, :author_id => 1, Chris@1494: :subject => 'child', :parent_issue_id => issue.id) Chris@1494: assert_save child Chris@1494: assert_equal issue, child.reload.parent Chris@1494: end Chris@1494: Chris@1494: def test_creating_a_child_in_an_invalid_project_should_not_validate Chris@1494: issue = Issue.generate! Chris@1494: child = Issue.new(:project_id => 2, :tracker_id => 1, :author_id => 1, Chris@1494: :subject => 'child', :parent_issue_id => issue.id) Chris@1494: assert !child.save Chris@1494: assert_not_equal [], child.errors[:parent_issue_id] Chris@1494: end Chris@1494: Chris@1494: def test_move_a_root_to_child Chris@1494: parent1 = Issue.generate! Chris@1494: parent2 = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent1.id) Chris@1494: Chris@1494: parent2.parent_issue_id = parent1.id Chris@1494: parent2.save! Chris@1494: child.reload Chris@1494: parent1.reload Chris@1494: parent2.reload Chris@1494: Chris@1494: assert_equal [parent1.id, 1, 6], [parent1.root_id, parent1.lft, parent1.rgt] Chris@1494: assert_equal [parent1.id, 4, 5], [parent2.root_id, parent2.lft, parent2.rgt] Chris@1494: assert_equal [parent1.id, 2, 3], [child.root_id, child.lft, child.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_move_a_child_to_root Chris@1494: parent1 = Issue.generate! Chris@1494: parent2 = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent1.id) Chris@1494: Chris@1494: child.parent_issue_id = nil Chris@1494: child.save! Chris@1494: child.reload Chris@1494: parent1.reload Chris@1494: parent2.reload Chris@1494: Chris@1494: assert_equal [parent1.id, 1, 2], [parent1.root_id, parent1.lft, parent1.rgt] Chris@1494: assert_equal [parent2.id, 1, 2], [parent2.root_id, parent2.lft, parent2.rgt] Chris@1494: assert_equal [child.id, 1, 2], [child.root_id, child.lft, child.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_move_a_child_to_another_issue Chris@1494: parent1 = Issue.generate! Chris@1494: parent2 = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent1.id) Chris@1494: Chris@1494: child.parent_issue_id = parent2.id Chris@1494: child.save! Chris@1494: child.reload Chris@1494: parent1.reload Chris@1494: parent2.reload Chris@1494: Chris@1494: assert_equal [parent1.id, 1, 2], [parent1.root_id, parent1.lft, parent1.rgt] Chris@1494: assert_equal [parent2.id, 1, 4], [parent2.root_id, parent2.lft, parent2.rgt] Chris@1494: assert_equal [parent2.id, 2, 3], [child.root_id, child.lft, child.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_move_a_child_with_descendants_to_another_issue Chris@1494: parent1 = Issue.generate! Chris@1494: parent2 = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent1.id) Chris@1494: grandchild = Issue.generate!(:parent_issue_id => child.id) Chris@1494: Chris@1494: parent1.reload Chris@1494: parent2.reload Chris@1494: child.reload Chris@1494: grandchild.reload Chris@1494: Chris@1494: assert_equal [parent1.id, 1, 6], [parent1.root_id, parent1.lft, parent1.rgt] Chris@1494: assert_equal [parent2.id, 1, 2], [parent2.root_id, parent2.lft, parent2.rgt] Chris@1494: assert_equal [parent1.id, 2, 5], [child.root_id, child.lft, child.rgt] Chris@1494: assert_equal [parent1.id, 3, 4], [grandchild.root_id, grandchild.lft, grandchild.rgt] Chris@1494: Chris@1494: child.reload.parent_issue_id = parent2.id Chris@1494: child.save! Chris@1494: child.reload Chris@1494: grandchild.reload Chris@1494: parent1.reload Chris@1494: parent2.reload Chris@1494: Chris@1494: assert_equal [parent1.id, 1, 2], [parent1.root_id, parent1.lft, parent1.rgt] Chris@1494: assert_equal [parent2.id, 1, 6], [parent2.root_id, parent2.lft, parent2.rgt] Chris@1494: assert_equal [parent2.id, 2, 5], [child.root_id, child.lft, child.rgt] Chris@1494: assert_equal [parent2.id, 3, 4], [grandchild.root_id, grandchild.lft, grandchild.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_move_a_child_with_descendants_to_another_project Chris@1494: parent1 = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent1.id) Chris@1494: grandchild = Issue.generate!(:parent_issue_id => child.id) Chris@1494: Chris@1494: child.reload Chris@1494: child.project = Project.find(2) Chris@1494: assert child.save Chris@1494: child.reload Chris@1494: grandchild.reload Chris@1494: parent1.reload Chris@1494: Chris@1494: assert_equal [1, parent1.id, 1, 2], [parent1.project_id, parent1.root_id, parent1.lft, parent1.rgt] Chris@1494: assert_equal [2, child.id, 1, 4], [child.project_id, child.root_id, child.lft, child.rgt] Chris@1494: assert_equal [2, child.id, 2, 3], [grandchild.project_id, grandchild.root_id, grandchild.lft, grandchild.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_moving_an_issue_to_a_descendant_should_not_validate Chris@1494: parent1 = Issue.generate! Chris@1494: parent2 = Issue.generate! Chris@1494: child = Issue.generate!(:parent_issue_id => parent1.id) Chris@1494: grandchild = Issue.generate!(:parent_issue_id => child.id) Chris@1494: Chris@1494: child.reload Chris@1494: child.parent_issue_id = grandchild.id Chris@1494: assert !child.save Chris@1494: assert_not_equal [], child.errors[:parent_issue_id] Chris@1494: end Chris@1494: Chris@1494: def test_updating_a_root_issue_should_not_trigger_update_nested_set_attributes_on_parent_change Chris@1494: issue = Issue.find(Issue.generate!.id) Chris@1494: issue.parent_issue_id = "" Chris@1494: issue.expects(:update_nested_set_attributes_on_parent_change).never Chris@1494: issue.save! Chris@1494: end Chris@1494: Chris@1494: def test_updating_a_child_issue_should_not_trigger_update_nested_set_attributes_on_parent_change Chris@1494: issue = Issue.find(Issue.generate!(:parent_issue_id => 1).id) Chris@1494: issue.parent_issue_id = "1" Chris@1494: issue.expects(:update_nested_set_attributes_on_parent_change).never Chris@1494: issue.save! Chris@1494: end Chris@1494: Chris@1494: def test_moving_a_root_issue_should_trigger_update_nested_set_attributes_on_parent_change Chris@1494: issue = Issue.find(Issue.generate!.id) Chris@1494: issue.parent_issue_id = "1" Chris@1494: issue.expects(:update_nested_set_attributes_on_parent_change).once Chris@1494: issue.save! Chris@1494: end Chris@1494: Chris@1494: def test_moving_a_child_issue_to_another_parent_should_trigger_update_nested_set_attributes_on_parent_change Chris@1494: issue = Issue.find(Issue.generate!(:parent_issue_id => 1).id) Chris@1494: issue.parent_issue_id = "2" Chris@1494: issue.expects(:update_nested_set_attributes_on_parent_change).once Chris@1494: issue.save! Chris@1494: end Chris@1494: Chris@1494: def test_moving_a_child_issue_to_root_should_trigger_update_nested_set_attributes_on_parent_change Chris@1494: issue = Issue.find(Issue.generate!(:parent_issue_id => 1).id) Chris@1494: issue.parent_issue_id = "" Chris@1494: issue.expects(:update_nested_set_attributes_on_parent_change).once Chris@1494: issue.save! Chris@1494: end Chris@1494: Chris@1494: def test_destroy_should_destroy_children Chris@1494: issue1 = Issue.generate! Chris@1494: issue2 = Issue.generate! Chris@1494: issue3 = Issue.generate!(:parent_issue_id => issue2.id) Chris@1494: issue4 = Issue.generate!(:parent_issue_id => issue1.id) Chris@1494: Chris@1494: issue3.init_journal(User.find(2)) Chris@1494: issue3.subject = 'child with journal' Chris@1494: issue3.save! Chris@1494: Chris@1494: assert_difference 'Issue.count', -2 do Chris@1494: assert_difference 'Journal.count', -1 do Chris@1494: assert_difference 'JournalDetail.count', -1 do Chris@1494: Issue.find(issue2.id).destroy Chris@1494: end Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: issue1.reload Chris@1494: issue4.reload Chris@1494: assert !Issue.exists?(issue2.id) Chris@1494: assert !Issue.exists?(issue3.id) Chris@1494: assert_equal [issue1.id, 1, 4], [issue1.root_id, issue1.lft, issue1.rgt] Chris@1494: assert_equal [issue1.id, 2, 3], [issue4.root_id, issue4.lft, issue4.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_destroy_child_should_update_parent Chris@1494: issue = Issue.generate! Chris@1494: child1 = Issue.generate!(:parent_issue_id => issue.id) Chris@1494: child2 = Issue.generate!(:parent_issue_id => issue.id) Chris@1494: Chris@1494: issue.reload Chris@1494: assert_equal [issue.id, 1, 6], [issue.root_id, issue.lft, issue.rgt] Chris@1494: Chris@1494: child2.reload.destroy Chris@1494: Chris@1494: issue.reload Chris@1494: assert_equal [issue.id, 1, 4], [issue.root_id, issue.lft, issue.rgt] Chris@1494: end Chris@1494: Chris@1494: def test_destroy_parent_issue_updated_during_children_destroy Chris@1494: parent = Issue.generate! Chris@1494: Issue.generate!(:start_date => Date.today, :parent_issue_id => parent.id) Chris@1494: Issue.generate!(:start_date => 2.days.from_now, :parent_issue_id => parent.id) Chris@1494: Chris@1494: assert_difference 'Issue.count', -3 do Chris@1494: Issue.find(parent.id).destroy Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: def test_destroy_child_issue_with_children Chris@1494: root = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'root') Chris@1494: child = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'child', :parent_issue_id => root.id) Chris@1494: leaf = Issue.create!(:project_id => 1, :author_id => 2, :tracker_id => 1, :subject => 'leaf', :parent_issue_id => child.id) Chris@1494: leaf.init_journal(User.find(2)) Chris@1494: leaf.subject = 'leaf with journal' Chris@1494: leaf.save! Chris@1494: Chris@1494: assert_difference 'Issue.count', -2 do Chris@1494: assert_difference 'Journal.count', -1 do Chris@1494: assert_difference 'JournalDetail.count', -1 do Chris@1494: Issue.find(child.id).destroy Chris@1494: end Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: root = Issue.find(root.id) Chris@1494: assert root.leaf?, "Root issue is not a leaf (lft: #{root.lft}, rgt: #{root.rgt})" Chris@1494: end Chris@1494: Chris@1494: def test_destroy_issue_with_grand_child Chris@1494: parent = Issue.generate! Chris@1494: issue = Issue.generate!(:parent_issue_id => parent.id) Chris@1494: child = Issue.generate!(:parent_issue_id => issue.id) Chris@1494: grandchild1 = Issue.generate!(:parent_issue_id => child.id) Chris@1494: grandchild2 = Issue.generate!(:parent_issue_id => child.id) Chris@1494: Chris@1494: assert_difference 'Issue.count', -4 do Chris@1494: Issue.find(issue.id).destroy Chris@1494: parent.reload Chris@1494: assert_equal [1, 2], [parent.lft, parent.rgt] Chris@1494: end Chris@1494: end Chris@1494: Chris@1494: def test_parent_priority_should_be_the_highest_child_priority Chris@1494: parent = Issue.generate!(:priority => IssuePriority.find_by_name('Normal')) Chris@1494: # Create children Chris@1494: child1 = Issue.generate!(:priority => IssuePriority.find_by_name('High'), :parent_issue_id => parent.id) Chris@1494: assert_equal 'High', parent.reload.priority.name Chris@1494: child2 = Issue.generate!(:priority => IssuePriority.find_by_name('Immediate'), :parent_issue_id => child1.id) Chris@1494: assert_equal 'Immediate', child1.reload.priority.name Chris@1494: assert_equal 'Immediate', parent.reload.priority.name Chris@1494: child3 = Issue.generate!(:priority => IssuePriority.find_by_name('Low'), :parent_issue_id => parent.id) Chris@1494: assert_equal 'Immediate', parent.reload.priority.name Chris@1494: # Destroy a child Chris@1494: child1.destroy Chris@1494: assert_equal 'Low', parent.reload.priority.name Chris@1494: # Update a child Chris@1494: child3.reload.priority = IssuePriority.find_by_name('Normal') Chris@1494: child3.save! Chris@1494: assert_equal 'Normal', parent.reload.priority.name Chris@1494: end Chris@1494: Chris@1494: def test_parent_dates_should_be_lowest_start_and_highest_due_dates Chris@1494: parent = Issue.generate! Chris@1494: Issue.generate!(:start_date => '2010-01-25', :due_date => '2010-02-15', :parent_issue_id => parent.id) Chris@1494: Issue.generate!( :due_date => '2010-02-13', :parent_issue_id => parent.id) Chris@1494: Issue.generate!(:start_date => '2010-02-01', :due_date => '2010-02-22', :parent_issue_id => parent.id) Chris@1494: parent.reload Chris@1494: assert_equal Date.parse('2010-01-25'), parent.start_date Chris@1494: assert_equal Date.parse('2010-02-22'), parent.due_date Chris@1494: end Chris@1494: Chris@1494: def test_parent_done_ratio_should_be_average_done_ratio_of_leaves Chris@1494: parent = Issue.generate! Chris@1494: Issue.generate!(:done_ratio => 20, :parent_issue_id => parent.id) Chris@1494: assert_equal 20, parent.reload.done_ratio Chris@1494: Issue.generate!(:done_ratio => 70, :parent_issue_id => parent.id) Chris@1494: assert_equal 45, parent.reload.done_ratio Chris@1494: Chris@1494: child = Issue.generate!(:done_ratio => 0, :parent_issue_id => parent.id) Chris@1494: assert_equal 30, parent.reload.done_ratio Chris@1494: Chris@1494: Issue.generate!(:done_ratio => 30, :parent_issue_id => child.id) Chris@1494: assert_equal 30, child.reload.done_ratio Chris@1494: assert_equal 40, parent.reload.done_ratio Chris@1494: end Chris@1494: Chris@1494: def test_parent_done_ratio_should_be_weighted_by_estimated_times_if_any Chris@1494: parent = Issue.generate! Chris@1494: Issue.generate!(:estimated_hours => 10, :done_ratio => 20, :parent_issue_id => parent.id) Chris@1494: assert_equal 20, parent.reload.done_ratio Chris@1494: Issue.generate!(:estimated_hours => 20, :done_ratio => 50, :parent_issue_id => parent.id) Chris@1494: assert_equal (50 * 20 + 20 * 10) / 30, parent.reload.done_ratio Chris@1494: end Chris@1494: Chris@1494: def test_parent_done_ratio_with_child_estimate_to_0_should_reach_100 Chris@1494: parent = Issue.generate! Chris@1494: issue1 = Issue.generate!(:parent_issue_id => parent.id) Chris@1494: issue2 = Issue.generate!(:parent_issue_id => parent.id, :estimated_hours => 0) Chris@1494: assert_equal 0, parent.reload.done_ratio Chris@1494: issue1.reload.update_attribute :status_id, 5 Chris@1494: assert_equal 50, parent.reload.done_ratio Chris@1494: issue2.reload.update_attribute :status_id, 5 Chris@1494: assert_equal 100, parent.reload.done_ratio Chris@1494: end Chris@1494: Chris@1494: def test_parent_estimate_should_be_sum_of_leaves Chris@1494: parent = Issue.generate! Chris@1494: Issue.generate!(:estimated_hours => nil, :parent_issue_id => parent.id) Chris@1494: assert_equal nil, parent.reload.estimated_hours Chris@1494: Issue.generate!(:estimated_hours => 5, :parent_issue_id => parent.id) Chris@1494: assert_equal 5, parent.reload.estimated_hours Chris@1494: Issue.generate!(:estimated_hours => 7, :parent_issue_id => parent.id) Chris@1494: assert_equal 12, parent.reload.estimated_hours Chris@1494: end Chris@1494: Chris@1494: def test_done_ratio_of_parent_with_a_child_with_estimated_time_at_0_should_not_exceed_100 Chris@1494: parent = Issue.generate! Chris@1494: Issue.generate!(:estimated_hours => 40, :parent_issue_id => parent.id) Chris@1494: Issue.generate!(:estimated_hours => 40, :parent_issue_id => parent.id) Chris@1494: Issue.generate!(:estimated_hours => 20, :parent_issue_id => parent.id) Chris@1494: Issue.generate!(:estimated_hours => 0, :parent_issue_id => parent.id) Chris@1494: parent.reload.children.each do |child| Chris@1494: child.update_attribute :status_id, 5 Chris@1494: end Chris@1494: assert_equal 100, parent.reload.done_ratio Chris@1494: end Chris@1494: Chris@1494: def test_move_parent_updates_old_parent_attributes Chris@1494: first_parent = Issue.generate! Chris@1494: second_parent = Issue.generate! Chris@1494: child = Issue.generate!(:estimated_hours => 5, :parent_issue_id => first_parent.id) Chris@1494: assert_equal 5, first_parent.reload.estimated_hours Chris@1494: child.update_attributes(:estimated_hours => 7, :parent_issue_id => second_parent.id) Chris@1494: assert_equal 7, second_parent.reload.estimated_hours Chris@1494: assert_nil first_parent.reload.estimated_hours Chris@1494: end Chris@1494: Chris@1494: def test_reschuling_a_parent_should_reschedule_subtasks Chris@1494: parent = Issue.generate! Chris@1494: c1 = Issue.generate!(:start_date => '2010-05-12', :due_date => '2010-05-18', :parent_issue_id => parent.id) Chris@1494: c2 = Issue.generate!(:start_date => '2010-06-03', :due_date => '2010-06-10', :parent_issue_id => parent.id) Chris@1494: parent.reload Chris@1494: parent.reschedule_on!(Date.parse('2010-06-02')) Chris@1494: c1.reload Chris@1494: assert_equal [Date.parse('2010-06-02'), Date.parse('2010-06-08')], [c1.start_date, c1.due_date] Chris@1494: c2.reload Chris@1494: assert_equal [Date.parse('2010-06-03'), Date.parse('2010-06-10')], [c2.start_date, c2.due_date] # no change Chris@1494: parent.reload Chris@1494: assert_equal [Date.parse('2010-06-02'), Date.parse('2010-06-10')], [parent.start_date, parent.due_date] Chris@1494: end Chris@1494: Chris@1494: def test_project_copy_should_copy_issue_tree Chris@1494: p = Project.create!(:name => 'Tree copy', :identifier => 'tree-copy', :tracker_ids => [1, 2]) Chris@1494: i1 = Issue.generate!(:project => p, :subject => 'i1') Chris@1494: i2 = Issue.generate!(:project => p, :subject => 'i2', :parent_issue_id => i1.id) Chris@1494: i3 = Issue.generate!(:project => p, :subject => 'i3', :parent_issue_id => i1.id) Chris@1494: i4 = Issue.generate!(:project => p, :subject => 'i4', :parent_issue_id => i2.id) Chris@1494: i5 = Issue.generate!(:project => p, :subject => 'i5') Chris@1494: c = Project.new(:name => 'Copy', :identifier => 'copy', :tracker_ids => [1, 2]) Chris@1494: c.copy(p, :only => 'issues') Chris@1494: c.reload Chris@1494: Chris@1494: assert_equal 5, c.issues.count Chris@1494: ic1, ic2, ic3, ic4, ic5 = c.issues.order('subject').all Chris@1494: assert ic1.root? Chris@1494: assert_equal ic1, ic2.parent Chris@1494: assert_equal ic1, ic3.parent Chris@1494: assert_equal ic2, ic4.parent Chris@1494: assert ic5.root? Chris@1494: end Chris@1494: end