* Dariusz Rumiński * * This source file is subject to the MIT license that is bundled * with this source code in the file LICENSE. */ namespace PhpCsFixer\Tests\Test; use PhpCsFixer\AbstractFixer; use PhpCsFixer\AbstractProxyFixer; use PhpCsFixer\Fixer\ConfigurableFixerInterface; use PhpCsFixer\Fixer\DeprecatedFixerInterface; use PhpCsFixer\Fixer\Whitespace\SingleBlankLineAtEofFixer; use PhpCsFixer\FixerConfiguration\FixerOptionInterface; use PhpCsFixer\FixerDefinition\CodeSampleInterface; use PhpCsFixer\FixerDefinition\FileSpecificCodeSampleInterface; use PhpCsFixer\FixerDefinition\VersionSpecificCodeSampleInterface; use PhpCsFixer\Linter\CachingLinter; use PhpCsFixer\Linter\Linter; use PhpCsFixer\Linter\LinterInterface; use PhpCsFixer\Linter\ProcessLinter; use PhpCsFixer\PhpunitConstraintIsIdenticalString\Constraint\IsIdenticalString; use PhpCsFixer\Preg; use PhpCsFixer\StdinFileInfo; use PhpCsFixer\Tests\Test\Assert\AssertTokensTrait; use PhpCsFixer\Tests\TestCase; use PhpCsFixer\Tokenizer\Token; use PhpCsFixer\Tokenizer\Tokens; /** * @author Dariusz Rumiński * * @internal */ abstract class AbstractFixerTestCase extends TestCase { use AssertTokensTrait; /** * @var null|LinterInterface */ protected $linter; /** * @var null|AbstractFixer */ protected $fixer; /** * do not modify this structure without prior discussion. * * @var array> */ private array $allowedRequiredOptions = [ 'header_comment' => ['header' => true], ]; /** * do not modify this structure without prior discussion. * * @var array */ private array $allowedFixersWithoutDefaultCodeSample = [ 'general_phpdoc_annotation_remove' => true, 'general_phpdoc_tag_rename' => true, ]; protected function setUp(): void { parent::setUp(); $this->linter = $this->getLinter(); $this->fixer = $this->createFixer(); } protected function tearDown(): void { parent::tearDown(); $this->linter = null; $this->fixer = null; } final public function testIsRisky(): void { if ($this->fixer->isRisky()) { self::assertValidDescription($this->fixer->getName(), 'risky description', $this->fixer->getDefinition()->getRiskyDescription()); } else { self::assertNull($this->fixer->getDefinition()->getRiskyDescription(), sprintf('[%s] Fixer is not risky so no description of it expected.', $this->fixer->getName())); } if ($this->fixer instanceof AbstractProxyFixer) { return; } $reflection = new \ReflectionMethod($this->fixer, 'isRisky'); // If fixer is not risky then the method `isRisky` from `AbstractFixer` must be used self::assertSame( !$this->fixer->isRisky(), AbstractFixer::class === $reflection->getDeclaringClass()->getName() ); } final public function testFixerDefinitions(): void { $fixerName = $this->fixer->getName(); $definition = $this->fixer->getDefinition(); $fixerIsConfigurable = $this->fixer instanceof ConfigurableFixerInterface; self::assertValidDescription($fixerName, 'summary', $definition->getSummary()); if (null !== $definition->getDescription()) { self::assertValidDescription($fixerName, 'description', $definition->getDescription()); } $samples = $definition->getCodeSamples(); self::assertNotEmpty($samples, sprintf('[%s] Code samples are required.', $fixerName)); $configSamplesProvided = []; $dummyFileInfo = new StdinFileInfo(); foreach ($samples as $sampleCounter => $sample) { self::assertInstanceOf(CodeSampleInterface::class, $sample, sprintf('[%s] Sample #%d', $fixerName, $sampleCounter)); self::assertIsInt($sampleCounter); $code = $sample->getCode(); self::assertNotEmpty($code, sprintf('[%s] Sample #%d', $fixerName, $sampleCounter)); self::assertStringStartsNotWith("\n", $code, sprintf('[%s] Sample #%d must not start with linebreak', $fixerName, $sampleCounter)); if (!$this->fixer instanceof SingleBlankLineAtEofFixer) { self::assertStringEndsWith("\n", $code, sprintf('[%s] Sample #%d must end with linebreak', $fixerName, $sampleCounter)); } $config = $sample->getConfiguration(); if (null !== $config) { self::assertTrue($fixerIsConfigurable, sprintf('[%s] Sample #%d has configuration, but the fixer is not configurable.', $fixerName, $sampleCounter)); $configSamplesProvided[$sampleCounter] = $config; } elseif ($fixerIsConfigurable) { if (!$sample instanceof VersionSpecificCodeSampleInterface) { self::assertArrayNotHasKey('default', $configSamplesProvided, sprintf('[%s] Multiple non-versioned samples with default configuration.', $fixerName)); } $configSamplesProvided['default'] = true; } if ($sample instanceof VersionSpecificCodeSampleInterface) { $supportedPhpVersions = [7_04_00, 8_00_00, 8_01_00, 8_02_00]; $hasSuitableSupportedVersion = false; foreach ($supportedPhpVersions as $version) { if ($sample->isSuitableFor($version)) { $hasSuitableSupportedVersion = true; } } self::assertTrue($hasSuitableSupportedVersion, 'Version specific code sample must be suitable for at least 1 supported PHP version.'); $hasUnsuitableSupportedVersion = false; foreach ($supportedPhpVersions as $version) { if (!$sample->isSuitableFor($version)) { $hasUnsuitableSupportedVersion = true; } } self::assertTrue($hasUnsuitableSupportedVersion, 'Version specific code sample must be unsuitable for at least 1 supported PHP version.'); if (!$sample->isSuitableFor(\PHP_VERSION_ID)) { continue; } } if ($fixerIsConfigurable) { // always re-configure as the fixer might have been configured with diff. configuration form previous sample $this->fixer->configure($config ?? []); } Tokens::clearCache(); $tokens = Tokens::fromCode($code); $this->fixer->fix( $sample instanceof FileSpecificCodeSampleInterface ? $sample->getSplFileInfo() : $dummyFileInfo, $tokens ); self::assertTrue($tokens->isChanged(), sprintf('[%s] Sample #%d is not changed during fixing.', $fixerName, $sampleCounter)); $duplicatedCodeSample = array_search( $sample, \array_slice($samples, 0, $sampleCounter), true ); self::assertFalse( $duplicatedCodeSample, sprintf('[%s] Sample #%d duplicates #%d.', $fixerName, $sampleCounter, $duplicatedCodeSample) ); } if ($fixerIsConfigurable) { if (isset($configSamplesProvided['default'])) { reset($configSamplesProvided); self::assertSame('default', key($configSamplesProvided), sprintf('[%s] First sample must be for the default configuration.', $fixerName)); } elseif (!isset($this->allowedFixersWithoutDefaultCodeSample[$fixerName])) { self::assertArrayHasKey($fixerName, $this->allowedRequiredOptions, sprintf('[%s] Has no sample for default configuration.', $fixerName)); } if (\count($configSamplesProvided) < 2) { self::fail(sprintf('[%s] Configurable fixer only provides a default configuration sample and none for its configuration options.', $fixerName)); } $options = $this->fixer->getConfigurationDefinition()->getOptions(); foreach ($options as $option) { self::assertMatchesRegularExpression('/^[a-z_]+[a-z]$/', $option->getName(), sprintf('[%s] Option %s is not snake_case.', $fixerName, $option->getName())); self::assertMatchesRegularExpression( '/^[A-Z].+\.$/s', $option->getDescription(), sprintf('[%s] Description of option "%s" must start with capital letter and end with dot.', $fixerName, $option->getName()) ); } } self::assertIsInt($this->fixer->getPriority()); } final public function testFixersAreFinal(): void { $reflection = new \ReflectionClass($this->fixer); self::assertTrue( $reflection->isFinal(), sprintf('Fixer "%s" must be declared "final".', $this->fixer->getName()) ); } final public function testDeprecatedFixersHaveCorrectSummary(): void { self::assertStringNotContainsString( 'DEPRECATED', $this->fixer->getDefinition()->getSummary(), 'Fixer cannot contain word "DEPRECATED" in summary' ); $reflection = new \ReflectionClass($this->fixer); $comment = $reflection->getDocComment(); if ($this->fixer instanceof DeprecatedFixerInterface) { self::assertIsString($comment, sprintf('Missing class PHPDoc for deprecated fixer "%s".', $this->fixer->getName())); self::assertStringContainsString('@deprecated', $comment); } elseif (\is_string($comment)) { self::assertStringNotContainsString('@deprecated', $comment); } } /** * Blur filter that find candidate fixer for performance optimization to use only `insertSlices` instead of multiple `insertAt` if there is no other collection manipulation. */ public function testFixerUseInsertSlicesWhenOnlyInsertionsArePerformed(): void { $reflection = new \ReflectionClass($this->fixer); $tokens = Tokens::fromCode(file_get_contents($reflection->getFileName())); $sequences = $this->findAllTokenSequences($tokens, [[T_VARIABLE, '$tokens'], [T_OBJECT_OPERATOR], [T_STRING]]); $usedMethods = array_unique(array_map(static function (array $sequence): string { $last = end($sequence); return $last->getContent(); }, $sequences)); // if there is no `insertAt`, it's not a candidate if (!\in_array('insertAt', $usedMethods, true)) { $this->expectNotToPerformAssertions(); return; } $usedMethods = array_filter($usedMethods, static fn (string $method): bool => !Preg::match('/^(count|find|generate|get|is|rewind)/', $method)); $allowedMethods = ['insertAt']; $nonAllowedMethods = array_diff($usedMethods, $allowedMethods); if ([] === $nonAllowedMethods) { $fixerName = $this->fixer->getName(); if (\in_array($fixerName, [ // DO NOT add anything to this list at ease, align with core contributors whether it makes sense to insert tokens individually or by bulk for your case. // The original list of the fixers being exceptions and insert tokens individually came from legacy reasons when it was the only available methods to insert tokens. 'blank_line_after_namespace', 'blank_line_after_opening_tag', 'blank_line_before_statement', 'class_attributes_separation', 'date_time_immutable', 'declare_strict_types', 'doctrine_annotation_braces', 'doctrine_annotation_spaces', 'final_internal_class', 'final_public_method_for_abstract_class', 'function_typehint_space', 'heredoc_indentation', 'method_chaining_indentation', 'native_constant_invocation', 'new_with_braces', 'no_short_echo_tag', 'not_operator_with_space', 'not_operator_with_successor_space', 'php_unit_internal_class', 'php_unit_no_expectation_annotation', 'php_unit_set_up_tear_down_visibility', 'php_unit_size_class', 'php_unit_test_annotation', 'php_unit_test_class_requires_covers', 'phpdoc_to_param_type', 'phpdoc_to_property_type', 'phpdoc_to_return_type', 'random_api_migration', 'semicolon_after_instruction', 'single_line_after_imports', 'static_lambda', 'strict_param', 'void_return', ], true)) { self::markTestIncomplete(sprintf('Fixer "%s" may be optimized to use `Tokens::insertSlices` instead of `%s`, please help and optimize it.', $fixerName, implode(', ', $allowedMethods))); } self::fail(sprintf('Fixer "%s" shall be optimized to use `Tokens::insertSlices` instead of `%s`.', $fixerName, implode(', ', $allowedMethods))); } $this->addToAssertionCount(1); } final public function testFixerConfigurationDefinitions(): void { if (!$this->fixer instanceof ConfigurableFixerInterface) { $this->expectNotToPerformAssertions(); // not applied to the fixer without configuration return; } $configurationDefinition = $this->fixer->getConfigurationDefinition(); foreach ($configurationDefinition->getOptions() as $option) { self::assertInstanceOf(FixerOptionInterface::class, $option); self::assertNotEmpty($option->getDescription()); self::assertSame( !isset($this->allowedRequiredOptions[$this->fixer->getName()][$option->getName()]), $option->hasDefault(), sprintf( $option->hasDefault() ? 'Option `%s` of fixer `%s` is wrongly listed in `$allowedRequiredOptions` structure, as it is not required. If you just changed that option to not be required anymore, please adjust mentioned structure.' : 'Option `%s` of fixer `%s` shall not be required. If you want to introduce new required option please adjust `$allowedRequiredOptions` structure.', $option->getName(), $this->fixer->getName() ) ); self::assertStringNotContainsString( 'DEPRECATED', $option->getDescription(), 'Option description cannot contain word "DEPRECATED"' ); } } protected function createFixer(): AbstractFixer { $fixerClassName = preg_replace('/^(PhpCsFixer)\\\\Tests(\\\\.+)Test$/', '$1$2', static::class); return new $fixerClassName(); } final protected static function getTestFile(string $filename = __FILE__): \SplFileInfo { static $files = []; return $files[$filename] ?? $files[$filename] = new \SplFileInfo($filename); } /** * Tests if a fixer fixes a given string to match the expected result. * * It is used both if you want to test if something is fixed or if it is not touched by the fixer. * It also makes sure that the expected output does not change when run through the fixer. That means that you * do not need two test cases like [$expected] and [$expected, $input] (where $expected is the same in both cases) * as the latter covers both of them. * This method throws an exception if $expected and $input are equal to prevent test cases that accidentally do * not test anything. * * @param string $expected The expected fixer output * @param null|string $input The fixer input, or null if it should intentionally be equal to the output * @param null|\SplFileInfo $file The file to fix, or null if unneeded */ protected function doTest(string $expected, ?string $input = null, ?\SplFileInfo $file = null): void { if ($expected === $input) { throw new \InvalidArgumentException('Input parameter must not be equal to expected parameter.'); } $file ??= self::getTestFile(); $fileIsSupported = $this->fixer->supports($file); if (null !== $input) { self::assertNull($this->lintSource($input)); Tokens::clearCache(); $tokens = Tokens::fromCode($input); if ($fileIsSupported) { self::assertTrue($this->fixer->isCandidate($tokens), 'Fixer must be a candidate for input code.'); self::assertFalse($tokens->isChanged(), 'Fixer must not touch Tokens on candidate check.'); $this->fixer->fix($file, $tokens); } self::assertThat( $tokens->generateCode(), new IsIdenticalString($expected), 'Code build on input code must match expected code.' ); self::assertTrue($tokens->isChanged(), 'Tokens collection built on input code must be marked as changed after fixing.'); $tokens->clearEmptyTokens(); self::assertSameSize( $tokens, array_unique(array_map(static fn (Token $token): string => spl_object_hash($token), $tokens->toArray())), 'Token items inside Tokens collection must be unique.' ); Tokens::clearCache(); $expectedTokens = Tokens::fromCode($expected); self::assertTokens($expectedTokens, $tokens); } self::assertNull($this->lintSource($expected)); Tokens::clearCache(); $tokens = Tokens::fromCode($expected); if ($fileIsSupported) { $this->fixer->fix($file, $tokens); } self::assertThat( $tokens->generateCode(), new IsIdenticalString($expected), 'Code build on expected code must not change.' ); self::assertFalse($tokens->isChanged(), 'Tokens collection built on expected code must not be marked as changed after fixing.'); } protected function lintSource(string $source): ?string { try { $this->linter->lintSource($source)->check(); } catch (\Exception $e) { return $e->getMessage()."\n\nSource:\n{$source}"; } return null; } protected static function assertCorrectCasing(string $needle, string $haystack, string $message): void { self::assertSame(substr_count(strtolower($haystack), strtolower($needle)), substr_count($haystack, $needle), $message); } private function getLinter(): LinterInterface { static $linter = null; if (null === $linter) { $linter = new CachingLinter( getenv('FAST_LINT_TEST_CASES') ? new Linter() : new ProcessLinter() ); } return $linter; } private static function assertValidDescription(string $fixerName, string $descriptionType, string $description): void { self::assertMatchesRegularExpression('/^[A-Z`].+\.$/s', $description, sprintf('[%s] The %s must start with capital letter or a ` and end with dot.', $fixerName, $descriptionType)); self::assertStringNotContainsString('phpdocs', $description, sprintf('[%s] `PHPDoc` must not be in the plural in %s.', $fixerName, $descriptionType)); self::assertCorrectCasing($description, 'PHPDoc', sprintf('[%s] `PHPDoc` must be in correct casing in %s.', $fixerName, $descriptionType)); self::assertCorrectCasing($description, 'PHPUnit', sprintf('[%s] `PHPUnit` must be in correct casing in %s.', $fixerName, $descriptionType)); self::assertFalse(strpos($descriptionType, '``'), sprintf('[%s] The %s must no contain sequential backticks.', $fixerName, $descriptionType)); } /** * @param list $sequence * * @return list> */ private function findAllTokenSequences(Tokens $tokens, array $sequence): array { $lastIndex = 0; $sequences = []; while ($found = $tokens->findSequence($sequence, $lastIndex)) { $keys = array_keys($found); $sequences[] = $found; $lastIndex = $keys[2]; } return $sequences; } }