[9.x] Add support for both single email string and array of email str…

…ings (#1810)

* Add support for both single email string and array of email strings

* Add tests to handle single and multiple email addresses

src/Config/NotificationMailConfig.php

@@ -8,7 +8,7 @@
 class NotificationMailConfig extends Data
 {
     protected function __construct(
-        public string $to,
+        public string|array $to,
         public NotificationMailSenderConfig $from,
     ) {}
 
@@ -19,8 +19,12 @@ protected function __construct(
      */
     public static function fromArray(array $data): self
     {
-        if (! filter_var($data['to'], FILTER_VALIDATE_EMAIL)) {
-            throw InvalidConfig::invalidEmail($data['to']);
+        $to = is_array($data['to']) ? $data['to'] : [$data['to']];
+
+        foreach ($to as $email) {
+            if (! filter_var($email, FILTER_VALIDATE_EMAIL)) {
+                throw InvalidConfig::invalidEmail($email);
+            }
         }
 
         return new self(

tests/Notifications/EventHandlerTest.php

@@ -6,6 +6,8 @@
 use Spatie\Backup\Events\BackupHasFailed;
 use Spatie\Backup\Notifications\Notifiable;
 use Spatie\Backup\Notifications\Notifications\BackupHasFailedNotification;
+use Spatie\Backup\Exceptions\InvalidConfig;
+use Spatie\Backup\Config\NotificationMailConfig;
 
 beforeEach(function () {
     Notification::fake();
@@ -50,6 +52,58 @@
     Notification::assertSentTimes(BackupHasFailedNotification::class, 1);
 });
 
+it('will accept a single email address', function () {
+    $data = [
+        'to' => '[email protected]',
+        'from' => [
+            'address' => '[email protected]',
+            'name' => 'Backup',
+        ],
+    ];
+
+    $config = NotificationMailConfig::fromArray($data);
+
+    expect($config->to)->toBe(['[email protected]']);
+});
+
+it('will accept multiple email addresses', function () {
+    $data = [
+        'to' => ['[email protected]', '[email protected]'],
+        'from' => [
+            'address' => '[email protected]',
+            'name' => 'Backup',
+        ],
+    ];
+
+    $config = NotificationMailConfig::fromArray($data);
+
+    expect($config->to)->toBe(['[email protected]', '[email protected]']);
+});
+
+it('will throw an exception for invalid email', function () {
+    $data = [
+        'to' => 'invalid-email',
+        'from' => [
+            'address' => '[email protected]',
+            'name' => 'Backup',
+        ],
+    ];
+
+    expect(fn() => NotificationMailConfig::fromArray($data))->toThrow(InvalidConfig::class);
+});
+
+it('will throw an exception for invalid email in array', function () {
+    $data = [
+        'to' => ['[email protected]', 'invalid-email'],
+        'from' => [
+            'address' => '[email protected]',
+            'name' => 'Backup',
+        ],
+    ];
+
+    expect(fn() => NotificationMailConfig::fromArray($data))->toThrow(InvalidConfig::class);
+});
+
 function fireBackupHasFailedEvent(): void
 {
     $exception = new Exception('Dummy exception');