Check if backup destination is reachable before trying to copy backup (…

…#1469)

* add check if backup destination is reachable before trying to copy zip file

* throw exception in catch statement to avoid wrong global status with 0 when an error occurred

src/Tasks/Backup/BackupJob.php

@@ -283,6 +283,10 @@ protected function copyToBackupDestinations(string $path): void
         $this->backupDestinations
             ->each(function (BackupDestination $backupDestination) use ($path) {
                 try {
+                    if (! $backupDestination->isReachable()) {
+                        throw new Exception("Could not connect to disk {$backupDestination->diskName()} because: {$backupDestination->connectionError()}");
+                    }
+
                     consoleOutput()->info("Copying zip to disk named {$backupDestination->diskName()}...");
 
                     $backupDestination->write($path);
@@ -294,6 +298,8 @@ protected function copyToBackupDestinations(string $path): void
                     consoleOutput()->error("Copying zip failed because: {$exception->getMessage()}.");
 
                     $this->sendNotification(new BackupHasFailed($exception, $backupDestination ?? null));
+
+                    throw $exception;
                 }
             });
     }