Force uppercase on methods/verbs.

Author: gwillz

src/Modes/RouterChunkedMode.php

@@ -44,6 +44,7 @@ class RouterChunkedMode extends Router
     /** @inheritdoc */
     public function find(string $method, string $path): ?Action
     {
+        $method = strtoupper($method);
         $route = "{$method} {$path}";
 
         foreach ($this->patterns as [$pattern, $rules]) {

src/Modes/RouterRegexMode.php

@@ -25,6 +25,8 @@ class RouterRegexMode extends Router
     /** @inheritdoc */
     public function find(string $method, string $path): ?Action
     {
+        $method = strtoupper($method);
+
         foreach ($this->routes as $rule => $target) {
             $pattern = $this->expandRule($rule);
 

src/Modes/RouterSingleMode.php

@@ -49,6 +49,8 @@ class RouterSingleMode extends Router
     /** @inheritdoc */
     public function find(string $method, string $path): ?Action
     {
+        $method = strtoupper($method);
+
         foreach ($this->patterns as $pattern => [$rule, $target]) {
             $matches = [];
             if (!preg_match($pattern, "{$method} {$path}", $matches)) continue;

tests/RouterTestCase.php

@@ -65,6 +65,47 @@ public function testMultiMethod()
     }
 
 
+    public function testCaseInsensitive()
+    {
+        $router = Router::create($this->router->config);
+        $router->config->case_insensitive = false;
+        $router->load($this->routes);
+
+        // Good.
+        $action = $router->find('GET', '/get/123');
+        $this->assertNotNull($action);
+        $this->assertEquals('get route', $action->target);
+
+        // Bad.
+        $action = $router->find('GET', '/GET/123');
+        $this->assertNull($action);
+
+        // Methods are not case sensitive.
+        $action = $router->find('get', '/get/123');
+        $this->assertNotNull($action);
+        $this->assertEquals('get route', $action->target);
+
+        $router = Router::create($router->config);
+        $router->config->case_insensitive = true;
+        $router->load($this->routes);
+
+        // Still good.
+        $action = $router->find('GET', '/get/123');
+        $this->assertNotNull($action);
+        $this->assertEquals('get route', $action->target);
+
+        // Now this works.
+        $action = $router->find('GET', '/GET/123');
+        $this->assertNotNull($action);
+        $this->assertEquals('get route', $action->target);
+
+        // Also still good.
+        $action = $router->find('get', '/GeT/123');
+        $this->assertNotNull($action);
+        $this->assertEquals('get route', $action->target);
+    }
+
+
     public function testVariables()
     {
         $action = $this->router->find('GET', '/abc/12345');