Bugzilla – Attachment 101989 Details for
Bug 22001
Add RaiseError and PrintError flags for all tests
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 22002: (RM follow-up) Allow RaiseError tests in Reports/Guided.t
Bug-22002-RM-follow-up-Allow-RaiseError-tests-in-R.patch (text/plain), 2.69 KB, created by
Martin Renvoize (ashimema)
on 2020-03-27 11:14:21 UTC
(
hide
)
Description:
Bug 22002: (RM follow-up) Allow RaiseError tests in Reports/Guided.t
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2020-03-27 11:14:21 UTC
Size:
2.69 KB
patch
obsolete
>From d40ad5f69a1caa6150e1d77b245fa9143a8dbd06 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Fri, 27 Mar 2020 11:12:41 +0000 >Subject: [PATCH] Bug 22002: (RM follow-up) Allow RaiseError tests in > Reports/Guided.t > >https://bugs.koha-community.org/show_bug.cgi?id=22001 >--- > t/db_dependent/Reports/Guided.t | 8 ++++++++ > 1 file changed, 8 insertions(+) > >diff --git a/t/db_dependent/Reports/Guided.t b/t/db_dependent/Reports/Guided.t >index f9c2b738f7..9251863b95 100644 >--- a/t/db_dependent/Reports/Guided.t >+++ b/t/db_dependent/Reports/Guided.t >@@ -367,6 +367,7 @@ count(h.reservedate) AS 'holds' > subtest 'Email report test' => sub { > > plan tests => 12; >+ my $dbh = C4::Context->dbh; > > my $id1 = $builder->build({ source => 'Borrower',value => { surname => 'mailer', email => 'a@b.com', emailpro => 'b@c.com' } })->{ borrowernumber }; > my $id2 = $builder->build({ source => 'Borrower',value => { surname => 'nomailer', email => undef, emailpro => 'd@e.com' } })->{ borrowernumber }; >@@ -399,10 +400,13 @@ subtest 'Email report test' => sub { > ($emails, $errors ) = C4::Reports::Guided::EmailReport({report_id => $report1, module => $letter1->{module}, code => $letter2->{code}}); > is( $errors->[0]{FATAL}, 'NO_LETTER', "Must have a letter that exists"); > >+ # for next test, we want to let execute_query capture any SQL errors >+ $dbh->{RaiseError} = 0; > warning_like { ($emails, $errors ) = C4::Reports::Guided::EmailReport({report_id => $report2, module => $letter1->{module} , code => $letter1->{code} }) } > qr/^DBD::mysql::st execute failed/, > 'Error from bad report'; > is( $errors->[0]{FATAL}, 'REPORT_FAIL', "Bad report returns failure"); >+ $dbh->{RaiseError} = 1; > > ($emails, $errors ) = C4::Reports::Guided::EmailReport({report_id => $report1, module => $letter1->{module} , code => $letter1->{code} }); > is( $errors->[0]{NO_FROM_COL} == 1 && $errors->[1]{NO_EMAIL_COL} == 2 && $errors->[2]{NO_FROM_COL} == 2, 1, "Correct warnings from the routine"); >@@ -429,6 +433,7 @@ subtest 'nb_rows() tests' => sub { > > plan tests => 3; > >+ my $dbh = C4::Context->dbh; > $schema->storage->txn_begin; > > my $items_count = Koha::Items->search->count; >@@ -448,12 +453,15 @@ subtest 'nb_rows() tests' => sub { > SELECT * items xxx > }; > >+ # for next test, we want to let execute_query capture any SQL errors >+ $dbh->{RaiseError} = 0; > warning_like > { $nb_rows = nb_rows( $bad_query ) } > qr/^DBD::mysql::st execute failed:/, > 'Bad queries raise a warning'; > > is( $nb_rows, 0, 'nb_rows returns 0 on bad queries' ); >+ $dbh->{RaiseError} = 0; > > $schema->storage->txn_rollback; > }; >-- >2.20.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 22001
:
83230
|
83231
|
83232
|
98910
|
98911
|
98912
|
99589
|
99590
|
99591
|
101989
|
101990
|
101991
|
102070
|
102140
|
102147
|
102149
|
102150
|
102151
|
102152