Add visibility modifiers for constants

Author: willemstuursma

classes/mutex/MemcachedMutex.php

@@ -19,12 +19,6 @@ class MemcachedMutex extends SpinlockMutex
      */
     private $memcache;
 
-    /**
-     * The memcache key prefix.
-     * @internal
-     */
-    const PREFIX = "lockd_";
-    
     /**
      * Sets the lock's name and the connected Memcached API.
      *

classes/mutex/SpinlockMutex.php

@@ -41,7 +41,7 @@ abstract class SpinlockMutex extends LockMutex
     /**
      * The prefix for the lock key.
      */
-    const PREFIX = "lock_";
+    private const PREFIX = "lock_";
 
     /**
      * Sets the timeout.
@@ -54,7 +54,7 @@ public function __construct(string $name, int $timeout = 3)
     {
         $this->timeout = $timeout;
         $this->loop    = new Loop($this->timeout);
-        $this->key     = static::PREFIX.$name;
+        $this->key     = self::PREFIX.$name;
     }
 
     protected function lock(): void

tests/mutex/MemcachedMutexTest.php

@@ -45,7 +45,7 @@ public function testFailAcquireLock()
     {
         $this->memcached->expects($this->atLeastOnce())
             ->method("add")
-            ->with("lockd_test", true, 2)
+            ->with("lock_test", true, 2)
             ->willReturn(false);
 
         $this->mutex->synchronized(function (): void {
@@ -62,12 +62,12 @@ public function testFailReleasingLock()
     {
         $this->memcached->expects($this->once())
             ->method("add")
-            ->with("lockd_test", true, 2)
+            ->with("lock_test", true, 2)
             ->willReturn(true);
 
         $this->memcached->expects($this->once())
             ->method("delete")
-            ->with("lockd_test")
+            ->with("lock_test")
             ->willReturn(false);
 
         $this->mutex->synchronized(function (): void {