Magium\Cli\Command\ApiPing

Complexity

Total Complexity

20

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	8

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	6	1
C	execute()	0	71	19

<?php

namespace Magium\Cli\Command;

use Guzzle\Http\Client;
use Magium\Cli\Command\Test\TestSkeleton;
use Magium\InvalidConfigurationException;
use Magium\NotFoundException;
use Magium\Util\Api\ApiConfiguration;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\ArrayInput;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

class ApiPing extends Command
{

    protected function configure()
    {
        $this->setName('api:ping');
        $this->setDescription('Pings the API ping endpoint to ensure that the key works');

    }

    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $test = new TestSkeleton();
        $test->configureDi();
        $api = $test->get('Magium\Util\Api\ApiConfiguration');
        /* @var $api ApiConfiguration */
        $api->setEnabled(true); // Gotta force this for this test

        $output->writeln('Sending un-authenticated payload...');
        $client = new Client('http://' . $api->getApiHostname(). '/api/ping');
        $request = $client->get();
        $response = $request->send();
        $output->writeln(
            'Checking for 200 status message... '
            . ($response->getStatusCode() == '200'?'OK':'Failed')
        );
        $output->writeln(
            'Checking for application/json content type... '
            . (stripos($response->getContentType(), 'application/json') !== false?'OK':'Failed')
        );
        $content = json_decode($response->getBody(), true);
        $output->writeln(
            'Checking for successful response... '
            . (is_array($content) && isset($content['status']) && $content['status'] === 'success'?'OK':'Failed')
        );

        $output->writeln('');
        $output->writeln('Attempting authenticated ping...');

        $request = $test->get('Magium\Util\Api\Request');
        /* @var $request \Magium\Util\Api\Request */
        $response = $request->fetch('/api/ping-authed');

        $output->writeln(
            'Checking for 200 status message... '
            . ($response->getStatusCode() == '200'?'OK':'Failed')
        );
        $output->writeln(
            'Checking for application/json content type... '
            . (stripos($response->getContentType(), 'application/json') !== false?'OK':'Failed')
        );
        $content = $request->getPayload($response);

It seems like $response defined by $request->fetch('/api/ping-authed') on line 58 can also be of type array or null; however, Magium\Util\Api\Request::getPayload() does only seem to accept object<Guzzle\Http\Message\Response>, maybe add an additional type check?

        $output->writeln(
            'Checking for successful response... '
            . (is_array($content) && isset($content['status']) && $content['status'] === 'success'?'OK':'Failed')
        );


        $output->writeln('');
        $output->writeln('Attempting authenticated echo...');
        $request = $test->get('Magium\Util\Api\Request');
        /* @var $request \Magium\Util\Api\Request */
        $response = $request->push('/api/echo-authed', ['message' => 'hello world']);
        $output->writeln(
            'Checking for 200 status message... '
            . ($response->getStatusCode() == '200'?'OK':'Failed')
        );
        $output->writeln(
            'Checking for application/json content type... '
            . (stripos($response->getContentType(), 'application/json') !== false?'OK':'Failed')
        );
        $content = $request->getPayload($response);

It seems like $response defined by $request->push('/api/echo-authed', array('message' => 'hello world')) on line 79 can also be of type array or null; however, Magium\Util\Api\Request::getPayload() does only seem to accept object<Guzzle\Http\Message\Response>, maybe add an additional type check?

        $output->writeln(
            'Checking for successful response... '
            . (is_array($content) && isset($content['status']) && $content['status'] === 'success'?'OK':'Failed')
        );
        $output->writeln(
            'Checking for matching echo message... '
            . (is_array($content) && isset($content['message']) && $content['message'] === 'hello world'?'OK':'Failed')
        );
    }
    
}