PR suggestion: make HookSupportData a real POJO

Author: alexandraoberaigner

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

@@ -33,8 +33,11 @@ public void updateEvaluationContext(HookSupportData hookSupportData, EvaluationC
      *                          created
      * @param evaluationContext the evaluation context which is
      */
-    public void setHookSupportDataHooks(HookSupportData hookSupportData, List<Hook> hooks,
-            SharedHookContext sharedContext, EvaluationContext evaluationContext) {
+    public void setHookSupportDataHooks(
+            HookSupportData hookSupportData,
+            List<Hook> hooks,
+            SharedHookContext sharedContext,
+            EvaluationContext evaluationContext) {
         List<Pair<Hook, HookContext>> hookContextPairs = new ArrayList<>();
         for (Hook hook : hooks) {
             if (hook.supportsFlagValueType(sharedContext.getType())) {

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

@@ -15,5 +15,4 @@ class HookSupportData {
     Map<String, Object> hints;
 
     HookSupportData() {}
-
 }

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

@@ -35,8 +35,8 @@ void shouldMergeEvaluationContextsOnBeforeHooksCorrectly() {
 
         var sharedContext = getBaseHookContextForType(FlagValueType.STRING);
         var hookSupportData = new HookSupportData();
-        hookSupport.setHookSupportDataHooks(hookSupportData,
-                Arrays.asList(hook1, hook2), sharedContext, baseEvalContext);
+        hookSupport.setHookSupportDataHooks(
+                hookSupportData, Arrays.asList(hook1, hook2), sharedContext, baseEvalContext);
         hookSupport.updateEvaluationContext(hookSupportData, baseEvalContext);
 
         hookSupport.executeBeforeHooks(hookSupportData);
@@ -55,8 +55,11 @@ void shouldAlwaysCallGenericHook(FlagValueType flagValueType) {
         Hook<?> genericHook = mockGenericHook();
 
         var hookSupportData = new HookSupportData();
-        hookSupport.setHookSupportDataHooks(hookSupportData,
-                List.of(genericHook), getBaseHookContextForType(flagValueType), ImmutableContext.EMPTY);
+        hookSupport.setHookSupportDataHooks(
+                hookSupportData,
+                List.of(genericHook),
+                getBaseHookContextForType(flagValueType),
+                ImmutableContext.EMPTY);
 
         callAllHooks(hookSupportData);
 
@@ -72,8 +75,8 @@ void shouldAlwaysCallGenericHook(FlagValueType flagValueType) {
     void shouldPassDataAcrossStages(FlagValueType flagValueType) {
         var testHook = new TestHookWithData();
         var hookSupportData = new HookSupportData();
-        hookSupport.setHookSupportDataHooks(hookSupportData,
-                List.of(testHook), getBaseHookContextForType(flagValueType), ImmutableContext.EMPTY);
+        hookSupport.setHookSupportDataHooks(
+                hookSupportData, List.of(testHook), getBaseHookContextForType(flagValueType), ImmutableContext.EMPTY);
 
         hookSupport.executeBeforeHooks(hookSupportData);
         assertHookData(testHook, "before");
@@ -98,8 +101,11 @@ void shouldIsolateDataBetweenHooks(FlagValueType flagValueType) {
         var testHook2 = new TestHookWithData(2);
 
         var hookSupportData = new HookSupportData();
-        hookSupport.setHookSupportDataHooks(hookSupportData,
-                List.of(testHook1, testHook2), getBaseHookContextForType(flagValueType), ImmutableContext.EMPTY);
+        hookSupport.setHookSupportDataHooks(
+                hookSupportData,
+                List.of(testHook1, testHook2),
+                getBaseHookContextForType(flagValueType),
+                ImmutableContext.EMPTY);
 
         callAllHooks(hookSupportData);