Fix #9 Add support for optional route parameters (#10)

Author: mnapoli

src/config.php

@@ -7,6 +7,7 @@
 use Invoker\Invoker;
 use Invoker\ParameterResolver\AssociativeArrayResolver;
 use Invoker\ParameterResolver\Container\TypeHintContainerResolver;
+use Invoker\ParameterResolver\DefaultValueResolver;
 use Invoker\ParameterResolver\ResolverChain;
 use Slim\Http\Headers;
 use Slim\Http\Request;
@@ -62,6 +63,8 @@
             new AssociativeArrayResolver,
             // Then inject services by type-hints for those that weren't resolved
             new TypeHintContainerResolver($c),
+            // Then fall back on parameters default values for optional route parameters
+            new DefaultValueResolver(),
         ];
         return new Invoker(new ResolverChain($resolvers), $c);
     },

tests/RoutingTest.php

@@ -42,6 +42,36 @@ public function injects_request_path_parameters()
         $this->assertEquals('Hello matt', $response->getBody()->__toString());
     }
 
+    /**
+     * @test
+     */
+    public function injects_optional_path_parameter()
+    {
+        $app = new App;
+        $app->get('/[{name}]', function ($response, $name = null) {
+            $response->getBody()->write('Hello ' . $name);
+            return $response;
+        });
+
+        $response = $app->callMiddlewareStack(RequestFactory::create('/matt'), new Response);
+        $this->assertEquals('Hello matt', (string) $response->getBody());
+    }
+
+    /**
+     * @test
+     */
+    public function injects_default_value_in_optional_path_parameter()
+    {
+        $app = new App;
+        $app->get('/[{name}]', function ($response, $name = 'john doe') {
+            $response->getBody()->write('Hello ' . $name);
+            return $response;
+        });
+
+        $response = $app->callMiddlewareStack(RequestFactory::create('/'), new Response);
+        $this->assertEquals('Hello john doe', (string) $response->getBody());
+    }
+
     /**
      * @test
      */