Softius\JenkinsJobMonitor\PushCommand

Complexity

Total Complexity

9

Size/Duplication

Total Lines	88
Duplicated Lines	13.64 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

81.08%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	12	1
A	execute()	0	18	2
A	getOptionLog()	12	12	4
A	setClient()	0	4	1
A	getClient()	0	4	1

<?php

namespace Softius\JenkinsJobMonitor;

use GuzzleHttp\ClientInterface;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class PushCommand
 * @package Softius\JenkinsJobMonitor
 */
class PushCommand extends Command
{
    /**
     * @var GuzzleHttp\ClientInterface
     */
    private $client;

    /**
     * Configures the push command.
     */
    protected function configure()
    {
        $this
            ->setName('push')
            ->setDescription('Sets external job result')
            ->addArgument('url', InputArgument::REQUIRED, 'Jenkins home URL')
            ->addArgument('job', InputArgument::REQUIRED, 'Job name as configured in Jenkins')
            ->addOption('display', null, InputOption::VALUE_OPTIONAL, '')
            ->addOption('description', null, InputOption::VALUE_OPTIONAL, '')
            ->addOption('duration', null, InputOption::VALUE_OPTIONAL, '')
            ->addOption('log', null, InputOption::VALUE_OPTIONAL, '');
    }

    /**
     * Executes the monitor command.
     *
     * @param InputInterface  $input  An InputInterface instance
     * @param OutputInterface $output An OutputInterface instance
     *
     * @return null|int null or 0 if everything went fine, or an error code
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $monitor = new JobMonitor(
            $input->getArgument('job'),
            $input->getOption('display'),
            $input->getOption('description')
        );
        $monitor->setLog($this->getOptionLog($input));

        try {
            $this->getClient()->send($monitor->getRequest(), ['base_uri' => $input->getArgument('url')]);
        } catch (\Exception $e) {
            $output->writeln(sprintf('<error>%s</error>', $e->getMessage()));
            return 1;
        }

        return 0;
    }

    /**
     * Returns the value for log option.
     *
     * If the log option is not provided, STDIN is being used instead.
     *
     * @param InputInterface $input
     * @throws \RuntimeException
     * @return mixed|string
     */
    public function getOptionLog(InputInterface $input)

This method seems to be duplicated in your project.

    {
        $log = $input->getOption('log');
        if (!$log && 0 === ftell(STDIN)) {
            $log = '';
            while (!feof(STDIN)) {
                $log .= fread(STDIN, 1024);
            }
        }

        return $log;
    }

    /**
     * @param GuzzleHttp\ClientInterface $client
     */
    public function setClient(ClientInterface $client)
    {
        $this->client = $client;

It seems like $client of type object<GuzzleHttp\ClientInterface> is incompatible with the declared type object<Softius\JenkinsJobMonitor\GuzzleHttp\ClientInterface> of property $client.

    }

    /**
     * @return Client
     */
    public function getClient()
    {
        return $this->client;
    }
}