RuleSetsTest.php 9.5 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301
  1. <?php
  2. declare(strict_types=1);
  3. /*
  4. * This file is part of PHP CS Fixer.
  5. *
  6. * (c) Fabien Potencier <fabien@symfony.com>
  7. * Dariusz Rumiński <dariusz.ruminski@gmail.com>
  8. *
  9. * This source file is subject to the MIT license that is bundled
  10. * with this source code in the file LICENSE.
  11. */
  12. namespace PhpCsFixer\Tests\RuleSet;
  13. use PhpCsFixer\Fixer\ConfigurableFixerInterface;
  14. use PhpCsFixer\Fixer\PhpUnit\PhpUnitTargetVersion;
  15. use PhpCsFixer\RuleSet\RuleSet;
  16. use PhpCsFixer\RuleSet\RuleSets;
  17. use PhpCsFixer\Tests\Test\TestCaseUtils;
  18. use PhpCsFixer\Tests\TestCase;
  19. /**
  20. * @author Dariusz Rumiński <dariusz.ruminski@gmail.com>
  21. *
  22. * @internal
  23. *
  24. * @covers \PhpCsFixer\RuleSet\RuleSets
  25. */
  26. final class RuleSetsTest extends TestCase
  27. {
  28. public function testGetSetDefinitionNames(): void
  29. {
  30. self::assertSame(
  31. array_keys(RuleSets::getSetDefinitions()),
  32. RuleSets::getSetDefinitionNames()
  33. );
  34. }
  35. public function testGetSetDefinitions(): void
  36. {
  37. $sets = RuleSets::getSetDefinitions();
  38. foreach ($sets as $name => $set) {
  39. self::assertIsString($name);
  40. self::assertStringStartsWith('@', $name);
  41. self::assertIsArray($set->getRules());
  42. self::assertSame($set, RuleSets::getSetDefinition($name));
  43. }
  44. }
  45. public function testGetUnknownSetDefinition(): void
  46. {
  47. $name = 'Unknown';
  48. $this->expectException(\InvalidArgumentException::class);
  49. $this->expectExceptionMessageMatches(sprintf('#^Set "%s" does not exist\.$#', $name));
  50. RuleSets::getSetDefinition($name);
  51. }
  52. /**
  53. * @dataProvider provideSetDefinitionNameCases
  54. */
  55. public function testHasIntegrationTest(string $setDefinitionName): void
  56. {
  57. $setsWithoutTests = [
  58. '@PER',
  59. '@PER:risky',
  60. '@PER-CS',
  61. '@PER-CS:risky',
  62. '@PHP56Migration',
  63. '@PHP56Migration:risky',
  64. '@PHP70Migration',
  65. '@PHP70Migration:risky',
  66. '@PHP71Migration',
  67. '@PHP71Migration:risky',
  68. '@PHP73Migration',
  69. '@PHP80Migration',
  70. '@PhpCsFixer',
  71. '@PhpCsFixer:risky',
  72. '@PHPUnit48Migration',
  73. '@PHPUnit55Migration:risky',
  74. '@PHPUnit75Migration:risky',
  75. '@PHPUnit84Migration:risky',
  76. '@PHPUnit100Migration:risky',
  77. '@PSR1',
  78. ];
  79. if (\in_array($setDefinitionName, $setsWithoutTests, true)) {
  80. self::markTestIncomplete(sprintf('Set "%s" has no integration test.', $setDefinitionName));
  81. }
  82. $setDefinitionFileNamePrefix = str_replace(':', '-', $setDefinitionName);
  83. $dir = __DIR__.'/../../tests/Fixtures/Integration/set';
  84. $file = sprintf('%s/%s.test', $dir, $setDefinitionFileNamePrefix);
  85. self::assertFileExists($file);
  86. self::assertFileExists(sprintf('%s/%s.test-in.php', $dir, $setDefinitionFileNamePrefix));
  87. self::assertFileExists(sprintf('%s/%s.test-out.php', $dir, $setDefinitionFileNamePrefix));
  88. $template = '--TEST--
  89. Integration of %s.
  90. --RULESET--
  91. {"%s": true}
  92. ';
  93. self::assertStringStartsWith(
  94. sprintf($template, $setDefinitionName, $setDefinitionName),
  95. file_get_contents($file)
  96. );
  97. }
  98. /**
  99. * @dataProvider provideSetDefinitionNameCases
  100. */
  101. public function testBuildInSetDefinitionNames(string $setName): void
  102. {
  103. self::assertStringStartsWith('@', $setName);
  104. }
  105. /**
  106. * @dataProvider provideSetDefinitionNameCases
  107. */
  108. public function testSetDefinitionsAreSorted(string $setDefinitionName): void
  109. {
  110. $setDefinition = RuleSets::getSetDefinitions()[$setDefinitionName]->getRules();
  111. $sortedSetDefinition = $setDefinition;
  112. $this->sort($sortedSetDefinition);
  113. self::assertSame($sortedSetDefinition, $setDefinition, sprintf(
  114. 'Failed to assert that the set definition for "%s" is sorted by key.',
  115. $setDefinitionName
  116. ));
  117. }
  118. public function testSetDefinitionsItselfIsSorted(): void
  119. {
  120. $setDefinition = array_keys(RuleSets::getSetDefinitions());
  121. $sortedSetDefinition = $setDefinition;
  122. natsort($sortedSetDefinition);
  123. self::assertSame($sortedSetDefinition, $setDefinition);
  124. }
  125. public static function provideSetDefinitionNameCases(): iterable
  126. {
  127. $setDefinitionNames = RuleSets::getSetDefinitionNames();
  128. return array_map(static fn (string $setDefinitionName): array => [$setDefinitionName], $setDefinitionNames);
  129. }
  130. /**
  131. * @dataProvider providePHPUnitMigrationTargetVersionsCases
  132. */
  133. public function testPHPUnitMigrationTargetVersions(string $setName): void
  134. {
  135. $ruleSet = new RuleSet([$setName => true]);
  136. foreach ($ruleSet->getRules() as $ruleName => $ruleConfig) {
  137. $targetVersion = $ruleConfig['target'] ?? $this->getDefaultPHPUnitTargetOfRule($ruleName);
  138. if (null === $targetVersion) {
  139. // fixer does not have "target" option
  140. $this->addToAssertionCount(1);
  141. continue;
  142. }
  143. self::assertPHPUnitVersionIsLargestAllowed($setName, $ruleName, $targetVersion);
  144. }
  145. }
  146. /**
  147. * @return iterable<array{string}>
  148. */
  149. public static function providePHPUnitMigrationTargetVersionsCases(): iterable
  150. {
  151. $setDefinitionNames = RuleSets::getSetDefinitionNames();
  152. $setDefinitionPHPUnitMigrationNames = array_filter($setDefinitionNames, static fn (string $setDefinitionName): bool => 1 === preg_match('/^@PHPUnit\d+Migration:risky$/', $setDefinitionName));
  153. return array_map(static fn (string $setDefinitionName): array => [$setDefinitionName], $setDefinitionPHPUnitMigrationNames);
  154. }
  155. private static function assertPHPUnitVersionIsLargestAllowed(string $setName, string $ruleName, string $actualTargetVersion): void
  156. {
  157. $maximumVersionForRuleset = preg_replace('/^@PHPUnit(\d+)(\d)Migration:risky$/', '$1.$2', $setName);
  158. $fixer = TestCaseUtils::getFixerByName($ruleName);
  159. self::assertInstanceOf(ConfigurableFixerInterface::class, $fixer, sprintf('The fixer "%s" shall be configurable.', $fixer->getName()));
  160. foreach ($fixer->getConfigurationDefinition()->getOptions() as $option) {
  161. if ('target' === $option->getName()) {
  162. $allowedVersionsForFixer = array_diff($option->getAllowedValues(), [PhpUnitTargetVersion::VERSION_NEWEST]);
  163. break;
  164. }
  165. }
  166. if (!isset($allowedVersionsForFixer)) {
  167. throw new \Exception(sprintf('The fixer "%s" does not have option "target".', $fixer->getName()));
  168. }
  169. /** @var list<string> $allowedVersionsForRuleset */
  170. $allowedVersionsForRuleset = array_filter(
  171. $allowedVersionsForFixer,
  172. static fn (string $version): bool => version_compare($maximumVersionForRuleset, $version) >= 0
  173. );
  174. self::assertTrue(\in_array($actualTargetVersion, $allowedVersionsForRuleset, true), sprintf(
  175. 'Rule "%s" (in rule set "%s") has target "%s", but the rule set is not allowing it (allowed are only "%s")',
  176. $fixer->getName(),
  177. $setName,
  178. $actualTargetVersion,
  179. implode('", "', $allowedVersionsForRuleset)
  180. ));
  181. rsort($allowedVersionsForRuleset);
  182. $maximumAllowedVersionForRuleset = reset($allowedVersionsForRuleset);
  183. self::assertSame($maximumAllowedVersionForRuleset, $actualTargetVersion, sprintf(
  184. 'Rule "%s" (in rule set "%s") has target "%s", but there is higher available target "%s"',
  185. $fixer->getName(),
  186. $setName,
  187. $actualTargetVersion,
  188. $maximumAllowedVersionForRuleset
  189. ));
  190. }
  191. /**
  192. * Sorts an array of rule set definitions recursively.
  193. *
  194. * Sometimes keys are all string, sometimes they are integers - we need to account for that.
  195. *
  196. * @param array<array-key, mixed> $data
  197. */
  198. private function sort(array &$data): void
  199. {
  200. $this->doSort($data, '');
  201. }
  202. /**
  203. * @param array<array-key, mixed> $data
  204. */
  205. private function doSort(array &$data, string $path): void
  206. {
  207. if ('ordered_imports.imports_order' === $path) { // order matters
  208. return;
  209. }
  210. $keys = array_keys($data);
  211. if ($this->allInteger($keys)) {
  212. sort($data);
  213. } else {
  214. ksort($data);
  215. }
  216. foreach ($data as $key => $value) {
  217. if (\is_array($value)) {
  218. $this->doSort(
  219. $data[$key],
  220. $path.('' !== $path ? '.' : '').$key
  221. );
  222. }
  223. }
  224. }
  225. /**
  226. * @param array<array-key, mixed> $values
  227. */
  228. private function allInteger(array $values): bool
  229. {
  230. foreach ($values as $value) {
  231. if (!\is_int($value)) {
  232. return false;
  233. }
  234. }
  235. return true;
  236. }
  237. private function getDefaultPHPUnitTargetOfRule(string $ruleName): ?string
  238. {
  239. $targetVersion = null;
  240. $fixer = TestCaseUtils::getFixerByName($ruleName);
  241. if ($fixer instanceof ConfigurableFixerInterface) {
  242. foreach ($fixer->getConfigurationDefinition()->getOptions() as $option) {
  243. if ('target' === $option->getName()) {
  244. $targetVersion = $option->getDefault();
  245. break;
  246. }
  247. }
  248. }
  249. return $targetVersion;
  250. }
  251. }