updated to minimum symfony 4.4 and php 7.1, fixed tests and removed deprecations

Author: rbaarsma

Tests/Factory/AbstractHttpMessageFactoryTest.php

@@ -34,7 +34,7 @@ abstract class AbstractHttpMessageFactoryTest extends TestCase
      */
     abstract protected function buildHttpMessageFactory();
 
-    public function setup()
+    public function setUp(): void
     {
         $this->factory = $this->buildHttpMessageFactory();
         $this->tmpDir = sys_get_temp_dir();
@@ -158,7 +158,7 @@ public function testCreateResponse()
         $this->assertEquals(['3.4'], $psrResponse->getHeader('X-Symfony'));
 
         $cookieHeader = $psrResponse->getHeader('Set-Cookie');
-        $this->assertInternalType('array', $cookieHeader);
+        $this->assertIsArray($cookieHeader);
         $this->assertCount(1, $cookieHeader);
         $this->assertRegExp('{city=Lille; expires=Wed, 13-Jan-2021 22:23:01 GMT;( max-age=\d+;)? path=/; httponly}i', $cookieHeader[0]);
     }

Tests/Factory/HttpFoundationFactoryTest.php

@@ -20,6 +20,7 @@
 use Symfony\Bridge\PsrHttpMessage\Tests\Fixtures\UploadedFile;
 use Symfony\Bridge\PsrHttpMessage\Tests\Fixtures\Uri;
 use Symfony\Component\HttpFoundation\Cookie;
+use Symfony\Component\HttpFoundation\File\Exception\FileException;
 
 /**
  * @author Kévin Dunglas <[email protected]>
@@ -32,7 +33,7 @@ class HttpFoundationFactoryTest extends TestCase
     /** @var string */
     private $tmpDir;
 
-    public function setup()
+    public function setUp(): void
     {
         $this->factory = new HttpFoundationFactory();
         $this->tmpDir = sys_get_temp_dir();
@@ -80,7 +81,7 @@ public function testCreateRequest()
         $this->assertEquals('France', $symfonyRequest->server->get('country'));
         $this->assertEquals('The body', $symfonyRequest->getContent());
         $this->assertEquals('1.0', $symfonyRequest->headers->get('X-Dunglas-API-Platform'));
-        $this->assertEquals(['a', 'b'], $symfonyRequest->headers->get('X-data', null, false));
+        $this->assertEquals(['a', 'b'], $symfonyRequest->headers->all('X-data'));
     }
 
     public function testCreateRequestWithNullParsedBody()
@@ -160,12 +161,11 @@ public function testCreateUploadedFile()
         $this->assertEquals('An uploaded file.', file_get_contents($this->tmpDir.'/'.$uniqid));
     }
 
-    /**
-     * @expectedException        \Symfony\Component\HttpFoundation\File\Exception\FileException
-     * @expectedExceptionMessage The file "e" could not be written on disk.
-     */
     public function testCreateUploadedFileWithError()
     {
+        $this->expectException(FileException::class);
+        $this->expectExceptionMessage('The file "e" could not be written on disk.');
+
         $uploadedFile = $this->createUploadedFile('Error.', UPLOAD_ERR_CANT_WRITE, 'e', 'text/plain');
         $symfonyUploadedFile = $this->callCreateUploadedFile($uploadedFile);
 

Tests/Functional/CovertTest.php

@@ -36,7 +36,7 @@ class CovertTest extends TestCase
 {
     private $tmpDir;
 
-    public function setup()
+    public function setUp(): void
     {
         if (!class_exists('Nyholm\Psr7\ServerRequest')) {
             $this->markTestSkipped('nyholm/psr7 is not installed.');

composer.json

@@ -18,10 +18,10 @@
     "require": {
         "php": "^7.1",
         "psr/http-message": "^1.0",
-        "symfony/http-foundation": "^3.4 || ^4.0"
+        "symfony/http-foundation": "^4.4 || ^5.0"
     },
     "require-dev": {
-        "symfony/phpunit-bridge": "^3.4.20 || ^4.0",
+        "symfony/phpunit-bridge": "^4.4 || ^5.0",
         "nyholm/psr7": "^1.1",
         "zendframework/zend-diactoros": "^1.4.1 || ^2.0"
     },

phpunit.xml.dist

@@ -8,7 +8,6 @@
          convertWarningsToExceptions="true"
          processIsolation="false"
          stopOnFailure="false"
-         syntaxCheck="false"
          bootstrap="vendor/autoload.php"
 >
     <testsuites>