From f3b83357baed2a19f329927f4eb4ce35c8d87a1a Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Fri, 26 Aug 2022 16:45:10 -0300 Subject: [PATCH] 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 Signed-off-by: Victor Grousset/tuxayo --- t/db_dependent/Koha/Items.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/db_dependent/Koha/Items.t b/t/db_dependent/Koha/Items.t index 5a1586b877..d5759488ef 100755 --- a/t/db_dependent/Koha/Items.t +++ b/t/db_dependent/Koha/Items.t @@ -1699,7 +1699,7 @@ subtest 'filter_by_visible_in_opac() tests' => sub { $rules = { itemlost => ['AB'] }; my $c = Koha::Items->filter_by_visible_in_opac->count; my @warnings = C4::Context->dbh->selectrow_array('SHOW WARNINGS'); - is( $warnings[2], q{Truncated incorrect DOUBLE value: 'AB'}); + like( $warnings[2], qr/Truncated incorrect (DOUBLE|DECIMAL) value: 'AB'/); $schema->storage->txn_rollback; }; -- 2.39.5