- * It is ok, if the method is expensive as it is executed in the background. All + * It is ok if the method is expensive as it is executed in the background. All * runtime exceptions will be * caught and logged. *
@@ -46,7 +46,7 @@ default void initialize(EvaluationContext evaluationContext) throws Exception { * Providers can overwrite this method, if they have special shutdown actions * needed. *- * It is ok, if the method is expensive as it is executed in the background. All + * It is ok if the method is expensive as it is executed in the background. All * runtime exceptions will be * caught and logged. *
@@ -57,7 +57,11 @@ default void shutdown() { /** * Returns a representation of the current readiness of the provider. - * Providers which do not implement this method are assumed to be ready immediately. + * 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}. + * + *Providers which do not implement this method are assumed to be ready immediately.
* * @return ProviderState */ diff --git a/src/test/java/dev/openfeature/sdk/EventProviderTest.java b/src/test/java/dev/openfeature/sdk/EventProviderTest.java index cb73b529..3744682b 100644 --- a/src/test/java/dev/openfeature/sdk/EventProviderTest.java +++ b/src/test/java/dev/openfeature/sdk/EventProviderTest.java @@ -121,6 +121,11 @@ public ProviderEvaluation