Merge pull request #6 from tigitz/master

Julien Neuhart · web-flow · commit c556fe974f49 · 2019-01-11T09:23:04.000+01:00
adding document creation from string
diff --git a/src/DocumentFactory.php b/src/DocumentFactory.php
@@ -4,6 +4,10 @@
 
 use GuzzleHttp\Psr7\LazyOpenStream;
 use Psr\Http\Message\StreamInterface;
+use Safe\Exceptions\FilesystemException;
+use function GuzzleHttp\Psr7\stream_for;
+use function Safe\fopen;
+use function Safe\fwrite;
 
 final class DocumentFactory
 {
@@ -26,4 +30,18 @@ public static function makeFromStream(string $fileName, StreamInterface $fileStr
     {
         return new Document($fileName, $fileStream);
     }
+
+    /**
+     * @param string $fileName
+     * @param string $string
+     * @return Document
+     * @throws FilesystemException
+     */
+    public static function makeFromString(string $fileName, string $string): Document
+    {
+        $fileStream = fopen('php://memory', 'rb+');
+        fwrite($fileStream, $string);
+
+        return new Document($fileName, stream_for($fileStream));
+    }
 }
diff --git a/tests/DocumentFactoryTest.php b/tests/DocumentFactoryTest.php
@@ -15,5 +15,8 @@ function testMake()
         // case 2: uses a stream.
         $document = DocumentFactory::makeFromStream('file.pdf', new LazyOpenStream(__DIR__ . '/assets/file.pdf', 'r'));
         $this->assertNotEmpty($document->getFileStream());
+        // case 3: uses a string
+        $document = DocumentFactory::makeFromString('index.html', '<html>foo</html>');
+        $this->assertNotEmpty($document->getFileStream());
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -15,5 +15,8 @@ function testMake()`
`15`	`15`	`// case 2: uses a stream.`
`16`	`16`	`$document = DocumentFactory::makeFromStream('file.pdf', new LazyOpenStream(__DIR__ . '/assets/file.pdf', 'r'));`
`17`	`17`	`$this->assertNotEmpty($document->getFileStream());`
	`18`	`+ // case 3: uses a string`
	`19`	`+ $document = DocumentFactory::makeFromString('index.html', '<html>foo</html>');`
	`20`	`+ $this->assertNotEmpty($document->getFileStream());`
`18`	`21`	`}`
`19`	`22`	`}`