From 0895ca8c31e5aedd31e44218b952738c9fffed21 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Fr=C3=A9d=C3=A9ric=20Demians?= Date: Wed, 30 Dec 2015 13:10:16 +0100 Subject: [PATCH] Bug 15429 UT for _parseletter modifying its parameter MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Before/after applying the related patch, do: prove -v t/db_dependent/Letters.t Signed-off-by: Jonathan Druart Signed-off-by: Kyle M Hall (cherry picked from commit fd07e3296b64a78edf1cf947a824147f1adbba1a) Signed-off-by: Julian Maurice (cherry picked from commit 2eac97ba65d869f34641f6eb7fd22047a104a794) Signed-off-by: Frédéric Demians --- t/db_dependent/Letters.t | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Letters.t b/t/db_dependent/Letters.t index b549a7bf72..59d52ad66c 100644 --- a/t/db_dependent/Letters.t +++ b/t/db_dependent/Letters.t @@ -18,7 +18,7 @@ # along with Koha; if not, see . use Modern::Perl; -use Test::More tests => 61; +use Test::More tests => 62; use Test::MockModule; use Test::Warn; @@ -292,6 +292,13 @@ is( $prepared_letter->{content}, $my_content_letter, 'GetPreparedLetter returns $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('claimacquisition','TESTACQCLAIM','Acquisition Claim','Item Not Received','<>|<>|Ordernumber <> (<>) (<> ordered)');}); +# Test that _parseletter doesn't modify its parameters bug 15429 +{ + my $values = { dateexpiry => '2015-12-13', }; + C4::Letters::_parseletter($prepared_letter, 'borrowers', $values); + is( $values->{dateexpiry}, '2015-12-13', "_parseletter doesn't modify its parameters" ); +} + my $booksellerid = C4::Bookseller::AddBookseller( { name => "my vendor", -- 2.39.5