Merge pull request #2949 from tarlepp/chore(ci)/disable-phpinsights

Chore(ci) - Disabled `PHP Insights` tool `ForbiddenSecurityIssues` check

Author: tarlepp

phpinsights.php

@@ -53,6 +53,8 @@
         //  ExampleInsight::class,
         NunoMaduro\PhpInsights\Domain\Insights\Composer\ComposerMustBeValid::class,
         NunoMaduro\PhpInsights\Domain\Insights\ForbiddenNormalClasses::class,
+        // Disabled for now because of the - https://github.com/nunomaduro/phpinsights/issues/704
+        NunoMaduro\PhpInsights\Domain\Insights\ForbiddenSecurityIssues::class,
         NunoMaduro\PhpInsights\Domain\Insights\ForbiddenTraits::class,
         NunoMaduro\PhpInsights\Domain\Sniffs\ForbiddenSetterSniff::class,
         ObjectCalisthenics\Sniffs\Classes\ForbiddenPublicPropertySniff::class,

tests/Integration/TestCase/RestIntegrationControllerTestCase.php

@@ -14,8 +14,6 @@
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
 use UnexpectedValueException;
 use function assert;
-use function gc_collect_cycles;
-use function gc_enable;
 use function mb_substr;
 use function sprintf;
 
@@ -40,8 +38,6 @@ abstract class RestIntegrationControllerTestCase extends KernelTestCase
     #[Override]
     protected function setUp(): void
     {
-        gc_enable();
-
         parent::setUp();
 
         $controller = static::getContainer()->get($this->controllerClass);
@@ -51,16 +47,6 @@ protected function setUp(): void
         $this->controller = $controller;
     }
 
-    #[Override]
-    protected function tearDown(): void
-    {
-        parent::tearDown();
-
-        unset($this->controller);
-
-        gc_collect_cycles();
-    }
-
     public function testThatGivenControllerIsCorrect(): void
     {
         $expected = mb_substr((new ReflectionClass($this))->getShortName(), 0, -4);