TestCase::rmdir() - Code Metrics - Inspection of "Utilize test utilities" - php-api-clients/resource-test-utilities - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a5d40d...abe42f )

by Cees-Jan

created 2016-11-20 12:08 UTC

TestCase::rmdir() B

↳ Parent: TestCase

Complexity

Conditions	5
Paths	5

Size

Total Lines	21
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	11
CRAP Score	5.246

Importance

Changes

Metric	Value
dl	0
loc	21
ccs	11
cts	14
cp	0.7856
rs	8.7624
c	0
b	0
f	0
cc	5
eloc	13
nc	5
nop	1
crap	5.246

<?php
declare(strict_types=1);

namespace ApiClients\Tools\ResourceTestUtilities;

use ApiClients\Foundation\Hydrator\Factory;
use ApiClients\Foundation\Hydrator\Options;
use ApiClients\Tools\TestUtilities\TestCase as BaseTestCase;

abstract class TestCase extends BaseTestCase
{
    const DEFAULT_GENERATED_CLASS_NAMESPACE = 'GHGC_%s';

    /**
     * @var string
     */
    private $tmpDir;


    /**
     * @var string
     */
    private $tmpNamespace;


    public function setUp()
    {
        parent::setUp();
        $crc32 = crc32(get_class($this));
        $this->tmpDir = sys_get_temp_dir() . DIRECTORY_SEPARATOR . uniqid('act-' . $crc32, true) . DIRECTORY_SEPARATOR;
        mkdir($this->tmpDir, 0777, true);
        $this->tmpNamespace = sprintf(
            static::DEFAULT_GENERATED_CLASS_NAMESPACE,
            crc32(uniqid((string)$crc32, true))
        );
    }

    public function tearDown()
    {
        parent::tearDown();
        $this->rmdir($this->tmpDir);
    }

    protected function rmdir($dir)
    {
        $directory = dir($dir);
        while (false !== ($entry = $directory->read())) {
            if (in_array($entry, ['.', '..'])) {
                continue;
            }

            if (is_dir($dir . $entry)) {
                $this->rmdir($dir . $entry . DIRECTORY_SEPARATOR);
                continue;
            }

            if (is_file($dir . $entry)) {
                unlink($dir . $entry);
                continue;
            }
        }
        $directory->close();
        rmdir($dir);
    }

    protected function getTmpDir(): string
    {
        return $this->tmpDir;
    }

    protected function getRandomNameSpace(): string
    {
        return $this->tmpNamespace;
    }

    public function hydrate($class, $json, $namespace)
    {
        return Factory::create([
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            Options::NAMESPACE => $namespace,
        ])->hydrateFQCN($class, $json);
    }
}


1		<?php
2		declare(strict_types=1);
3
4		namespace ApiClients\Tools\ResourceTestUtilities;
5
6		use ApiClients\Foundation\Hydrator\Factory;
7		use ApiClients\Foundation\Hydrator\Options;
8		use ApiClients\Tools\TestUtilities\TestCase as BaseTestCase;
9
10		abstract class TestCase extends BaseTestCase
11		{
12		const DEFAULT_GENERATED_CLASS_NAMESPACE = 'GHGC_%s';
13
14		/**
15		* @var string
16		*/
17		private $tmpDir;
		0 ignored issues – show Comprehensibility introduced 2016-11-20 12:10 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
18
19		/**
20		* @var string
21		*/
22		private $tmpNamespace;
		0 ignored issues – show Comprehensibility introduced 2016-11-20 12:10 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history...
23
24	6	public function setUp()
25		{
26	6	parent::setUp();
27	6	$crc32 = crc32(get_class($this));
28	6	$this->tmpDir = sys_get_temp_dir() . DIRECTORY_SEPARATOR . uniqid('act-' . $crc32, true) . DIRECTORY_SEPARATOR;
29	6	mkdir($this->tmpDir, 0777, true);
30	6	$this->tmpNamespace = sprintf(
31	6	static::DEFAULT_GENERATED_CLASS_NAMESPACE,
32	6	crc32(uniqid((string)$crc32, true))
33		);
34	6	}
35
36	6	public function tearDown()
37		{
38	6	parent::tearDown();
39	6	$this->rmdir($this->tmpDir);
40	6	}
41
42	6	protected function rmdir($dir)
43		{
44	6	$directory = dir($dir);
45	6	while (false !== ($entry = $directory->read())) {
46	6	if (in_array($entry, ['.', '..'])) {
47	6	continue;
48		}
49
50	6	if (is_dir($dir . $entry)) {
51	6	$this->rmdir($dir . $entry . DIRECTORY_SEPARATOR);
52	6	continue;
53		}
54
55		if (is_file($dir . $entry)) {
56		unlink($dir . $entry);
57		continue;
58		}
59		}
60	6	$directory->close();
61	6	rmdir($dir);
62	6	}
63
64		protected function getTmpDir(): string
65		{
66		return $this->tmpDir;
67		}
68
69		protected function getRandomNameSpace(): string
70		{
71		return $this->tmpNamespace;
72		}
73
74		public function hydrate($class, $json, $namespace)
75		{
76		return Factory::create([
		0 ignored issues – show Documentation introduced 2016-11-20 11:51 UTC by Report Bug Copy Issue Report `array(\ApiClients\Founda...AMESPACE => $namespace)` is of type `array`, but the function expects a `object<Interop\Container\ContainerInterface>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
77		Options::NAMESPACE => $namespace,
78		])->hydrateFQCN($class, $json);
79		}
80		}
81

php-api-clients / resource-test-utilities

GitHub Access Token became invalid

Push — master ( a5d40d...abe42f )

TestCase::rmdir() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like