FilesystemStreamProxyTest::tearDown() - Code Metrics - Inspection of "Add more tests (#232)" - yiisoft/yii-debug - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d5232f...8b0585 )

by Dmitriy

created 2024-01-05 13:30 UTC

FilesystemStreamProxyTest::tearDown() A

↳ Parent: FilesystemStreamProxyTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
nc	1
nop	0
dl	0
loc	3
rs	10
c	1
b	0
f	0

<?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
{
    protected function tearDown(): void
    {
        FilesystemStreamProxy::unregister();
    }

    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;

        $proxy->dir_rewinddir();

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


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Yiisoft\Yii\Debug\Tests\Unit\Collector\Stream;
6
7			use PHPUnit\Framework\TestCase;
8			use Yiisoft\Yii\Debug\Collector\Stream\FilesystemStreamProxy;
9
10			final class FilesystemStreamProxyTest extends TestCase
11			{
12			protected function tearDown(): void
13			{
14			FilesystemStreamProxy::unregister();
15			}
16
17			public function testRegisteredTwice()
18			{
19			FilesystemStreamProxy::unregister();
20			$this->assertFalse(FilesystemStreamProxy::$registered);
21			FilesystemStreamProxy::register();
22			$this->assertTrue(FilesystemStreamProxy::$registered);
23			FilesystemStreamProxy::register();
24			$this->assertTrue(FilesystemStreamProxy::$registered);
25			}
26
27			public function testProxyAccess()
28			{
29			$proxy = new FilesystemStreamProxy();
30			FilesystemStreamProxy::register();
31			$handle = opendir(sys_get_temp_dir());
32
33			$firstElement = readdir($handle);
34			$secondElement = readdir($handle);
35
36			$this->assertNotSame($firstElement, $secondElement);
37			rewinddir($handle);
38			$this->assertEquals($firstElement, readdir($handle));
39
40			$proxy->decorated->stream = $handle;
			0 ignored issues – show Bug introduced 2024-01-05 08:17 UTC by Report Bug Copy Issue Report Accessing `stream` on the interface `Yiisoft\Yii\Debug\Helper...\StreamWrapperInterface` suggest that you code against a concrete implementation. How about adding an `instanceof` check? Loading history...
41			$proxy->dir_rewinddir();
42
43			$this->assertEquals($firstElement, readdir($handle));
44			}
45			}
46

yiisoft / yii-debug

Push — master ( d5232f...8b0585 )

FilesystemStreamProxyTest::tearDown() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like