Log successful release of Lock

Author: PabloKowalczyk

Lock.php

@@ -236,6 +236,8 @@ public function release()
             if ($this->store->exists($this->key)) {
                 throw new LockReleasingException(sprintf('Failed to release the "%s" lock, the resource is still locked.', $this->key));
             }
+
+            $this->logger->debug('Successfully released the "{resource}" lock.', ['resource' => $this->key]);
         } catch (LockReleasingException $e) {
             $this->logger->notice('Failed to release the "{resource}" lock.', ['resource' => $this->key]);
             throw $e;

Tests/LockTest.php

@@ -12,6 +12,7 @@
 namespace Symfony\Component\Lock\Tests;
 
 use PHPUnit\Framework\TestCase;
+use Psr\Log\AbstractLogger;
 use Psr\Log\LoggerInterface;
 use Symfony\Component\Lock\BlockingSharedLockStoreInterface;
 use Symfony\Component\Lock\BlockingStoreInterface;
@@ -22,6 +23,7 @@
 use Symfony\Component\Lock\PersistingStoreInterface;
 use Symfony\Component\Lock\SharedLockStoreInterface;
 use Symfony\Component\Lock\Store\ExpiringStoreTrait;
+use Symfony\Component\Lock\Store\InMemoryStore;
 
 /**
  * @author Jérémy Derussé <jeremy@derusse.com>
@@ -366,6 +368,34 @@ public function testReleaseThrowsAndLog()
         $lock->release();
     }
 
+    public function testSuccessReleaseLog()
+    {
+        $key = new Key((string) random_int(100, 1000));
+        $store = new InMemoryStore();
+        $logger = new class() extends AbstractLogger {
+            private array $logs = [];
+
+            public function log($level, $message, array $context = []): void
+            {
+                $this->logs[] = [
+                    $level,
+                    (string) $message,
+                    $context,
+                ];
+            }
+
+            public function logs(): array
+            {
+                return $this->logs;
+            }
+        };
+        $lock = new Lock($key, $store, 10, true);
+        $lock->setLogger($logger);
+        $lock->release();
+
+        $this->assertSame([['debug', 'Successfully released the "{resource}" lock.', ['resource' => $key]]], $logger->logs());
+    }
+
     /**
      * @dataProvider provideExpiredDates
      */