Merge pull request #609 from mpociot/analysis-qognB5

Apply fixes from StyleCI

config/apidoc.php

@@ -1,7 +1,7 @@
 <?php
 
 return [
-    /**
+    /*
      * The type of documentation output to generate.
      * - "static" will generate a static HTMl page in the /public/docs folder,
      * - "laravel" will generate the documentation as a Blade view,

src/ApiDoc.php

@@ -7,11 +7,10 @@ use Illuminate\Support\Facades\Storage;
 
 class ApiDoc
 {
-    public static function routes($path = "/doc")
+    public static function routes($path = '/doc')
     {
         return Route::get("$path{format?}", function (?string $format = null) {
-            if ($format == ".json") {
-
+            if ($format == '.json') {
                 return response(
                     Storage::disk('local')->get('apidoc/collection.json'),
                     200,
@@ -20,7 +19,7 @@ class ApiDoc
                 );
             }
 
-            return view("apidoc.index");
+            return view('apidoc.index');
         })->name('apidoc');
     }
 }

src/Commands/GenerateDocumentation.php

@@ -9,8 +9,8 @@ use Illuminate\Console\Command;
 use Mpociot\ApiDoc\Tools\Flags;
 use Mpociot\ApiDoc\Tools\Utils;
 use Mpociot\Reflection\DocBlock;
-use Mpociot\ApiDoc\Writing\Writer;
 use Illuminate\Support\Collection;
+use Mpociot\ApiDoc\Writing\Writer;
 use Illuminate\Support\Facades\URL;
 use Mpociot\ApiDoc\Extracting\Generator;
 use Mpociot\ApiDoc\Matching\RouteMatcher;
@@ -71,7 +71,6 @@ class GenerateDocumentation extends Command
         $generator = new Generator($this->docConfig);
         $parsedRoutes = $this->processRoutes($generator, $routes);
 
-
         $groupedRoutes = collect($parsedRoutes)
             ->groupBy('metadata.groupName')
             ->sortBy(static function ($group) {
@@ -155,5 +154,4 @@ class GenerateDocumentation extends Command
 
         return true;
     }
-
 }

src/Extracting/Generator.php

@@ -2,13 +2,13 @@
 
 namespace Mpociot\ApiDoc\Extracting;
 
-use Mpociot\ApiDoc\Tools\DocumentationConfig;
-use Mpociot\ApiDoc\Tools\Utils;
 use ReflectionClass;
 use ReflectionMethod;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Str;
 use Illuminate\Routing\Route;
+use Mpociot\ApiDoc\Tools\Utils;
+use Mpociot\ApiDoc\Tools\DocumentationConfig;
 
 class Generator
 {

src/Extracting/RouteDocBlocker.php

@@ -2,9 +2,9 @@
 
 namespace Mpociot\ApiDoc\Extracting;
 
-use Mpociot\ApiDoc\Tools\Utils;
 use ReflectionClass;
 use Illuminate\Routing\Route;
+use Mpociot\ApiDoc\Tools\Utils;
 use Mpociot\Reflection\DocBlock;
 
 /**

src/Extracting/Strategies/BodyParameters/GetFromBodyParamTag.php

@@ -7,10 +7,10 @@ use ReflectionMethod;
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
+use Mpociot\ApiDoc\Extracting\ParamHelpers;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Dingo\Api\Http\FormRequest as DingoFormRequest;
-use Mpociot\ApiDoc\Extracting\ParamHelpers;
 use Illuminate\Foundation\Http\FormRequest as LaravelFormRequest;
 
 class GetFromBodyParamTag extends Strategy

src/Extracting/Strategies/Metadata/GetFromDocBlocks.php

@@ -7,8 +7,8 @@ use ReflectionMethod;
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 
 class GetFromDocBlocks extends Strategy
 {

src/Extracting/Strategies/QueryParameters/GetFromQueryParamTag.php

@@ -8,10 +8,10 @@ use Illuminate\Support\Str;
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
+use Mpociot\ApiDoc\Extracting\ParamHelpers;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Dingo\Api\Http\FormRequest as DingoFormRequest;
-use Mpociot\ApiDoc\Extracting\ParamHelpers;
 use Illuminate\Foundation\Http\FormRequest as LaravelFormRequest;
 
 class GetFromQueryParamTag extends Strategy

src/Extracting/Strategies/Responses/ResponseCalls.php

@@ -9,8 +9,8 @@ use Illuminate\Http\Response;
 use Illuminate\Routing\Route;
 use Mpociot\ApiDoc\Tools\Flags;
 use Mpociot\ApiDoc\Tools\Utils;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Extracting\ParamHelpers;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 
 /**
  * Make a call to the route and retrieve its response.

src/Extracting/Strategies/Responses/UseApiResourceTags.php

@@ -15,9 +15,9 @@ use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
 use Illuminate\Database\Eloquent\Model;
 use League\Fractal\Resource\Collection;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
 use Illuminate\Http\Resources\Json\JsonResource;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Illuminate\Http\Resources\Json\ResourceCollection;
 
 /**

src/Extracting/Strategies/Responses/UseResponseFileTag.php

@@ -5,8 +5,8 @@ namespace Mpociot\ApiDoc\Extracting\Strategies\Responses;
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 
 /**
  * Get a response from from a file in the docblock ( @responseFile ).

src/Extracting/Strategies/Responses/UseResponseTag.php

@@ -5,8 +5,8 @@ namespace Mpociot\ApiDoc\Extracting\Strategies\Responses;
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 
 /**
  * Get a response from the docblock ( @response ).

src/Extracting/Strategies/Responses/UseTransformerTags.php

@@ -15,8 +15,8 @@ use League\Fractal\Resource\Item;
 use Mpociot\Reflection\DocBlock\Tag;
 use Illuminate\Database\Eloquent\Model;
 use League\Fractal\Resource\Collection;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 
 /**
  * Parse a transformer response from the docblock ( @transformer || @transformercollection ).

src/Extracting/Strategies/UrlParameters/GetFromUrlParamTag.php

@@ -8,10 +8,10 @@ use Illuminate\Support\Str;
 use Illuminate\Routing\Route;
 use Mpociot\Reflection\DocBlock;
 use Mpociot\Reflection\DocBlock\Tag;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
+use Mpociot\ApiDoc\Extracting\ParamHelpers;
 use Mpociot\ApiDoc\Extracting\RouteDocBlocker;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Dingo\Api\Http\FormRequest as DingoFormRequest;
-use Mpociot\ApiDoc\Extracting\ParamHelpers;
 use Illuminate\Foundation\Http\FormRequest as LaravelFormRequest;
 
 class GetFromUrlParamTag extends Strategy

src/Matching/RouteMatcher.php

@@ -19,8 +19,7 @@ class RouteMatcher
      */
     protected $routeRules;
 
-
-    public function __construct(array $routeRules = [], string $router = "laravel")
+    public function __construct(array $routeRules = [], string $router = 'laravel')
     {
         $this->router = $router;
         $this->routeRules = $routeRules;
@@ -29,6 +28,7 @@ class RouteMatcher
     public function getRoutes()
     {
         $usingDingoRouter = strtolower($this->router) == 'dingo';
+
         return $this->getRoutesToBeDocumented($this->routeRules, $usingDingoRouter);
     }
 

src/Writing/PostmanCollectionWriter.php

@@ -40,7 +40,7 @@ class PostmanCollectionWriter
         $collection = [
             'variables' => [],
             'info' => [
-                'name' => config('apidoc.postman.name') ?: config('app.name') . ' API',
+                'name' => config('apidoc.postman.name') ?: config('app.name').' API',
                 '_postman_id' => Uuid::uuid4()->toString(),
                 'description' => config('apidoc.postman.description') ?: '',
                 'schema' => 'https://schema.getpostman.com/json/collection/v2.0.0/collection.json',
@@ -55,11 +55,11 @@ class PostmanCollectionWriter
                         return [
                             'name' => $route['metadata']['title'] != '' ? $route['metadata']['title'] : url($route['uri']),
                             'request' => [
-                                'url' => url($route['uri']) . (collect($route['queryParameters'])->isEmpty()
+                                'url' => url($route['uri']).(collect($route['queryParameters'])->isEmpty()
                                         ? ''
-                                        : ('?' . implode('&', collect($route['queryParameters'])->map(function ($parameter, $key) {
-                                                return urlencode($key) . '=' . urlencode($parameter['value'] ?? '');
-                                            })->all()))),
+                                        : ('?'.implode('&', collect($route['queryParameters'])->map(function ($parameter, $key) {
+                                            return urlencode($key).'='.urlencode($parameter['value'] ?? '');
+                                        })->all()))),
                                 'method' => $route['methods'][0],
                                 'header' => collect($route['headers'])
                                     ->union([

src/Writing/Writer.php

@@ -1,9 +1,7 @@
 <?php
 
-
 namespace Mpociot\ApiDoc\Writing;
 
-
 use Illuminate\Console\Command;
 use Illuminate\Support\Collection;
 use Illuminate\Support\Facades\Storage;
@@ -68,8 +66,8 @@ class Writer
         // go in resources/views/apidoc/ and storage/app/apidoc/ respectively.
         $isStatic = $this->config->get('type') === 'static';
 
-        $sourceOutputPath = "resources/docs";
-        $outputPath = $isStatic ? "public/docs" : "resources/views/apidoc";
+        $sourceOutputPath = 'resources/docs';
+        $outputPath = $isStatic ? 'public/docs' : 'resources/views/apidoc';
 
         $this->writeMarkdownAndSourceFiles($this->routes, $sourceOutputPath);
 
@@ -85,8 +83,8 @@ class Writer
      */
     public function writeMarkdownAndSourceFiles(Collection $parsedRoutes, string $sourceOutputPath)
     {
-        $targetFile = $sourceOutputPath . '/source/index.md';
-        $compareFile = $sourceOutputPath . '/source/.compare.md';
+        $targetFile = $sourceOutputPath.'/source/index.md';
+        $compareFile = $sourceOutputPath.'/source/.compare.md';
 
         $infoText = view('apidoc::partials.info')
             ->with('outputPath', 'docs')
@@ -110,14 +108,14 @@ class Writer
 
             $parsedRouteOutput->transform(function (Collection $routeGroup) use ($generatedDocumentation, $compareDocumentation) {
                 return $routeGroup->transform(function (array $route) use ($generatedDocumentation, $compareDocumentation) {
-                    if (preg_match('/<!-- START_' . $route['id'] . ' -->(.*)<!-- END_' . $route['id'] . ' -->/is', $generatedDocumentation, $existingRouteDoc)) {
-                        $routeDocumentationChanged = (preg_match('/<!-- START_' . $route['id'] . ' -->(.*)<!-- END_' . $route['id'] . ' -->/is', $compareDocumentation, $lastDocWeGeneratedForThisRoute) && $lastDocWeGeneratedForThisRoute[1] !== $existingRouteDoc[1]);
+                    if (preg_match('/<!-- START_'.$route['id'].' -->(.*)<!-- END_'.$route['id'].' -->/is', $generatedDocumentation, $existingRouteDoc)) {
+                        $routeDocumentationChanged = (preg_match('/<!-- START_'.$route['id'].' -->(.*)<!-- END_'.$route['id'].' -->/is', $compareDocumentation, $lastDocWeGeneratedForThisRoute) && $lastDocWeGeneratedForThisRoute[1] !== $existingRouteDoc[1]);
                         if ($routeDocumentationChanged === false || $this->forceIt) {
                             if ($routeDocumentationChanged) {
-                                $this->output->warn('Discarded manual changes for route [' . implode(',', $route['methods']) . '] ' . $route['uri']);
+                                $this->output->warn('Discarded manual changes for route ['.implode(',', $route['methods']).'] '.$route['uri']);
                             }
                         } else {
-                            $this->output->warn('Skipping modified route [' . implode(',', $route['methods']) . '] ' . $route['uri']);
+                            $this->output->warn('Skipping modified route ['.implode(',', $route['methods']).'] '.$route['uri']);
                             $route['modified_output'] = $existingRouteDoc[0];
                         }
                     }
@@ -140,9 +138,9 @@ class Writer
             ->with('showPostmanCollectionButton', $this->shouldGeneratePostmanCollection)
             ->with('parsedRoutes', $parsedRouteOutput);
 
-        $this->output->info('Writing index.md and source files to: ' . $sourceOutputPath);
+        $this->output->info('Writing index.md and source files to: '.$sourceOutputPath);
 
-        if (!is_dir($sourceOutputPath)) {
+        if (! is_dir($sourceOutputPath)) {
             $documentarian = new Documentarian();
             $documentarian->create($sourceOutputPath);
         }
@@ -163,14 +161,14 @@ class Writer
 
         file_put_contents($compareFile, $compareMarkdown);
 
-        $this->output->info('Wrote index.md and source files to: ' . $sourceOutputPath);
+        $this->output->info('Wrote index.md and source files to: '.$sourceOutputPath);
     }
 
     public function generateMarkdownOutputForEachRoute(Collection $parsedRoutes, array $settings): Collection
     {
         $parsedRouteOutput = $parsedRoutes->map(function (Collection $routeGroup) use ($settings) {
             return $routeGroup->map(function (array $route) use ($settings) {
-                if (count($route['cleanBodyParameters']) && !isset($route['headers']['Content-Type'])) {
+                if (count($route['cleanBodyParameters']) && ! isset($route['headers']['Content-Type'])) {
                     // Set content type if the user forgot to set it
                     $route['headers']['Content-Type'] = 'application/json';
                 }
@@ -183,6 +181,7 @@ class Writer
                 return $route;
             });
         });
+
         return $parsedRouteOutput;
     }
 
@@ -225,11 +224,12 @@ class Writer
      */
     protected function getMarkdownToPrepend(string $sourceOutputPath): string
     {
-        $prependFile = $sourceOutputPath . '/source/prepend.md';
+        $prependFile = $sourceOutputPath.'/source/prepend.md';
         $prependFileContents = file_exists($prependFile)
-            ? file_get_contents($prependFile) . "\n" : '';
+            ? file_get_contents($prependFile)."\n" : '';
+
         return $prependFileContents;
-}
+    }
 
     /**
      * @param string $sourceOutputPath
@@ -238,16 +238,17 @@ class Writer
      */
     protected function getMarkdownToAppend(string $sourceOutputPath): string
     {
-        $appendFile = $sourceOutputPath . '/source/append.md';
+        $appendFile = $sourceOutputPath.'/source/append.md';
         $appendFileContents = file_exists($appendFile)
-            ? "\n" . file_get_contents($appendFile) : '';
+            ? "\n".file_get_contents($appendFile) : '';
+
         return $appendFileContents;
-}
+    }
 
     protected function copyAssetsFromSourceFolderToPublicFolder(string $sourceOutputPath, bool $isStatic = true): void
     {
-        $publicPath = "public/docs";
-        if (!is_dir($publicPath)) {
+        $publicPath = 'public/docs';
+        if (! is_dir($publicPath)) {
             mkdir($publicPath, 0777, true);
             mkdir("{$publicPath}/css");
             mkdir("{$publicPath}/js");
@@ -270,7 +271,7 @@ class Writer
             rename("{$sourceOutputPath}/index.html", "{$outputPath}/index.html");
         } else {
             // Move output to resources/views
-            if (!is_dir($outputPath)) {
+            if (! is_dir($outputPath)) {
                 mkdir($outputPath);
             }
             rename("{$sourceOutputPath}/index.html", "$outputPath/index.blade.php");

tests/GenerateDocumentationTest.php

@@ -136,7 +136,7 @@ class GenerateDocumentationTest extends TestCase
     /** @test */
     public function can_parse_partial_resource_routes()
     {
-            RouteFacade::resource('/api/users', TestResourceController::class)
+        RouteFacade::resource('/api/users', TestResourceController::class)
                 ->only(['index', 'create']);
 
         config(['apidoc.routes.0.match.prefixes' => ['api/*']]);
@@ -152,7 +152,7 @@ class GenerateDocumentationTest extends TestCase
         $generatedMarkdown = __DIR__.'/../resources/docs/source/index.md';
         $this->assertFilesHaveSameContent($fixtureMarkdown, $generatedMarkdown);
 
-            RouteFacade::apiResource('/api/users', TestResourceController::class)
+        RouteFacade::apiResource('/api/users', TestResourceController::class)
                 ->only(['index', 'create']);
 
         $this->artisan('apidoc:generate');

tests/Unit/GeneratorPluginSystemTestCase.php

@@ -6,10 +6,10 @@ use ReflectionClass;
 use ReflectionMethod;
 use Illuminate\Routing\Route;
 use Mpociot\ApiDoc\Extracting\Generator;
-use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 use Mpociot\ApiDoc\Tools\DocumentationConfig;
 use Mpociot\ApiDoc\Tests\Fixtures\TestController;
 use Mpociot\ApiDoc\ApiDocGeneratorServiceProvider;
+use Mpociot\ApiDoc\Extracting\Strategies\Strategy;
 
 class GeneratorPluginSystemTestCase extends LaravelGeneratorTest
 {

tests/Unit/RouteMatcherTest.php

@@ -56,17 +56,17 @@ class RouteMatcherTest extends TestCase
         $routeRules[0]['match']['prefixes'] = ['*'];
 
         $routeRules[0]['match']['domains'] = ['*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(12, $routes);
 
         $routeRules[0]['match']['domains'] = ['domain1.*', 'domain2.*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(12, $routes);
 
         $routeRules[0]['match']['domains'] = ['domain1.*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(6, $routes);
         foreach ($routes as $route) {
@@ -74,7 +74,7 @@ class RouteMatcherTest extends TestCase
         }
 
         $routeRules[0]['match']['domains'] = ['domain2.*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(6, $routes);
         foreach ($routes as $route) {
@@ -121,17 +121,17 @@ class RouteMatcherTest extends TestCase
         $routeRules[0]['match']['domains'] = ['*'];
 
         $routeRules[0]['match']['prefixes'] = ['*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(12, $routes);
 
         $routeRules[0]['match']['prefixes'] = ['prefix1/*', 'prefix2/*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(8, $routes);
 
         $routeRules[0]['match']['prefixes'] = ['prefix1/*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(4, $routes);
         foreach ($routes as $route) {
@@ -139,7 +139,7 @@ class RouteMatcherTest extends TestCase
         }
 
         $routeRules[0]['match']['prefixes'] = ['prefix2/*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(4, $routes);
         foreach ($routes as $route) {
@@ -154,7 +154,7 @@ class RouteMatcherTest extends TestCase
         $routeRules[0]['match']['versions'] = ['v2'];
         $routeRules[0]['match']['domains'] = ['*'];
         $routeRules[0]['match']['prefixes'] = ['*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(6, $routes);
         foreach ($routes as $route) {
@@ -164,7 +164,7 @@ class RouteMatcherTest extends TestCase
         $routeRules[0]['match']['versions'] = ['v1', 'v2'];
         $routeRules[0]['match']['domains'] = ['*'];
         $routeRules[0]['match']['prefixes'] = ['*'];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(18, $routes);
     }
@@ -200,7 +200,7 @@ class RouteMatcherTest extends TestCase
                 'include' => [$mustInclude],
             ],
         ];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $oddRuleOut = collect($routes)->filter(function ($route) use ($mustInclude) {
             return $route['route']->getName() === $mustInclude;
@@ -241,7 +241,7 @@ class RouteMatcherTest extends TestCase
                 'include' => [$includePattern],
             ],
         ];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $oddRuleOut = collect($routes)->filter(function ($route) use ($mustInclude) {
             return in_array($route['route']->getName(), $mustInclude);
@@ -280,7 +280,7 @@ class RouteMatcherTest extends TestCase
                 'exclude' => [$mustNotInclude],
             ],
         ];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $oddRuleOut = collect($routes)->filter(function ($route) use ($mustNotInclude) {
             return $route['route']->getName() === $mustNotInclude;
@@ -321,7 +321,7 @@ class RouteMatcherTest extends TestCase
                 'exclude' => [$excludePattern],
             ],
         ];
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $oddRuleOut = collect($routes)->filter(function ($route) use ($mustNotInclude) {
             return in_array($route['route']->getName(), $mustNotInclude);
@@ -386,7 +386,7 @@ class RouteMatcherTest extends TestCase
             ],
         ];
 
-        $matcher = new RouteMatcher($routeRules, "dingo");
+        $matcher = new RouteMatcher($routeRules, 'dingo');
         $routes = $matcher->getRoutes();
         $this->assertCount(18, $routes);