Disable bugged rules

Author: GromNaN

rector.php

@@ -1,6 +1,8 @@
 <?php
 
 use Rector\Config\RectorConfig;
+use Rector\Php56\Rector\FunctionLike\AddDefaultValueForUndefinedVariableRector;
+use Rector\Php71\Rector\FuncCall\RemoveExtraParametersRector;
 use Rector\Set\ValueObject\LevelSetList;
 
 return static function (RectorConfig $rectorConfig): void {
@@ -11,6 +13,16 @@
         __DIR__ . '/tools',
     ]);
 
-    // define sets of rules
     $rectorConfig->sets([LevelSetList::UP_TO_PHP_72]);
+
+    $rectorConfig->skip([
+        // Falsely detect unassigned variables in code paths stopped by PHPUnit\Framework\Assert::markTestSkipped()
+        AddDefaultValueForUndefinedVariableRector::class => [
+            __DIR__ . '/tests/',
+        ],
+        // @see https://github.com/phpstan/phpstan-src/pull/2429
+        RemoveExtraParametersRector::class => [
+            __DIR__ . '/src/Operation/',
+        ],
+    ]);
 };

tests/SpecTests/ClientSideEncryptionSpecTest.php

@@ -2017,7 +2017,6 @@ private function createTestCollection(?stdClass $encryptedFields = null, ?stdCla
 
     private function encryptCorpusValue(string $fieldName, stdClass $data, ClientEncryption $clientEncryption)
     {
-        $encrypted = null;
         $encryptionOptions = [
             'algorithm' => $data->algo === 'rand' ? ClientEncryption::AEAD_AES_256_CBC_HMAC_SHA_512_RANDOM : ClientEncryption::AEAD_AES_256_CBC_HMAC_SHA_512_DETERMINISTIC,
         ];

tests/UnifiedSpecTests/Constraint/Matches.php

@@ -368,6 +368,7 @@ private function doToString()
         return 'matches ' . $this->exporter()->export($this->value);
     }
 
+    /** @psalm-return never-return */
     private static function failAt(string $message, string $keyPath): void
     {
         $prefix = empty($keyPath) ? '' : sprintf('Field path "%s": ', $keyPath);

tests/UnifiedSpecTests/Operation.php

@@ -172,7 +172,6 @@ public function assert(bool $rethrowExceptions = false): void
 
     private function execute()
     {
-        $result = null;
         $this->context->setActiveClient(null);
 
         if ($this->isTestRunnerOperation) {