Merge pull request #8075 from annando/html-escaping

[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 dd38172b38d0b0c7e4d225dd119cfd819998fc50..70d7819d7318a739e5f0ebf5b42da73c34d7f254 100644 (file)
--- a/tests/src/Core/Lock/LockTest.php
+++ b/tests/src/Core/Lock/LockTest.php
@@ -38,7 +38,7 @@ abstract class LockTest extends MockedTest
        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'));
        }
@@ -49,10 +49,10 @@ abstract class LockTest extends MockedTest
        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));
        }
 
        /**
@@ -61,9 +61,9 @@ abstract class LockTest extends MockedTest
        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'));
        }
 
@@ -72,9 +72,9 @@ abstract class LockTest extends MockedTest
         */
        public function testReleaseAll()
        {
-               $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->isLocked('foo'));
                $this->assertTrue($this->instance->isLocked('bar'));
@@ -95,11 +95,11 @@ abstract class LockTest extends MockedTest
                $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'));
@@ -117,9 +117,9 @@ 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'));
        }
 
@@ -128,9 +128,9 @@ abstract class LockTest extends MockedTest
         */
        public function testGetLocks()
        {
-               $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->isLocked('foo'));
                $this->assertTrue($this->instance->isLocked('bar'));
@@ -148,9 +148,9 @@ abstract class LockTest extends MockedTest
         */
        public function testGetLocksWithPrefix()
        {
-               $this->assertTrue($this->instance->acquireLock('foo', 1));
-               $this->assertTrue($this->instance->acquireLock('test1', 1));
-               $this->assertTrue($this->instance->acquireLock('test2', 1));
+               $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'));
@@ -174,8 +174,8 @@ abstract class LockTest extends MockedTest
                $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'));
@@ -190,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'));
+       }
 }