Merge pull request #127 from DirectoryTree/bug-126

Fix null date handling in `HasParsedMessage`

Author: stevebauman

src/HasParsedMessage.php

@@ -32,11 +32,15 @@ public function date(): ?CarbonInterface
             return null;
         }
 
-        if ($header instanceof DateHeader) {
-            return Carbon::instance($header->getDateTime());
+        if (! $header instanceof DateHeader) {
+            return null;
+        }
+
+        if (! $date = $header->getDateTime()) {
+            return null;
         }
 
-        return null;
+        return Carbon::instance($date);
     }
 
     /**

tests/Unit/FileMessageTest.php

@@ -29,6 +29,24 @@
     expect($message->flags())->toBe([]);
 });
 
+test('it handles invalid date header that returns null from getDateTime', function () {
+    $contents = <<<'EOT'
+    From: "John Doe" <[email protected]>
+    To: "Jane Roe" <[email protected]>
+    Subject: Test Subject
+    Date: Invalid Date String
+    Message-ID: <[email protected]>
+    MIME-Version: 1.0
+    Content-Type: text/plain; charset="UTF-8"
+
+    Hello World
+    EOT;
+
+    $message = new FileMessage($contents);
+
+    expect($message->date())->toBeNull();
+});
+
 test('it can parse a standard EML message and read basic headers', function () {
     $contents = <<<'EOT'
     From: "John Doe" <[email protected]>