diff options
| author | Roeland Jago Douma <roeland@famdouma.nl> | 2018-06-19 16:39:51 +0200 |
|---|---|---|
| committer | Roeland Jago Douma <roeland@famdouma.nl> | 2018-07-04 08:28:33 +0200 |
| commit | 6a0c54d5bfd70baeed2438ac05278a9b4cb73d88 (patch) | |
| tree | cd061e0570ff628076c7841e11e2e91f70576ad1 /tests/Settings | |
| parent | 0cf0bcba616811248e051a469360055845daed18 (diff) | |
| download | nextcloud-server-6a0c54d5bfd70baeed2438ac05278a9b4cb73d88.tar.gz nextcloud-server-6a0c54d5bfd70baeed2438ac05278a9b4cb73d88.zip | |
Add warning to setup checks if the default mailer is still php
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Diffstat (limited to 'tests/Settings')
| -rw-r--r-- | tests/Settings/Controller/CheckSetupControllerTest.php | 27 |
1 files changed, 26 insertions, 1 deletions
diff --git a/tests/Settings/Controller/CheckSetupControllerTest.php b/tests/Settings/Controller/CheckSetupControllerTest.php index 470bc9cde64..057774a45ba 100644 --- a/tests/Settings/Controller/CheckSetupControllerTest.php +++ b/tests/Settings/Controller/CheckSetupControllerTest.php @@ -119,7 +119,22 @@ class CheckSetupControllerTest extends TestCase { $this->lockingProvider, $this->dateTimeFormatter, ]) - ->setMethods(['isReadOnlyConfig', 'hasValidTransactionIsolationLevel', 'hasFileinfoInstalled', 'hasWorkingFileLocking', 'getLastCronInfo', 'getSuggestedOverwriteCliURL', 'getOutdatedCaches', 'getCurlVersion', 'isPhpOutdated', 'isOpcacheProperlySetup', 'hasFreeTypeSupport', 'hasMissingIndexes', 'isSqliteUsed'])->getMock(); + ->setMethods([ + 'isReadOnlyConfig', + 'hasValidTransactionIsolationLevel', + 'hasFileinfoInstalled', + 'hasWorkingFileLocking', + 'getLastCronInfo', + 'getSuggestedOverwriteCliURL', + 'getOutdatedCaches', + 'getCurlVersion', + 'isPhpOutdated', + 'isOpcacheProperlySetup', + 'hasFreeTypeSupport', + 'hasMissingIndexes', + 'isSqliteUsed', + 'isPhpMailerUsed', + ])->getMock(); } public function testIsInternetConnectionWorkingDisabledViaConfig() { @@ -352,6 +367,10 @@ class CheckSetupControllerTest extends TestCase { ->method('linkToDocs') ->with('admin-db-conversion') ->willReturn('http://docs.example.org/server/go.php?to=admin-db-conversion'); + $this->urlGenerator->expects($this->at(6)) + ->method('getAbsoluteURL') + ->with('index.php/settings/admin') + ->willReturn('https://server/index.php/settings/admin'); $this->checkSetupController ->method('hasFreeTypeSupport') ->willReturn(false); @@ -392,6 +411,10 @@ class CheckSetupControllerTest extends TestCase { 'relativeTime' => '2 hours ago', 'backgroundJobsUrl' => 'https://example.org', ]); + $this->checkSetupController + ->expects($this->once()) + ->method('isPhpMailerUsed') + ->willReturn(false); $this->checker ->expects($this->once()) ->method('hasPassedCheck') @@ -434,6 +457,8 @@ class CheckSetupControllerTest extends TestCase { 'isSqliteUsed' => false, 'databaseConversionDocumentation' => 'http://docs.example.org/server/go.php?to=admin-db-conversion', 'missingIndexes' => [], + 'isPhpMailerUsed' => false, + 'mailSettingsDocumentation' => 'https://server/index.php/settings/admin', ] ); $this->assertEquals($expected, $this->checkSetupController->check()); |