https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42359 --- Comment #2 from Andrew Fuerste-Henry <andrew@bywatersolutions.com> --- Created attachment 197364 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=197364&action=edit Fix Guided.t update_sql test input The test was feeding update_sql object field names instead of the parameter names it actually expects. This let savedsql get cleared during the update. The test was updated to pass the expected inputs so it still checks cache_expiry without creating the warning in the delete log path. Test plan: 1. Confirm the issue by enabling the ReportsLog preference in Koha UI andin ktd shell running prove t/db_dependent/Reports/Guided.t 2. Apply the patch. 3. Run the tests again with prove t/db_dependent/Reports/Guided.t 4. The tests should pass. 5. Disable the preference and run again. 6. Tests should pass. 7. Sign off if steps above check out. Signed-off-by: Andrew Fuerste Henry <andrew@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.