From ff7938cff203b7ebc3a85e716b218e33833423fc 2015-12-20 14:07:12
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: 2015-12-20 14:07:12
Subject: [PATCH] IssuePriority.position_name not recalculated every time it should (#21504).

Patch by Felix Schäfer.

git-svn-id: http://svn.redmine.org/redmine/trunk@14989 e93f8b46-1217-0410-a6f0-8f06a7374b81

---

diff --git a/test/unit/issue_priority_test.rb b/test/unit/issue_priority_test.rb
index e9d44b6..afce368 100644
--- a/test/unit/issue_priority_test.rb
+++ b/test/unit/issue_priority_test.rb
@@ -99,6 +99,20 @@ class IssuePriorityTest < ActiveSupport::TestCase
     assert_equal %w(lowest default high4 high3 high2 highest), IssuePriority.active.to_a.sort.map(&:position_name)
   end
 
+  def test_moving_a_priority_should_update_position_names
+    prio = IssuePriority.first
+    prio.move_to = 'lowest'
+    prio.reload
+    assert_equal 'highest', prio.position_name
+  end
+
+  def test_deactivating_a_priority_should_update_position_names
+    prio = IssuePriority.active.order(:position).last
+    prio.active = false
+    prio.save
+    assert_equal 'highest', IssuePriority.active.order(:position).last.position_name
+  end
+
   def test_destroying_a_priority_should_update_position_names
     IssuePriority.find_by_position_name('highest').destroy
     assert_equal %w(lowest default high2 highest), IssuePriority.active.to_a.sort.map(&:position_name)