ApiClients\Foundation\Transport\Factory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	7

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	20	3
A	createFromReactHttpClient()	0	22	1

<?php
declare(strict_types=1);

namespace ApiClients\Foundation\Transport;

use GuzzleHttp\Client as GuzzleClient;
use GuzzleHttp\HandlerStack;
use React\Dns\Resolver\Resolver;
use React\EventLoop\Factory as LoopFactory;
use React\EventLoop\LoopInterface;
use React\HttpClient\Client as HttpClient;
use React\HttpClient\Factory as HttpClientFactory;
use React\Dns\Resolver\Factory as ResolverFactory;
use WyriHaximus\React\GuzzlePsr7\HttpClientAdapter;

class Factory
{
    /**
     * @param LoopInterface|null $loop
     * @param array $options
     * @return Client
     */
    public static function create(LoopInterface $loop = null, array $options = []): Client
    {
        if (!($loop instanceof LoopInterface)) {
            $loop = LoopFactory::create();
        }

        if (!isset($options['dns'])) {
            $options['dns'] = '8.8.8.8';
        }

        $resolver = (new ResolverFactory())->createCached($options['dns'], $loop);
        $httpClient = (new HttpClientFactory())->create($loop, $resolver);

        return self::createFromReactHttpClient(
            $httpClient,
            $resolver,
            $loop,
            $options
        );
    }

    /**
     * @param HttpClient $httpClient
     * @param Resolver $resolver
     * @param LoopInterface|null $loop
     * @param array $options
     * @return Client
     */
    public static function createFromReactHttpClient(
        HttpClient $httpClient,
        Resolver $resolver,
        LoopInterface $loop = null,
        array $options = []
    ): Client {
        return new Client(
            $loop,

It seems like $loop defined by parameter $loop on line 54 can be null; however, ApiClients\Foundation\Transport\Client::__construct() does not accept null, maybe add an additional type check?

            new GuzzleClient(
                [
                    'handler' => HandlerStack::create(
                        new HttpClientAdapter(
                            $loop,

It seems like $loop defined by parameter $loop on line 54 can be null; however, WyriHaximus\React\GuzzlePsr7\HttpClientAdapter::__construct() does not accept null, maybe add an additional type check?

                            $httpClient,
                            $resolver
                        )
                    ),
                ]
            ),
            $options
        );
    }
}