Disallow creating inverse relates issue relations (#27663).

Patch by Gregor Schmidt.


git-svn-id: http://svn.redmine.org/redmine/trunk@17056 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Go MAEDA
2017-12-06 09:01:07 +00:00
parent 0ed07156ad
commit ae842a73db
3 changed files with 25 additions and 3 deletions

View File

@@ -82,8 +82,8 @@ class IssueRelationsControllerTest < Redmine::ControllerTest
assert_equal 'text/javascript', response.content_type
end
relation = IssueRelation.order('id DESC').first
assert_equal 3, relation.issue_from_id
assert_equal 1, relation.issue_to_id
assert_equal 1, relation.issue_from_id
assert_equal 3, relation.issue_to_id
assert_include 'Bug #1', response.body
end

View File

@@ -65,6 +65,20 @@ class IssueRelationTest < ActiveSupport::TestCase
assert_equal from, relation.issue_to
end
def test_cannot_create_inverse_relates_relations
from = Issue.find(1)
to = Issue.find(2)
relation1 = IssueRelation.new :issue_from => from, :issue_to => to,
:relation_type => IssueRelation::TYPE_RELATES
assert relation1.save
relation2 = IssueRelation.new :issue_from => to, :issue_to => from,
:relation_type => IssueRelation::TYPE_RELATES
assert !relation2.save
assert_not_equal [], relation2.errors[:base]
end
def test_follows_relation_should_not_be_reversed_if_validation_fails
from = Issue.find(1)
to = Issue.find(2)