Bugzilla – Attachment 105253 Details for
Bug 24612
expirationdate blank if patron has more than one item from bib on hold
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24612: Fix existing ReserveSlip tests
Bug-24612-Fix-existing-ReserveSlip-tests.patch (text/plain), 2.75 KB, created by
Jonathan Druart
on 2020-05-22 09:20:41 UTC
(
hide
)
Description:
Bug 24612: Fix existing ReserveSlip tests
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2020-05-22 09:20:41 UTC
Size:
2.75 KB
patch
obsolete
>From 443c374a8fc091ee9b26dc0dce544d0cdccbcf2f Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Fri, 22 May 2020 11:20:00 +0200 >Subject: [PATCH] Bug 24612: Fix existing ReserveSlip tests > >--- > t/db_dependent/Reserves/ReserveSlip.t | 38 +++------------------------ > 1 file changed, 3 insertions(+), 35 deletions(-) > >diff --git a/t/db_dependent/Reserves/ReserveSlip.t b/t/db_dependent/Reserves/ReserveSlip.t >index 9b28179fcc..cd7a1936b4 100644 >--- a/t/db_dependent/Reserves/ReserveSlip.t >+++ b/t/db_dependent/Reserves/ReserveSlip.t >@@ -19,7 +19,7 @@ > > use Modern::Perl; > >-use Test::More tests => 5; >+use Test::More tests => 3; > use t::lib::TestBuilder; > > use C4::Reserves qw( ReserveSlip ); >@@ -118,48 +118,16 @@ my $letter = $builder->build( > is ( ReserveSlip(), undef, "No hold slip returned if invalid or undef borrowernumber and/or biblionumber" ); > is ( ReserveSlip({ > branchcode => $library->{branchcode}, >- borrowernumber => $patron->{borrowernumber}, >- biblionumber => $biblio->{biblionumber}, >+ reserve_id => $hold1->reserve_id, > })->{code}, > 'HOLD_SLIP', "Get a hold slip from library, patron and biblio" ); > > is (ReserveSlip({ > branchcode => $library->{branchcode}, >- borrowernumber => $patron->{borrowernumber}, >- biblionumber => $biblio->{biblionumber}, >+ reserve_id => $hold1->reserve_id, > })->{content}, > "Hold found for $patron->{firstname}: Please pick up $biblio->{title} with barcode $item1->{barcode} at $library->{branchcode}.", "Hold slip contains correctly parsed content"); > >-is_deeply( >- ReserveSlip({ >- branchcode => $library->{branchcode}, >- borrowernumber => $patron->{borrowernumber}, >- biblionumber => $biblio->{biblionumber}, >- }), >- ReserveSlip({ >- branchcode => $library->{branchcode}, >- borrowernumber => $patron->{borrowernumber}, >- biblionumber => $biblio->{biblionumber}, >- itemnumber => $item1->{itemnumber}, >- barcode => $item1->{barcode}, >- }), >- "No item as param generate hold slip from first item in reserves"); >- >-isnt ( >- ReserveSlip({ >- branchcode => $library->{branchcode}, >- borrowernumber => $patron->{borrowernumber}, >- biblionumber => $biblio->{biblionumber}, >- })->{content}, >- ReserveSlip({ >- branchcode => $library->{branchcode}, >- borrowernumber => $patron->{borrowernumber}, >- biblionumber => $biblio->{biblionumber}, >- itemnumber => $item2->{itemnumber}, >- barcode => $item2->{barcode}, >- })->{content}, >- "Item and/or barcode as params return correct pickup item in hold slip"); >- > $schema->storage->txn_rollback; > > 1; >-- >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 24612
:
98738
|
98739
|
102818
|
102819
|
102820
|
104803
|
104804
|
104805
|
105202
|
105203
|
105204
|
105253
|
105254
|
105775