Project

General

Profile

Actions

Patch #40801

closed

Add missing fixture journal_details to JournalsHelperTest

Added by Mizuki ISHIKAWA 7 months ago. Updated 7 months ago.

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

0%

Estimated time:

Description

A test failure occurred as follows:

Failure:
JournalsHelperTest#test_journal_thumbnail_attachments_should_be_in_the_same_order_as_the_journal_details [/usr/src/redmine/test/helpers/journals_helper_test.rb:87]:
Expected: 2
  Actual: 3

rails test test/helpers/journals_helper_test.rb:65

I cannot reproduce it because I forgot to seed, but I think the issue is caused by the absence of journal_details in the fixtures, despite it being used in the test at https://github.com/redmine/redmine/blob/master/test/helpers/journals_helper_test.rb#L87.
The following patch should fix it:
diff --git a/test/helpers/journals_helper_test.rb b/test/helpers/journals_helper_test.rb
index a33313e78..55fb1260a 100644
--- a/test/helpers/journals_helper_test.rb
+++ b/test/helpers/journals_helper_test.rb
@@ -29,7 +29,8 @@ class JournalsHelperTest < Redmine::HelperTest
            :enabled_modules,
            :custom_fields,
            :attachments,
-           :versions
+           :versions,
+           :journal_details

   def test_journal_thumbnail_attachments_should_return_thumbnailable_attachments
     skip unless convert_installed?

Actions #1

Updated by Go MAEDA 7 months ago

  • Target version set to 6.0.0
Actions #2

Updated by Go MAEDA 7 months ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed the patch in r22874. Thank you.

Actions

Also available in: Atom PDF