Merge pull request #13 from Crell/registry-has

Add has() method to the registry to make it easier to avoid double-registering
elazar · Aug 15, 2024 · 13bf51e · 13bf51e
2 parents a0bed9c + 8fe0be6
commit 13bf51e
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/FilesystemRegistry.php b/src/FilesystemRegistry.php
@@ -42,6 +42,11 @@ public function get(
         return $this->filesystems[$protocol];
     }
 
+    public function has(string $protocol): bool
+    {
+        return array_key_exists($protocol, $this->filesystems);
+    }
+
     private function checkForProtocol(string $protocol): void
     {
         if (!isset($this->filesystems[$protocol])) {

diff --git a/tests/FilesystemRegistryTest.php b/tests/FilesystemRegistryTest.php
@@ -53,3 +53,11 @@
     FlystreamException::class,
     'Specified protocol is not registered: foo'
 );
+
+it('knows when a protocol is already registered', function () {
+    expect($this->registry->has('beep'))->toEqual(false);
+    $this->registry->register('beep', $this->filesystem);
+    expect($this->registry->has('beep'))->toEqual(true);
+    $this->registry->unregister('beep');
+    expect($this->registry->has('beep'))->toEqual(false);
+});