Bug 40541

Summary: Add new line at the end of the files when missing
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: Pushed to main --- QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: m.de.rooy
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39324
GIT URL: Change sponsored?: ---
Patch complexity: Small patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
25.11.00
Circulation function:
Attachments: Bug 40541: Add missing new line at the end of files when missing
Bug 40541: Add missing new line at the end of files when missing
Bug 40541: Add missing new line at the end of files when missing
Bug 40541: Add missing new line at the end of files when missing

Description Jonathan Druart 2025-07-29 09:50:26 UTC
We have insert_final_newline = true to ensure files will contain a new line at the end.
However we have seen commits with "No new line" added in the git log.

We should have a xt test to prevent more occurrences.

We should also have a test at the QA script level: https://gitlab.com/koha-community/qa-test-tools/-/issues/96
Comment 1 Jonathan Druart 2025-07-29 09:55:25 UTC
Created attachment 184789 [details] [review]
Bug 40541: Add missing new line at the end of files when missing

We have insert_final_newline = true to ensure files will contain a new line at the end.
However we have seen commits with "No new line" added in the git log.

We should have a xt test to prevent more occurrences.

We should also have a test at the QA script level: https://gitlab.com/koha-community/qa-test-tools/-/issues/96

Test plan:
  prove xt/find-missing-new-lines-at-the-end-of-file.t
should return green.

Remove the new line at the end of a file, eg.
vim -b mainpage.pl
:set noeol
:wq

Now
  prove xt/find-missing-new-lines-at-the-end-of-file.t
is failing on mainpage.pl.
Comment 2 Jonathan Druart 2025-07-29 10:20:29 UTC
Created attachment 184791 [details] [review]
Bug 40541: Add missing new line at the end of files when missing

We have insert_final_newline = true to ensure files will contain a new line at the end.
However we have seen commits with "No new line" added in the git log.

We should have a xt test to prevent more occurrences.

We should also have a test at the QA script level: https://gitlab.com/koha-community/qa-test-tools/-/issues/96

Test plan:
  prove xt/find-missing-new-lines-at-the-end-of-file.t
should return green.

Remove the new line at the end of a file, eg.
vim -b mainpage.pl
:set noeol
:wq

Now
  prove xt/find-missing-new-lines-at-the-end-of-file.t
is failing on mainpage.pl.
Comment 3 Owen Leonard 2025-07-29 11:32:19 UTC
Created attachment 184794 [details] [review]
Bug 40541: Add missing new line at the end of files when missing

We have insert_final_newline = true to ensure files will contain a new
line at the end. However we have seen commits with "No new line" added
in the git log.

We should have a xt test to prevent more occurrences.

We should also have a test at the QA script level:
https://gitlab.com/koha-community/qa-test-tools/-/issues/96

Test plan:
  prove xt/find-missing-new-lines-at-the-end-of-file.t
should return green.

Remove the new line at the end of a file, eg.
vim -b mainpage.pl
:set noeol
:wq

Now
  prove xt/find-missing-new-lines-at-the-end-of-file.t
is failing on mainpage.pl.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 4 Marcel de Rooy 2025-07-29 13:48:27 UTC
Created attachment 184807 [details] [review]
Bug 40541: Add missing new line at the end of files when missing

We have insert_final_newline = true to ensure files will contain a new
line at the end. However we have seen commits with "No new line" added
in the git log.

We should have a xt test to prevent more occurrences.

We should also have a test at the QA script level:
https://gitlab.com/koha-community/qa-test-tools/-/issues/96

Test plan:
  prove xt/find-missing-new-lines-at-the-end-of-file.t
should return green.

Remove the new line at the end of a file, eg.
vim -b mainpage.pl
:set noeol
:wq

Now
  prove xt/find-missing-new-lines-at-the-end-of-file.t
is failing on mainpage.pl.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Lucas Gass (lukeg) 2025-07-31 18:47:37 UTC
Nice work everyone!

Pushed to main for 25.11