Artack\Dsn\Field\DateFieldTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testWithDateTime()	0	12	1
A	testWithImmutableDateTime()	0	12	1

<?php

declare(strict_types=1);

namespace Artack\Dsn\Field;

use PHPUnit\Framework\TestCase;

class DateFieldTest extends TestCase
{
    public function testWithDateTime(): void
    {
        $timeZone = new \DateTimeZone('Europe/Zurich');

        $dateTimeFirst = (new \DateTime('2019-02-25 09:15:30'))->setTimezone($timeZone);
        $dateTimeSecond = (clone $dateTimeFirst)->modify('+2 hour');

        $dateField = new DateField('name', $dateTimeFirst);
        $this->assertEquals($dateTimeFirst, $dateField->getDateTime());

        $dateField->setDateTime($dateTimeSecond);
        $this->assertEquals($dateTimeSecond, $dateField->getDateTime());
    }

    public function testWithImmutableDateTime(): void
    {
        $timeZone = new \DateTimeZone('Europe/Zurich');

        $dateTimeFirst = (new \DateTimeImmutable('2019-02-25 09:15:30'))->setTimezone($timeZone);
        $dateTimeSecond = $dateTimeFirst->modify('+2 hour');

        $dateField = new DateField('name', $dateTimeFirst);
        $this->assertEquals($dateTimeFirst, $dateField->getDateTime());

        $dateField->setDateTime($dateTimeSecond);

It seems like $dateTimeSecond can also be of type false; however, parameter $dateTime of Artack\Dsn\Field\DateField::setDateTime() does only seem to accept DateTimeInterface, maybe add an additional type check?

        $this->assertEquals($dateTimeSecond, $dateField->getDateTime());
    }
}