Atrapalo\PHPTools\Enum\EnumPHPDocTester

Complexity

Total Complexity

9

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

92.86%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
	enumClass()	0	1	?
A	staticAccessByPhpDoc()	0	15	3
A	accessByPhpDoc()	0	21	4
A	createValidEnum()	0	6	1
A	snakeCaseToCamelCase()	0	6	1

<?php

namespace Atrapalo\PHPTools\Enum;

use Atrapalo\PHPTools\Parser\PHPDocClass;

/**
 * Class EnumPHPDocTester
 * @package Atrapalo\PHPTools\Enum
 *
 * @author Guillermo González <guillermo.gonzalez@atrapalo.com>
 */
trait EnumPHPDocTester
{
    /**
     * @return string
     */
    abstract protected function enumClass(): string;

    /**
     * @test
     */
    public function staticAccessByPhpDoc()
    {
        $className = $this->enumClass();

        $staticMethods = PHPDocClass::staticMethods($className);
        if (!empty($staticMethods)) {
            foreach ($staticMethods as $staticMethod) {
                $data = call_user_func([$className, $staticMethod->name()]);

                $this->assertInstanceOf($className, $data);

It seems like assertInstanceOf() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            }
        } else {
            $this->markTestSkipped('Skipped because no static methods were found for '.get_called_class());

It seems like markTestSkipped() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * @test
     */
    public function accessByPhpDoc()
    {
        $className = $this->enumClass();

        $methods = PHPDocClass::methods($className);
        if (!empty($methods)) {

            $enum = $this->createValidEnum($className);
            $actualMethod = $this->snakeCaseToCamelCase("IS_".$enum->key());
            foreach ($methods as $method) {
                $methodName = $method->name();
                if ($methodName === $actualMethod) {
                    $this->assertTrue($enum->$methodName());

It seems like assertTrue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                } else {
                    $this->assertFalse($enum->$methodName());

It seems like assertFalse() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                }
            }
        } else {
            $this->markTestSkipped('Skipped because no methods were found for '.get_called_class());

It seems like markTestSkipped() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * @param Enum|string $className
     * @return Enum
     */
    private function createValidEnum($className)
    {
        $values = $className::toArray();

        return new $className($values[array_rand($values)]);
    }

    /**
     * @param string $string
     * @return string
     */
    private function snakeCaseToCamelCase(string $string): string
    {
        return preg_replace_callback('/_(.?)/', function($matches) {
            return ucfirst($matches[1]);
        }, strtolower($string));
    }
}