HMoragrega\PhpSpec\DataProvider\Parser\ExampleParser

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getDataProvider()	0	14	3

<?php declare(strict_types=1);

namespace HMoragrega\PhpSpec\DataProvider\Parser;

use PhpSpec\Loader\Node\ExampleNode;

class ExampleParser
{
    private const DATA_PROVIDER_REGEX = '/@dataProvider ([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)/';

    /**
     * Returns the data provider method looking for the @dataProvider annotation
     *
     * @param ExampleNode $example
     *
     * @return string|null
     */
    public function getDataProvider(ExampleNode $example): ?string
    {
        $method = $example->getFunctionReflection();

        $docComment = $method->getDocComment();
        if (false === $docComment) {
            return null;
        }

        if (!preg_match(self::DATA_PROVIDER_REGEX, $docComment, $matches)) {

It seems like $docComment can also be of type true; however, parameter $subject of preg_match() does only seem to accept string, maybe add an additional type check?

            return null;
        }

        return $matches[1];
    }
}