Merge pull request #8072 from nupplaphil/task/Cache_to_DI

[friendica.git] / tests / src / Core / Lock / LockTest.php

diff --git a/tests/src/Core/Lock/LockTest.php b/tests/src/Core/Lock/LockTest.php
index 59d6ee2d5d824f7b45cba698efde6db55a4b7654..70d7819d7318a739e5f0ebf5b42da73c34d7f254 100644 (file)
--- a/tests/src/Core/Lock/LockTest.php
+++ b/tests/src/Core/Lock/LockTest.php
@@ -3,23 +3,16 @@
 namespace Friendica\Test\src\Core\Lock;
 
 use Friendica\Test\MockedTest;
-use Friendica\Test\Util\AppMockTrait;
-use Friendica\Test\Util\VFSTrait;
-use Psr\Log\LoggerInterface;
-use Psr\Log\NullLogger;
 
 abstract class LockTest extends MockedTest
 {
-       use VFSTrait;
-       use AppMockTrait;
-
        /**
         * @var int Start time of the mock (used for time operations)
         */
        protected $startTime = 1417011228;
 
        /**
-        * @var \Friendica\Core\Lock\ILockDriver
+        * @var \Friendica\Core\Lock\ILock
         */
        protected $instance;
 
@@ -27,35 +20,25 @@ abstract class LockTest extends MockedTest
 
        protected function setUp()
        {
-               // Reusable App object
-               $this->setUpVfsDir();
-               $this->mockApp($this->root);
-               $this->app
-                       ->shouldReceive('getHostname')
-                       ->andReturn('friendica.local');
-
-               $logger = new NullLogger();
-               $this->dice->shouldReceive('create')
-                          ->with(LoggerInterface::class)
-                          ->andReturn($logger);
-
                parent::setUp();
+
                $this->instance = $this->getInstance();
-               $this->instance->releaseAll();
+               $this->instance->releaseAll(true);
        }
 
        protected function tearDown()
        {
-               $this->instance->releaseAll();
+               $this->instance->releaseAll(true);
                parent::tearDown();
        }
 
        /**
         * @small
         */
-       public function testLock() {
+       public function testLock()
+       {
                $this->assertFalse($this->instance->isLocked('foo'));
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
+               $this->assertTrue($this->instance->acquire('foo', 1));
                $this->assertTrue($this->instance->isLocked('foo'));
                $this->assertFalse($this->instance->isLocked('bar'));
        }
@@ -63,32 +46,35 @@ abstract class LockTest extends MockedTest
        /**
         * @small
         */
-       public function testDoubleLock() {
+       public function testDoubleLock()
+       {
                $this->assertFalse($this->instance->isLocked('foo'));
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
+               $this->assertTrue($this->instance->acquire('foo', 1));
                $this->assertTrue($this->instance->isLocked('foo'));
                // We already locked it
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
+               $this->assertTrue($this->instance->acquire('foo', 1));
        }
 
        /**
         * @small
         */
-       public function testReleaseLock() {
+       public function testReleaseLock()
+       {
                $this->assertFalse($this->instance->isLocked('foo'));
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
+               $this->assertTrue($this->instance->acquire('foo', 1));
                $this->assertTrue($this->instance->isLocked('foo'));
-               $this->instance->releaseLock('foo');
+               $this->instance->release('foo');
                $this->assertFalse($this->instance->isLocked('foo'));
        }
 
        /**
         * @small
         */
-       public function testReleaseAll() {
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
-               $this->assertTrue($this->instance->acquireLock('bar', 1));
-               $this->assertTrue($this->instance->acquireLock('nice', 1));
+       public function testReleaseAll()
+       {
+               $this->assertTrue($this->instance->acquire('foo', 1));
+               $this->assertTrue($this->instance->acquire('bar', 1));
+               $this->assertTrue($this->instance->acquire('nice', 1));
 
                $this->assertTrue($this->instance->isLocked('foo'));
                $this->assertTrue($this->instance->isLocked('bar'));
@@ -104,15 +90,16 @@ abstract class LockTest extends MockedTest
        /**
         * @small
         */
-       public function testReleaseAfterUnlock() {
+       public function testReleaseAfterUnlock()
+       {
                $this->assertFalse($this->instance->isLocked('foo'));
                $this->assertFalse($this->instance->isLocked('bar'));
                $this->assertFalse($this->instance->isLocked('nice'));
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
-               $this->assertTrue($this->instance->acquireLock('bar', 1));
-               $this->assertTrue($this->instance->acquireLock('nice', 1));
+               $this->assertTrue($this->instance->acquire('foo', 1));
+               $this->assertTrue($this->instance->acquire('bar', 1));
+               $this->assertTrue($this->instance->acquire('nice', 1));
 
-               $this->assertTrue($this->instance->releaseLock('foo'));
+               $this->assertTrue($this->instance->release('foo'));
 
                $this->assertFalse($this->instance->isLocked('foo'));
                $this->assertTrue($this->instance->isLocked('bar'));
@@ -130,24 +117,65 @@ abstract class LockTest extends MockedTest
        public function testReleaseWitTTL()
        {
                $this->assertFalse($this->instance->isLocked('test'));
-               $this->assertTrue($this->instance->acquireLock('test', 1, 10));
+               $this->assertTrue($this->instance->acquire('test', 1, 10));
                $this->assertTrue($this->instance->isLocked('test'));
-               $this->assertTrue($this->instance->releaseLock('test'));
+               $this->assertTrue($this->instance->release('test'));
                $this->assertFalse($this->instance->isLocked('test'));
        }
 
+       /**
+        * @small
+        */
+       public function testGetLocks()
+       {
+               $this->assertTrue($this->instance->acquire('foo', 1));
+               $this->assertTrue($this->instance->acquire('bar', 1));
+               $this->assertTrue($this->instance->acquire('nice', 1));
+
+               $this->assertTrue($this->instance->isLocked('foo'));
+               $this->assertTrue($this->instance->isLocked('bar'));
+               $this->assertTrue($this->instance->isLocked('nice'));
+
+               $locks = $this->instance->getLocks();
+
+               $this->assertContains('foo', $locks);
+               $this->assertContains('bar', $locks);
+               $this->assertContains('nice', $locks);
+       }
+
+       /**
+        * @small
+        */
+       public function testGetLocksWithPrefix()
+       {
+               $this->assertTrue($this->instance->acquire('foo', 1));
+               $this->assertTrue($this->instance->acquire('test1', 1));
+               $this->assertTrue($this->instance->acquire('test2', 1));
+
+               $this->assertTrue($this->instance->isLocked('foo'));
+               $this->assertTrue($this->instance->isLocked('test1'));
+               $this->assertTrue($this->instance->isLocked('test2'));
+
+               $locks = $this->instance->getLocks('test');
+
+               $this->assertContains('test1', $locks);
+               $this->assertContains('test2', $locks);
+               $this->assertNotContains('foo', $locks);
+       }
+
        /**
         * @medium
         */
-       function testLockTTL() {
+       function testLockTTL()
+       {
                $this->markTestSkipped('taking too much time without mocking');
 
                $this->assertFalse($this->instance->isLocked('foo'));
                $this->assertFalse($this->instance->isLocked('bar'));
 
                // TODO [nupplaphil] - Because of the Datetime-Utils for the database, we have to wait a FULL second between the checks to invalidate the db-locks/cache
-               $this->assertTrue($this->instance->acquireLock('foo', 2, 1));
-               $this->assertTrue($this->instance->acquireLock('bar', 2, 3));
+               $this->assertTrue($this->instance->acquire('foo', 2, 1));
+               $this->assertTrue($this->instance->acquire('bar', 2, 3));
 
                $this->assertTrue($this->instance->isLocked('foo'));
                $this->assertTrue($this->instance->isLocked('bar'));
@@ -162,4 +190,13 @@ abstract class LockTest extends MockedTest
                $this->assertFalse($this->instance->isLocked('foo'));
                $this->assertFalse($this->instance->isLocked('bar'));
        }
+
+       /**
+        * Test if releasing a non-existing lock doesn't throw errors
+        */
+       public function testReleaseLockWithoutLock()
+       {
+               $this->assertFalse($this->instance->isLocked('wrongLock'));
+               $this->assertFalse($this->instance->release('wrongLock'));
+       }
 }