https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39522 Jacob O'Mara <jacob.omara@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #189538|0 |1 is obsolete| | --- Comment #27 from Jacob O'Mara <jacob.omara@openfifth.co.uk> --- Created attachment 189546 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=189546&action=edit Bug 39522: Add unit tests for valuebuilder plugin hooks This patch adds comprehensive unit tests for the new valuebuilder plugin functionality. The tests verify that plugins can provide valuebuilders through the get_valuebuilder() method and that FrameworkPlugin can correctly load and integrate with plugin-based valuebuilders. The tests use a dedicated TestValuebuilder plugin to ensure realistic testing scenarios without relying on mocks, following the pattern established in other plugin hook tests. Test plan: 1. Run the new test file: prove t/db_dependent/Koha/Plugins/Valuebuilder_hooks.t 2. Verify all tests pass (9 tests total) 3. Run all plugin tests to ensure no regressions: prove t/db_dependent/Koha/Plugins/ 4. Verify all plugin tests still pass (113 tests across 19 files) 5. Test the specific functionality: - Verify plugins without get_valuebuilder() method are not found as valuebuilders - Verify plugins with get_valuebuilder() method are correctly discovered - Verify FrameworkPlugin can load plugin-based valuebuilders - Verify JavaScript generation works with plugin valuebuilders - Verify plugin enable/disable affects valuebuilder availability 6. Confirm FrameworkPlugin integration tests still pass: prove t/db_dependent/FrameworkPlugin.t Signed-off-by: Catherine Small <c.small@rcs.ac.uk> Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk> -- You are receiving this mail because: You are watching all bug changes.