Assertion::reflectionGenerator() - Code Metrics - Inspection of "Moved all BC compliance and `Changes` instance to..." - Roave/BackwardCompatibilityCheck - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#108)

by Marco

created 2018-09-02 19:00 UTC

Assertion::reflectionGenerator() A

↳ Parent: Assertion

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	5
nc	2
nop	0
dl	0
loc	10
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace RoaveTest\BackwardCompatibility;

use Generator;
use PHPUnit\Framework\Assert;
use ReflectionProperty;
use Roave\BackwardCompatibility\Changes;

abstract class Assertion
{
    /** @var ReflectionProperty|null */
    private static $reflectionGenerator;

    final private function __construct()
    {
    }

    public static function assertChangesEqual(
        Changes $expected,
        Changes $actual,
        string $message = ''
    ) : void {
        Assert::assertInstanceOf(
            Generator::class,
            self::reflectionGenerator()->getValue($actual),
            'Generator must NOT be exhausted'
        );
        // Forces eager initialisation of the `Changes` instances, allowing us to compare them by value
        Assert::assertSame(count($expected), count($actual));
        Assert::assertEquals($expected, $actual, $message);
    }

    private static function reflectionGenerator() : ReflectionProperty {
        if (self::$reflectionGenerator) {
            return self::$reflectionGenerator;
        }

        self::$reflectionGenerator = new ReflectionProperty(Changes::class, 'generator');

        self::$reflectionGenerator->setAccessible(true);

        return self::$reflectionGenerator;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace RoaveTest\BackwardCompatibility;
6
7			use Generator;
8			use PHPUnit\Framework\Assert;
9			use ReflectionProperty;
10			use Roave\BackwardCompatibility\Changes;
11
12			abstract class Assertion
13			{
14			/** @var ReflectionProperty\|null */
15			private static $reflectionGenerator;
16
17			final private function __construct()
18			{
19			}
20
21			public static function assertChangesEqual(
22			Changes $expected,
23			Changes $actual,
24			string $message = ''
25			) : void {
26			Assert::assertInstanceOf(
27			Generator::class,
28			self::reflectionGenerator()->getValue($actual),
29			'Generator must NOT be exhausted'
30			);
31			// Forces eager initialisation of the `Changes` instances, allowing us to compare them by value
32			Assert::assertSame(count($expected), count($actual));
33			Assert::assertEquals($expected, $actual, $message);
34			}
35
36			private static function reflectionGenerator() : ReflectionProperty {
37			if (self::$reflectionGenerator) {
38			return self::$reflectionGenerator;
39			}
40
41			self::$reflectionGenerator = new ReflectionProperty(Changes::class, 'generator');
42
43			self::$reflectionGenerator->setAccessible(true);
44
45			return self::$reflectionGenerator;
46			}
47			}
48

Roave / BackwardCompatibilityCheck

Pull Request — master (#108)

Assertion::reflectionGenerator() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like