Disallow extra fields in config to protect from typos (#52)

src/Route/InvalidRouteDefinitionException.php

@@ -0,0 +1,20 @@
+<?php declare(strict_types = 1);
+
+namespace BrandEmbassy\Slim\Route;
+
+use RuntimeException;
+use function implode;
+use function sprintf;
+
+final class InvalidRouteDefinitionException extends RuntimeException
+{
+    /**
+     * @param string[] $path
+     */
+    public function __construct(array $path, string $hint)
+    {
+        parent::__construct(
+            sprintf('Unexpected route definition key in "%s", did you mean "%s"?', implode(' › ', $path), $hint)
+        );
+    }
+}

src/Route/RouteDefinition.php

@@ -10,6 +10,14 @@ final class RouteDefinition
     public const IGNORE_VERSION_MIDDLEWARE_GROUP = 'ignoreVersionMiddlewareGroup';
     public const NAME = 'name';
 
+    public const ALL_DEFINED_KEYS = [
+        self::SERVICE,
+        self::MIDDLEWARES,
+        self::MIDDLEWARE_GROUPS,
+        self::IGNORE_VERSION_MIDDLEWARE_GROUP,
+        self::NAME,
+    ];
+
     /**
      * @var string
      */

src/Route/RouteRegister.php

@@ -5,7 +5,9 @@
 use BrandEmbassy\Slim\Middleware\BeforeRouteMiddlewares;
 use BrandEmbassy\Slim\Middleware\MiddlewareGroups;
 use Slim\Interfaces\RouterInterface;
+use function array_keys;
 use function array_merge_recursive;
+use function levenshtein;
 
 final class RouteRegister
 {
@@ -66,6 +68,8 @@ public function register(string $apiNamespace, string $routePattern, array $rout
                 continue;
             }
 
+            $this->detectTyposInRouteConfiguration([$apiNamespace, $routePattern, $method], $routeDefinitionData);
+
             $routeDefinition = $this->routeDefinitionFactory->create($method, $routeDefinitionData);
 
             $routeName = $routeDefinition->getName() ?? $resolveRoutePath;
@@ -121,4 +125,24 @@ private function getEmptyRouteDefinitionData(): array
             RouteDefinition::NAME => null,
         ];
     }
+
+
+    /**
+     * @param string[] $path
+     * @param mixed[] $routeDefinitionData
+     */
+    private function detectTyposInRouteConfiguration(array $path, array $routeDefinitionData): void
+    {
+        $usedKeys = array_keys($routeDefinitionData);
+        foreach ($usedKeys as $usedKey) {
+            foreach (RouteDefinition::ALL_DEFINED_KEYS as $definedKey) {
+                $levenshteinDistance = levenshtein($usedKey, $definedKey);
+                if ($levenshteinDistance > 0 && $levenshteinDistance < 2) {
+                    $path[] = $usedKey;
+
+                    throw new InvalidRouteDefinitionException($path, $definedKey);
+                }
+            }
+        }
+    }
 }

tests/SlimApplicationFactoryTest.php

@@ -233,4 +233,14 @@ private function prepareEnvironment(string $requestMethod, string $requestUrlPat
             $_SERVER[$name] = $value;
         }
     }
+
+
+    public function testRouteConfigWillFailWhenMisconfigured(): void
+    {
+        $this->expectExceptionMessage(
+            'Unexpected route definition key in "app › /hello-world › get › middleware", did you mean "middlewares"?'
+        );
+
+        SlimAppTester::createSlimApp(__DIR__ . '/typo-in-config.neon');
+    }
 }

tests/typo-in-config.neon

@@ -0,0 +1,21 @@
+extensions:
+    slimApi: \BrandEmbassy\Slim\DI\SlimApiExtension
+
+
+services:
+    - BrandEmbassyTest\Slim\Sample\BeforeRouteMiddleware
+    - BrandEmbassyTest\Slim\Sample\HelloWorldRoute
+
+slimApi:
+    apiPrefix: '/tests'
+    slimConfiguration:
+        settings:
+            removeDefaultHandlers: true
+
+    routes:
+        "app":
+            "/hello-world":
+                get:
+                    service: BrandEmbassyTest\Slim\Sample\HelloWorldRoute
+                    middleware:
+                        - BrandEmbassyTest\Slim\Sample\BeforeRouteMiddleware