Andi\KickBoxBundle\Tests\Http\ClientTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes	2
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	5	1
A	testVerify()	0	17	1
A	testKickboxApiException()	0	14	1
A	testEmptyContentException()	0	11	1
A	getResponse()	0	8	1
A	getErrorResponse()	0	8	1

<?php

/*
 * This file is part of the Kickbox Bundle.
 *
 * (c) Abdoul Ndiaye <abdoul.nd@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Andi\KickBoxBundle\Tests\Http;

use Andi\KickBoxBundle\Exception\EmptyContentException;
use Andi\KickBoxBundle\Exception\KickBoxApiException;
use Andi\KickBoxBundle\Factory\ResponseFactory;
use Andi\KickBoxBundle\Http\Client;
use Andi\KickBoxBundle\Http\Response;
use Andi\KickBoxBundle\Tests\Factory\ResponseFactoryTest;
use GuzzleHttp\Psr7\Response as HttpResponse;
use GuzzleHttp\Psr7\Request;
use GuzzleHttp\Psr7\Stream;
use Phake;
use GuzzleHttp\Client as HttpClient;

/**
 * Class ClientTest
 *
 * @author Abdoul Ndiaye <abdoul.nd@gmail.com>
 *
 * @see    Andi\KickBoxBundle\Http\Client
 */
class ClientTest extends \PHPUnit_Framework_TestCase
{
    /**
     * @var HttpClient|\Phake_IMock
     */
    protected $httpClient;

    /**
     * @var ResponseFactory
     */
    protected $responseFactory;

    /**
     * @{@inheritdoc}
     */
    protected function setUp()
    {
        $this->httpClient      = Phake::mock(HttpClient::class);
        $this->responseFactory = new ResponseFactory();
    }

    public function testVerify()
    {
        Phake::when($this->httpClient)->get(

It seems like $this->httpClient can also be of type object<GuzzleHttp\Client>; however, Phake::when() does only seem to accept object<Phake_IMock>, maybe add an additional type check?

            'my_end_point',
            [
                'query' => [
                    'email'   => 'ab@cd.com',
                    'apikey'  => 'my_key',
                    'timeout' => 7000,
                ]
            ]
        )->thenReturn($this->getResponse());

        $client = new Client($this->httpClient, $this->responseFactory, 'my_end_point', 'my_key');

It seems like $this->httpClient can also be of type object<Phake_IMock>; however, Andi\KickBoxBundle\Http\Client::__construct() does only seem to accept object<GuzzleHttp\Client>, maybe add an additional type check?

        $this->assertInstanceOf(Response::class, $client->verify('ab@cd.com', 7000));
    }

    public function testKickboxApiException()
    {
        Phake::when($this->httpClient)->get(Phake::anyParameters())->thenThrow(

It seems like $this->httpClient can also be of type object<GuzzleHttp\Client>; however, Phake::when() does only seem to accept object<Phake_IMock>, maybe add an additional type check?

            new KickBoxApiException('expectedMessage', new Request('GET', 'error'), $this->getErrorResponse())
        );

        $this->setExpectedException(
            KickBoxApiException::class,
            'An error occurred during the api call : exception message'
        );

        $client = new Client($this->httpClient, $this->responseFactory, '', '');

It seems like $this->httpClient can also be of type object<Phake_IMock>; however, Andi\KickBoxBundle\Http\Client::__construct() does only seem to accept object<GuzzleHttp\Client>, maybe add an additional type check?

        $client->verify('');
    }

    public function testEmptyContentException()
    {
        Phake::when($this->httpClient)->get(Phake::anyParameters())->thenReturn(new HttpResponse(400));

It seems like $this->httpClient can also be of type object<GuzzleHttp\Client>; however, Phake::when() does only seem to accept object<Phake_IMock>, maybe add an additional type check?

        $this->setExpectedException(
            EmptyContentException::class
        );

        $client = new Client($this->httpClient, $this->responseFactory, '', '');

It seems like $this->httpClient can also be of type object<Phake_IMock>; however, Andi\KickBoxBundle\Http\Client::__construct() does only seem to accept object<GuzzleHttp\Client>, maybe add an additional type check?

        $client->verify('');
    }

    /**
     * @return HttpResponse
     */
    protected function getResponse()
    {
        return new HttpResponse(
            200,
            ResponseFactoryTest::$goodHeaders,
            json_encode(ResponseFactoryTest::$goodParameters)
        );
    }

    /**
     * @return HttpResponse
     */
    protected function getErrorResponse()
    {
        return new HttpResponse(
            500,
            [],
            json_encode(['message' => 'exception message'])
        );
    }
}