Tests\Koded\Http\Client\ClientTestCaseTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	test_read_on_success()	0	8	1
A	test_should_exit_on_bad_url()	0	7	1
A	test_should_exit_on_bad_request()	0	11	1
A	tearDown()	0	3	1

<?php

namespace Tests\Koded\Http\Client;

use Koded\Http\Interfaces\{HttpRequestClient, HttpStatus};
use Koded\Http\Uri;
use function Koded\Http\create_stream;

/**
 * Trait ClientTestCaseTrait ensures some consistent behaviour
 * across the HTTP client implementations.
 *
 */

/**
 * @group internet
 */
trait ClientTestCaseTrait
{
    private ?HttpRequestClient $SUT;

    public function test_read_on_success()
    {
        $this->markTestSkipped();

It seems like markTestSkipped() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $response = $this->SUT->read();

The method read() does not exist on null.

        $this->assertSame(HttpStatus::OK, $response->getStatusCode(), (string)$response->getBody());

It seems like assertSame() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->assertStringContainsString('text/html', $response->getHeaderLine('Content-Type'));

It seems like assertStringContainsString() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->assertGreaterThan(0, (string)$response->getBody()->getSize());

It seems like assertGreaterThan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function test_should_exit_on_bad_url()
    {
        $this->expectException(\InvalidArgumentException::class);

It seems like expectException() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->expectExceptionMessage('Please provide a valid URI');

It seems like expectExceptionMessage() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->expectExceptionCode(HttpStatus::BAD_REQUEST);

It seems like expectExceptionCode() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->SUT->withUri(new Uri('scheme://host:port'));
    }

    public function test_should_exit_on_bad_request()
    {
        /** @var HttpRequestClient $SUT */
        $SUT = $this->SUT->withBody(create_stream(json_encode(['foo' => 'bar'])));

        $badResponse = $SUT->read();

        $this->assertSame(HttpStatus::BAD_REQUEST, $badResponse->getStatusCode(), get_class($SUT));
        $this->assertSame($badResponse->getHeaderLine('Content-type'), 'application/problem+json');
        $this->assertStringContainsString('failed to open stream: you should not set the message body with safe HTTP methods',
            (string)$badResponse->getBody());
    }

    protected function tearDown(): void
    {
        $this->SUT = null;
    }
}