Verify that a type name is specified before overriding class's type name

Fixes folkloreinc#365
sburba · Aug 1, 2018 · 98127db · 98127db
1 parent 171631a
commit 98127db
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 4 deletions.
diff --git a/src/Folklore/GraphQL/GraphQL.php b/src/Folklore/GraphQL/GraphQL.php
@@ -67,7 +67,7 @@ public function schema($schema = null)
                 $this->typesInstances[$name] = $objectType;
                 $types[] = $objectType;
 
-                $this->addType($type, $name);
+                $this->addType($type, is_numeric($name) ? null : $name);
             }
         } else {
             foreach ($this->types as $name => $type) {

diff --git a/tests/GraphQLTest.php b/tests/GraphQLTest.php
@@ -77,14 +77,21 @@ public function testSchemaWithArray()
                 'updateExampleCustom' => UpdateExampleMutation::class
             ],
             'types' => [
-                CustomExampleType::class
+                CustomExampleType::class,
+                AnotherCustomExampleType::class
             ]
         ]);
-
+
+        $graphql_types = GraphQL::getTypes();
+        $schema_types = $schema->getTypeMap();
+
         $this->assertGraphQLSchema($schema);
         $this->assertGraphQLSchemaHasQuery($schema, 'examplesCustom');
         $this->assertGraphQLSchemaHasMutation($schema, 'updateExampleCustom');
-        $this->assertArrayHasKey('CustomExample', $schema->getTypeMap());
+        $this->assertArrayHasKey('CustomExample', $schema_types);
+        $this->assertArrayHasKey('AnotherCustomExample', $schema_types);
+        $this->assertArrayHasKey('CustomExample', $graphql_types);
+        $this->assertArrayHasKey('AnotherCustomExample', $graphql_types);
     }
 
     /**

diff --git a/tests/Objects/AnotherCustomExampleType.php b/tests/Objects/AnotherCustomExampleType.php
@@ -0,0 +1,23 @@
+<?php
+
+use GraphQL\Type\Definition\Type;
+use Folklore\GraphQL\Support\Type as GraphQLType;
+
+class AnotherCustomExampleType extends GraphQLType
+{
+
+    protected $attributes = [
+        'name' => 'AnotherCustomExample',
+        'description' => 'An example'
+    ];
+
+    public function fields()
+    {
+        return [
+            'test' => [
+                'type' => Type::string(),
+                'description' => 'A test field'
+            ]
+        ];
+    }
+}