Merge pull request #1 from brnck/master

Change Lock to LockInterface

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## 0.2.0
+### Changed 
+- `LockManager` now takes and return `LockInterface` instead of `Lock` which implements it 
+
 ## 0.1.1
 ### Changed
 - Downgraded `phpunit` to `^6.0` to have PHP7.0 dev compatibility

src/Service/LockManager.php

@@ -4,7 +4,7 @@
 
 use Symfony\Component\Lock\Exception\LockAcquiringException;
 use Symfony\Component\Lock\Factory;
-use Symfony\Component\Lock\Lock;
+use Symfony\Component\Lock\LockInterface;
 
 class LockManager
 {
@@ -19,12 +19,12 @@ public function __construct(
         $this->ttl = $ttl;
     }
 
-    public function createLock(string $resource): Lock
+    public function createLock(string $resource): LockInterface
     {
         return $this->lockFactory->createLock($resource, $this->ttl);
     }
 
-    public function acquire(Lock $lock): bool
+    public function acquire(LockInterface $lock): bool
     {
         foreach (range(1, $this->ttl) as $waited) {
             if ($lock->acquire()) {
@@ -41,15 +41,15 @@ public function acquire(Lock $lock): bool
         throw new LockAcquiringException('Failed to acquire lock');
     }
 
-    public function createAcquired(string $resource): Lock
+    public function createAcquired(string $resource): LockInterface
     {
         $lock = $this->createLock($resource);
         $this->acquire($lock);
 
         return $lock;
     }
 
-    public function release(Lock $lock)
+    public function release(LockInterface $lock)
     {
         $lock->release();
     }