Fix repeated attachment content reads

Author: stevebauman

src/Attachment.php

@@ -117,6 +117,10 @@ public function contentDisposition(): string
      */
     public function contents(): string
     {
+        if ($this->contentStream->isSeekable()) {
+            $this->contentStream->rewind();
+        }
+
         return $this->contentStream->getContents();
     }
 

tests/Unit/AttachmentTest.php

@@ -1,20 +1,44 @@
 <?php
 
 use DirectoryTree\ImapEngine\Attachment;
-use GuzzleHttp\Psr7\LazyOpenStream;
+use GuzzleHttp\Psr7\Utils;
 
 test('extension', function () {
-    $stream = new LazyOpenStream('test.jpg', 'r');
+    $stream = Utils::streamFor('');
 
     $ext = (new Attachment('test.jpg', null, 'image/jpeg', 'attachment', $stream))->extension();
 
     expect($ext)->toBe('jpg');
 });
 
 test('extension with content type', function () {
-    $stream = new LazyOpenStream('test', 'r');
+    $stream = Utils::streamFor('');
 
     $ext = (new Attachment('test', null, 'image/jpeg', 'attachment', $stream))->extension();
 
     expect($ext)->toBe('jpg');
 });
+
+test('contents can be read multiple times', function () {
+    $stream = Utils::streamFor('Hello World!');
+
+    $attachment = new Attachment('test.txt', null, 'text/plain', 'attachment', $stream);
+
+    expect($attachment->contents())->toBe('Hello World!');
+    expect($attachment->contents())->toBe('Hello World!');
+});
+
+test('save writes contents after reading contents', function () {
+    $stream = Utils::streamFor('Hello World!');
+
+    $attachment = new Attachment('test.txt', null, 'text/plain', 'attachment', $stream);
+
+    $path = tempnam(sys_get_temp_dir(), 'imap-engine-attachment-');
+
+    $attachment->contents();
+    $attachment->save($path);
+
+    expect(file_get_contents($path))->toBe('Hello World!');
+
+    unlink($path);
+});