]> git.koha-community.org Git - koha.git/commit
Bug 31473: Fix fragile test about bad OpacHiddenItems conf
authorTomas Cohen Arazi <tomascohen@theke.io>
Fri, 26 Aug 2022 19:45:10 +0000 (16:45 -0300)
committerVictor Grousset/tuxayo <victor@tuxayo.net>
Thu, 8 Sep 2022 03:31:38 +0000 (05:31 +0200)
commitf3b83357baed2a19f329927f4eb4ce35c8d87a1a
tree445f0b2896726708e3fa9b3617968f645fe5a78e
parent4086c1c4ff57c46881e4e44e06672eae7091373c
Bug 31473: Fix fragile test about bad OpacHiddenItems conf

There's been a behavior change in recent MariaDB that made some cases
deal with truncated data in DOUBLE context instead of DECIMAL.

Probably this:
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-27380

This made the affected test fail consistently on those versions.

This trick for checking OpacHiddenItems is correct (as introduced on bug
28125 doesn't really make use of the warning, just prints it in
about.pl) so one option was to just check for a warning.

I decided to keep the test, but add the optional DECIMAL|DOUBLE check on
the qr. This way other eventual changes will make it fail and serve as a
warning in case something more relevant changes.

To test:
1. Apply this patch
2. Run:
   $ kshell
  k$ t/db_dependent/Koha/Items.t
=> SUCCESS: Tests pass!
3. Sign off :-D

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
t/db_dependent/Koha/Items.t