Browse Source

Fix failing tests in ViewBuilder.

Can't use assertSame() as response is immutable.
mark_story 8 years ago
parent
commit
2fa8735ad2

+ 1 - 1
src/Error/ExceptionRenderer.php

@@ -200,8 +200,8 @@ class ExceptionRenderer implements ExceptionRendererInterface
         if ($unwrapped instanceof CakeException && $isDebug) {
             $this->controller->set($unwrapped->getAttributes());
         }
-
         $this->controller->response = $response;
+
         return $this->_outputMessage($template);
     }
 

+ 1 - 1
src/Routing/Filter/AssetFilter.php

@@ -131,7 +131,7 @@ class AssetFilter extends DispatcherFilter
     protected function _deliverAsset(ServerRequest $request, Response $response, $assetFile, $ext)
     {
         $compressionEnabled = $response->compress();
-        if ($response->getType($ext) === $ext) {
+        if ($response->getType() === $ext) {
             $contentType = 'application/octet-stream';
             $agent = $request->getEnv('HTTP_USER_AGENT');
             if (preg_match('%Opera(/| )([0-9].[0-9]{1,2})%', $agent) || preg_match('/MSIE ([0-9].[0-9]{1,2})/', $agent)) {

+ 5 - 3
tests/TestCase/View/ViewBuilderTest.php

@@ -14,6 +14,8 @@
  */
 namespace Cake\Test\TestCase\View;
 
+use Cake\Http\Response;
+use Cake\Http\ServerRequest;
 use Cake\TestSuite\TestCase;
 use Cake\View\ViewBuilder;
 
@@ -108,8 +110,8 @@ class ViewBuilderTest extends TestCase
      */
     public function testBuildComplete()
     {
-        $request = $this->getMockBuilder('Cake\Http\ServerRequest')->getMock();
-        $response = $this->getMockBuilder('Cake\Http\Response')->getMock();
+        $request = new ServerRequest();
+        $response = new Response();
         $events = $this->getMockBuilder('Cake\Event\EventManager')->getMock();
 
         $builder = new ViewBuilder();
@@ -136,7 +138,7 @@ class ViewBuilderTest extends TestCase
         $this->assertEquals('TestPlugin', $view->plugin);
         $this->assertEquals('TestTheme', $view->theme);
         $this->assertSame($request, $view->request);
-        $this->assertSame($response, $view->response);
+        $this->assertInstanceOf(Response::class, $view->response);
         $this->assertSame($events, $view->getEventManager());
         $this->assertSame(['one' => 'value'], $view->viewVars);
         $this->assertInstanceOf('Cake\View\Helper\HtmlHelper', $view->Html);