Merge pull request #888 from hello-liang-shan/master

Fix "No configuration for schema '' found" when route prefix is empty string

Author: mfn

src/GraphQLController.php

@@ -19,7 +19,7 @@ class GraphQLController extends Controller
     public function query(Request $request, RequestParser $parser, Repository $config, GraphQL $graphql): JsonResponse
     {
         $routePrefix = $config->get('graphql.route.prefix', 'graphql');
-        $schemaName = $this->findSchemaNameInRequest($request, "$routePrefix/") ?? $config->get('graphql.default_schema', 'default');
+        $schemaName = $this->findSchemaNameInRequest($request, "$routePrefix/") ?: $config->get('graphql.default_schema', 'default');
 
         $operations = $parser->parseRequest($request);
 

tests/Unit/EmptyRoutePrefixTest.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types = 1);
+namespace Rebing\GraphQL\Tests\Unit;
+
+use Rebing\GraphQL\Tests\TestCase;
+
+class EmptyRoutePrefixTest extends TestCase
+{
+    protected function getEnvironmentSetUp($app): void
+    {
+        parent::getEnvironmentSetUp($app);
+
+        $app['config']->set('graphql.route.prefix', '');
+    }
+
+    public function testEmptyRoutePrefix():void
+    {
+        $response = $this->call('GET', '/', [
+            'query' => $this->queries['examples'],
+        ]);
+
+        self::assertEquals(200, $response->getStatusCode());
+    }
+}