ApiClients\Tools\ResourceTestUtilities\AbstractResourceTest

Complexity

Total Complexity

29

Size/Duplication

Total Lines	216
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	8

Test Coverage

Coverage

0%

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
	getSyncAsync()	0	1	?
	getClass()	0	1	?
	getNamespace()	0	1	?
A	provideProperties()	0	4	1
A	providePropertiesIncompatible()	0	4	1
B	providePropertiesGenerator()	0	44	9
A	testProperties()	0	19	2
A	testPropertiesIncompatible()	0	29	4
A	testInterface()	0	9	1
A	generateTypeValues()	0	33	5
A	getDocBlock()	0	8	2
A	getAnnotation()	0	32	4

<?php
declare(strict_types=1);

namespace ApiClients\Tools\ResourceTestUtilities;

use ApiClients\Foundation\Hydrator\Annotation\Rename;
use ApiClients\Foundation\Hydrator\AnnotationInterface;
use ApiClients\Foundation\Resource\ResourceInterface;
use Doctrine\Common\Annotations\AnnotationReader;
use Doctrine\Common\Inflector\Inflector;
use phpDocumentor\Reflection\DocBlock;
use phpDocumentor\Reflection\DocBlockFactory;
use ReflectionClass;
use ReflectionProperty;
use TypeError;

abstract class AbstractResourceTest extends TestCase
{
    abstract public function getSyncAsync(): string;

    abstract public function getClass(): string;

    abstract public function getNamespace(): string;

    public function provideProperties(): array
    {
        return [[$this->providePropertiesGenerator('compatible')]];
    }

    public function providePropertiesIncompatible(): array
    {
        return [[$this->providePropertiesGenerator('incompatible')]];
    }

    public function providePropertiesGenerator(string $typeMethod): array
    {
        $yield = [];
        $class = new ReflectionClass($this->getClass());

        $jsonTemplate = [];
        foreach ($class->getProperties() as $property) {
            $key = $property->getName();
            $renamed = self::GetAnnotation($property->getDeclaringClass()->getName(), Rename::class);

Consider using $property->class. There is an issue with getName() and APC-enabled PHP versions.

            if ($renamed instanceof Rename && $renamed->has($key)) {
                $key = $renamed->get($key);
            }

            $jsonTemplate[$key] = '';
        }

        foreach ($class->getProperties() as $property) {
            $method = Inflector::camelize($property->getName());
            $docBlock = $this->getDocBlock($property->getDocComment());

            $varTag = $docBlock->getTagsByName('var');
            if (count($varTag) !== 1) {
                continue;
            }

            $varTag = $varTag[0];
            $scalar = (string)$varTag->getType();
            if ($scalar === '') {
                continue;
            }

            if (!Types::has($scalar)) {
                continue;
            }

            $type = Types::get($scalar);
            foreach ($this->generateTypeValues($type, $property, $method, $typeMethod, $jsonTemplate) as $item) {
                $yield[] = $item;
            }
        }

        return $yield;
    }

    /**
     * @dataProvider provideProperties
     * @param mixed $args
     */
    public function testProperties($args)
    {
        foreach ($args as $arg) {
            list($property, $method, $type, $json, $value) = $arg;

The assignment to $property is unused. Consider omitting it like so list($first,,$third).

            $class = $this->getClass();
            $resource = $this->hydrate(
                str_replace(
                    $this->getNamespace(),
                    $this->getNamespace() . '\\Async',
                    $class
                ),
                $json,
                $this->getNamespace(),
                'Async'
            );
            $this->assertSame($value, $resource->{$method}());
            $this->assertInternalType($type->scalar(), $resource->{$method}());
        }
    }

    /**
     * @dataProvider providePropertiesIncompatible
     * @param mixed $args
     */
    public function testPropertiesIncompatible($args)
    {
        foreach ($args as $arg) {
            list($property, $method, $type, $json, $value) = $arg;

The assignment to $property is unused. Consider omitting it like so list($first,,$third).

The assignment to $type is unused. Consider omitting it like so list($first,,$third).

            try {
                $class = $this->getClass();
                $resource = $this->hydrate(
                    str_replace(
                        $this->getNamespace(),
                        $this->getNamespace() . '\\Async',
                        $class
                    ),
                    $json,
                    $this->getNamespace(),
                    'Async'
                );

                if ($value !== $resource->{$method}()) {
                    throw new TypeError();
                }
            } catch (\Throwable $t) {
                $this->assertTrue(true);
                continue;
            }

            $this->fail('We should not reach this');
        }
    }

    public function testInterface()
    {
        $this->assertTrue(
            is_subclass_of(

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \ApiClients\Foundation\Resource\ResourceInterface::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

                $this->getClass(),
                ResourceInterface::class
            )
        );
    }

    protected function generateTypeValues(
        Type $type,
        ReflectionProperty $property,
        string $method,
        string $typeMethod,
        array $jsonTemplate
    ): array {
        $yield = [];
        $json = $jsonTemplate;
        foreach ($type->$typeMethod() as $typeClass) {
            $methodType = Types::get(constant($typeClass . '::SCALAR'));
            foreach ($methodType->generate(3) as $value) {
                $key = $property->getName();

                $renamed = self::GetAnnotation($property->getDeclaringClass()->getName(), Rename::class);

Consider using $property->class. There is an issue with getName() and APC-enabled PHP versions.

                if ($renamed instanceof Rename && $renamed->has($key)) {
                    $key = $renamed->get($key);
                }

                $json[$key] = $value;

                $yield[] = [
                    $property->getName(), // Name of the property to assign data to
                    $method,              // Method to call verifying that data
                    $type,                // The different types of data associated with this field
                    $json,                // JSON to use during testing
                    $value,               // Value to check against
                ];
            }
        }

        return $yield;
    }

    /**
     * @param $docBlockContents
     * @return DocBlock
     */
    protected function getDocBlock(string $docBlockContents): DocBlock
    {
        if (class_exists('phpDocumentor\Reflection\DocBlockFactory')) {
            return DocBlockFactory::createInstance()->create($docBlockContents);
        }

        return new DocBlock($docBlockContents);
    }

    /**
     * @param  string                   $class
     * @param  string                   $annotationClass
     * @return null|AnnotationInterface
     */
    protected static function getAnnotation(string $class, string $annotationClass)
    {
        $annotationReader = new AnnotationReader();

        $annotation = $annotationReader
            ->getClassAnnotation(
                new ReflectionClass($class),
                $annotationClass
            )
        ;

        if ($annotation === null) {
            return null;
        }

        if (get_class($annotation) === $annotationClass) {
            return $annotation;
        }

        $class = get_parent_class($class);

        if ($class === false) {
            return null;
        }

        return $annotationReader
            ->getClassAnnotation(
                new ReflectionClass($class),
                $annotationClass
            )
            ;
    }
}