From 60c88bfa2f6c23c58ebd953eca2d0c29f6f2dee3 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 27 Mar 2020 17:09:27 +0100 Subject: [PATCH] Bug 24614: Fix Reports/Guided.t DBD::mysql::st execute failed: Column 'public' cannot be null [for Statement "UPDATE `saved_sql` SET `last_modified` = ?, `notes` = ?, `public` = ?, `report_group` = ?, `report_name` = ?, `report_subgroup` = ?, `savedsql` = ? WHERE ( `id` = ? )" with ParamValues: 0='2020-03-27T16:03:04', 1=undef, 2=undef, 3=undef, 4='Just another report', 5=undef, 6=undef, 7=25] at /usr/share/perl5/DBIx/Class/Storage/DBI.pm line 1836. # Looks like you planned 17 tests but ran 6. update_sql expect all the paramters to be set, otherwise it will be nulled. The best way to fix it is at test level. There is only 1 occurrence in controller/module, and it sends all the parameters. That is the correct way to do and will make things easier when we will removed them to use Koha::Reports directly Signed-off-by: Martin Renvoize Signed-off-by: Joy Nelson --- t/db_dependent/Reports/Guided.t | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/t/db_dependent/Reports/Guided.t b/t/db_dependent/Reports/Guided.t index bb832af84d..c298335d0e 100644 --- a/t/db_dependent/Reports/Guided.t +++ b/t/db_dependent/Reports/Guided.t @@ -217,7 +217,7 @@ subtest 'get_saved_reports' => sub { ok( 0 < scalar @{ get_saved_reports( $report_ids[0] ) }, "filter takes report id" ); my $r1 = Koha::Reports->find($report_ids[0]); - $r1 = update_sql($r1->id, { borrowernumber => $r1->borrowernumber, name => 'Just another report' }); + $r1 = update_sql($r1->id, { %{$r1->unblessed}, borrowernumber => $r1->borrowernumber, name => 'Just another report' }); is( $r1->cache_expiry, 300, 'cache_expiry has the correct default value, from DBMS' ); #Test delete_report -- 2.39.5