JumpCloud\Response\Factory\AuthorizationResponseFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	create()	0	8	2

<?php declare(strict_types=1);

namespace JumpCloud\Response\Factory;

use Psr\Http\Message\ResponseInterface as HttpResponseInterface;
use JumpCloud\Response\ResponseInterface;
use JumpCloud\Response\AuthorizationResponse;

/**
 * Class AuthorizationResponseFactory
 * @package JumpCloud\Factory
 */
class AuthorizationResponseFactory implements ResponseFactoryInterface
{
    /**
     * @param HttpResponseInterface $response
     *
     * @return ResponseInterface
     */
    public function create(HttpResponseInterface $response): ResponseInterface
    {
        if ($response->getStatusCode() === 200) {
            return new AuthorizationResponse($response, AuthorizationResponse::AUTHORISED);

$response of type object<Psr\Http\Message\ResponseInterface> is not a sub-type of object<GuzzleHttp\Psr7\Response>. It seems like you assume a concrete implementation of the interface Psr\Http\Message\ResponseInterface to be always present.

        }

        return new AuthorizationResponse($response, AuthorizationResponse::UNAUTHORISED);

$response of type object<Psr\Http\Message\ResponseInterface> is not a sub-type of object<GuzzleHttp\Psr7\Response>. It seems like you assume a concrete implementation of the interface Psr\Http\Message\ResponseInterface to be always present.

    }
}