Project

General

Profile

Actions

Patch #36241

closed

MenuManagerTest randomly fails

Added by Mizuki ISHIKAWA almost 3 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

This test rarely fails.
This problem occurs because Project.visible(user).ids does not contain an ORDER clause for sorting and does not guarantee the order.

Failure:
MenuManagerTest#test_cross_project_menu_should_hide_item_if_module_is_not_enabled_for_any_project [/usr/src/redmine/test/integration/lib/redmine/menu_manager_test.rb:105]:
Expected: [1, 3, 4, 6]
  Actual: [6, 1, 3, 4]

rails test test/integration/lib/redmine/menu_manager_test.rb:103

Solved by the following patch.

diff --git a/test/integration/lib/redmine/menu_manager_test.rb b/test/integration/lib/redmine/menu_manager_test.rb
index 9bac57ad49..0456bc50e7 100644
--- a/test/integration/lib/redmine/menu_manager_test.rb
+++ b/test/integration/lib/redmine/menu_manager_test.rb
@@ -102,7 +102,7 @@ class MenuManagerTest < Redmine::IntegrationTest

   def test_cross_project_menu_should_hide_item_if_module_is_not_enabled_for_any_project
     user = User.find_by_login('dlopper')
-    assert_equal [1, 3, 4, 6], Project.visible(user).ids
+    assert_equal [1, 3, 4, 6], Project.visible(user).ids.sort

     # gantt and news are not enabled for any visible project
     Project.find(1).enabled_module_names = %w(issue_tracking calendar)


Related issues

Related to Redmine - Defect #36718: Menu Manager Test randomly failsClosed

Actions
Actions

Also available in: Atom PDF