Yiisoft\Yii\Debug\Tests\Unit\Collector\Stream\FilesystemStreamProxyTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testRegisteredTwice()	0	8	1
A	testProxyAccess()	0	17	1

<?php

declare(strict_types=1);

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

use PHPUnit\Framework\TestCase;
use Yiisoft\Yii\Debug\Collector\Stream\FilesystemStreamProxy;

final class FilesystemStreamProxyTest extends TestCase
{
    public function testRegisteredTwice()
    {
        FilesystemStreamProxy::unregister();
        $this->assertFalse(FilesystemStreamProxy::$registered);
        FilesystemStreamProxy::register();
        $this->assertTrue(FilesystemStreamProxy::$registered);
        FilesystemStreamProxy::register();
        $this->assertTrue(FilesystemStreamProxy::$registered);
    }

    public function testProxyAccess()
    {
        $proxy = new FilesystemStreamProxy();
        FilesystemStreamProxy::register();
        $handle = opendir(sys_get_temp_dir());

        $firstElement = readdir($handle);
        $secondElement = readdir($handle);

        $this->assertNotSame($firstElement, $secondElement);
        rewinddir($handle);
        $this->assertEquals($firstElement, readdir($handle));

        $proxy->decorated->stream = $handle;

Accessing stream on the interface Yiisoft\Yii\Debug\Helper\StreamWrapper\StreamWrapperInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        $proxy->dir_rewinddir();

        $this->assertEquals($firstElement, readdir($handle));
    }
}