feat: provide a base AbstractProvider for provider implementations to…

… use
open-feature · Oct 30, 2022 · 78be67c · 78be67c
1 parent 2344c4a
commit 78be67c
Show file tree

Hide file tree

Showing 2 changed files with 59 additions and 29 deletions.
diff --git a/src/implementation/provider/AbstractProvider.php b/src/implementation/provider/AbstractProvider.php
@@ -0,0 +1,57 @@
+<?php
+
+declare(strict_types=1);
+
+namespace OpenFeature\implementation\provider;
+
+use OpenFeature\implementation\common\Metadata;
+use OpenFeature\interfaces\common\Metadata as MetadataInterface;
+use OpenFeature\interfaces\flags\EvaluationContext;
+use OpenFeature\interfaces\hooks\Hook;
+use OpenFeature\interfaces\provider\Provider;
+use OpenFeature\interfaces\provider\ResolutionDetails as ResolutionDetailsInterface;
+use Psr\Log\LoggerAwareTrait;
+
+abstract class AbstractProvider implements Provider
+{
+    use LoggerAwareTrait;
+
+    protected static string $NAME = 'AbstractProvider';
+
+    /** @var Hook[] $hooks */
+    private array $hooks = [];
+
+    public function getMetadata(): MetadataInterface
+    {
+        return new Metadata(self::$NAME);
+    }
+
+    abstract public function resolveBooleanValue(string $flagKey, bool $defaultValue, ?EvaluationContext $context = null): ResolutionDetailsInterface;
+
+    abstract public function resolveStringValue(string $flagKey, string $defaultValue, ?EvaluationContext $context = null): ResolutionDetailsInterface;
+
+    abstract public function resolveIntegerValue(string $flagKey, int $defaultValue, ?EvaluationContext $context = null): ResolutionDetailsInterface;
+
+    abstract public function resolveFloatValue(string $flagKey, float $defaultValue, ?EvaluationContext $context = null): ResolutionDetailsInterface;
+
+    /**
+     * @inheritdoc
+     */
+    abstract public function resolveObjectValue(string $flagKey, $defaultValue, ?EvaluationContext $context = null): ResolutionDetailsInterface;
+
+    /**
+     * @return Hook[]
+     */
+    public function getHooks(): array
+    {
+        return $this->hooks;
+    }
+
+    /**
+     * @param Hook[] $hooks
+     */
+    public function setHooks(array $hooks): void
+    {
+        $this->hooks = $hooks;
+    }
+}
diff --git a/src/implementation/provider/NoOpProvider.php b/src/implementation/provider/NoOpProvider.php
@@ -4,24 +4,13 @@
 
 namespace OpenFeature\implementation\provider;
 
-use OpenFeature\implementation\common\Metadata;
-use OpenFeature\interfaces\common\Metadata as MetadataInterface;
 use OpenFeature\interfaces\flags\EvaluationContext;
-use OpenFeature\interfaces\hooks\Hook;
 use OpenFeature\interfaces\provider\Provider;
 use OpenFeature\interfaces\provider\ResolutionDetails as ResolutionDetailsInterface;
-use Psr\Log\LoggerAwareTrait;
 
-class NoOpProvider implements Provider
+class NoOpProvider extends AbstractProvider implements Provider
 {
-    use LoggerAwareTrait;
-
-    private static string $NAME = 'NoOpProvider';
-
-    public function getMetadata(): MetadataInterface
-    {
-        return new Metadata(self::$NAME);
-    }
+    protected static string $NAME = 'NoOpProvider';
 
     public function resolveBooleanValue(string $flagKey, bool $defaultValue, ?EvaluationContext $context = null): ResolutionDetailsInterface
     {
@@ -50,20 +39,4 @@ public function resolveObjectValue(string $flagKey, $defaultValue, ?EvaluationCo
     {
         return ResolutionDetailsFactory::fromSuccess($defaultValue);
     }
-
-    /**
-     * @return Hook[]
-     */
-    public function getHooks(): array
-    {
-        return [];
-    }
-
-    /**
-     * @param Hook[] $hooks
-     */
-    public function setHooks(array $hooks): void
-    {
-        // no-op
-    }
 }