Spiral\Tests\Framework\Http\SapiTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Importance

Changes

0

11 Methods

Rating	Name	Duplication	Size	Complexity
A	hp$0 ➔ testCantServe()	0	3	1
A	hp$0 ➔ emit()	0	4	1
A	hp$0 ➔ setUp()	0	12	1
A	hp$0 ➔ testDispatch()	0	10	1
A	hp$0 ➔ testDispatchError()	0	20	1
A	hp$0 ➔ testDispatchNativeError()	0	24	1
	testDispatch()	0	10	?
	setUp()	0	12	?
	testDispatchError()	0	20	?
	testDispatchNativeError()	0	24	?
	testCantServe()	0	3	?

<?php

/**
 * Spiral Framework.
 *
 * @license   MIT
 * @author    Anton Titov (Wolfy-J)
 */

declare(strict_types=1);

namespace Spiral\Tests\Framework\Http;

use Psr\Http\Message\ResponseInterface;
use Spiral\Boot\DirectoriesInterface;
use Spiral\Files\FilesInterface;
use Spiral\Http\EmitterInterface;
use Spiral\Http\SapiDispatcher;
use Spiral\Tests\Framework\ConsoleTest;

class SapiTest extends ConsoleTest
{
    /** @var EmitterInterface */
    private $bufferEmitter;

    public function setUp(): void
    {
        $this->bufferEmitter = new class() implements EmitterInterface {
            public $response;

            public function emit(ResponseInterface $response): bool
            {
                $this->response = $response;
                return true;
            }
        };
        parent::setUp();
    }

    public function testCantServe(): void
    {
        $this->assertFalse($this->app->get(SapiDispatcher::class)->canServe());
    }

    public function testDispatch(): void
    {
        $e = $this->bufferEmitter;

        $app = $this->makeApp();

        $_SERVER['REQUEST_URI'] = '/index/dave';
        $app->get(SapiDispatcher::class)->serve($e);

        $this->assertSame('Hello, dave.', (string)$e->response->getBody());

Accessing response on the interface Spiral\Http\EmitterInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    public function testDispatchError(): void
    {
        $e = $this->bufferEmitter;

        $files = $this->app->get(FilesInterface::class)->getFiles(
            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
        );

        $this->assertCount(0, $files);

        $_SERVER['REQUEST_URI'] = '/error';
        $this->app->get(SapiDispatcher::class)->serve($e);

        $files = $this->app->get(FilesInterface::class)->getFiles(
            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
        );

        $this->assertCount(1, $files);

        $this->assertStringContainsString('500', (string)$e->response->getBody());

Accessing response on the interface Spiral\Http\EmitterInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    public function testDispatchNativeError(): void
    {
        $e = $this->bufferEmitter;

        $app = $this->makeApp([
            'DEBUG' => true
        ]);

        $files = $app->get(FilesInterface::class)->getFiles(
            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
        );

        $this->assertCount(0, $files);

        $_SERVER['REQUEST_URI'] = '/error';
        $app->get(SapiDispatcher::class)->serve($e);

        $files = $app->get(FilesInterface::class)->getFiles(
            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
        );

        $this->assertCount(1, $files);

        $this->assertStringContainsString('undefined', (string)$e->response->getBody());

Accessing response on the interface Spiral\Http\EmitterInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }
}