[Lock] Prevent store exception break combined store

symfony · nicolas-grekas · Dec 14, 2020 · Dec 11, 2020 · Dec 14, 2020 · 0daff35bae09f8feda23bcfed081c797fc1aa526
commit 0daff35bae09f8feda23bcfed081c797fc1aa526
@@ -171,9 +171,14 @@ public function exists(Key $key)
        $storesCount = \count($this->stores);

        foreach ($this->stores as $store) {
-            if ($store->exists($key)) {
-                ++$successCount;
-            } else {
+            try {
+                if ($store->exists($key)) {
+                    ++$successCount;
+                } else {
+                    ++$failureCount;
+                }
+            } catch (\Exception $e) {
+                $this->logger->debug('One store failed to check the "{resource}" lock.', ['resource' => $key, 'store' => $store, 'exception' => $e]);
                ++$failureCount;
            }


@@ -351,4 +351,29 @@ public function testDeleteDontStopOnFailure()

        $this->store->delete($key);
    }
+
+    public function testExistsDontStopOnFailure()
+    {
+        $key = new Key(uniqid(__METHOD__, true));
+
+        $this->strategy
+            ->expects($this->any())
+            ->method('canBeMet')
+            ->willReturn(true);
+        $this->strategy
+            ->expects($this->any())
+            ->method('isMet')
+            ->willReturn(false);
+        $this->store1
+            ->expects($this->once())
+            ->method('exists')
+            ->willThrowException(new \Exception());
+        $this->store2
+            ->expects($this->once())
+            ->method('exists')
+            ->with($key)
+            ->willReturn(false);
+
+        $this->assertFalse($this->store->exists($key));
+    }
 }