diff options
author | Roeland Jago Douma <roeland@famdouma.nl> | 2019-08-19 19:35:47 +0200 |
---|---|---|
committer | Roeland Jago Douma <roeland@famdouma.nl> | 2019-08-19 19:38:43 +0200 |
commit | 7927aebdeb3a1cf0f7e953ed3adfd1f34898d98c (patch) | |
tree | 9c34d43901b390923ee97943306f2771dde9590f /tests | |
parent | 1521da14ff292cb257f442a5f16c41b29faec337 (diff) | |
download | nextcloud-server-7927aebdeb3a1cf0f7e953ed3adfd1f34898d98c.tar.gz nextcloud-server-7927aebdeb3a1cf0f7e953ed3adfd1f34898d98c.zip |
Fix report of phpstan in Limiter
* unneeded arguments to constructor
* added return types
* let automatic DI do its work
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/lib/Security/RateLimiting/LimiterTest.php | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/tests/lib/Security/RateLimiting/LimiterTest.php b/tests/lib/Security/RateLimiting/LimiterTest.php index 80b63ebb391..5d42104ba82 100644 --- a/tests/lib/Security/RateLimiting/LimiterTest.php +++ b/tests/lib/Security/RateLimiting/LimiterTest.php @@ -31,10 +31,6 @@ use OCP\IUserSession; use Test\TestCase; class LimiterTest extends TestCase { - /** @var IUserSession|\PHPUnit_Framework_MockObject_MockObject */ - private $userSession; - /** @var IRequest|\PHPUnit_Framework_MockObject_MockObject */ - private $request; /** @var ITimeFactory|\PHPUnit_Framework_MockObject_MockObject */ private $timeFactory; /** @var IBackend|\PHPUnit_Framework_MockObject_MockObject */ @@ -45,14 +41,10 @@ class LimiterTest extends TestCase { public function setUp() { parent::setUp(); - $this->userSession = $this->createMock(IUserSession::class); - $this->request = $this->createMock(IRequest::class); $this->timeFactory = $this->createMock(ITimeFactory::class); $this->backend = $this->createMock(IBackend::class); $this->limiter = new Limiter( - $this->userSession, - $this->request, $this->timeFactory, $this->backend ); |