https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19279 --- Comment #16 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Victor Grousset/tuxayo from comment #15)
Hi, I think it's a good idea to *always* instigate to add tests when fixing existing code. In this case the changes seems too complex to make this testable without messing with too much stuff. So in cases (like this) where the fix gets stuck for a long time due to requiring tests, if we are confident about the fix, can we allow it as it is?
I think at the moment we can't be sure if this works correctly. I am also a bit doubtful about the test plan - it doesn't even mention the EasyAnalytics feature this seems to be used for? I'd at least like a real good test plan here. -- You are receiving this mail because: You are watching all bug changes.