AbacaphiliacTest\Doctrine\PsrSqlLoggerTest

Complexity

Total Complexity

7

Size/Duplication

Total Lines	154
Duplicated Lines	36.36 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	6	1
A	getRecordByIndex()	0	8	1
A	testLogsQuery()	0	24	1
A	testLogsDuration()	27	27	1
A	testSharedQueryId()	29	29	1
A	testQueryIdChanges()	0	36	1
A	testInvalidLogLevel()	0	6	1

<?php

namespace AbacaphiliacTest\Doctrine;

use Abacaphiliac\Doctrine\PsrSqlLogger;
use PHPUnit\Framework\TestCase;
use Psr\Log\LogLevel;
use Psr\Log\NullLogger;
use Psr\Log\Test\TestLogger;

/**
 * @covers \Abacaphiliac\Doctrine\PsrSqlLogger
 */
class PsrSqlLoggerTest extends TestCase
{
    /** @var PsrSqlLogger */
    private $sut;

    /** @var TestLogger */
    private $logger;

    /** @var string */
    private $sql = 'SELECT * FROM users WHERE id = :id';

    protected function setUp(): void
    {
        $this->logger = new TestLogger();

        $this->sut = new PsrSqlLogger($this->logger);
    }

    private function getRecordByIndex(int $index): \stdClass
    {
        $record = $this->logger->records[$index];

        self::assertIsArray($record);

        return (object) $record;
    }

    public function testLogsQuery()
    {
        self::assertCount(0, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $this->sut->startQuery(
            $this->sql,
            [
                ':id' => 1234,
            ],
            [
                ':id' => \PDO::PARAM_INT,
            ]
        );

        self::assertCount(1, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $log = $this->getRecordByIndex(0);

        self::assertSame(LogLevel::INFO, (string) $log->level);
        self::assertSame('Query started', (string) $log->message);
        self::assertNotEmpty($log->context['query_id']);
        self::assertSame($this->sql, $log->context['sql']);
        self::assertSame([':id' => \PDO::PARAM_INT], $log->context['types']);
    }

    public function testLogsDuration()

This method seems to be duplicated in your project.

    {
        self::assertCount(0, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $this->sut->startQuery(
            $this->sql,
            [
                ':id' => 1234,
            ],
            [
                ':id' => \PDO::PARAM_INT,
            ]
        );

        $this->sut->stopQuery();

        self::assertCount(2, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $log = $this->getRecordByIndex(1);

        self::assertSame(LogLevel::INFO, (string) $log->level);
        self::assertSame('Query finished', (string) $log->message);
        self::assertNotEmpty($log->context['query_id']);
        self::assertIsFloat($log->context['start']);
        self::assertIsFloat($log->context['stop']);
        self::assertIsFloat($log->context['duration_s']);
    }

    public function testSharedQueryId()

This method seems to be duplicated in your project.

    {
        self::assertCount(0, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $this->sut->startQuery(
            $this->sql,
            [
                ':id' => 1234,
            ],
            [
                ':id' => \PDO::PARAM_INT,
            ]
        );

        $this->sut->stopQuery();

        self::assertCount(2, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $startLog = $this->getRecordByIndex(0);

        self::assertInstanceOf(\stdClass::class, $startLog);

        $queryId = $startLog->context['query_id'];
        self::assertNotEmpty($queryId);

        $stopLog = $this->getRecordByIndex(1);

        self::assertSame($queryId, $stopLog->context['query_id']);
    }

    public function testQueryIdChanges()
    {
        self::assertCount(0, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $this->sut->startQuery(
            $this->sql,
            [
                ':id' => 1234,
            ],
            [
                ':id' => \PDO::PARAM_INT,
            ]
        );

        $this->sut->startQuery(
            $this->sql,
            [
                ':id' => 2345,
            ],
            [
                ':id' => \PDO::PARAM_INT,
            ]
        );

        self::assertCount(2, $this->logger->records);

$this->logger->records is of type array, but the function expects a object<Countable>|object<PHPUnit\Framework\iterable>.

        $firstLog = $this->getRecordByIndex(0);

        $queryId = $firstLog->context['query_id'];
        self::assertNotEmpty($queryId);

        $secondLog = $this->getRecordByIndex(1);

        self::assertNotEmpty($secondLog->context['query_id']);
        self::assertNotEquals($queryId, $secondLog->context['query_id']);
    }

    public function testInvalidLogLevel()
    {
        self::expectException(\InvalidArgumentException::class);

        new PsrSqlLogger(new NullLogger(), 'InvalidLevel');
    }
}