Bug 20204

Summary: FrameworkPlugin.t should not depend on CPL branch
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: Test SuiteAssignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 20204: Add TestBuilder to replace CPL branch

Description Marcel de Rooy 2018-02-15 08:17:36 UTC
Triggered by:
<Joubu> could you take a look at the noise of t/db_dependent/FrameworkPlugin.t?

I took a look.
And saw something else :)
If you dont have CPL, the test crashes.

Note that the noise is not from the test, but from poorly written plugins. Especially the unimarc plugins generate much uninitialized value and substr outside of string warnings. We should not have added Modern::Perl in one pass imo, but consider them one by one.. See bug 19996.
I have been working on those plugins, but stranded so far on lack of support.
Comment 1 Marcel de Rooy 2018-02-15 08:25:36 UTC
Created attachment 71640 [details] [review]
Bug 20204: Add TestBuilder to replace CPL branch

Remove CPL; without this branch, the test crashes.
Rearrange modules at the start.
Add Koha::Database calls.
Move a few global my vars to our level.
Remove intermediate rollbacks.

Test plan:
[1] Run prove t/db_dependent/FrameworkPlugin.t
    Ignore noise from unimarc plugins :)

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 2 Marcel de Rooy 2018-02-15 08:27:33 UTC
Signing off myself: trivial edits.
Comment 3 Jonathan Druart 2018-02-15 14:25:23 UTC
Patch pushed to master for 18.05

Thanks Marcel