Remove support for Lumen

Supporting Lumen without a dedicated framework for testing is a nightmare,
so many unresolved issues and very few contributions over the years led
me to believe we're better off removing support.

The readme never went beyond calling it "experimental".

This PR targets the next major release due to this breaking change.

Author: mfn

.github/ISSUE_TEMPLATE/1_Bug_report.md

@@ -8,7 +8,6 @@ labels: bug
 ### Versions:
 - graphql-laravel Version: #.#.#
 - Laravel Version: #.#.# <!-- remove if not applicable -->
-- Lumen Version: #.#.# <!-- remove if not applicable -->
 - PHP Version: #.#.#
 
 ### Description:

.github/ISSUE_TEMPLATE/3_Support_question.md

@@ -11,7 +11,6 @@ Alternative you can also use our Slack workspace: https://join.slack.com/t/rebin
 ### Versions:
 - graphql-laravel Version: #.#.#
 - Laravel Version: #.#.# <!-- remove if not applicable -->
-- Lumen Version: #.#.# <!-- remove if not applicable -->
 - PHP Version: #.#.#
 
 ### Question:

.github/workflows/integration_tests.yml

@@ -18,10 +18,7 @@ on:
 
 jobs:
   integration:
-    strategy:
-      matrix:
-        integration: [laravel, lumen]
-    name: ${{ matrix.integration }}
+    name: laravel
     runs-on: ubuntu-20.04
     env:
       COMPOSER_NO_INTERACTION: 1
@@ -52,4 +49,4 @@ jobs:
 
       - run: composer install --prefer-dist --no-suggest
 
-      - run: tests/integration-${{ matrix.integration }}.sh
+      - run: tests/integration-laravel.sh

CHANGELOG.md

@@ -4,6 +4,9 @@ CHANGELOG
 [Next release](https://github.com/rebing/graphql-laravel/compare/7.0.0...master)
 --------------
 
+### BREAKING CHANGE
+- Support for Lumen has been removed
+
 2021-04-03, 7.0.0
 -----------------
 ## Breaking changes

README.md

@@ -32,7 +32,7 @@ It offers following features and improvements over the original package by
 
 ### Dependencies:
 
-* [Laravel 6.0+](https://github.com/laravel/laravel) or [Lumen](https://github.com/laravel/lumen)
+* [Laravel 6.0+](https://github.com/laravel/laravel)
 * [GraphQL PHP](https://github.com/webonyx/graphql-php)
 
 
@@ -55,37 +55,7 @@ Review the configuration file:
 config/graphql.php
 ```
 
-#### Lumen (experimental!)
-
-**1.** Add the following service provider to the `bootstrap/app.php` file
-```php
-$app->register(Rebing\GraphQL\GraphQLLumenServiceProvider::class);
-```
-
-**2.** Publish the configuration file
-```bash
-$ php artisan graphql:publish
-```
-
-**3.** Add the configuration to the `bootstrap/app.php` file
-    *Important:* this needs to be before the registration of the service provider
-```php
-$app->configure('graphql');
-...
-$app->register(Rebing\GraphQL\GraphQLLumenServiceProvider::class);
-```
-
-**4.** Review the configuration file
-```php
-config/graphql.php
-```
-
 The default GraphiQL view makes use of the global `csrf_token()` helper function.
-Out of the box, this function is not available in Lumen.
-
-To work this around:
-- Point to your local GraphiQL view: change `graphql.view` to `'vendor/graphql/graphiql'`
-- Modify your file `resources/views/vendor/graphql/graphiql.php` and remove the call
 
 ## Usage
 
@@ -94,7 +64,6 @@ To work this around:
       - [Dependencies:](#dependencies)
       - [Installation:](#installation)
         - [Laravel 6.0+](#laravel)
-        - [Lumen (experimental!)](#lumen-experimental)
   - [Usage](#usage)
     - [Concepts](#concepts)
     - [Data loading](#data-loading)

phpstan-baseline.neon

@@ -480,11 +480,6 @@ parameters:
 			count: 1
 			path: src/Support/UploadType.php
 
-		-
-			message: "#^PHPDoc tag @var for variable \\$router contains unknown class Laravel\\\\Lumen\\\\Routing\\\\Router\\.$#"
-			count: 2
-			path: src/routes.php
-
 		-
 			message: "#^PHPDoc tag @var for variable \\$queryTypesMap has no value type specified in iterable type array\\.$#"
 			count: 1
@@ -495,16 +490,6 @@ parameters:
 			count: 1
 			path: src/routes.php
 
-		-
-			message: "#^Call to method get\\(\\) on an unknown class Laravel\\\\Lumen\\\\Routing\\\\Router\\.$#"
-			count: 3
-			path: src/routes.php
-
-		-
-			message: "#^Call to method post\\(\\) on an unknown class Laravel\\\\Lumen\\\\Routing\\\\Router\\.$#"
-			count: 3
-			path: src/routes.php
-
 		-
 			message: "#^Property Rebing\\\\GraphQL\\\\Tests\\\\Database\\\\AuthorizeArgsTests\\\\GraphQLContext\\:\\:\\$data has no typehint specified\\.$#"
 			count: 1

src/GraphQL.php

@@ -472,12 +472,6 @@ public static function routeNameTransformer(string $name, string $schemaParamete
         $routeName = null;
 
         if (count($multiLevelPath) > 1) {
-            if (Helpers::isLumen()) {
-                array_walk($multiLevelPath, function (string &$multiName): void {
-                    $multiName = "$multiName:$multiName";
-                });
-            }
-
             foreach ($multiLevelPath as $multiName) {
                 $routeName = ! $routeName ? null : $routeName.'/';
                 $routeName =
@@ -486,7 +480,7 @@ public static function routeNameTransformer(string $name, string $schemaParamete
             }
         }
 
-        return $routeName ?: preg_replace($schemaParameterPattern, '{'.(Helpers::isLumen() ? "schema:$name" : $name).'}', $queryRoute);
+        return $routeName ?: preg_replace($schemaParameterPattern, '{'.$name.'}', $queryRoute);
     }
 
     /**

src/GraphQLController.php

@@ -28,7 +28,7 @@ public function query(Request $request, string $schema = null): JsonResponse
 
         // If there are multiple route params we can expect that there
         // will be a schema name that has to be built
-        $routeParameters = $this->getRouteParameters($request);
+        $routeParameters = $request->route()->parameters;
         if (count($routeParameters) > 1) {
             $schema = implode('/', $routeParameters);
         }
@@ -101,20 +101,4 @@ public function graphiql(Request $request, string $schema = null): View
             'schema' => $schema,
         ]);
     }
-
-    /**
-     * @param  Request  $request
-     * @return array<string,string>
-     */
-    protected function getRouteParameters(Request $request): array
-    {
-        if (Helpers::isLumen()) {
-            /** @var array<int,mixed> $route */
-            $route = $request->route();
-
-            return $route[2] ?? [];
-        }
-
-        return $request->route()->parameters;
-    }
 }