Project

General

Profile

Actions
Copy link

Defect #44438

closed

DateCalculationTest is not run by `rails test`

Added by Kenta Kumojima 21 days ago. Updated 19 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Go MAEDA
Category:
Code cleanup/refactoring
Target version:
6.1.5
Resolution:
Fixed
Affected version:

Description

DateCalculationTest is not executed by `rails test` because its file name does not match the test file loading pattern.

This patch renames the file so that it is detected and executed by `rails test`.

Files

rename_date_calculation_test.patch (662 Bytes) rename_date_calculation_test.patch Kenta Kumojima, 2026-09-09 09:48
Actions
Copy link
 #1

Updated by Go MAEDA 21 days ago

  • Tracker changed from Patch to Defect
  • Status changed from New to Confirmed
  • Target version set to 6.1.5
Actions
Copy link
 #2

Updated by Go MAEDA 20 days ago

  • Subject changed from DateCalculationTest is not run by rails test to DateCalculationTest is not run by `rails test`
  • Status changed from Confirmed to Resolved
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the fix in r25064. Thank you for catching and reporting the issue.

After the fix, I ran bin/rails test and confirmed in log/test.log that DateCalculationTest was run:

$ grep DateCalculationTest log/test.log
I, [2026-09-10T09:51:19.591995 #32500]  INFO -- : Redmine::Utils::DateCalculationTest: test_add_working_days_without_non_working_week_days
I, [2026-09-10T09:51:19.676314 #32500]  INFO -- : Redmine::Utils::DateCalculationTest: test_working_days_with_non_working_week_days
I, [2026-09-10T09:51:19.711604 #32500]  INFO -- : Redmine::Utils::DateCalculationTest: test_working_days_without_non_working_week_days
I, [2026-09-10T09:51:19.796717 #32500]  INFO -- : Redmine::Utils::DateCalculationTest: test_add_working_days_with_non_working_week_days
Actions
Copy link
 #3

Updated by Go MAEDA 19 days ago

  • Status changed from Resolved to Closed

Merged the fix into the stable branches in r25066 and r25067.

Actions
Copy link

Also available in: Atom PDF