guilien-patch-1 branch with improved exception messages (#496)

* Update CannotMapTypeException.php

* Update IteratorTypeMapperTest.php

* Update CannotMapTypeException.php

* Update FieldsBuilderTest.php

* Lock in doctrine/annotations version as newest version breaks

Co-authored-by: Julien Neuhart <j.neuhart@thecodingmachine.com>

Author: oojacoboo

composer.json

@@ -12,7 +12,7 @@
     "require": {
         "php": ">=7.4",
         "ext-json": "*",
-        "doctrine/annotations": "^1.13",
+        "doctrine/annotations": "1.13.2",
         "composer/package-versions-deprecated": "^1.8",
         "phpdocumentor/reflection-docblock": "^4.3 || ^5.0",
         "phpdocumentor/type-resolver": "^1.4",

src/Mappers/CannotMapTypeException.php

@@ -42,7 +42,7 @@ public static function createForType(string $className): self
      */
     public static function createForInputType(string $className): self
     {
-        return new self('cannot map class "' . $className . '" to a known GraphQL input type. Check your TypeMapper configuration.');
+        return new self('cannot map class "' . $className . '" to a known GraphQL input type. Are you missing a @Factory annotation? If you have a @Factory annotation, is it in a namespace analyzed by GraphQLite?');
     }
 
     public static function createForName(string $name): self

tests/FieldsBuilderTest.php

@@ -508,7 +508,7 @@ public function testQueryProviderWithInvalidInputType(): void
         $queryProvider = $this->buildFieldsBuilder();
 
         $this->expectException(CannotMapTypeException::class);
-        $this->expectExceptionMessage('For parameter $foo, in TheCodingMachine\GraphQLite\Fixtures\TestControllerWithInvalidInputType::test, cannot map class "Exception" to a known GraphQL input type. Check your TypeMapper configuration.');
+        $this->expectExceptionMessage('For parameter $foo, in TheCodingMachine\GraphQLite\Fixtures\TestControllerWithInvalidInputType::test, cannot map class "Exception" to a known GraphQL input type. Are you missing a @Factory annotation? If you have a @Factory annotation, is it in a namespace analyzed by GraphQLite?');
         $queryProvider->getQueries($controller);
     }
 

tests/Mappers/Root/IteratorTypeMapperTest.php

@@ -19,7 +19,7 @@ public function testInputIterator(): void
 
         // A type like ArrayObject|int[] CAN be mapped to an output type, but NOT to an input type.
         $this->expectException(CannotMapTypeException::class);
-        $this->expectExceptionMessage('cannot map class "ArrayObject" to a known GraphQL input type. Check your TypeMapper configuration.');
+        $this->expectExceptionMessage('cannot map class "ArrayObject" to a known GraphQL input type. Are you missing a @Factory annotation? If you have a @Factory annotation, is it in a namespace analyzed by GraphQLite?');
         $typeMapper->toGraphQLInputType($this->resolveType('ArrayObject|int[]'), null, 'foo', new ReflectionMethod(__CLASS__, 'testInputIterator'), new DocBlock());
     }