Bugzilla – Attachment 74728 Details for
Bug 19204
Fines in days restriction calculation is not taking calendar into account
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 19204: Add a test: holiday is expiration date
Bug-19204-Add-a-test-holiday-is-expiration-date.patch (text/plain), 1.90 KB, created by
Jonathan Druart
on 2018-04-23 13:29:21 UTC
(
hide
)
Description:
Bug 19204: Add a test: holiday is expiration date
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2018-04-23 13:29:21 UTC
Size:
1.90 KB
patch
obsolete
>From 80ae1d4b28bb9a248bb780548b6fca79392c0f9a Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 23 Apr 2018 18:52:00 -0300 >Subject: [PATCH] Bug 19204: Add a test: holiday is expiration date > >--- > t/db_dependent/Circulation.t | 22 +++++++++++++++++++++- > 1 file changed, 21 insertions(+), 1 deletion(-) > >diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t >index 294e622611..d8de2e220b 100755 >--- a/t/db_dependent/Circulation.t >+++ b/t/db_dependent/Circulation.t >@@ -1764,7 +1764,7 @@ subtest 'AddReturn + CumulativeRestrictionPeriods' => sub { > }; > > subtest 'AddReturn + suspension_chargeperiod' => sub { >- plan tests => 10; >+ plan tests => 12; > > my $library = $builder->build( { source => 'Branch' } ); > my $patron = $builder->build( { source => 'Borrower', value => { categorycode => $patron_category->{categorycode} } } ); >@@ -1895,6 +1895,26 @@ subtest 'AddReturn + suspension_chargeperiod' => sub { > expiration_date => $expected_expiration, > } > ); >+ >+ # Adding another holiday, day of expiration date >+ my $expected_expiration_dt = dt_from_string($expected_expiration); >+ $calendar->insert_single_holiday( >+ day => $expected_expiration_dt->day, >+ month => $expected_expiration_dt->month, >+ year => $expected_expiration_dt->year, >+ title => 'holidayTest_exp', >+ description => 'holidayDesc on expiration date' >+ ); >+ # Expiration date will be the day after >+ test_debarment_on_checkout( >+ { >+ item => $item_1, >+ library => $library, >+ patron => $patron, >+ due_date => $five_days_ago, >+ expiration_date => $expected_expiration_dt->clone->add( days => 1 ), >+ } >+ ); > }; > > subtest 'AddReturn | is_overdue' => sub { >-- >2.11.0
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 19204
:
74726
|
74727
|
74728
|
74730
|
74733
|
74734
|
74735
|
74736
|
74777
|
74778
|
74779
|
74780
|
74798
|
74799
|
74800
|
74801