keep it simple

Author: jrushlow

tests/Maker/MakeEntityTest.php

@@ -30,20 +30,10 @@ private function createMakeEntityTest(bool $withDatabase = true): MakerTestDetai
     {
         return $this->createMakerTest()
             ->preRun(function (MakerTestRunner $runner) use ($withDatabase) {
-                $config = $runner->readYaml('config/packages/doctrine.yaml');
-                $appMapping = $config['doctrine']['orm']['mappings']['App'] ?? [];
-
                 /* @legacy Refactor when annotations are no longer supported. */
-                if (isset($appMapping['type']) && $this->useAttributes($runner)) {
-                    // We want to use attributes by default in tests if they're supported.
-                    $runner->replaceInFile(
-                        'config/packages/doctrine.yaml',
-                        'type: annotation',
-                        'type: attribute'
-                    );
-                } elseif (!empty($appMapping) && !$this->useAttributes($runner)) {
-                    // Flex assumes attributes are supported based on package versions.
-                    // When PHP <8.0, we have to use annotations.
+                if (!$this->useAttributes($runner)) {
+                    // Attributes are only supported w/ PHP 8, FrameworkBundle >=5.2,
+                    // ORM >=2.9, & DoctrineBundle >=2.4
                     $runner->modifyYamlFile('config/packages/doctrine.yaml', function (array $data) {
                         $data['doctrine']['orm']['mappings']['App']['type'] = 'annotation';