Attachment 28132 Details for Bug 12214 – [PASSED QA] Bug 12214: add regression test for reporting error when running report with SQL error

[patch] [PASSED QA] Bug 12214: add regression test for reporting error when running report with SQL error

PASSED-QA-Bug-12214-add-regression-test-for-report.patch (text/plain), 1.47 KB, created by Kyle M Hall (khall) on 2014-05-09 13:36:21 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-05-09 13:36:21 UTC

Size: 1.47 KB

patch

obsolete

>From 82ce5946cd6324dba7dd69a37ea6158e6adf3f0e Mon Sep 17 00:00:00 2001
>From: Galen Charlton <gmc@esilibrary.com>
>Date: Thu, 8 May 2014 16:09:07 +0000
>Subject: [PATCH] [PASSED QA] Bug 12214: add regression test for reporting error when running report with SQL error
>
>To test:
>
>[1] Run prove -v t/db_dependent/Reports_Guided.t. Last test should
>    fail.
>[2] Apply the main patch.
>[3] Run step 1 again.  This time, all tests should pass.
>
>Signed-off-by: Galen Charlton <gmc@esilibrary.com>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> t/db_dependent/Reports_Guided.t |   15 ++++++++++++++-
> 1 files changed, 14 insertions(+), 1 deletions(-)
>
>diff --git a/t/db_dependent/Reports_Guided.t b/t/db_dependent/Reports_Guided.t
>index ad34dbd..6633c2f 100755
>--- a/t/db_dependent/Reports_Guided.t
>+++ b/t/db_dependent/Reports_Guided.t
>@@ -5,7 +5,7 @@
> 
> use Modern::Perl;
> 
>-use Test::More tests => 14;
>+use Test::More tests => 15;
> 
> use C4::Context;
> 
>@@ -85,6 +85,19 @@ is_deeply(
>     'running a query with a parameter returned the expected result'
> );
> 
>+# for next test, we want to let execute_query capture any SQL errors
>+$dbh->{RaiseError} = 0;
>+my $errors;
>+($sth, $errors) = execute_query(
>+    'SELECT surname FRM borrowers',  # error in the query is intentional
>+    0,
>+    10,
>+);
>+ok(
>+    defined($errors) && exists($errors->{queryerr}),
>+    'attempting to run a report with an SQL syntax error returns error message (Bug 12214)'
>+);
>+
> #End transaction
> $dbh->rollback;
> 
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 12214: 28044 | 28115 | 28116 | 28117 | 28132 | 28133 | 28134 | 28135 | 28136