Reworked LanguageTransformerTest (#1492)

* Reworked LanguageTransformerTest

* [PHPStan] regenerated baseline

Author: ciastektk

phpstan-baseline.neon

@@ -15876,66 +15876,6 @@ parameters:
 			count: 1
 			path: tests/lib/Form/DataTransformer/FieldValueTransformerTest.php
 
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testReverseTransformWithLanguageCode\(\) has no return type specified\.$#'
-			identifier: missingType.return
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testReverseTransformWithNotFoundException\(\) has no return type specified\.$#'
-			identifier: missingType.return
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testReverseTransformWithNull\(\) has no return type specified\.$#'
-			identifier: missingType.return
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testTransform\(\) has no return type specified\.$#'
-			identifier: missingType.return
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testTransform\(\) has parameter \$expected with no type specified\.$#'
-			identifier: missingType.parameter
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testTransform\(\) has parameter \$value with no type specified\.$#'
-			identifier: missingType.parameter
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testTransformWithInvalidInput\(\) has no return type specified\.$#'
-			identifier: missingType.return
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:testTransformWithInvalidInput\(\) has parameter \$value with no type specified\.$#'
-			identifier: missingType.parameter
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:transformDataProvider\(\) return type has no value type specified in iterable type array\.$#'
-			identifier: missingType.iterableValue
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
-		-
-			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LanguageTransformerTest\:\:transformWithInvalidInputDataProvider\(\) return type has no value type specified in iterable type array\.$#'
-			identifier: missingType.iterableValue
-			count: 1
-			path: tests/lib/Form/DataTransformer/LanguageTransformerTest.php
-
 		-
 			message: '#^Method Ibexa\\Tests\\AdminUi\\Form\\DataTransformer\\LocationTransformerTest\:\:testReverseTransformWithId\(\) has no return type specified\.$#'
 			identifier: missingType.return

tests/lib/Form/DataTransformer/LanguageTransformerTest.php

@@ -17,17 +17,23 @@
 
 class LanguageTransformerTest extends TestCase
 {
+    /** @var \Ibexa\Contracts\Core\Repository\LanguageService&\PHPUnit\Framework\MockObject\MockObject */
+    private LanguageService $languageService;
+
+    protected function setUp(): void
+    {
+        $this->languageService = $this->createMock(LanguageService::class);
+    }
+
     /**
      * @dataProvider transformDataProvider
      *
-     * @param $value
-     * @param $expected
+     * @param \Ibexa\Contracts\Core\Repository\Values\Content\Language|null $value
+     * @param string|null $expected
      */
-    public function testTransform($value, $expected)
+    public function testTransform($value, $expected): void
     {
-        /** @var \Ibexa\Contracts\Core\Repository\LanguageService|\PHPUnit\Framework\MockObject\MockObject $languageService */
-        $languageService = $this->createMock(LanguageService::class);
-        $transformer = new LanguageTransformer($languageService);
+        $transformer = new LanguageTransformer($this->languageService);
 
         $result = $transformer->transform($value);
 
@@ -37,68 +43,67 @@ public function testTransform($value, $expected)
     /**
      * @dataProvider transformWithInvalidInputDataProvider
      *
-     * @param $value
+     * @param mixed $value
      */
-    public function testTransformWithInvalidInput($value)
+    public function testTransformWithInvalidInput($value): void
     {
-        /** @var \Ibexa\Contracts\Core\Repository\LanguageService|\PHPUnit\Framework\MockObject\MockObject $languageService */
-        $languageService = $this->createMock(LanguageService::class);
-        $transformer = new LanguageTransformer($languageService);
+        $transformer = new LanguageTransformer($this->languageService);
 
         $this->expectException(TransformationFailedException::class);
         $this->expectExceptionMessage('Expected a ' . Language::class . ' object.');
 
+        /** @phpstan-ignore method.resultUnused */
         $transformer->transform($value);
     }
 
-    public function testReverseTransformWithLanguageCode()
+    public function testReverseTransformWithLanguageCode(): void
     {
-        /** @var \Ibexa\Contracts\Core\Repository\LanguageService|\PHPUnit\Framework\MockObject\MockObject $languageService */
-        $languageService = $this->createMock(LanguageService::class);
-        $languageService->expects(self::once())
+        $this->languageService
+            ->expects(self::once())
             ->method('loadLanguage')
             ->with('eng-GB')
             ->willReturn(new Language(['languageCode' => 'eng-GB']));
 
-        $transformer = new LanguageTransformer($languageService);
+        $transformer = new LanguageTransformer($this->languageService);
 
         $result = $transformer->reverseTransform('eng-GB');
 
         $this->assertEquals(new Language(['languageCode' => 'eng-GB']), $result);
     }
 
-    public function testReverseTransformWithNull()
+    public function testReverseTransformWithNull(): void
     {
-        /** @var \Ibexa\Contracts\Core\Repository\LanguageService|\PHPUnit\Framework\MockObject\MockObject $languageService */
-        $languageService = $this->createMock(LanguageService::class);
-        $languageService->expects(self::never())
+        $this->languageService
+            ->expects(self::never())
             ->method('loadLanguageById');
 
-        $transformer = new LanguageTransformer($languageService);
+        $transformer = new LanguageTransformer($this->languageService);
 
         $result = $transformer->reverseTransform(null);
 
         $this->assertNull($result);
     }
 
-    public function testReverseTransformWithNotFoundException()
+    public function testReverseTransformWithNotFoundException(): void
     {
-        $this->expectException(TransformationFailedException::class);
-        $this->expectExceptionMessage('Language not found');
-
-        /** @var \Ibexa\Contracts\Core\Repository\LanguageService|\PHPUnit\Framework\MockObject\MockObject $languageService */
-        $languageService = $this->createMock(LanguageService::class);
-        $languageService->method('loadLanguage')
+        $this->languageService
+            ->method('loadLanguage')
             ->will($this->throwException(new class('Language not found') extends NotFoundException {
             }));
 
-        $transformer = new LanguageTransformer($languageService);
+        $transformer = new LanguageTransformer($this->languageService);
+
+        $this->expectException(TransformationFailedException::class);
+        $this->expectExceptionMessage('Language not found');
 
         $transformer->reverseTransform('pol-PL');
     }
 
     /**
-     * @return array
+     * @return array<string, array{
+     *     \Ibexa\Contracts\Core\Repository\Values\Content\Language|null,
+     *     string|null,
+     * }>
      */
     public function transformDataProvider(): array
     {
@@ -111,7 +116,7 @@ public function transformDataProvider(): array
     }
 
     /**
-     * @return array
+     * @return array<string, mixed>
      */
     public function transformWithInvalidInputDataProvider(): array
     {