Merge branch '4.4'

* 4.4:
  Minor fixes
  [Mailer] fixed dispatcher not available in Mailer
  [HttpClient] Minor fixes
  Use namespaced Phpunit classes
  Add polyfill for PhpUnit namespace
  [Messenger] Fixed ConsumeMessagesCommand configuration
  [Form] remove leftover int child phpdoc
  Support DateTimeInterface in IntlDateFormatter::format
  [PhpUnitBridge] fixed PHPUnit 8.3 compatibility: method handleError was renamed to __invoke
  [Yaml] Removed unused $nullAsTilde property
  [Security] add support for opportunistic password migrations
  [Lock] Legacy test should implement legacy interface
  fixed phpdocs
  Use PHPunit assertion
  [Intl] Order alpha2 to alpha3 mapping + phpdoc fixes

Author: nicolas-grekas

Tests/FilesystemTest.php

@@ -153,7 +153,7 @@ public function testCopyCreatesTargetDirectoryIfItDoesNotExist()
 
         $this->filesystem->copy($sourceFilePath, $targetFilePath);
 
-        $this->assertTrue(is_dir($targetFileDirectory));
+        $this->assertDirectoryExists($targetFileDirectory);
         $this->assertFileExists($targetFilePath);
         $this->assertStringEqualsFile($targetFilePath, 'SOURCE FILE');
     }
@@ -185,7 +185,7 @@ public function testMkdirCreatesDirectoriesRecursively()
 
         $this->filesystem->mkdir($directory);
 
-        $this->assertTrue(is_dir($directory));
+        $this->assertDirectoryExists($directory);
     }
 
     public function testMkdirCreatesDirectoriesFromArray()
@@ -197,9 +197,9 @@ public function testMkdirCreatesDirectoriesFromArray()
 
         $this->filesystem->mkdir($directories);
 
-        $this->assertTrue(is_dir($basePath.'1'));
-        $this->assertTrue(is_dir($basePath.'2'));
-        $this->assertTrue(is_dir($basePath.'3'));
+        $this->assertDirectoryExists($basePath.'1');
+        $this->assertDirectoryExists($basePath.'2');
+        $this->assertDirectoryExists($basePath.'3');
     }
 
     public function testMkdirCreatesDirectoriesFromTraversableObject()
@@ -211,9 +211,9 @@ public function testMkdirCreatesDirectoriesFromTraversableObject()
 
         $this->filesystem->mkdir($directories);
 
-        $this->assertTrue(is_dir($basePath.'1'));
-        $this->assertTrue(is_dir($basePath.'2'));
-        $this->assertTrue(is_dir($basePath.'3'));
+        $this->assertDirectoryExists($basePath.'1');
+        $this->assertDirectoryExists($basePath.'2');
+        $this->assertDirectoryExists($basePath.'3');
     }
 
     public function testMkdirCreatesDirectoriesFails()
@@ -347,7 +347,7 @@ public function testRemoveCleansInvalidLinks()
 
         // create symlink to dir using trailing forward slash
         $this->filesystem->symlink($basePath.'dir/', $basePath.'dir-link');
-        $this->assertTrue(is_dir($basePath.'dir-link'));
+        $this->assertDirectoryExists($basePath.'dir-link');
 
         // create symlink to nonexistent dir
         rmdir($basePath.'dir');
@@ -807,7 +807,7 @@ public function testRemoveSymlink()
 
         $this->assertFalse(is_link($link));
         $this->assertFalse(is_file($link));
-        $this->assertFalse(is_dir($link));
+        $this->assertDirectoryNotExists($link);
     }
 
     public function testSymlinkIsOverwrittenIfPointsToDifferentTarget()
@@ -1136,8 +1136,8 @@ public function testMirrorCopiesFilesAndDirectoriesRecursively()
 
         $this->filesystem->mirror($sourcePath, $targetPath);
 
-        $this->assertTrue(is_dir($targetPath));
-        $this->assertTrue(is_dir($targetPath.'directory'));
+        $this->assertDirectoryExists($targetPath);
+        $this->assertDirectoryExists($targetPath.'directory');
         $this->assertFileEquals($file1, $targetPath.'directory'.\DIRECTORY_SEPARATOR.'file1');
         $this->assertFileEquals($file2, $targetPath.'file2');
 
@@ -1170,7 +1170,7 @@ public function testMirrorCreatesEmptyDirectory()
 
         $this->filesystem->mirror($sourcePath, $targetPath);
 
-        $this->assertTrue(is_dir($targetPath));
+        $this->assertDirectoryExists($targetPath);
 
         $this->filesystem->remove($sourcePath);
     }
@@ -1189,7 +1189,7 @@ public function testMirrorCopiesLinks()
 
         $this->filesystem->mirror($sourcePath, $targetPath);
 
-        $this->assertTrue(is_dir($targetPath));
+        $this->assertDirectoryExists($targetPath);
         $this->assertFileEquals($sourcePath.'file1', $targetPath.'link1');
         $this->assertTrue(is_link($targetPath.\DIRECTORY_SEPARATOR.'link1'));
     }
@@ -1209,7 +1209,7 @@ public function testMirrorCopiesLinkedDirectoryContents()
 
         $this->filesystem->mirror($sourcePath, $targetPath);
 
-        $this->assertTrue(is_dir($targetPath));
+        $this->assertDirectoryExists($targetPath);
         $this->assertFileEquals($sourcePath.'/nested/file1.txt', $targetPath.'link1/file1.txt');
         $this->assertTrue(is_link($targetPath.\DIRECTORY_SEPARATOR.'link1'));
     }
@@ -1233,7 +1233,7 @@ public function testMirrorCopiesRelativeLinkedContents()
 
         $this->filesystem->mirror($sourcePath, $targetPath);
 
-        $this->assertTrue(is_dir($targetPath));
+        $this->assertDirectoryExists($targetPath);
         $this->assertFileEquals($sourcePath.'/nested/file1.txt', $targetPath.'link1/file1.txt');
         $this->assertTrue(is_link($targetPath.\DIRECTORY_SEPARATOR.'link1'));
         $this->assertEquals('\\' === \DIRECTORY_SEPARATOR ? realpath($sourcePath.'\nested') : 'nested', readlink($targetPath.\DIRECTORY_SEPARATOR.'link1'));
@@ -1256,7 +1256,7 @@ public function testMirrorContentsWithSameNameAsSourceOrTargetWithoutDeleteOptio
 
         chdir($oldPath);
 
-        $this->assertTrue(is_dir($targetPath));
+        $this->assertDirectoryExists($targetPath);
         $this->assertFileExists($targetPath.'source');
         $this->assertFileExists($targetPath.'target');
     }
@@ -1281,7 +1281,7 @@ public function testMirrorContentsWithSameNameAsSourceOrTargetWithDeleteOption()
 
         chdir($oldPath);
 
-        $this->assertTrue(is_dir($targetPath));
+        $this->assertDirectoryExists($targetPath);
         $this->assertFileExists($targetPath.'source');
         $this->assertFileNotExists($targetPath.'target');
     }