fix tests

liip · Aug 3, 2019 · 33c290f · 33c290f
1 parent 6683cac
commit 33c290f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/Tests/Binary/Loader/ChainLoaderTest.php b/Tests/Binary/Loader/ChainLoaderTest.php
@@ -164,7 +164,7 @@ private function getChainLoader(array $paths = [], array $loaders = null): Chain
      * @param FileBinary|mixed $return
      * @param string|null      $message
      */
-    private function assertValidLoaderFindReturn($return, string $message = null): void
+    private function assertValidLoaderFindReturn($return, string $message = ''): void
     {
         $this->assertInstanceOf(FileBinary::class, $return, $message);
         $this->assertStringStartsWith('text/', $return->getMimeType(), $message);

diff --git a/Tests/Binary/Loader/FileSystemLoaderTest.php b/Tests/Binary/Loader/FileSystemLoaderTest.php
@@ -205,7 +205,7 @@ private function getFileSystemLoader(array $roots = [], LocatorInterface $locato
      * @param FileBinary|mixed $return
      * @param string|null      $message
      */
-    private function assertValidLoaderFindReturn($return, $message = null)
+    private function assertValidLoaderFindReturn($return, string $message = ''): void
     {
         $this->assertInstanceOf(FileBinary::class, $return, $message);
         $this->assertStringStartsWith('text/', $return->getMimeType(), $message);

diff --git a/Tests/DependencyInjection/Compiler/AbstractCompilerPassTestCase.php b/Tests/DependencyInjection/Compiler/AbstractCompilerPassTestCase.php
@@ -94,7 +94,7 @@ protected function expectContainerLogMethodCalledOnce(\PHPUnit_Framework_MockObj
      * @param Definition  $definition
      * @param string|null $message
      */
-    protected function assertDefinitionMethodCallsNone(Definition $definition, $message = null)
+    protected function assertDefinitionMethodCallsNone(Definition $definition, $message = '')
     {
         $this->assertDefinitionMethodCallCount(0, $definition, $message);
     }
@@ -104,7 +104,7 @@ protected function assertDefinitionMethodCallsNone(Definition $definition, $mess
      * @param Definition  $definition
      * @param string|null $message
      */
-    protected function assertDefinitionMethodCallCount($expect, Definition $definition, $message = null)
+    protected function assertDefinitionMethodCallCount($expect, Definition $definition, $message = '')
     {
         $this->assertCount($expect, $definition->getMethodCalls(), $message);
     }

diff --git a/Tests/Functional/Binary/Locator/FileSystemLocatorTest.php b/Tests/Functional/Binary/Locator/FileSystemLocatorTest.php
@@ -100,7 +100,7 @@ private function assertBarBundleResourcesExist(LocatorInterface $locator, $only
      * @param string           $expectedContents
      * @param string|null      $message
      */
-    private function assertLocatedFileContentsStartsWith(LocatorInterface $locator, $filePath, $expectedContents, $message = null)
+    private function assertLocatedFileContentsStartsWith(LocatorInterface $locator, $filePath, $expectedContents, $message = '')
     {
         $this->assertStringStartsWith($expectedContents, file_get_contents($locator->locate($filePath)), $message);
     }