merge default backup config with user config

Author: mho22

src/Config/Config.php

@@ -27,10 +27,10 @@ public static function fromArray(array $data): self
         $source = require realpath(__DIR__.'/../../config/backup.php');
 
         return new self(
-            backup: BackupConfig::fromArray(array_merge($source['backup'], $data['backup'])),
-            notifications: NotificationsConfig::fromArray(array_merge($source['notifications'], $data['notifications'])),
-            monitoredBackups: MonitoredBackupsConfig::fromArray($data['monitor_backups'] ?? $source['notifications']),
-            cleanup: CleanupConfig::fromArray(array_merge($source['cleanup'], $data['cleanup']))
+            backup: BackupConfig::fromArray(array_merge($source['backup'], $data['backup'] ?? [])),
+            notifications: NotificationsConfig::fromArray(array_merge($source['notifications'], $data['notifications'] ?? [])),
+            monitoredBackups: MonitoredBackupsConfig::fromArray($data['monitor_backups'] ?? $source['monitor_backups']),
+            cleanup: CleanupConfig::fromArray(array_merge($source['cleanup'], $data['cleanup'] ?? []))
         );
     }
 }

tests/ConfigTest.php

@@ -0,0 +1,29 @@
+<?php
+
+use Spatie\Backup\Config\Config;
+use Spatie\Backup\Config\BackupConfig;
+use Spatie\Backup\Config\NotificationsConfig;
+use Spatie\Backup\Config\MonitoredBackupsConfig;
+use Spatie\Backup\Config\CleanupConfig;
+
+beforeEach(function () {
+    config()->set('backup', []);
+});
+
+it('returns default backup config if no backup config file exist', function () {
+    $config = Config::fromArray(config('backup'));
+
+    expect($config->backup)->toBeInstanceOf(BackupConfig::class);
+    expect($config->notifications)->toBeInstanceOf(NotificationsConfig::class);
+    expect($config->monitoredBackups)->toBeInstanceOf(MonitoredBackupsConfig::class);
+    expect($config->cleanup)->toBeInstanceOf(CleanupConfig::class);
+});
+
+it('returns a merged backup config made with minimal config and default config file', function () {
+    config()->set('backup.backup.name', 'foo');
+
+    $config = Config::fromArray(config('backup'));
+
+    expect($config->backup)->toBeInstanceOf(BackupConfig::class);
+    expect($config->backup->name)->toBe('foo');
+});