Bug 25723: Remove tests for removed method
authorMartin Renvoize <martin.renvoize@ptfs-europe.com>
Wed, 17 Jun 2020 12:46:34 +0000 (13:46 +0100)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 20 Jul 2020 15:45:31 +0000 (17:45 +0200)
We removed the exception_holidays (and single_holidays) methods as part
of this patchset. As such, we should no longer test them ;)

Signed-off-by: Emma Perks <Emma.Perks2@uhb.nhs.uk>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

t/db_dependent/Holidays.t

index 5b3ecb0..ced8bd5 100755 (executable)
@@ -17,7 +17,7 @@
 
 use Modern::Perl;
 
-use Test::More tests => 17;
+use Test::More tests => 12;
 
 use DateTime;
 use DateTime::TimeZone;
@@ -37,7 +37,7 @@ my $schema = Koha::Database->new->schema;
 my $dbh = C4::Context->dbh;
 my $builder = t::lib::TestBuilder->new;
 
-subtest 'exception_holidays() tests' => sub {
+subtest 'is_holiday timezone tests' => sub {
 
     plan tests => 1;
 
@@ -160,47 +160,6 @@ C4::Calendar->new( branchcode => $branch_2 )->insert_single_holiday(
 is( Koha::Calendar->new( branchcode => $branch_2 )->is_holiday( $today ), 1, "Today is a holiday for $branch_2" );
 is( Koha::Calendar->new( branchcode => $branch_1 )->is_holiday( $today ), 0, "Today is not a holiday for $branch_1");
 
-# Few tests for exception holidays
-my ( $diff, $cal, $special );
-$dbh->do("DELETE FROM special_holidays");
-_add_exception( $today, $branch_1, 'Today' );
-$cal = Koha::Calendar->new( branchcode => $branch_1 );
-$special = $cal->exception_holidays;
-is( keys %{$special}, 1, 'One exception holiday added' );
-
-my $tomorrow= dt_from_string();
-$tomorrow->add_duration( DateTime::Duration->new(days => 1) );
-_add_exception( $tomorrow, $branch_1, 'Tomorrow' );
-$cal = Koha::Calendar->new( branchcode => $branch_1 );
-$special = $cal->exception_holidays;
-is( keys %{$special}, 2, 'Set of exception holidays contains two dates' );
-
-$diff = $today->delta_days( $today )->in_units('days');
-is( $diff, 0, 'Lowest exception holiday is today' );
-$diff = $tomorrow->delta_days( $tomorrow )->in_units('days');
-is( $diff, 0, 'Highest exception holiday is tomorrow' );
-
-C4::Calendar->new( branchcode => $branch_1 )->delete_holiday(
-    weekday => $tomorrow->day_of_week,
-    day     => $tomorrow->day,
-    month   => $tomorrow->month,
-    year    => $tomorrow->year,
-);
-$cal = Koha::Calendar->new( branchcode => $branch_1 );
-$special = $cal->exception_holidays;
-is( keys %{$special}, 1, 'Set of exception holidays back to one' );
-
-sub _add_exception {
-    my ( $dt, $branch, $descr ) = @_;
-    C4::Calendar->new( branchcode => $branch )->insert_exception_holiday(
-        day         => $dt->day,
-        month       => $dt->month,
-        year        => $dt->year,
-        title       => $descr,
-        description => $descr,
-    );
-}
-
 $schema->storage->txn_rollback;
 
 subtest 'copy_to_branch' => sub {