fixup: apply changes

Signed-off-by: Simon Schrottner <simon.schrottner@dynatrace.com>

Author: aepfli

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

@@ -3,7 +3,7 @@
 import java.util.HashMap;
 import java.util.Map;
 
-@SuppressWarnings({ "PMD.BeanMembersShouldSerialize", "checkstyle:MissingJavadocType" })
+@SuppressWarnings({"PMD.BeanMembersShouldSerialize", "checkstyle:MissingJavadocType"})
 abstract class AbstractStructure implements Structure {
 
     protected final Map<String, Value> attributes;
@@ -23,12 +23,11 @@ abstract class AbstractStructure implements Structure {
      */
     @Override
     public Map<String, Object> asObjectMap() {
-        return attributes
-                .entrySet()
-                .stream()
+        return attributes.entrySet().stream()
                 // custom collector, workaround for Collectors.toMap in JDK8
                 // https://bugs.openjdk.org/browse/JDK-8148463
-                .collect(HashMap::new,
+                .collect(
+                        HashMap::new,
                         (accumulated, entry) -> accumulated.put(entry.getKey(), convertValue(entry.getValue())),
                         HashMap::putAll);
     }

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

@@ -3,7 +3,7 @@
 /**
  * An extension point which can run around flag resolution. They are intended to be used as a way to add custom logic
  * to the lifecycle of flag evaluation.
- * 
+ *
  * @see Hook
  */
 public interface BooleanHook extends Hook<Boolean> {

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

@@ -3,7 +3,7 @@
 /**
  * An extension point which can run around flag resolution. They are intended to be used as a way to add custom logic
  * to the lifecycle of flag evaluation.
- * 
+ *
  * @see Hook
  */
 public interface DoubleHook extends Hook<Double> {
@@ -12,4 +12,4 @@ public interface DoubleHook extends Hook<Double> {
     default boolean supportsFlagValueType(FlagValueType flagValueType) {
         return FlagValueType.DOUBLE == flagValueType;
     }
-}
+}

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

@@ -2,5 +2,11 @@
 
 @SuppressWarnings("checkstyle:MissingJavadocType")
 public enum ErrorCode {
-    PROVIDER_NOT_READY, FLAG_NOT_FOUND, PARSE_ERROR, TYPE_MISMATCH, TARGETING_KEY_MISSING, INVALID_CONTEXT, GENERAL
+    PROVIDER_NOT_READY,
+    FLAG_NOT_FOUND,
+    PARSE_ERROR,
+    TYPE_MISMATCH,
+    TARGETING_KEY_MISSING,
+    INVALID_CONTEXT,
+    GENERAL
 }

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

@@ -6,56 +6,56 @@
  * Interface for attaching event handlers.
  */
 public interface EventBus<T> {
-    
+
     /**
      * Add a handler for the {@link ProviderEvent#PROVIDER_READY} event.
      * Shorthand for {@link #on(ProviderEvent, Consumer)}
-     * 
+     *
      * @param handler behavior to add with this event
      * @return this
      */
     T onProviderReady(Consumer<EventDetails> handler);
-    
+
     /**
      * Add a handler for the {@link ProviderEvent#PROVIDER_CONFIGURATION_CHANGED} event.
      * Shorthand for {@link #on(ProviderEvent, Consumer)}
-     * 
+     *
      * @param handler behavior to add with this event
      * @return this
      */
     T onProviderConfigurationChanged(Consumer<EventDetails> handler);
-    
+
     /**
      * Add a handler for the {@link ProviderEvent#PROVIDER_STALE} event.
      * Shorthand for {@link #on(ProviderEvent, Consumer)}
-     * 
+     *
      * @param handler behavior to add with this event
      * @return this
      */
     T onProviderError(Consumer<EventDetails> handler);
-    
+
     /**
      * Add a handler for the {@link ProviderEvent#PROVIDER_ERROR} event.
      * Shorthand for {@link #on(ProviderEvent, Consumer)}
-     * 
+     *
      * @param handler behavior to add with this event
      * @return this
      */
     T onProviderStale(Consumer<EventDetails> handler);
 
     /**
      * Add a handler for the specified {@link ProviderEvent}.
-     * 
+     *
      * @param event event type
      * @param handler behavior to add with this event
      * @return this
      */
     T on(ProviderEvent event, Consumer<EventDetails> handler);
-    
+
     /**
      * Remove the previously attached handler by reference.
      * If the handler doesn't exists, no-op.
-     * 
+     *
      * @param event event type
      * @param handler to be removed
      * @return this

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

@@ -17,9 +17,7 @@ static EventDetails fromProviderEventDetails(ProviderEventDetails providerEventD
     }
 
     static EventDetails fromProviderEventDetails(
-            ProviderEventDetails providerEventDetails,
-            String providerName,
-            String domain) {
+            ProviderEventDetails providerEventDetails, String providerName, String domain) {
         return builder()
                 .domain(domain)
                 .providerName(providerName)

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

@@ -26,7 +26,7 @@ public abstract class EventProvider implements FeatureProvider {
 
     /**
      * "Attach" this EventProvider to an SDK, which allows events to propagate from this provider.
-     * No-op if the same onEmit is already attached. 
+     * No-op if the same onEmit is already attached.
      *
      * @param onEmit the function to run when a provider emits events.
      * @throws IllegalStateException if attempted to bind a new emitter for already bound provider
@@ -50,7 +50,7 @@ void detach() {
 
     /**
      * Emit the specified {@link ProviderEvent}.
-     * 
+     *
      * @param event   The event type
      * @param details The details of the event
      */
@@ -63,7 +63,7 @@ public void emit(ProviderEvent event, ProviderEventDetails details) {
     /**
      * Emit a {@link ProviderEvent#PROVIDER_READY} event.
      * Shorthand for {@link #emit(ProviderEvent, ProviderEventDetails)}
-     * 
+     *
      * @param details The details of the event
      */
     public void emitProviderReady(ProviderEventDetails details) {
@@ -74,7 +74,7 @@ public void emitProviderReady(ProviderEventDetails details) {
      * Emit a
      * {@link ProviderEvent#PROVIDER_CONFIGURATION_CHANGED}
      * event. Shorthand for {@link #emit(ProviderEvent, ProviderEventDetails)}
-     * 
+     *
      * @param details The details of the event
      */
     public void emitProviderConfigurationChanged(ProviderEventDetails details) {
@@ -84,7 +84,7 @@ public void emitProviderConfigurationChanged(ProviderEventDetails details) {
     /**
      * Emit a {@link ProviderEvent#PROVIDER_STALE} event.
      * Shorthand for {@link #emit(ProviderEvent, ProviderEventDetails)}
-     * 
+     *
      * @param details The details of the event
      */
     public void emitProviderStale(ProviderEventDetails details) {
@@ -94,7 +94,7 @@ public void emitProviderStale(ProviderEventDetails details) {
     /**
      * Emit a {@link ProviderEvent#PROVIDER_ERROR} event.
      * Shorthand for {@link #emit(ProviderEvent, ProviderEventDetails)}
-     * 
+     *
      * @param details The details of the event
      */
     public void emitProviderError(ProviderEventDetails details) {

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

@@ -1,7 +1,5 @@
 package dev.openfeature.sdk;
 
-import lombok.extern.slf4j.Slf4j;
-
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -13,6 +11,7 @@
 import java.util.concurrent.Executors;
 import java.util.concurrent.TimeUnit;
 import java.util.function.Consumer;
+import lombok.extern.slf4j.Slf4j;
 
 /**
  * Util class for storing and running handlers.
@@ -35,74 +34,68 @@ class EventSupport {
     /**
      * Run all the event handlers associated with this domain.
      * If the domain is null, handlers attached to unnamed clients will run.
-     * 
+     *
      * @param domain       the domain to run event handlers for, or null
      * @param event        the event type
      * @param eventDetails the event details
      */
     public void runClientHandlers(String domain, ProviderEvent event, EventDetails eventDetails) {
-        domain = Optional.ofNullable(domain)
-                .orElse(defaultClientUuid);
+        domain = Optional.ofNullable(domain).orElse(defaultClientUuid);
 
         // run handlers if they exist
         Optional.ofNullable(handlerStores.get(domain))
                 .filter(store -> Optional.of(store).isPresent())
                 .map(store -> store.handlerMap.get(event))
-                .ifPresent(handlers -> handlers
-                        .forEach(handler -> runHandler(handler, eventDetails)));
+                .ifPresent(handlers -> handlers.forEach(handler -> runHandler(handler, eventDetails)));
     }
 
     /**
      * Run all the API (global) event handlers.
-     * 
+     *
      * @param event        the event type
      * @param eventDetails the event details
      */
     public void runGlobalHandlers(ProviderEvent event, EventDetails eventDetails) {
-        globalHandlerStore.handlerMap.get(event)
-                .forEach(handler -> {
-                    runHandler(handler, eventDetails);
-                });
+        globalHandlerStore.handlerMap.get(event).forEach(handler -> {
+            runHandler(handler, eventDetails);
+        });
     }
 
     /**
      * Add a handler for the specified domain, or all unnamed clients.
-     * 
+     *
      * @param domain     the domain to add handlers for, or else unnamed
      * @param event      the event type
      * @param handler    the handler function to run
      */
     public void addClientHandler(String domain, ProviderEvent event, Consumer<EventDetails> handler) {
-        final String name = Optional.ofNullable(domain)
-                .orElse(defaultClientUuid);
+        final String name = Optional.ofNullable(domain).orElse(defaultClientUuid);
 
         // lazily create and cache a HandlerStore if it doesn't exist
-        HandlerStore store = Optional.ofNullable(this.handlerStores.get(name))
-                .orElseGet(() -> {
-                    HandlerStore newStore = new HandlerStore();
-                    this.handlerStores.put(name, newStore);
-                    return newStore;
-                });
+        HandlerStore store = Optional.ofNullable(this.handlerStores.get(name)).orElseGet(() -> {
+            HandlerStore newStore = new HandlerStore();
+            this.handlerStores.put(name, newStore);
+            return newStore;
+        });
         store.addHandler(event, handler);
     }
 
     /**
      * Remove a client event handler for the specified event type.
-     * 
+     *
      * @param domain     the domain of the client handler to remove, or null to remove
      *                   from unnamed clients
      * @param event      the event type
      * @param handler    the handler ref to be removed
      */
     public void removeClientHandler(String domain, ProviderEvent event, Consumer<EventDetails> handler) {
-        domain = Optional.ofNullable(domain)
-                .orElse(defaultClientUuid);
+        domain = Optional.ofNullable(domain).orElse(defaultClientUuid);
         this.handlerStores.get(domain).removeHandler(event, handler);
     }
 
     /**
      * Add a global event handler of the specified event type.
-     * 
+     *
      * @param event   the event type
      * @param handler the handler to be added
      */
@@ -112,7 +105,7 @@ public void addGlobalHandler(ProviderEvent event, Consumer<EventDetails> handler
 
     /**
      * Remove a global event handler for the specified event type.
-     * 
+     *
      * @param event   the event type
      * @param handler the handler ref to be removed
      */
@@ -122,7 +115,7 @@ public void removeGlobalHandler(ProviderEvent event, Consumer<EventDetails> hand
 
     /**
      * Get all domain names for which we have event handlers registered.
-     * 
+     *
      * @return set of domain names
      */
     public Set<String> getAllDomainNames() {
@@ -131,7 +124,7 @@ public Set<String> getAllDomainNames() {
 
     /**
      * Run the passed handler on the taskExecutor.
-     * 
+     *
      * @param handler      the handler to run
      * @param eventDetails the event details
      */

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

@@ -60,13 +60,12 @@ default void shutdown() {
      * If the provider needs to be initialized, it should return {@link ProviderState#NOT_READY}.
      * If the provider is in an error state, it should return {@link ProviderState#ERROR}.
      * If the provider is functioning normally, it should return {@link ProviderState#READY}.
-     * 
+     *
      * <p><i>Providers which do not implement this method are assumed to be ready immediately.</i></p>
-     * 
+     *
      * @return ProviderState
      */
     default ProviderState getState() {
         return ProviderState.READY;
     }
-
 }