diff --git a/apps/sharebymail/tests/CapabilitiesTest.php b/apps/sharebymail/tests/CapabilitiesTest.php index dc642a16ec..2bf05f188b 100644 --- a/apps/sharebymail/tests/CapabilitiesTest.php +++ b/apps/sharebymail/tests/CapabilitiesTest.php @@ -26,19 +26,28 @@ namespace OCA\ShareByMail\Tests; use OCA\ShareByMail\Capabilities; +use OCA\ShareByMail\Settings\SettingsManager; use Test\TestCase; class CapabilitiesTest extends TestCase { /** @var Capabilities */ private $capabilities; + /** @var SettingsManager */ + private $settingsManager; + protected function setUp(): void { parent::setUp(); - $this->capabilities = new Capabilities(); + + $this->settingsManager = $this::createMock(SettingsManager::class); + $this->capabilities = new Capabilities($this->settingsManager); } public function testGetCapabilities() { + $this->settingsManager->method('enforcePasswordProtection') + ->willReturn(false); + $capabilities = [ 'files_sharing' => [ @@ -46,7 +55,7 @@ class CapabilitiesTest extends TestCase { [ 'enabled' => true, 'upload_files_drop' => ['enabled' => true], - 'password' => ['enabled' => true], + 'password' => ['enabled' => true, 'enforced' => false], 'expire_date' => ['enabled' => true] ] ] diff --git a/apps/sharebymail/tests/SettingsTest.php b/apps/sharebymail/tests/SettingsTest.php deleted file mode 100644 index 427dc256d6..0000000000 --- a/apps/sharebymail/tests/SettingsTest.php +++ /dev/null @@ -1,97 +0,0 @@ - - * - * @author Bjoern Schiessle - * @author Christoph Wurst - * @author Roeland Jago Douma - * - * @license GNU AGPL version 3 or any later version - * - * This program is free software: you can redistribute it and/or modify - * it under the terms of the GNU Affero General Public License as - * published by the Free Software Foundation, either version 3 of the - * License, or (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU Affero General Public License for more details. - * - * You should have received a copy of the GNU Affero General Public License - * along with this program. If not, see . - * - */ - -namespace OCA\ShareByMail\Tests; - -use OCA\ShareByMail\Settings; -use OCA\ShareByMail\Settings\SettingsManager; -use Test\TestCase; - -class SettingsTest extends TestCase { - - /** @var Settings */ - private $instance; - - /** @var SettingsManager | \PHPUnit_Framework_MockObject_MockObject */ - private $settingsManager; - - protected function setUp(): void { - parent::setUp(); - - $this->settingsManager = $this->getMockBuilder(SettingsManager::class) - ->disableOriginalConstructor()->getMock(); - $this->instance = new Settings($this->settingsManager); - } - - public function testAnnounceShareProvider() { - $before = [ - 'oc_appconfig' => - json_encode([ - 'key1' => 'value1', - 'key2' => 'value2' - ]), - 'oc_foo' => 'oc_bar' - ]; - - $after = [ - 'oc_appconfig' => - json_encode([ - 'key1' => 'value1', - 'key2' => 'value2', - 'shareByMailEnabled' => true - ]), - 'oc_foo' => 'oc_bar' - ]; - - $this->instance->announceShareProvider(['array' => &$before]); - $this->assertSame($after, $before); - } - - - public function testAnnounceShareByMailSettings() { - $this->settingsManager->expects($this->once())->method('enforcePasswordProtection')->willReturn(true); - $before = [ - 'oc_appconfig' => - json_encode([ - 'key1' => 'value1', - 'key2' => 'value2' - ]), - 'oc_foo' => 'oc_bar' - ]; - - $after = [ - 'oc_appconfig' => - json_encode([ - 'key1' => 'value1', - 'key2' => 'value2', - 'shareByMail' => ['enforcePasswordProtection' => true] - ]), - 'oc_foo' => 'oc_bar' - ]; - - $this->instance->announceShareByMailSettings(['array' => &$before]); - $this->assertSame($after, $before); - } -} diff --git a/apps/sharebymail/tests/ShareByMailProviderTest.php b/apps/sharebymail/tests/ShareByMailProviderTest.php index 61c76ea1f3..aeafc86c50 100644 --- a/apps/sharebymail/tests/ShareByMailProviderTest.php +++ b/apps/sharebymail/tests/ShareByMailProviderTest.php @@ -624,6 +624,8 @@ class ShareByMailProviderTest extends TestCase { $node = $this->getMockBuilder(File::class)->getMock(); $node->expects($this->any())->method('getName')->willReturn('filename'); + $this->settingsManager->method('sendPasswordByMail')->willReturn(true); + $originalShare = $this->getMockBuilder(IShare::class)->getMock(); $originalShare->expects($this->any())->method('getSharedWith')->willReturn('receiver@example.com'); $originalShare->expects($this->any())->method('getNode')->willReturn($node);