From 61e445da88f17b282bffc974b741a0d38e8ee366 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Thu, 14 Jun 2018 11:31:32 +0200 Subject: [PATCH] Add PublicShareControllerTests Signed-off-by: Roeland Jago Douma --- .../AppFramework/PublicShareController.php | 2 +- .../Controller/PublicShareControllerTest.php | 102 ++++++++++++++++++ 2 files changed, 103 insertions(+), 1 deletion(-) create mode 100644 tests/lib/AppFramework/Controller/PublicShareControllerTest.php diff --git a/lib/public/AppFramework/PublicShareController.php b/lib/public/AppFramework/PublicShareController.php index 64fbe9df40..d0e54a0295 100644 --- a/lib/public/AppFramework/PublicShareController.php +++ b/lib/public/AppFramework/PublicShareController.php @@ -74,7 +74,7 @@ abstract class PublicShareController extends Controller { * * @since 14.0.0 */ - public function getToken(): string { + final public function getToken(): string { return $this->token; } diff --git a/tests/lib/AppFramework/Controller/PublicShareControllerTest.php b/tests/lib/AppFramework/Controller/PublicShareControllerTest.php new file mode 100644 index 0000000000..eff7563cc4 --- /dev/null +++ b/tests/lib/AppFramework/Controller/PublicShareControllerTest.php @@ -0,0 +1,102 @@ + + * + * @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 Test\AppFramework\Controller; + +use OC\AppFramework\Middleware\PublicShare\Exceptions\NeedAuthenticationException; +use OC\AppFramework\Middleware\PublicShare\PublicShareMiddleware; +use OCP\AppFramework\AuthPublicShareController; +use OCP\AppFramework\Controller; +use OCP\AppFramework\Http\NotFoundResponse; +use OCP\AppFramework\Http\RedirectResponse; +use OCP\AppFramework\PublicShareController; +use OCP\Files\NotFoundException; +use OCP\IConfig; +use OCP\IRequest; +use OCP\ISession; +use OCP\IURLGenerator; + + +class PublicShareControllerTest extends \Test\TestCase { + + /** @var IRequest|\PHPUnit_Framework_MockObject_MockObject */ + private $request; + /** @var ISession|\PHPUnit_Framework_MockObject_MockObject */ + private $session; + + /** @var PublicShareController|\PHPUnit_Framework_MockObject_MockObject */ + private $controller; + + + protected function setUp() { + parent::setUp(); + + $this->request = $this->createMock(IRequest::class); + $this->session = $this->createMock(ISession::class); + + $this->controller = $this->getMockBuilder(PublicShareController::class) + ->setConstructorArgs([ + 'app', + $this->request, + $this->session + ])->getMock(); + } + + public function testGetToken() { + $this->controller->setToken('test'); + $this->assertEquals('test', $this->controller->getToken()); + } + + public function dataIsAuthenticated() { + return [ + [false, 'token1', 'token1', 'hash1', 'hash1', true], + [false, 'token1', 'token1', 'hash1', 'hash2', true], + [false, 'token1', 'token2', 'hash1', 'hash1', true], + [false, 'token1', 'token2', 'hash1', 'hash2', true], + [ true, 'token1', 'token1', 'hash1', 'hash1', true], + [ true, 'token1', 'token1', 'hash1', 'hash2', false], + [ true, 'token1', 'token2', 'hash1', 'hash1', false], + [ true, 'token1', 'token2', 'hash1', 'hash2', false], + ]; + } + + /** + * @dataProvider dataIsAuthenticated + */ + public function testIsAuthenticatedNotPasswordProtected(bool $protected, string $token1, string $token2, string $hash1, string $hash2, bool $expected) { + $this->controller->method('isPasswordProtected') + ->willReturn($protected); + + $this->session->method('get') + ->willReturnMap([ + ['public_link_authenticated_token', $token1], + ['public_link_authenticated_password_hash', $hash1], + ]); + + $this->controller->setToken($token2); + $this->controller->method('getPasswordHash') + ->willReturn($hash2); + + $this->assertEquals($expected, $this->controller->isAuthenticated()); + } + +}