Yiisoft\Yii\Debug\Tests\Collector\AbstractCollectorTestCase

Complexity

Total Complexity

5

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	checkIndexData()	0	3	1
A	testCollect()	0	17	3
A	checkCollectedData()	0	3	1

<?php

declare(strict_types=1);

namespace Yiisoft\Yii\Debug\Tests\Collector;

use PHPUnit\Framework\TestCase;
use Yiisoft\Yii\Debug\Collector\CollectorInterface;
use Yiisoft\Yii\Debug\Collector\IndexCollectorInterface;

abstract class AbstractCollectorTestCase extends TestCase
{
    public function testCollect(): void
    {
        $indexData = null;
        $collector = $this->getCollector();

        $collector->startup();
        $this->collectTestData($collector);
        $data = $collector->getCollected();
        if ($collector instanceof IndexCollectorInterface) {
            $indexData = $collector->getIndexData();
        }
        $collector->shutdown();

        $this->assertSame($collector::class, $collector->getName());
        $this->checkCollectedData($data);
        if ($collector instanceof IndexCollectorInterface) {
            $this->checkIndexData($indexData);

It seems like $indexData can also be of type null; however, parameter $data of Yiisoft\Yii\Debug\Tests\Collector\AbstractCollectorTestCase::checkIndexData() does only seem to accept array, maybe add an additional type check?

        }
    }

    abstract protected function getCollector(): CollectorInterface;

    abstract protected function collectTestData(CollectorInterface $collector): void;

    protected function checkCollectedData(array $data): void
    {
        $this->assertNotEmpty($data);
    }

    protected function checkIndexData(array $data): void
    {
        $this->assertNotEmpty($data);
    }
}