Add return types to JsonSerializable implementations

derrabus · derrabus · commit 89c7cca9284b · 2021-06-05T22:22:57.000+02:00
diff --git a/Tests/JsonResponseTest.php b/Tests/JsonResponseTest.php
@@ -224,9 +224,6 @@ public function testSetContentJsonSerializeError()
     {
         $this->expectException(\Exception::class);
         $this->expectExceptionMessage('This error is expected');
-        if (!interface_exists(\JsonSerializable::class, false)) {
-            $this->markTestSkipped('JsonSerializable is required.');
-        }
 
         $serializable = new JsonSerializableObject();
 
@@ -280,12 +277,10 @@ public function testConstructorWithObjectWithoutToStringMethodThrowsAnException(
     }
 }
 
-if (interface_exists(\JsonSerializable::class, false)) {
-    class JsonSerializableObject implements \JsonSerializable
+class JsonSerializableObject implements \JsonSerializable
+{
+    public function jsonSerialize(): array
     {
-        public function jsonSerialize()
-        {
-            throw new \Exception('This error is expected');
-        }
+        throw new \Exception('This error is expected');
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -224,9 +224,6 @@ public function testSetContentJsonSerializeError()`
`224`	`224`	`{`
`225`	`225`	`$this->expectException(\Exception::class);`
`226`	`226`	`$this->expectExceptionMessage('This error is expected');`
`227`		`- if (!interface_exists(\JsonSerializable::class, false)) {`
`228`		`- $this->markTestSkipped('JsonSerializable is required.');`
`229`		`- }`
`230`	`227`
`231`	`228`	`$serializable = new JsonSerializableObject();`
`232`	`229`
`@@ -280,12 +277,10 @@ public function testConstructorWithObjectWithoutToStringMethodThrowsAnException(`
`280`	`277`	`}`
`281`	`278`	`}`
`282`	`279`
`283`		`-if (interface_exists(\JsonSerializable::class, false)) {`
`284`		`- class JsonSerializableObject implements \JsonSerializable`
	`280`	`+class JsonSerializableObject implements \JsonSerializable`
	`281`	`+{`
	`282`	`+ public function jsonSerialize(): array`
`285`	`283`	`{`
`286`		`- public function jsonSerialize()`
`287`		`- {`
`288`		`- throw new \Exception('This error is expected');`
`289`		`- }`
	`284`	`+ throw new \Exception('This error is expected');`
`290`	`285`	`}`
`291`	`286`	`}`