Merge pull request #124 from DirectoryTree/bug-123

Cast `$flags` to array when appending new message

Author: stevebauman

src/MessageQuery.php

@@ -75,7 +75,7 @@ public function get(): MessageCollection
     public function append(string $message, mixed $flags = null): int
     {
         $response = $this->connection()->append(
-            $this->folder->path(), $message, Str::enums($flags),
+            $this->folder->path(), $message, (array) Str::enums($flags),
         );
 
         return (int) $response // TAG4 OK [APPENDUID <uidvalidity> <uid>] APPEND completed.

tests/Unit/MessageQueryTest.php

@@ -3,6 +3,7 @@
 use DirectoryTree\ImapEngine\Connection\ImapConnection;
 use DirectoryTree\ImapEngine\Connection\ImapQueryBuilder;
 use DirectoryTree\ImapEngine\Connection\Streams\FakeStream;
+use DirectoryTree\ImapEngine\Enums\ImapFlag;
 use DirectoryTree\ImapEngine\Folder;
 use DirectoryTree\ImapEngine\Mailbox;
 use DirectoryTree\ImapEngine\MessageQuery;
@@ -181,3 +182,25 @@ function query(?Mailbox $mailbox = null): MessageQuery
     // Should only process the first chunk (page 1)
     expect($processedChunks)->toBe([1]);
 });
+
+test('append with single flag converts to array', function (mixed $flag) {
+    $stream = new FakeStream;
+    $stream->open();
+
+    $stream->feed([
+        '* OK Welcome to IMAP',
+        'TAG1 OK Logged in',
+        'TAG2 OK [APPENDUID 1234567890 1] APPEND completed',
+    ]);
+
+    $mailbox = Mailbox::make();
+    $mailbox->connect(new ImapConnection($stream));
+
+    $folder = new Folder($mailbox, 'INBOX');
+    $query = new MessageQuery($folder, new ImapQueryBuilder);
+
+    $uid = $query->append('Hello world', $flag);
+
+    expect($uid)->toBe(1);
+    $stream->assertWritten('TAG2 APPEND "INBOX" (\\Seen) "Hello world"');
+})->with([ImapFlag::Seen, '\\Seen']);