https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10190 --- Comment #658 from Chloé Zermatten <chloe.zermatten@openfifth.co.uk> --- Created attachment 196868 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196868&action=edit Bug 10190: fix failing Permissions.t and overhaul CirculationTriggers Cypress spec Permissions.t: - Remove edit_notice_status_triggers from userpermissions.sql so fresh installs no longer seed this deprecated permission (existing installs are handled by the atomic update) - Add a defensive DELETE in the test to cover DBs that pre-date the atomic update CirculationTriggers_spec.ts: - Fix getMockCirculationRule: the helper was returning the old {rule_name, rule_value, branchcode, ...} format; update to the current API format {context: {...}, overdue_N_delay: ..., ...} - Remove assertions for UI elements that do not exist (success/error flash messages, "Existing triggers" heading, "rule set will be removed" text) - Fix reset confirmation text to match actual component output - Fix duplicate patron_category_id "ST" in mock patron categories - Remove aspirational tests that checked non-implemented behaviour - Add describe block for the no-default-triggers warning (no rules anywhere, rules only at specific libraries, library link switches the filter) - Add regression test for NaN trigger number when adding first rule for a library Signed-off-by: Michaela <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.