SteamMarketProviders\ParserManager\Http\Http

Complexity

Total Complexity

7

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Test Coverage

Coverage

17.65%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setStrategy()	0	5	1
A	__construct()	0	4	2
A	sendRequest()	0	20	4

<?php

declare(strict_types=1);

namespace SteamMarketProviders\ParserManager\Http;

use stdClass;
use SteamMarketProviders\ParserManager\Contract\StrategyInterface;
use SteamMarketProviders\ParserManager\Exception\HttpException;
use SteamMarketProviders\ParserManager\Http\Strategy\GuzzleStrategy;
use Throwable;

final class Http
{
    /**
     * @param StrategyInterface|null $strategy
     */
    public function __construct(private null|StrategyInterface $strategy = null)
    {
        if (!$this->strategy) {
            $this->strategy = new GuzzleStrategy();
        }
    }

    /**
     * @param StrategyInterface $strategy
     * @return $this
     */
    public function setStrategy(StrategyInterface $strategy): Http
    {
        $this->strategy = $strategy;

        return $this;
    }

    /**
     * @param string $url
     * @return stdClass
     * @throws HttpException
     */
    public function sendRequest(string $url): stdClass
    {
        try {
            $response = $this->strategy->sendRequest($url);

The method sendRequest() does not exist on null.

            $code = $response->getStatus();
            if ($code !== 200) {
                throw new HttpException("");
            }

            $response = json_decode($response->getBody());

It seems like $response->getBody() can also be of type callable and resource; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

            if (!$response->success) {
                throw new HttpException("");
            }
        } catch (Throwable $throwable) {
            throw new HttpException($throwable->getMessage());
        }

        return $response;
    }
}