Merge pull request #153 from ppi/fixing-app-run-test

Switching app->run() to app->dispatch() on unit tests to avoid STDOUT output

src/App.php

@@ -246,9 +246,6 @@ public function run(HttpRequest $request = null, HttpResponse $response = null)
             $this->boot();
         }
 
-        $request = $request ?: HttpRequest::createFromGlobals();
-        $response = $response ?: new HttpResponse();
-
         $response = $this->dispatch($request, $response);
         $response->send();
 

tests/AppTest.php

@@ -10,11 +10,12 @@
 
 namespace PPI\FrameworkTest;
 
-use PPI\Framework\Module\Controller\ControllerResolver;
 use PPI\Framework\ServiceManager\ServiceManager;
 use PPI\FrameworkTest\Fixtures\AppForDispatchTest;
 use PPI\FrameworkTest\Fixtures\AppForTest;
 use Symfony\Component\HttpFoundation\Response;
+use PPI\Framework\Http\Request as HttpRequest;
+use PPI\Framework\Http\Response as HttpResponse;
 
 /**
  * Class AppTest.
@@ -94,7 +95,7 @@ public function testGetName()
         $this->assertEquals('testName', $app->getName());
     }
 
-    public function testRun()
+    public function testDispatch()
     {
         $app = new AppForDispatchTest(array(
             'environment'   => 'test',
@@ -123,7 +124,10 @@ public function testRun()
         $sm->set('ControllerResolver', $mockControllerResolver);
         $app->setServiceManager($sm);
 
-        $response = $app->run();
+        $request = HttpRequest::createFromGlobals();
+        $response = new HttpResponse();
+
+        $response = $app->dispatch($request, $response);
         $this->assertInstanceOf('\Symfony\Component\HttpFoundation\Response', $response);
         $this->assertEquals($response->getContent(), 'Working Response');
     }