chore: don't use a mock in testIsSuccessfulException

Author: alexislefebvre

tests/App/Controller/DefaultController.php

@@ -107,4 +107,9 @@ public function embeddedAction(): Response
     {
         return new Response('Embedded Content', Response::HTTP_OK);
     }
+
+    public function exceptionAction(): Response
+    {
+        throw new \Exception('foo');
+    }
 }

tests/App/routing.yml

@@ -25,3 +25,7 @@ liipfunctionaltestbundle_json:
     defaults: { _controller: 'Liip\Acme\Tests\App\Controller\DefaultController::jsonAction' }
     requirements:
         userId: \d+
+
+liipfunctionaltestbundle_exception:
+    path:  /exception
+    defaults: { _controller: 'Liip\Acme\Tests\App\Controller\DefaultController::exceptionAction' }

tests/Test/WebTestCaseTest.php

@@ -16,7 +16,6 @@
 use Doctrine\Common\Annotations\Annotation\IgnoreAnnotation;
 use Liip\Acme\Tests\App\AppKernel;
 use Liip\FunctionalTestBundle\Test\WebTestCase;
-use function method_exists;
 use PHPUnit\Framework\AssertionFailedError;
 
 /**
@@ -243,25 +242,16 @@ public function test404Error(): void
      */
     public function testIsSuccessfulException(): void
     {
-        $mockBuilder = $this->getMockBuilder('Symfony\Component\HttpFoundation\Response')
-            ->disableOriginalConstructor();
-        if (method_exists($mockBuilder, 'setMethods')) {
-            $mockBuilder->setMethods(['getContent']);
-        } else { // phpunit 10
-            $mockBuilder->onlyMethods(['getContent']);
-        }
-        $response = $mockBuilder
-            ->getMock();
+        $path = '/exception';
 
-        $response->expects($this->any())
-            ->method('getContent')
-            ->will($this->throwException(new \Exception('foo')));
+        $client = static::makeClient();
+        $client->request('GET', $path);
 
         try {
-            $this->isSuccessful($response);
+            $this->isSuccessful($client->getResponse());
         } catch (AssertionFailedError $e) {
             $string = <<<'EOF'
-The Response was not successful: foo
+The Response was not successful: foo (500 Internal Server Error)
 Failed asserting that false is true.
 EOF;
             $this->assertSame($string, $e->getMessage());