IssuesControllerTest#test_index_sort_by_total_estimated_hours tests nothing (#30474).

Patch by Go MAEDA.


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

test/fixtures/issues.yml

@@ -15,6 +15,7 @@ issues_001:
   status_id: 1
   start_date: <%= 1.day.ago.to_date.to_s(:db) %>
   due_date: <%= 10.day.from_now.to_date.to_s(:db) %>
+  estimated_hours: 200.0
   root_id: 1
   lft: 1
   rgt: 2
@@ -35,6 +36,7 @@ issues_002:
   status_id: 2
   start_date: <%= 2.day.ago.to_date.to_s(:db) %>
   due_date:
+  estimated_hours: 0.5
   root_id: 2
   lft: 1
   rgt: 2
@@ -56,6 +58,7 @@ issues_003:
   status_id: 1
   start_date: <%= 15.day.ago.to_date.to_s(:db) %>
   due_date: <%= 5.day.ago.to_date.to_s(:db) %>
+  estimated_hours: 1.0
   root_id: 3
   lft: 1
   rgt: 2
@@ -90,6 +93,7 @@ issues_005:
   assigned_to_id:
   author_id: 2
   status_id: 1
+  estimated_hours: 2.0
   root_id: 5
   lft: 1
   rgt: 2

test/functional/issues_controller_test.rb

@@ -1067,6 +1067,8 @@ class IssuesControllerTest < Redmine::ControllerTest
       }
     assert_response :success
     hours = issues_in_list.map(&:total_estimated_hours)
+    # Removes nil because the position of NULL is database dependent
+    hours.compact!
     assert_equal hours.sort.reverse, hours
   end