[HttpFoundation] Remove always false condition in BinaryFileResponse

Author: alexandre-daubois

src/Symfony/Component/HttpFoundation/BinaryFileResponse.php

@@ -110,8 +110,8 @@ public function getFile(): File
      */
     public function setChunkSize(int $chunkSize): static
     {
-        if ($chunkSize < 1 || $chunkSize > \PHP_INT_MAX) {
-            throw new \LogicException('The chunk size of a BinaryFileResponse cannot be less than 1 or greater than PHP_INT_MAX.');
+        if ($chunkSize < 1) {
+            throw new \InvalidArgumentException('The chunk size of a BinaryFileResponse cannot be less than 1.');
         }
 
         $this->chunkSize = $chunkSize;

src/Symfony/Component/HttpFoundation/Tests/BinaryFileResponseTest.php

@@ -455,4 +455,14 @@ public function testCreateFromTemporaryFile()
         $string = ob_get_clean();
         $this->assertSame('foo,bar', $string);
     }
+
+    public function testSetChunkSizeTooSmall()
+    {
+        $response = new BinaryFileResponse(__DIR__.'/File/Fixtures/test.gif');
+
+        $this->expectException(\InvalidArgumentException::class);
+        $this->expectExceptionMessage('The chunk size of a BinaryFileResponse cannot be less than 1.');
+
+        $response->setChunkSize(0);
+    }
 }