https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14806

--- Comment #38 from Lari Strand <[email protected]> ---
So I assume every single test needs to start with enabling the BorrowersLog
syspref.

"Also, has_digest and takes_days (added to GetMessagingPreferences and returned
by GetPatronMessagingPreferences) aren't mentioned in the commit message and
don't appear to be used or needed."

There is a test which verifies that by changing the values for (or
disabling/enabling) has_digest and/or takes_days are being recorded (test
"SetMessagingPreference detects metadata changes"). I don't quite understand
what you mean by them "not being used" or "not needed". They are needed and the
UI saves the changes to them and they are being used by the system. I just
didn't specifically mention that you need to check that logging tracks their
values being changed separately without making other modifications to a message
transport. I can add that to the test plan though.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to