minor symfony#54909 [HttpFoundation] add test for JSON response with null as content (xabbuh)

xabbuh · xabbuh · commit 6d21a599741f · 2024-05-15T09:32:23.000+02:00
This PR was merged into the 5.4 branch. Discussion ---------- [HttpFoundation] add test for JSON response with null as content | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | symfony#11679 (comment) | License | MIT Commits ------- 5c37277 add test for JSON response with null as content
diff --git a/src/Symfony/Component/HttpFoundation/Tests/JsonResponseTest.php b/src/Symfony/Component/HttpFoundation/Tests/JsonResponseTest.php
@@ -294,6 +294,14 @@ public function testConstructorWithObjectWithoutToStringMethodThrowsAnException(
 
         new JsonResponse(new \stdClass(), 200, [], true);
     }
+
+    public function testSetDataWithNull()
+    {
+        $response = new JsonResponse();
+        $response->setData(null);
+
+        $this->assertSame('null', $response->getContent());
+    }
 }
 
 class JsonSerializableObject implements \JsonSerializable

Original file line number	Diff line number	Diff line change
`@@ -294,6 +294,14 @@ public function testConstructorWithObjectWithoutToStringMethodThrowsAnException(`
`294`	`294`
`295`	`295`	`new JsonResponse(new \stdClass(), 200, [], true);`
`296`	`296`	`}`
	`297`	`+`
	`298`	`+ public function testSetDataWithNull()`
	`299`	`+ {`
	`300`	`+ $response = new JsonResponse();`
	`301`	`+ $response->setData(null);`
	`302`	`+`
	`303`	`+ $this->assertSame('null', $response->getContent());`
	`304`	`+ }`
`297`	`305`	`}`
`298`	`306`
`299`	`307`	`class JsonSerializableObject implements \JsonSerializable`