QuickPay\Tests\RequestTest

Complexity

Total Complexity

11

Size/Duplication

Total Lines	113
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	5	1
A	testResponseInstance()	0	6	1
A	testBadAuthentication()	0	9	1
A	testSuccessfulGetResponse()	0	6	1
A	testFailedGetResponse()	0	6	1
A	testSuccesfulPostResponse()	0	6	1
A	testFailedPostResponse()	0	6	1
A	testBasket()	0	30	1
A	testStandardHeaders()	0	5	1
A	testAdditionalHeadersOnInit()	0	8	1
A	testAdditionalHeadersAfterInit()	0	8	1

<?php

namespace QuickPay\Tests;

use QuickPay\API\Client;
use QuickPay\API\Request;
use QuickPay\API\Response;

class RequestTest extends \PHPUnit_Framework_TestCase
{
    protected $request;

    public function setUp()
    {
        $this->client = new Client();

The property client does not exist. Did you maybe forget to declare it?

        $this->request = new Request($this->client);
    }

    public function testResponseInstance()
    {
        $pingResponse = $this->request->get('/ping');

        $this->assertTrue(($pingResponse instanceof Response));
    }

    public function testBadAuthentication()
    {
        $client = new Client(':foo');
        $request = new Request($client);

        $response = $request->get('/ping');

        $this->assertEquals(401, $response->httpStatus());
    }

    public function testSuccessfulGetResponse()
    {
        $pingResponse = $this->request->get('/ping');

        $this->assertTrue($pingResponse->isSuccess());
    }

    public function testFailedGetResponse()
    {
        $pingResponse = $this->request->get('/foobar');

        $this->assertFalse($pingResponse->isSuccess());
    }

    public function testSuccesfulPostResponse()
    {
        $pingResponse = $this->request->post('/ping');

        $this->assertTrue($pingResponse->isSuccess());
    }

    public function testFailedPostResponse()
    {
        $pingResponse = $this->request->post('/foobar');

        $this->assertFalse($pingResponse->isSuccess());
    }

    /**
     * Test function added to make sure that issue gh-54 is fixed.
     */
    public function testBasket()
    {
        $basket = [];
        $basket[0] = [
            'qty' => 1,
            'item_no' => 2,
            'item_name' => 'Test 1',
            'item_price' => 100,
            'vat_rate' => 0.25,
        ];
        $basket[1] = [
            'qty' => 1,
            'item_no' => 2,
            'item_name' => 'Test 2',
            'item_price' => 100,
            'vat_rate' => 0.25,
        ];

        $form = [
            'currency' => 'DKK',
            'order_id' => 1,
            'basket' => $basket,
        ];

        $query = $this->request->httpBuildQuery($form);

        $expected = 'currency=DKK&order_id=1&basket[][qty]=1&basket[][item_no]=2&basket[][item_name]=Test 1&basket[][item_price]=100&basket[][vat_rate]=0.25&basket[][qty]=1&basket[][item_no]=2&basket[][item_name]=Test 2&basket[][item_price]=100&basket[][vat_rate]=0.25';

        $this->assertEquals(urldecode($query), $expected);
    }

    public function testStandardHeaders()
    {
        $pingResponse = $this->request->get('/ping');

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

        $this->assertContains('Accept: application/json', curl_getinfo($this->client->ch, CURLINFO_HEADER_OUT));
    }

    public function testAdditionalHeadersOnInit()
    {
        $extra_headers = array('SpecialVar: foo');
        $client = new Client(null, $extra_headers);
        $request = new Request($client);
        $pingResponse = $request->get('/ping');

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

        $this->assertContains($extra_headers[0], curl_getinfo($client->ch, CURLINFO_HEADER_OUT));
    }

    public function testAdditionalHeadersAfterInit()
    {
        $extra_headers = array('NewVar: foo');
        $this->client->setHeaders($extra_headers);
        $request = new Request($this->client);
        $pingResponse = $request->get('/ping');

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

        $this->assertContains($extra_headers[0], curl_getinfo($this->client->ch, CURLINFO_HEADER_OUT));
    }
}