HttpStreamCollectorTest::dataSkipCollectOnMatchIgnoreReferences() - Code Metrics - Inspection of "Increase test coverage (#215)" - yiisoft/yii-debug - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( bace86...bb4cad )

by Dmitriy

created 2023-05-29 15:48 UTC

dataSkipCollectOnMatchIgnoreReferences() A

↳ Parent: HttpStreamCollectorTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	70
Code Lines	46

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	46
nc	1
nop	0
dl	0
loc	70
rs	9.1781
c	0
b	0
f	0

How to fix Long Method

<?php

declare(strict_types=1);

namespace Yiisoft\Yii\Debug\Tests\Unit\Collector;

use Yiisoft\Yii\Debug\Collector\CollectorInterface;
use Yiisoft\Yii\Debug\Collector\Stream\HttpStreamCollector;
use Yiisoft\Yii\Debug\Tests\Shared\AbstractCollectorTestCase;

final class HttpStreamCollectorTest extends AbstractCollectorTestCase
{
    /**
     * @param HttpStreamCollector $collector
     */
    protected function collectTestData(CollectorInterface $collector): void
    {
        $collector->collect(
                    collect(
            operation: 'read',
            path: __FILE__,
            args: ['arg1' => 'v1', 'arg2' => 'v2'],
        );
        $collector->collect(
            operation: 'read',
            path: __FILE__,
            args: ['arg3' => 'v3', 'arg4' => 'v4'],
        );
    }

    /**
     * @dataProvider dataSkipCollectOnMatchIgnoreReferences
     */
    public function testSkipCollectOnMatchIgnoreReferences(
        string $url,
        callable $before,
        array $ignoredPathPatterns,
        array $ignoredClasses,
        array $ignoredUrls,
        callable $operation,
        callable $after,
        array|callable $assertResult,
    ): void {
        $before($url);

        try {
            $collector = new HttpStreamCollector(
                ignoredPathPatterns: $ignoredPathPatterns,
                ignoredClasses: $ignoredClasses,
                ignoredUrls: $ignoredUrls,
            );
            $collector->startup();

            $operation($url);

            $collected = $collector->getCollected();
            $collector->shutdown();
        } finally {
            $after($url);
        }
        if (is_array($assertResult)) {

            $this->assertSame($assertResult, $collected);
        } else {
            $assertResult($url, $collected);
        }
    }

    public function dataSkipCollectOnMatchIgnoreReferences(): iterable
    {
        $httpStreamBefore = function (string $url) {
        $httpStreamBefore = function (/** @scrutinizer ignore-unused */ string $url) {
        };
        $httpStreamOperation = function (string $url) {
            $stream = fopen($url, 'r');
            fread($stream, 4);
            ftell($stream);
            feof($stream);
            fstat($stream);
            fclose($stream);
        };
        $httpStreamAfter = $httpStreamBefore;

        yield 'file stream matched' => [
            $url = 'http://example.com',
            $httpStreamBefore,
            [],
            [],
            [],
            $httpStreamOperation,
            $httpStreamAfter,
            function (string $url, array $collected) {
                $this->assertArrayHasKey('read', $collected);
                $this->assertIsArray($collected['read']);
                $this->assertCount(1, $collected['read']);

                $readItem = $collected['read'][0];
                $this->assertSame($url, $readItem['uri']);
                $this->assertArrayHasKey('args', $readItem);

                $readItemArgs = $readItem['args'];
                $this->assertCount(3, $readItemArgs);

                $this->assertSame('GET', $readItemArgs['method']);
                $this->assertIsArray($readItemArgs['response_headers']);
                $this->assertNotEmpty($readItemArgs['response_headers']);
                $this->assertIsArray($readItemArgs['request_headers']);
                $this->assertEmpty($readItemArgs['request_headers']);
            },
        ];
        yield 'file stream ignored by path' => [
            $url,
            $httpStreamBefore,
            ['/' . basename(__FILE__, '.php') . '/'],
            [],
            [],
            $httpStreamOperation,
            $httpStreamAfter,
            [],
        ];
        yield 'file stream ignored by class' => [
            $url,
            $httpStreamBefore,
            [],
            [self::class],
            [],
            $httpStreamOperation,
            $httpStreamAfter,
            [],
        ];
        yield 'file stream ignored by url' => [
            $url,
            $httpStreamBefore,
            [],
            [],
            ['/example/'],
            $httpStreamOperation,
            $httpStreamAfter,
            [],
        ];
    }

    protected function getCollector(): CollectorInterface
    {
        return new HttpStreamCollector();
    }

    protected function checkCollectedData(array $data): void
    {
        parent::checkCollectedData($data);
        $collected = $data;
        $this->assertCount(1, $collected);

        $this->assertCount(2, $collected['read']);
        $this->assertEquals([
            ['uri' => __FILE__, 'args' => ['arg1' => 'v1', 'arg2' => 'v2']],
            ['uri' => __FILE__, 'args' => ['arg3' => 'v3', 'arg4' => 'v4']],
        ], $collected['read']);
    }

    protected function checkSummaryData(array $data): void
    {
        parent::checkSummaryData($data);
        $this->assertArrayHasKey('http_stream', $data);
        $this->assertEquals(['read' => 2], $data['http_stream']);
    }
}


yiisoft / yii-debug

Push — master ( bace86...bb4cad )

dataSkipCollectOnMatchIgnoreReferences() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like