fixup: more tests for safety with claude.ai

Signed-off-by: Simon Schrottner <[email protected]>

Author: aepfli

openfeature-api/pom.xml

@@ -15,6 +15,8 @@
     <name>OpenFeature Java API</name>
     <description>OpenFeature Java API - Core contracts and interfaces for feature flag evaluation</description>
 
+    <version>0.0.1</version>
+
     <properties>
         <module-name>dev.openfeature.api</module-name>
     </properties>

openfeature-api/src/main/java/dev/openfeature/api/AbstractEventProvider.java

@@ -71,6 +71,9 @@ public void shutdown() {
      * @param details The details of the event
      */
     public Awaitable emit(final ProviderEvent event, final ProviderEventDetails details) {
+        if (eventEmitter == null) {
+            return null;
+        }
         return eventEmitter.emit(event, details);
     }
 

openfeature-api/src/main/java/dev/openfeature/api/evaluation/ProviderEvaluation.java

@@ -1,7 +1,6 @@
 package dev.openfeature.api.evaluation;
 
 import dev.openfeature.api.ErrorCode;
-import dev.openfeature.api.Reason;
 import dev.openfeature.api.types.Metadata;
 
 /**
@@ -17,10 +16,10 @@ static <T> ProviderEvaluation<T> of(T value, String variant, String reason, Meta
 
     static <T> ProviderEvaluation<T> of(
             T value, String variant, String reason, ErrorCode errorCode, String errorMessage, Metadata flagMetadata) {
-        return new DefaultProviderEvaluation<T>(value, variant, reason, errorCode, errorMessage, flagMetadata);
+        return new DefaultProviderEvaluation<>(value, variant, reason, errorCode, errorMessage, flagMetadata);
     }
 
-    static <T> ProviderEvaluation<T> of(ErrorCode errorCode, String errorMessage) {
-        return of(null, null, Reason.ERROR.toString(), errorCode, errorMessage, null);
+    static <T> ProviderEvaluation<T> of(ErrorCode errorCode, String errorMessage, Metadata flagMetadata) {
+        return of(null, null, null, errorCode, errorMessage, flagMetadata);
     }
 }

openfeature-api/src/main/java/dev/openfeature/api/lifecycle/DefaultHookContext.java

@@ -0,0 +1,70 @@
+package dev.openfeature.api.lifecycle;
+
+import dev.openfeature.api.FlagValueType;
+import dev.openfeature.api.evaluation.EvaluationContext;
+import dev.openfeature.api.types.ClientMetadata;
+import dev.openfeature.api.types.ProviderMetadata;
+
+/**
+ * A default implementation of {@link HookContext}.
+ */
+final class DefaultHookContext<T> implements HookContext<T> {
+
+    private final String flagKey;
+    private final T defaultValue;
+    private final FlagValueType type;
+    private final ProviderMetadata providerMetadata;
+    private final ClientMetadata clientMetadata;
+    private final EvaluationContext evaluationContext;
+    private final HookData hookData = new DefaultHookData();
+
+    DefaultHookContext(
+            String flagKey,
+            T defaultValue,
+            FlagValueType type,
+            ProviderMetadata providerMetadata,
+            ClientMetadata clientMetadata,
+            EvaluationContext evaluationContext) {
+        this.flagKey = flagKey;
+        this.defaultValue = defaultValue;
+        this.type = type;
+        this.providerMetadata = providerMetadata;
+        this.clientMetadata = clientMetadata;
+        this.evaluationContext = evaluationContext;
+    }
+
+    @Override
+    public String getFlagKey() {
+        return flagKey;
+    }
+
+    @Override
+    public FlagValueType getType() {
+        return type;
+    }
+
+    @Override
+    public T getDefaultValue() {
+        return defaultValue;
+    }
+
+    @Override
+    public EvaluationContext getCtx() {
+        return evaluationContext;
+    }
+
+    @Override
+    public ClientMetadata getClientMetadata() {
+        return clientMetadata;
+    }
+
+    @Override
+    public ProviderMetadata getProviderMetadata() {
+        return providerMetadata;
+    }
+
+    @Override
+    public HookData getHookData() {
+        return hookData;
+    }
+}

openfeature-api/src/main/java/dev/openfeature/api/lifecycle/DefaultHookData.java

@@ -6,7 +6,7 @@
 /**
  * Default implementation of HookData.
  */
-class DefaultHookData implements HookData {
+final class DefaultHookData implements HookData {
     Map<String, Object> data;
 
     @Override

openfeature-api/src/main/java/dev/openfeature/api/lifecycle/HookContext.java

@@ -11,6 +11,17 @@
  */
 public interface HookContext<T> {
 
+    static <T> HookContext<T> of(
+            final String flagKey,
+            final T defaultValue,
+            FlagValueType type,
+            ProviderMetadata providerMetadata,
+            ClientMetadata clientMetadata,
+            EvaluationContext evaluationContext) {
+        return new DefaultHookContext<>(
+                flagKey, defaultValue, type, providerMetadata, clientMetadata, evaluationContext);
+    }
+
     String getFlagKey();
 
     FlagValueType getType();
@@ -23,7 +34,5 @@ public interface HookContext<T> {
 
     ProviderMetadata getProviderMetadata();
 
-    default HookData getHookData() {
-        return null;
-    }
+    HookData getHookData();
 }