Attachment 53726 Details for Bug 16942 – Bug 16942 [QA Followup] - Add unit test

[patch] Bug 16942 [QA Followup] - Add unit test

Bug-16942-QA-Followup---Add-unit-test.patch (text/plain), 2.04 KB, created by Kyle M Hall (khall) on 2016-07-26 15:03:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2016-07-26 15:03:06 UTC

Size: 2.04 KB

patch

obsolete

>From e095da27612b6843648fb032f720480d1d3e7b16 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Tue, 26 Jul 2016 15:00:58 +0000
>Subject: [PATCH] Bug 16942 [QA Followup] - Add unit test
>
>---
> C4/Reserves.pm                           |  2 +-
> t/db_dependent/Letters/TemplateToolkit.t | 19 ++++++++++++++++++-
> 2 files changed, 19 insertions(+), 2 deletions(-)
>
>diff --git a/C4/Reserves.pm b/C4/Reserves.pm
>index 79fb11a..66d2747 100644
>--- a/C4/Reserves.pm
>+++ b/C4/Reserves.pm
>@@ -2013,7 +2013,7 @@ sub _koha_notify_reserve {
>             'biblio'         => $biblionumber,
>             'biblioitems'    => $biblionumber,
>             'reserves'       => $reserve,
>-            'items', $reserve->{'itemnumber'},
>+            'items'          => $reserve->{'itemnumber'},
>         },
>         substitute => { today => output_pref( { dt => dt_from_string, dateonly => 1 } ) },
>     );
>diff --git a/t/db_dependent/Letters/TemplateToolkit.t b/t/db_dependent/Letters/TemplateToolkit.t
>index c4ecc5f..4169556 100644
>--- a/t/db_dependent/Letters/TemplateToolkit.t
>+++ b/t/db_dependent/Letters/TemplateToolkit.t
>@@ -18,7 +18,7 @@
> # along with Koha; if not, see <http://www.gnu.org/licenses>.
> 
> use Modern::Perl;
>-use Test::More tests => 13;
>+use Test::More tests => 14;
> 
> use MARC::Record;
> 
>@@ -188,6 +188,23 @@ $prepared_letter = GetPreparedLetter(
> );
> is( $prepared_letter->{content}, $hold->id(), 'Hold object used correctly' );
> 
>+# Bug 16942
>+$prepared_letter = GetPreparedLetter(
>+    (
>+        module      => 'test',
>+        letter_code => 'TEST_HOLD',
>+        tables      => {
>+            'branches'    => $library,
>+            'borrowers'   => $patron,
>+            'biblio'      => $biblio->id,
>+            'biblioitems' => $biblioitem->id,
>+            'reserves'    => $hold->unblessed,
>+            'items'       => $hold->itemnumber,
>+        }
>+    )
>+);
>+is( $prepared_letter->{content}, $hold->id(), 'Hold object used correctly' );
>+
> $sth->execute( "TEST_SERIAL", "[% serial.id %]" );
> $prepared_letter = GetPreparedLetter(
>     (
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 16942: 53497 | 53499 | 53726 | 53737 | 53738 | 54569 | 54570