Mezon\CustomClient\Tests\CurlWrapperUnitTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testGetRequest()	0	6	1
A	testPostRequest()	0	12	1

<?php

namespace Mezon\CustomClient\Tests;

use PHPUnit\Framework\TestCase;
use Mezon\CustomClient\CurlWrapper;

class CurlWrapperUnitTest extends TestCase
{

    /**
     * Testing GET requests
     */
    public function testGetRequest()
    {
        list ($body, $code) = CurlWrapper::sendRequest('http://google.com', [], 'GET');

        $this->assertStringContainsString('', $body, 'Invalid HTML was returned');

It seems like $body can also be of type boolean; however, parameter $haystack of PHPUnit\Framework\Assert::assertStringContainsString() does only seem to accept string, maybe add an additional type check?

        $this->assertEquals(301, $code, 'Invalid HTTP code');
    }

    /**
     * Testing POST requests
     */
    public function testPostRequest()
    {
        list ($body, $code) = CurlWrapper::sendRequest(
            'http://google.com',
            [],
            'POST',
            [
                'data' => 1
            ]);

        $this->assertStringContainsString('', $body, 'Invalid HTML was returned');

It seems like $body can also be of type boolean; however, parameter $haystack of PHPUnit\Framework\Assert::assertStringContainsString() does only seem to accept string, maybe add an additional type check?

        $this->assertEquals(405, $code, 'Invalid HTTP code');
    }
}