issue_relation_test.rb
236 lines
| 8.6 KiB
| text/x-ruby
|
RubyLexer
Jean-Philippe Lang
|
r3076 | # Redmine - project management software | ||
|
Jean-Philippe Lang
|
r14856 | # Copyright (C) 2006-2016 Jean-Philippe Lang | ||
|
Jean-Philippe Lang
|
r3076 | # | ||
| # 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. | ||||
Toshi MARUYAMA
|
r6628 | # | ||
|
Jean-Philippe Lang
|
r3076 | # 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. | ||||
|
Toshi MARUYAMA
|
r6628 | # | ||
|
Jean-Philippe Lang
|
r3076 | # 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. | ||||
Jean-Baptiste Barth
|
r4395 | require File.expand_path('../../test_helper', __FILE__) | ||
|
Jean-Philippe Lang
|
r3076 | |||
| class IssueRelationTest < ActiveSupport::TestCase | ||||
|
Toshi MARUYAMA
|
r10361 | fixtures :projects, | ||
| :users, | ||||
| :roles, | ||||
| :members, | ||||
| :member_roles, | ||||
| :issues, | ||||
| :issue_statuses, | ||||
| :issue_relations, | ||||
| :enabled_modules, | ||||
| :enumerations, | ||||
|
Toshi MARUYAMA
|
r11839 | :trackers, | ||
| :projects_trackers | ||||
|
Jean-Philippe Lang
|
r3076 | |||
|
Jean-Philippe Lang
|
r11411 | include Redmine::I18n | ||
|
Jean-Philippe Lang
|
r3076 | def test_create | ||
| from = Issue.find(1) | ||||
| to = Issue.find(2) | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Toshi MARUYAMA
|
r7479 | relation = IssueRelation.new :issue_from => from, :issue_to => to, | ||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
|
Jean-Philippe Lang
|
r3076 | assert relation.save | ||
| relation.reload | ||||
| assert_equal IssueRelation::TYPE_PRECEDES, relation.relation_type | ||||
| assert_equal from, relation.issue_from | ||||
| assert_equal to, relation.issue_to | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r6057 | def test_create_minimum | ||
| relation = IssueRelation.new :issue_from => Issue.find(1), :issue_to => Issue.find(2) | ||||
| assert relation.save | ||||
| assert_equal IssueRelation::TYPE_RELATES, relation.relation_type | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r3076 | def test_follows_relation_should_be_reversed | ||
| from = Issue.find(1) | ||||
| to = Issue.find(2) | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Toshi MARUYAMA
|
r7480 | relation = IssueRelation.new :issue_from => from, :issue_to => to, | ||
| :relation_type => IssueRelation::TYPE_FOLLOWS | ||||
|
Jean-Philippe Lang
|
r3076 | assert relation.save | ||
| relation.reload | ||||
| assert_equal IssueRelation::TYPE_PRECEDES, relation.relation_type | ||||
| assert_equal to, relation.issue_from | ||||
| assert_equal from, relation.issue_to | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r3077 | def test_follows_relation_should_not_be_reversed_if_validation_fails | ||
| from = Issue.find(1) | ||||
| to = Issue.find(2) | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Toshi MARUYAMA
|
r7481 | relation = IssueRelation.new :issue_from => from, :issue_to => to, | ||
| :relation_type => IssueRelation::TYPE_FOLLOWS, | ||||
| :delay => 'xx' | ||||
|
Jean-Philippe Lang
|
r3077 | assert !relation.save | ||
| assert_equal IssueRelation::TYPE_FOLLOWS, relation.relation_type | ||||
| assert_equal from, relation.issue_from | ||||
| assert_equal to, relation.issue_to | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r3615 | def test_relation_type_for | ||
| from = Issue.find(1) | ||||
| to = Issue.find(2) | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Toshi MARUYAMA
|
r7482 | relation = IssueRelation.new :issue_from => from, :issue_to => to, | ||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
|
Jean-Philippe Lang
|
r3615 | assert_equal IssueRelation::TYPE_PRECEDES, relation.relation_type_for(from) | ||
| assert_equal IssueRelation::TYPE_FOLLOWS, relation.relation_type_for(to) | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r4353 | def test_set_issue_to_dates_without_issue_to | ||
|
Toshi MARUYAMA
|
r7483 | r = IssueRelation.new(:issue_from => Issue.new(:start_date => Date.today), | ||
| :relation_type => IssueRelation::TYPE_PRECEDES, | ||||
| :delay => 1) | ||||
|
Jean-Philippe Lang
|
r4353 | assert_nil r.set_issue_to_dates | ||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r4353 | def test_set_issue_to_dates_without_issues | ||
| r = IssueRelation.new(:relation_type => IssueRelation::TYPE_PRECEDES, :delay => 1) | ||||
| assert_nil r.set_issue_to_dates | ||||
| end | ||||
|
Toshi MARUYAMA
|
r6628 | |||
|
Jean-Philippe Lang
|
r4601 | def test_validates_circular_dependency | ||
| IssueRelation.delete_all | ||||
|
Toshi MARUYAMA
|
r10362 | assert IssueRelation.create!( | ||
| :issue_from => Issue.find(1), :issue_to => Issue.find(2), | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
| ) | ||||
| assert IssueRelation.create!( | ||||
| :issue_from => Issue.find(2), :issue_to => Issue.find(3), | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
| ) | ||||
| r = IssueRelation.new( | ||||
| :issue_from => Issue.find(3), :issue_to => Issue.find(1), | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
| ) | ||||
|
Jean-Philippe Lang
|
r4601 | assert !r.save | ||
|
Toshi MARUYAMA
|
r11840 | assert_not_equal [], r.errors[:base] | ||
|
Jean-Philippe Lang
|
r4601 | end | ||
|
Jean-Baptiste Barth
|
r6004 | |||
|
Jean-Philippe Lang
|
r11411 | def test_validates_circular_dependency_of_subtask | ||
| set_language_if_valid 'en' | ||||
| issue1 = Issue.generate! | ||||
| issue2 = Issue.generate! | ||||
| IssueRelation.create!( | ||||
| :issue_from => issue1, :issue_to => issue2, | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
| ) | ||||
| child = Issue.generate!(:parent_issue_id => issue2.id) | ||||
| issue1.reload | ||||
| child.reload | ||||
| r = IssueRelation.new( | ||||
| :issue_from => child, :issue_to => issue1, | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
| ) | ||||
| assert !r.save | ||||
| assert_include 'This relation would create a circular dependency', r.errors.full_messages | ||||
| end | ||||
|
Jean-Philippe Lang
|
r11462 | def test_subtasks_should_allow_precedes_relation | ||
| parent = Issue.generate! | ||||
| child1 = Issue.generate!(:parent_issue_id => parent.id) | ||||
| child2 = Issue.generate!(:parent_issue_id => parent.id) | ||||
| r = IssueRelation.new( | ||||
| :issue_from => child1, :issue_to => child2, | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES | ||||
| ) | ||||
| assert r.valid? | ||||
| assert r.save | ||||
| end | ||||
|
Jean-Baptiste Barth
|
r6004 | def test_validates_circular_dependency_on_reverse_relations | ||
| IssueRelation.delete_all | ||||
|
Toshi MARUYAMA
|
r10362 | assert IssueRelation.create!( | ||
| :issue_from => Issue.find(1), :issue_to => Issue.find(3), | ||||
| :relation_type => IssueRelation::TYPE_BLOCKS | ||||
| ) | ||||
| assert IssueRelation.create!( | ||||
| :issue_from => Issue.find(1), :issue_to => Issue.find(2), | ||||
| :relation_type => IssueRelation::TYPE_BLOCKED | ||||
| ) | ||||
| r = IssueRelation.new( | ||||
| :issue_from => Issue.find(2), :issue_to => Issue.find(1), | ||||
| :relation_type => IssueRelation::TYPE_BLOCKED | ||||
| ) | ||||
|
Jean-Baptiste Barth
|
r6004 | assert !r.save | ||
|
Toshi MARUYAMA
|
r11840 | assert_not_equal [], r.errors[:base] | ||
|
Jean-Baptiste Barth
|
r6004 | end | ||
|
Toshi MARUYAMA
|
r11655 | |||
|
Jean-Philippe Lang
|
r13152 | def test_create_with_initialized_journals_should_create_journals | ||
|
Toshi MARUYAMA
|
r11655 | from = Issue.find(1) | ||
| to = Issue.find(2) | ||||
| from_journals = from.journals.size | ||||
| to_journals = to.journals.size | ||||
| relation = IssueRelation.new(:issue_from => from, :issue_to => to, | ||||
| :relation_type => IssueRelation::TYPE_PRECEDES) | ||||
|
Jean-Philippe Lang
|
r13152 | relation.init_journals User.find(1) | ||
|
Toshi MARUYAMA
|
r11655 | assert relation.save | ||
| from.reload | ||||
| to.reload | ||||
| relation.reload | ||||
| assert_equal from.journals.size, (from_journals + 1) | ||||
| assert_equal to.journals.size, (to_journals + 1) | ||||
| assert_equal 'relation', from.journals.last.details.last.property | ||||
|
Jean-Philippe Lang
|
r12141 | assert_equal 'precedes', from.journals.last.details.last.prop_key | ||
|
Toshi MARUYAMA
|
r11655 | assert_equal '2', from.journals.last.details.last.value | ||
| assert_nil from.journals.last.details.last.old_value | ||||
| assert_equal 'relation', to.journals.last.details.last.property | ||||
|
Jean-Philippe Lang
|
r12141 | assert_equal 'follows', to.journals.last.details.last.prop_key | ||
|
Toshi MARUYAMA
|
r11655 | assert_equal '1', to.journals.last.details.last.value | ||
| assert_nil to.journals.last.details.last.old_value | ||||
| end | ||||
|
Jean-Philippe Lang
|
r13152 | def test_destroy_with_initialized_journals_should_create_journals | ||
|
Toshi MARUYAMA
|
r11655 | relation = IssueRelation.find(1) | ||
| from = relation.issue_from | ||||
| to = relation.issue_to | ||||
| from_journals = from.journals.size | ||||
| to_journals = to.journals.size | ||||
|
Jean-Philippe Lang
|
r13152 | relation.init_journals User.find(1) | ||
|
Toshi MARUYAMA
|
r11655 | assert relation.destroy | ||
| from.reload | ||||
| to.reload | ||||
| assert_equal from.journals.size, (from_journals + 1) | ||||
| assert_equal to.journals.size, (to_journals + 1) | ||||
| assert_equal 'relation', from.journals.last.details.last.property | ||||
|
Jean-Philippe Lang
|
r12141 | assert_equal 'blocks', from.journals.last.details.last.prop_key | ||
|
Toshi MARUYAMA
|
r11655 | assert_equal '9', from.journals.last.details.last.old_value | ||
| assert_nil from.journals.last.details.last.value | ||||
| assert_equal 'relation', to.journals.last.details.last.property | ||||
|
Jean-Philippe Lang
|
r12141 | assert_equal 'blocked', to.journals.last.details.last.prop_key | ||
|
Toshi MARUYAMA
|
r11655 | assert_equal '10', to.journals.last.details.last.old_value | ||
| assert_nil to.journals.last.details.last.value | ||||
| end | ||||
|
Jean-Philippe Lang
|
r13181 | |||
| def test_to_s_should_return_the_relation_string | ||||
| set_language_if_valid 'en' | ||||
| relation = IssueRelation.find(1) | ||||
| assert_equal "Blocks #9", relation.to_s(relation.issue_from) | ||||
| assert_equal "Blocked by #10", relation.to_s(relation.issue_to) | ||||
| end | ||||
| def test_to_s_without_argument_should_return_the_relation_string_for_issue_from | ||||
| set_language_if_valid 'en' | ||||
| relation = IssueRelation.find(1) | ||||
| assert_equal "Blocks #9", relation.to_s | ||||
| end | ||||
| def test_to_s_should_accept_a_block_as_custom_issue_formatting | ||||
| set_language_if_valid 'en' | ||||
| relation = IssueRelation.find(1) | ||||
| assert_equal "Blocks Bug #9", relation.to_s {|issue| "#{issue.tracker} ##{issue.id}"} | ||||
| end | ||||
|
Jean-Philippe Lang
|
r3076 | end | ||