Merge 10dc4e4 into 69878ea

tests/Assets/PhpFactoryFunctionOverrides.php

@@ -0,0 +1,27 @@
+<?php
+/**
+ * Slim Framework (https://slimframework.com)
+ *
+ * @license https://github.com/slimphp/Slim-Psr7/blob/master/LICENSE.md (MIT License)
+ */
+
+declare(strict_types=1);
+
+namespace Slim\Psr7\Factory;
+
+/**
+ * Return `false` if the parameter is "non-readable", otherwise the function would return the value from
+ * the php built-in function.
+ *
+ * @param string $filename
+ *
+ * @return bool
+ */
+function is_readable(string $filename): bool
+{
+    if ('non-readable' === $filename) {
+        return false;
+    }
+
+    return \is_readable($filename);
+}

tests/Factory/UploadedFileFactoryTest.php

@@ -10,6 +10,9 @@
 namespace Slim\Tests\Psr7\Factory;
 
 use Interop\Http\Factory\UploadedFileFactoryTestCase;
+use InvalidArgumentException;
+use Prophecy\Argument\Token\ExactValueToken;
+use Prophecy\Prophecy\MethodProphecy;
 use Psr\Http\Message\StreamInterface;
 use Slim\Psr7\Factory\StreamFactory;
 use Slim\Psr7\Factory\UploadedFileFactory;
@@ -36,4 +39,46 @@ protected function createStream($content)
 
         return (new StreamFactory())->createStreamFromResource($resource);
     }
+
+    /**
+     * Prophesize a `\Psr\Http\Message\StreamInterface` with a `getMetadata` method prophecy.
+     *
+     * @param string $argKey Argument for the method prophecy.
+     * @param mixed $returnValue Return value of the `getMetadata` method.
+     *
+     * @return StreamInterface
+     */
+    protected function prophesizeStreamInterfaceWithGetMetadataMethod(string $argKey, $returnValue): StreamInterface
+    {
+        $prophecy = $this->prophesize(StreamInterface::class);
+        $mp = new MethodProphecy($prophecy, 'getMetadata', [new ExactValueToken($argKey)]);
+        $mp->shouldBeCalled();
+        $mp->willReturn($returnValue);
+
+        /** @var StreamInterface $upload */
+        $upload = $prophecy->reveal();
+        return $upload;
+    }
+
+    /**
+     * @expectedException InvalidArgumentException
+     * @expectedExceptionMessage File is not readable.
+     */
+    public function testCreateUploadedFileWithInvalidUri()
+    {
+        $this->factory->createUploadedFile(
+            $this->prophesizeStreamInterfaceWithGetMetadataMethod('uri', null)
+        );
+    }
+
+    /**
+     * @expectedException InvalidArgumentException
+     * @expectedExceptionMessage File is not readable.
+     */
+    public function testCreateUploadedFileWithNonReadableFile()
+    {
+        $this->factory->createUploadedFile(
+            $this->prophesizeStreamInterfaceWithGetMetadataMethod('uri', 'non-readable')
+        );
+    }
 }

tests/bootstrap.php

@@ -11,3 +11,4 @@
 
 require __DIR__ . '/Assets/PhpGetAllHeaders.php';
 require __DIR__ . '/Assets/PhpFunctionOverrides.php';
+require __DIR__ . '/Assets/PhpFactoryFunctionOverrides.php';