Fix Magento Unit Tests to be compatible with PHP 8.1

Author: anzin

app/code/Magento/CatalogWidget/Test/Unit/Block/Product/Widget/ConditionsTest.php

@@ -223,6 +223,7 @@ public function testRender()
         $this->contextMock->expects($this->once())->method('getEnginePool')->willReturn($templateEnginePoolMock);
         $templateEnginePoolMock->expects($this->once())->method('get')->willReturn($templateEngineMock);
         $templateEngineMock->expects($this->once())->method('render')->willReturn('html');
+        $resolverMock->method('getTemplateFileName')->willReturn('');
 
         $this->widgetConditions = $this->objectManagerHelper->getObject(
             Conditions::class,

app/code/Magento/Fedex/Test/Unit/Model/CarrierTest.php

@@ -455,7 +455,7 @@ public function testGetTrackingErrorResponse()
         $this->trackErrorFactory->expects($this->once())
             ->method('create')
             ->willReturn($error);
-
+        $this->serializer->method('serialize')->willReturn('');
         $this->carrier->getTracking($tracking);
         $tracks = $this->carrier->getResult()->getAllTrackings();
 

app/code/Magento/ImportExport/Test/Unit/Model/ImportTest.php

@@ -940,6 +940,7 @@ public function testCreateHistoryReportThrowException()
             ->expects($this->never())
             ->method('getRelativePath');
         $phrase = $this->createMock(Phrase::class);
+        $phrase->method('render')->willReturn('');
         $this->_driver
             ->expects($this->any())
             ->method('fileGetContents')

app/code/Magento/Reports/Test/Unit/Model/ResourceModel/Order/CollectionTest.php

@@ -247,6 +247,7 @@ public function testPrepareSummary($useAggregatedData, $mainTable, $isFilter, $g
         $orderMock = $this->getMockBuilder(Order::class)
             ->disableOriginalConstructor()
             ->getMock();
+        $orderMock->method('getStoreTZOffsetQuery')->willReturn('');
 
         $this->orderFactoryMock
             ->expects($this->any())

lib/internal/Magento/Framework/App/DeploymentConfig.php

@@ -226,7 +226,7 @@ private function flattenParams(array $params, $path = null, array &$flattenResul
                 // allow reading values from env variables
                 // value need to be specified in %env(NAME, "default value")% format
                 // like #env(DB_PASSWORD), #env(DB_NAME, "test")
-                if (preg_match(self::ENV_NAME_PATTERN, $param, $matches)) {
+                if ($param !== null && preg_match(self::ENV_NAME_PATTERN, $param, $matches)) {
                     $param = getenv($matches['name']) ?: ($matches['default'] ?? null);
                 }
 

lib/internal/Magento/Framework/Data/Test/Unit/Form/Element/MultiselectTest.php

@@ -73,6 +73,7 @@ public function testHiddenFieldPresentInMultiSelect()
         $fieldId = 'fieldId';
         $this->_model->setCanBeEmpty(true);
         $this->_model->setName($fieldName);
+        $this->_model->setValue('');
         $this->_model->setId($fieldId);
         $elementHtml = $this->_model->getElementHtml();
         $this->assertStringContainsString(
@@ -91,6 +92,7 @@ public function testHiddenDisabledFieldPresentInMultiSelect()
         $fieldName = 'fieldName';
         $this->_model->setDisabled(true);
         $this->_model->setName($fieldName);
+        $this->_model->setValue('');
         $elementHtml = $this->_model->getElementHtml();
         $this->assertStringContainsString('<input type="hidden" name="' . $fieldName . '_disabled"', $elementHtml);
     }
@@ -106,6 +108,7 @@ public function testHiddenDisabledFieldNotPresentInMultiSelect()
         $fieldName = 'fieldName';
         $this->_model->setDisabled(false);
         $this->_model->setName($fieldName);
+        $this->_model->setValue('');
         $elementHtml = $this->_model->getElementHtml();
         $this->assertStringNotContainsString('<input type="hidden" name="' . $fieldName . '_disabled"', $elementHtml);
     }

lib/internal/Magento/Framework/Filesystem/Driver/File.php

@@ -3,6 +3,7 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+
 namespace Magento\Framework\Filesystem\Driver;
 
 use Magento\Framework\Exception\FileSystemException;
@@ -175,10 +176,13 @@ public function isDirectory($path)
     public function fileGetContents($path, $flag = null, $context = null)
     {
         $filename = $this->getScheme() . $path;
+
         if (!$this->stateful) {
             clearstatcache(false, $filename);
         }
+        $flag = $flag ?? false;
         $result = @file_get_contents($filename, $flag, $context);
+
         if (false === $result) {
             throw new FileSystemException(
                 new Phrase(
@@ -624,10 +628,13 @@ public function touch($path, $modificationTime = null)
      */
     public function filePutContents($path, $content, $mode = null)
     {
+        $mode = $mode ?? 0;
         $result = @file_put_contents($this->getScheme() . $path, $content, $mode);
+
         if ($this->stateful) {
             clearstatcache(true, $this->getScheme() . $path);
         }
+
         if ($result === false) {
             throw new FileSystemException(
                 new Phrase(
@@ -636,6 +643,7 @@ public function filePutContents($path, $content, $mode = null)
                 )
             );
         }
+
         return $result;
     }
 

lib/internal/Magento/Framework/View/Test/Unit/Asset/RepositoryTest.php

@@ -372,12 +372,10 @@ public function testCreateRelated($filePath, $resultFilePath, $module)
 
         $originalAssetMock = $this->getMockBuilder(File::class)
             ->disableOriginalConstructor()
-            ->setMethods(['getModule', 'getContext'])
+            ->onlyMethods(['getModule', 'getContext', 'getFilePath'])
             ->getMock();
-        $originalAssetMock
-            ->expects($this->any())
-            ->method('getContext')
-            ->willReturn($originalContextMock);
+        $originalAssetMock->method('getContext')->willReturn($originalContextMock);
+        $originalAssetMock->method('getFilePath')->willReturn('');
 
         $assetMock = $this->getMockBuilder(File::class)
             ->disableOriginalConstructor()