add hook data spec test

Signed-off-by: Alexandra Oberaigner <alexandra.oberaigner@dynatrace.com>

Author: alexandraoberaigner

src/main/java/dev/openfeature/sdk/DefaultHookData.java

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

src/test/java/dev/openfeature/sdk/HookSpecTest.java

@@ -6,6 +6,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertInstanceOf;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doThrow;
@@ -801,4 +802,28 @@ void doesnt_use_finally() {
                         Hook.class.getMethod("finallyAfter", HookContext.class, FlagEvaluationDetails.class, Map.class))
                 .doesNotThrowAnyException();
     }
+
+    @Specification(
+            number = "4.6.1",
+            text = "hook data MUST be a structure supporting the definition of arbitrary "
+                    + "properties, with keys of type string, and values of any type.")
+    @Test
+    void hook_data_structure() {
+        // Arrange
+        HookData hookData = new DefaultHookData();
+
+        // Act - Add arbitrary properties to hook data
+        hookData.set("stringKey", "StringValue"); // String value
+        hookData.set("intKey", 42); // Integer value
+        hookData.set("doubleKey", 3.14); // Double value
+        hookData.set("objectKey", new Object()); // Object value
+        hookData.set("nullKey", null); // Null value
+
+        // Assert - Retrieve and validate the properties
+        assertEquals("StringValue", hookData.get("stringKey"));
+        assertEquals(42, hookData.get("intKey"));
+        assertEquals(3.14, hookData.get("doubleKey"));
+        assertNotNull(hookData.get("objectKey"));
+        assertNull(hookData.get("nullKey"));
+    }
 }

src/test/java/dev/openfeature/sdk/OpenFeatureClientTest.java

@@ -111,13 +111,14 @@ void shouldNotCallEvaluationMethodsWhenProviderIsInNotReadyState() {
     @ValueSource(booleans = {true, false})
     @DisplayName("Should support usage of HookData with/without error")
     void shouldSupportUsageOfHookData(boolean isError) {
-        FeatureProvider provider = new TestEventsProvider(5000);
         OpenFeatureAPI api = new OpenFeatureAPI();
+        FeatureProvider provider;
         if (isError) {
-            api.setProvider("shouldSupportUsageOfHookData", provider);
+            provider = new AlwaysBrokenWithExceptionProvider();
         } else {
-            api.setProviderAndWait("shouldSupportUsageOfHookData", provider);
+            provider = new TestEventsProvider(5000);
         }
+        api.setProviderAndWait("shouldSupportUsageOfHookData", provider);
 
         var testHook = new TestHookWithData("test-data");
         api.addHooks(testHook);