Fix

Author: VincentLanglet

tests/Admin/AdminTest.php

@@ -13,11 +13,13 @@
 
 namespace Sonata\ClassificationBundle\Tests\Admin;
 
+use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 use Sonata\AdminBundle\Admin\AbstractAdmin;
 use Sonata\AdminBundle\Admin\AdminExtensionInterface;
 use Sonata\ClassificationBundle\Admin\ContextAdmin;
 use Sonata\ClassificationBundle\Admin\ContextAwareAdmin;
+use Sonata\ClassificationBundle\Model\ContextAwareInterface;
 use Sonata\ClassificationBundle\Model\ContextManagerInterface;
 
 final class AdminTest extends TestCase
@@ -44,13 +46,15 @@ public function testGetPersistentParametersWithNoExtension(): void
             'hide_context' => 0,
         ];
 
+        /** @phpstan-extends ContextAwareAdmin<ContextAwareInterface> */
         $admin = new class($this->contextManager) extends ContextAwareAdmin {};
 
         static::assertSame($expected, $admin->getPersistentParameters());
     }
 
     public function testGetPersistentParametersWithValidExtension(): void
     {
+        /** @phpstan-extends ContextAwareAdmin<ContextAwareInterface> */
         $admin = new class($this->contextManager) extends ContextAwareAdmin {
             public function __construct(ContextManagerInterface $contextManager)
             {

tests/Controller/CategoryAdminControllerTest.php

@@ -266,7 +266,7 @@ public function testTreeAction(string|false $context, array $categories): void
         $categoriesMock = [];
         foreach ($categories as $category) {
             $categoryMock = new class extends Category {
-                public function getId()
+                public function getId(): int
                 {
                     return 42;
                 }

tests/Entity/CategoryTest.php

@@ -29,7 +29,7 @@ public function testSetterGetter(): void
         $context = $this->createMock(ContextInterface::class);
 
         $category = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -67,14 +67,14 @@ public function getId()
     public function testParent(): void
     {
         $parent = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 33;
             }
         };
 
         $category = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -87,19 +87,19 @@ public function getId()
     public function testChildren(): void
     {
         $cat1 = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 1;
             }
         };
         $cat2 = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 2;
             }
         };
         $cat3 = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 3;
             }
@@ -108,7 +108,7 @@ public function getId()
         $context = $this->createMock(ContextInterface::class);
 
         $category = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -136,7 +136,7 @@ public function getId()
     public function testPrePersist(): void
     {
         $category = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -150,7 +150,7 @@ public function getId()
     public function testPreUpdate(): void
     {
         $category = new class extends Category {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }

tests/Model/CategoryTest.php

@@ -29,7 +29,7 @@ public function testSetterGetter(): void
         $context = $this->createMock(ContextInterface::class);
 
         $collection = new class extends Collection {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -65,7 +65,7 @@ public function getId()
     public function testPrePersist(): void
     {
         $collection = new class extends Collection {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -79,7 +79,7 @@ public function getId()
     public function testPreUpdate(): void
     {
         $collection = new class extends Collection {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }

tests/Model/CollectionTest.php

@@ -29,7 +29,7 @@ public function testSetterGetter(): void
         $context = $this->createMock(ContextInterface::class);
 
         $tag = new class extends Tag {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }
@@ -63,7 +63,7 @@ public function getId()
     public function testPreUpdate(): void
     {
         $tag = new class extends Tag {
-            public function getId()
+            public function getId(): int
             {
                 return 42;
             }