PSR7CsrfTest\Exception\SessionAttributeNotFoundExceptionTest

Complexity

Total Complexity

1

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	testFromInvalidExpirationTime()	0	17	1

<?php

declare(strict_types=1);

namespace PSR7CsrfTest\Exception;

use PHPUnit\Framework\TestCase;
use Psr\Http\Message\ServerRequestInterface;
use PSR7Csrf\Exception\ExceptionInterface;
use PSR7Csrf\Exception\SessionAttributeNotFoundException;
use UnexpectedValueException;

/**
 * @covers \PSR7Csrf\Exception\SessionAttributeNotFoundException
 */
final class SessionAttributeNotFoundExceptionTest extends TestCase
{
    public function testFromInvalidExpirationTime()
    {
        /* @var $request ServerRequestInterface|\PHPUnit_Framework_MockObject_MockObject */
        $request = $this->createMock(ServerRequestInterface::class);

        $request->expects(self::any())->method('getAttributes')->willReturn(['foo' => 'bar', 'baz' => 'tab']);

        $exception = SessionAttributeNotFoundException::fromAttributeNameAndRequest('foo', $request);

It seems like $request defined by $this->createMock(\Psr\Http\Message\ServerRequestInterface::class) on line 21 can also be of type object<PHPUnit_Framework_MockObject_MockObject>; however, PSR7Csrf\Exception\SessionAttributeNotFoundException::fromAttributeNameAndRequest() does only seem to accept object<Psr\Http\Message\ServerRequestInterface>, maybe add an additional type check?

        self::assertInstanceOf(SessionAttributeNotFoundException::class, $exception);
        self::assertInstanceOf(UnexpectedValueException::class, $exception);
        self::assertInstanceOf(ExceptionInterface::class, $exception);
        self::assertSame(
            'Provided request contains no matching session attribute "foo", attributes ["foo","baz"] exist',
            $exception->getMessage()
        );
    }
}