Runner should not accept a request to run()

That's what the factory is for!

Author: weierophinney

src/RequestHandlerRunner.php

@@ -73,10 +73,10 @@ public function __construct(
     /**
      * Run the application
      */
-    public function run(ServerRequestInterface $request = null) : void
+    public function run() : void
     {
         try {
-            $request = $request ?: ($this->serverRequestFactory)();
+            $request = ($this->serverRequestFactory)();
         } catch (Throwable $e) {
             // Error in generating the request
             $this->emitMarshalServerRequestException($e);

test/RequestHandlerRunnerTest.php

@@ -80,34 +80,4 @@ public function testRunPassesRequestGeneratedByRequestFactoryToHandleWhenNoReque
 
         $this->assertNull($runner->run());
     }
-
-    public function testRunPassesRequestPassedDuringInvocationToHandler()
-    {
-        $request = $this->prophesize(ServerRequestInterface::class)->reveal();
-
-        $serverRequestFactory = function () {
-            Assert::fail('Should never hit server request factory');
-        };
-
-        $errorResponseGenerator = function ($e) {
-            Assert::fail('Should never hit error response generator');
-        };
-
-        $response = $this->prophesize(ResponseInterface::class)->reveal();
-
-        $handler = $this->prophesize(RequestHandlerInterface::class);
-        $handler->handle($request)->willReturn($response);
-
-        $emitter = $this->prophesize(EmitterInterface::class);
-        $emitter->emit($response)->shouldBeCalled();
-
-        $runner = new RequestHandlerRunner(
-            $handler->reveal(),
-            $emitter->reveal(),
-            $serverRequestFactory,
-            $errorResponseGenerator
-        );
-
-        $this->assertNull($runner->run($request));
-    }
 }