RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\FunctionBased\RequiredParameterAmountIncreasedTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	127
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	testDiffs()	0	13	1
B	functionsToBeTested()	0	103	1

<?php

declare(strict_types=1);

namespace RoaveTest\ApiCompare\Comparator\BackwardsCompatibility\FunctionBased;

use PHPUnit\Framework\TestCase;
use Roave\ApiCompare\Change;
use Roave\ApiCompare\Comparator\BackwardsCompatibility\FunctionBased\RequiredParameterAmountIncreased;
use Roave\BetterReflection\BetterReflection;
use Roave\BetterReflection\Reflection\ReflectionFunctionAbstract;
use Roave\BetterReflection\Reflector\ClassReflector;
use Roave\BetterReflection\Reflector\FunctionReflector;
use Roave\BetterReflection\SourceLocator\Type\StringSourceLocator;
use function array_map;
use function iterator_to_array;

/**
 * @covers \Roave\ApiCompare\Comparator\BackwardsCompatibility\FunctionBased\RequiredParameterAmountIncreased
 */
final class RequiredParameterAmountIncreasedTest extends TestCase
{
    /**
     * @dataProvider functionsToBeTested
     *
     * @param string[] $expectedMessages
     */
    public function testDiffs(
        ReflectionFunctionAbstract $fromFunction,
        ReflectionFunctionAbstract $toFunction,
        array $expectedMessages
    ) : void {
        $changes = (new RequiredParameterAmountIncreased())
            ->compare($fromFunction, $toFunction);

        self::assertSame(
            $expectedMessages,
            array_map(function (Change $change) : string {
                return $change->__toString();
            }, iterator_to_array($changes))
        );
    }

    /** @return (string[]|ReflectionFunctionAbstract)[][] */
    public function functionsToBeTested() : array
    {
        $astLocator = (new BetterReflection())->astLocator();

        $fromLocator = new StringSourceLocator(
            <<<'PHP'
<?php

namespace {
   function parametersIncreased($a, $b, $c) {}
   function parametersReduced($a, $b, $c) {}
   function parameterNamesChanged($a, $b, $c) {}
   function optionalParameterAdded($a, $b, $c) {}
   function noParametersToOneParameter() {}
   function variadicParameterAdded($a, $b) {}
   function variadicParameterMoved($a, ...$b) {}
   function optionalParameterAddedInBetween($a, $b, $c) {}
   function untouched($a, $b, $c) {}
}

namespace N1 {
   class C {
       static function changed1($a, $b, $c) {}
       function changed2($a, $b, $c) {}
   }
}
PHP
            ,
            $astLocator
        );

        $toLocator = new StringSourceLocator(
            <<<'PHP'
<?php

namespace {
   function parametersIncreased($a, $b, $c, $d) {}
   function parametersReduced($a, $b) {}
   function parameterNamesChanged($d, $e, $f) {}
   function optionalParameterAdded($a, $b, $c, $d = null) {}
   function noParametersToOneParameter($a) {}
   function variadicParameterAdded($a, $b, ...$c) {}
   function variadicParameterMoved($a, $b, ...$b) {}
   function optionalParameterAddedInBetween($a, $b = null, $c, $d) {}
   function untouched($a, $b, $c) {}
}

namespace N1 {
   class C {
       static function changed1($a, $b, $c, $d) {}
       function changed2($a, $b, $c, $d) {}
   }
}
PHP
            ,
            $astLocator
        );

        $fromClassReflector = new ClassReflector($fromLocator);
        $toClassReflector   = new ClassReflector($toLocator);
        $fromReflector      = new FunctionReflector($fromLocator, $fromClassReflector);
        $toReflector        = new FunctionReflector($toLocator, $toClassReflector);

        $functions = [
            'parametersIncreased'             => ['[BC] CHANGED: The number of required arguments for parametersIncreased() increased from 3 to 4'],
            'parametersReduced'               => [],
            'parameterNamesChanged'           => [],
            'optionalParameterAdded'          => [],
            'noParametersToOneParameter'      => ['[BC] CHANGED: The number of required arguments for noParametersToOneParameter() increased from 0 to 1'],
            'variadicParameterAdded'          => [],
            'variadicParameterMoved'          => ['[BC] CHANGED: The number of required arguments for variadicParameterMoved() increased from 1 to 2'],
            'optionalParameterAddedInBetween' => ['[BC] CHANGED: The number of required arguments for optionalParameterAddedInBetween() increased from 3 to 4'],
            'untouched'                       => [],
        ];

        return array_merge(
            array_combine(
                array_keys($functions),
                array_map(
                    function (string $function, array $errorMessages) use ($fromReflector, $toReflector) : array {
                        return [
                            $fromReflector->reflect($function),
                            $toReflector->reflect($function),
                            $errorMessages,
                        ];
                    },
                    array_keys($functions),
                    $functions
                )
            ),
            [
                'N1\C::changed1' => [
                    $fromClassReflector->reflect('N1\C')->getMethod('changed1'),
                    $toClassReflector->reflect('N1\C')->getMethod('changed1'),
                    [
                        '[BC] CHANGED: The number of required arguments for N1\C::changed1() increased from 3 to 4',
                    ],
                ],
                'N1\C#changed2'  => [
                    $fromClassReflector->reflect('N1\C')->getMethod('changed2'),
                    $toClassReflector->reflect('N1\C')->getMethod('changed2'),
                    [
                        '[BC] CHANGED: The number of required arguments for N1\C#changed2() increased from 3 to 4',
                    ],
                ],
            ]
        );
    }
}