Bug 24326

Summary: Remove advance_notices_digest.t
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: Test SuiteAssignee: Chris Cormack <chris>
Status: In Discussion --- QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: andreas.jonsson, dcook, jonathan.druart, m.de.rooy, martin.renvoize, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33900
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 20478    
Bug Blocks:    
Attachments: Bug 24326: The garbage man is your friend

Description Katrin Fischer 2020-01-02 07:10:44 UTC
(In reply to Jonathan Druart from comment #26)
> There is something terribly wrong with this test file, the cronjob script is
> run outside the transaction and so changes are effective.
Comment 1 Marcel de Rooy 2021-10-06 13:21:30 UTC
Created attachment 125800 [details] [review]
Bug 24326: The garbage man is your friend
Comment 2 Marcel de Rooy 2021-10-06 13:23:04 UTC
Talking about bug titles, dcook ?
Comment 3 Jonathan Druart 2021-10-06 14:40:15 UTC
I do agree those tests should not have been pushed. 3 years later we are still missing the "clean ups could be handled separately" part.

However the tests are there and can be useful, Jenkins could back us is something is going wrong. But for devs it's really bad.

We could use the RUN_TESTS_AND_EXIT flag set by Jenkins: The tests would warn and exit early (exit 0) if the env var is not set.

Something better to suggest?
Comment 4 Katrin Fischer 2023-12-31 11:57:21 UTC
What is the issue with the tests?
Comment 5 Jonathan Druart 2024-01-04 09:09:24 UTC
See comment 0:
> the cronjob script is run outside the transaction and so changes are effective.
Comment 6 Katrin Fischer 2024-01-06 10:02:48 UTC
I didn't find any occurrence of RUN_TESTS_AND_EXIT in the codebase, so not sure if we have done similar before, but if we can make sure these are only run on by the automated tests, it would be helpful.