ApiClients\Client\Travis\CommandBus\Handler\JobLogHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	9

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	handle()	0	33	2

<?php declare(strict_types=1);

namespace ApiClients\Client\Travis\CommandBus\Handler;

use ApiClients\Client\Pusher\AsyncClient as PusherAsyncClient;
use ApiClients\Client\Pusher\Event;
use ApiClients\Client\Pusher\Service\SharedAppClientService;
use ApiClients\Client\Travis\ApiSettings;
use ApiClients\Client\Travis\CommandBus\Command\JobLogCommand;
use ApiClients\Client\Travis\Resource\LogLineInterface;
use ApiClients\Foundation\Hydrator\Hydrator;
use React\Promise\PromiseInterface;
use Rx\Observable;
use Rx\ObserverInterface;
use function React\Promise\resolve;

final class JobLogHandler
{
    /**
     * @var SharedAppClientService
     */
    private $pusher;

    /**
     * @var Hydrator
     */
    private $hydrator;

    /**
     * JobLogHandler constructor.
     * @param SharedAppClientService $pusher
     * @param Hydrator               $hydrator
     */
    public function __construct(SharedAppClientService $pusher, Hydrator $hydrator)
    {
        $this->pusher = $pusher;
        $this->hydrator = $hydrator;
    }

    /**
     * Fetch the given repository and hydrate it.
     *
     * @param  JobLogCommand    $command
     * @return PromiseInterface
     */
    public function handle(JobLogCommand $command): PromiseInterface
    {
        return $this->pusher->share(
            ApiSettings::PUSHER_KEY
        )->then(function (PusherAsyncClient $pusher) use ($command) {
            return resolve(Observable::create(function (
                ObserverInterface $observer
            ) use (
                $pusher,
                $command
            ) {
                $subscription = $pusher->channel('job-' . (string)$command->getId())->filter(function (Event $event) {

$subscription is not used, you could remove the assignment.

                    return $event->getEvent() === 'job:log';
                })->map(function (Event $event) {
                    return $this->hydrator->hydrate(LogLineInterface::HYDRATE_CLASS, $event->getData());
                })->subscribe(
                    function (LogLineInterface $line) use ($observer, &$subscription) {
                        $observer->onNext($line);

                        if ($line->final()) {
                            $subscription->dispose();
                        }
                    },
                    function ($error) use ($observer) {
                        $observer->onError($error);
                    },
                    function () use ($observer) {
                        $observer->onComplete();

The method onComplete() does not exist on Rx\ObserverInterface. Did you maybe mean onCompleted()?

                    }
                );
            }));
        });
    }
}