bug #760 only use attributes if PHP >= 8 && Symfony >= 5.2 (jrushlow)

This PR was squashed before being merged into the 1.0-dev branch.

Discussion
----------

only use attributes if PHP >= 8 && Symfony >= 5.2

fixes #750

Commits
-------

4dd942f only use attributes if PHP >= 8 && Symfony >= 5.2

Author: weaverryan

src/Util/PhpCompatUtil.php

@@ -12,6 +12,7 @@
 namespace Symfony\Bundle\MakerBundle\Util;
 
 use Symfony\Bundle\MakerBundle\FileManager;
+use Symfony\Component\HttpKernel\Kernel;
 
 /**
  * @author Jesse Rushlow <[email protected]>
@@ -32,7 +33,7 @@ public function canUseAttributes(): bool
     {
         $version = $this->getPhpVersion();
 
-        return version_compare($version, '8alpha', '>=');
+        return version_compare($version, '8alpha', '>=') && Kernel::VERSION_ID >= 50200;
     }
 
     protected function getPhpVersion(): string

tests/Util/PhpVersionTest.php

@@ -14,6 +14,7 @@
 use PHPUnit\Framework\TestCase;
 use Symfony\Bundle\MakerBundle\FileManager;
 use Symfony\Bundle\MakerBundle\Util\PhpCompatUtil;
+use Symfony\Component\HttpKernel\Kernel;
 
 /**
  * @author Jesse Rushlow <[email protected]>
@@ -52,6 +53,15 @@ public function testUsesPhpPlatformFromComposerJsonFile(string $version, bool $e
 
         $result = $version->canUseAttributes();
 
+        /*
+         * Symfony 5.2 is required to compare the result. Otherwise it will always
+         * return false regardless of the PHP Version. If the test suite is run w/
+         * Symfony < 5.2, we assert false here but still rely on the assertions above.
+         */
+        if (Kernel::VERSION_ID < 50200) {
+            $expectedResult = false;
+        }
+
         self::assertSame($expectedResult, $result);
     }