From 4caae91b832b1684351c16f28ee3a8c597b70889 Mon Sep 17 00:00:00 2001
From: Joas Schilling <coding@schilljs.com>
Date: Fri, 17 Mar 2017 13:45:25 +0100
Subject: [PATCH] Fail when the test mail could not be sent

Signed-off-by: Joas Schilling <coding@schilljs.com>
---
 settings/Controller/MailSettingsController.php | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/settings/Controller/MailSettingsController.php b/settings/Controller/MailSettingsController.php
index 108ac3f393a..8137b4da53c 100644
--- a/settings/Controller/MailSettingsController.php
+++ b/settings/Controller/MailSettingsController.php
@@ -153,7 +153,10 @@ class MailSettingsController extends Controller {
 				$message->setFrom([$this->defaultMailAddress]);
 				$message->setSubject($this->l10n->t('test email settings'));
 				$message->setPlainBody('If you received this email, the settings seem to be correct.');
-				$this->mailer->send($message);
+				$errors = $this->mailer->send($message);
+				if (!empty($errors)) {
+					throw new \RuntimeException($this->l10n->t('Mail could not be sent. Check your mail server log'));
+				}
 			} catch (\Exception $e) {
 				return [
 					'data' => [
-- 
GitLab