Merge pull request #1157 from cfv1000/master

fix issue with parsing invalid files

src/Config/Parser/MetadataParser/MetadataParser.php

@@ -134,6 +134,8 @@ private static function processFile(SplFileInfo $file, ContainerBuilder $contain
             }
 
             return $preProcess ? self::$map->toArray() : $gqlTypes;
+        } catch (ReflectionException $e) {
+            return $gqlTypes;
         } catch (\InvalidArgumentException $e) {
             throw new InvalidArgumentException(sprintf('Failed to parse GraphQL metadata from file "%s".', $file), $e->getCode(), $e);
         }

tests/Config/Parser/MetadataParserTest.php

@@ -615,4 +615,30 @@ public function testInvalidProviderMutationOnQuery(): void
             $this->assertMatchesRegularExpression('/try to add a mutation on type "RootQuery2"/', $e->getPrevious()->getMessage());
         }
     }
+
+    public function testInvalidPhpFiles(): void
+    {
+        $files = [
+            __DIR__.'/fixtures/annotations/Invalid/HasNoClass.php',
+            __DIR__.'/fixtures/annotations/Invalid/EmptyPhpFile.php',
+            __DIR__.'/fixtures/annotations/Invalid/NotAPhpFile',
+            __DIR__.'/fixtures/annotations/Type/RootQuery.php',
+        ];
+        $this->parser('reset', $this->parserConfig);
+
+        foreach ($files as $file) {
+            $this->parser('preParse', new SplFileInfo($file), $this->containerBuilder, $this->parserConfig);
+        }
+
+        $config = [];
+        foreach ($files as $file) {
+            $config += self::cleanConfig($this->parser('parse', new SplFileInfo($file), $this->containerBuilder, $this->parserConfig));
+        }
+
+        $this->assertSame([
+            'RootQuery' => [
+                'type' => 'object',
+            ],
+        ], $config);
+    }
 }

tests/Config/Parser/fixtures/annotations/Invalid/EmptyPhpFile.php

@@ -0,0 +1,3 @@
+<?php
+
+declare(strict_types=1);

tests/Config/Parser/fixtures/annotations/Invalid/HasNoClass.php

@@ -0,0 +1,5 @@
+<?php
+
+declare(strict_types=1);
+
+return function (): void {};