HansOtt\Holiday\Calendar\Belgium\ChristmasDay

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	1
A	isOfficial()	0	4	1

<?php

namespace HansOtt\Holiday\Calendar\Belgium;

use DateTimeImmutable;
use HansOtt\Holiday\Calendar\Year;
use HansOtt\Holiday\HolidayAbstract;

final class ChristmasDay extends HolidayAbstract
{
    public function __construct(Year $year)
    {
        $time = sprintf('%s-12-25', $year);
        $startsAt = new DateTimeImmutable($time);

        parent::__construct(
            $startsAt,
            \HansOtt\Holiday\getEndOfDay($startsAt)

It seems like \HansOtt\Holiday\getEndOfDay($startsAt) targeting HansOtt\Holiday\getEndOfDay() can also be of type false; however, HansOtt\Holiday\HolidayAbstract::__construct() does only seem to accept object<DateTimeImmutable>, did you maybe forget to handle an error condition?

        );
    }

    public function isOfficial()
    {
        return true;
    }
}