Merge pull request #3 from Codeception/symfony5-failed

Fixed _failed handling with Symfony 5

Author: Naktibalda

src/Codeception/Lib/InnerBrowser.php

@@ -21,6 +21,7 @@
 use Codeception\Util\ReflectionHelper;
 use Codeception\Util\Uri;
 use Symfony\Component\BrowserKit\Cookie;
+use Symfony\Component\BrowserKit\Exception\BadMethodCallException;
 use Symfony\Component\DomCrawler\Crawler;
 use Symfony\Component\DomCrawler\Field\ChoiceFormField;
 use Symfony\Component\DomCrawler\Field\FileFormField;
@@ -73,8 +74,14 @@ public function _failed(TestInterface $test, $fail)
             'application/xml' => 'xml',
             'text/plain' => 'txt'
         ];
-        
-        $internalResponse = $this->client->getInternalResponse();
+
+        try {
+            $internalResponse = $this->client->getInternalResponse();
+        } catch (BadMethodCallException $e) {
+            //Symfony 5 throws exception if request() method threw an exception.
+            //The "request()" method must be called before "Symfony\Component\BrowserKit\AbstractBrowser::getInternalResponse()"
+            $internalResponse = false;
+        }
 
         $responseContentType = $internalResponse ? $internalResponse->getHeader('content-type') : '';
         list($responseMimeType) = explode(';', $responseContentType);