fix test under php 8.0

sonata-project · Aug 29, 2020 · 44f3082 · 44f3082
1 parent 0c42eda
commit 44f3082
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 12 deletions.
diff --git a/tests/Admin/AdminTest.php b/tests/Admin/AdminTest.php
@@ -694,11 +694,20 @@ public function testToString(): void
         $s = new FooToString();
         $this->assertSame('salut', $admin->toString($s));
 
+        $this->assertSame('', $admin->toString(false));
+    }
+
+    public function testToStringNull(): void
+    {
+        if (\PHP_VERSION_ID >= 80000) {
+            $this->markTestSkipped('PHP 8.0 does not allow __toString() method to return null');
+        }
+
+        $admin = new PostAdmin('sonata.post.admin.post', 'NewsBundle\Entity\Post', 'Sonata\NewsBundle\Controller\PostAdminController');
+
         // To string method is implemented, but returns null
         $s = new FooToStringNull();
         $this->assertNotEmpty($admin->toString($s));
-
-        $this->assertSame('', $admin->toString(false));
     }
 
     public function testIsAclEnabled(): void

diff --git a/tests/Command/SetupAclCommandTest.php b/tests/Command/SetupAclCommandTest.php
@@ -77,14 +77,4 @@ public function testExecuteWithException1(): void
             $commandTester->getDisplay()
         );
     }
-
-    public function testExecuteWithException2(): void
-    {
-        $pool = new Pool($this->container, '', '');
-
-        $this->expectException(\TypeError::class);
-        $this->expectExceptionMessage(sprintf('Argument 2 passed to %s::__construct() must implement interface %s, instance of %s given', SetupAclCommand::class, AdminAclManipulatorInterface::class, \stdClass::class));
-
-        new SetupAclCommand($pool, new \stdClass());
-    }
 }