Allow to definte the factories in the constructor

CHANGELOG.md

@@ -5,6 +5,16 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## [Unreleased]
+
+### Fixed
+
+- Use `phpstan` as a dev dependency to detect bugs
+
+### Added
+
+- Added `responseFactory` option to `__construct`
+
 ## [1.0.0] - 2018-09-23
 
 ### Added

composer.json

@@ -24,7 +24,8 @@
         "phpunit/phpunit": "^6.0",
         "zendframework/zend-diactoros": "^1.3",
         "friendsofphp/php-cs-fixer": "^2.0",
-        "squizlabs/php_codesniffer": "^3.0"
+        "squizlabs/php_codesniffer": "^3.0",
+        "phpstan/phpstan": "^0.9.2|^0.10.3"
     },
     "autoload": {
         "psr-4": {
@@ -42,6 +43,7 @@
             "phpcs"
         ],
         "cs-fix": "php-cs-fixer fix .",
-        "coverage": "phpunit --coverage-html=coverage"
+        "coverage": "phpunit --coverage-html=coverage",
+        "analyse": "phpstan analyse --no-progress --level 7 src"
     }
 }

src/BasePathRouter.php

@@ -4,6 +4,7 @@
 namespace Middlewares;
 
 use Middlewares\Utils\Factory;
+use Psr\Http\Message\ResponseFactoryInterface;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
 use Psr\Http\Server\MiddlewareInterface;
@@ -26,14 +27,20 @@ class BasePathRouter implements MiddlewareInterface
      */
     private $continueOnError = false;
 
+    /**
+     * @var ResponseFactoryInterface
+     */
+    private $responseFactory;
+
     /**
      * @var string Attribute name for handler reference
      */
     private $attribute = 'request-handler';
 
-    public function __construct(array $middlewares)
+    public function __construct(array $middlewares, ResponseFactoryInterface $responseFactory = null)
     {
         $this->middlewares = $middlewares;
+        $this->responseFactory = $responseFactory ?: Factory::getResponseFactory();
 
         // Make sure the longest path prefixes are matched first
         // (otherwise, a path /foo would always match, even when /foo/bar
@@ -100,7 +107,7 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
             return $handler->handle($request);
         }
 
-        return Factory::createResponse(404);
+        return $this->responseFactory->createResponse(404);
     }
 
     private function unprefixedRequest(ServerRequestInterface $request, string $prefix): ServerRequestInterface