make FeatureProviderStateManager a true wrapper, without implementing FeatureProvider, fix codestyle

chrfwow · chrfwow · commit b0d66a5253d8 · 2024-09-18T09:09:18.000+02:00
Signed-off-by: christian.lutnik &lt;christian.lutnik@dynatrace.com&gt;
diff --git a/src/main/java/dev/openfeature/sdk/FeatureProviderStateManager.java b/src/main/java/dev/openfeature/sdk/FeatureProviderStateManager.java
@@ -76,7 +76,7 @@ FeatureProvider getProvider() {
         return delegate;
     }
 
-    public boolean hasSameProvider(FeatureProvider featureProvider){
+    public boolean hasSameProvider(FeatureProvider featureProvider) {
         return this.delegate.equals(featureProvider);
     }
 }
diff --git a/src/main/java/dev/openfeature/sdk/ProviderRepository.java b/src/main/java/dev/openfeature/sdk/ProviderRepository.java
@@ -87,7 +87,10 @@ public ProviderState getProviderState(FeatureProvider featureProvider) {
     }
 
     public ProviderState getProviderState(String domain) {
-        return Optional.ofNullable(domain).map(this.stateManagers::get).orElse(this.defaultStateManger.get()).getState();
+        return Optional.ofNullable(domain)
+                .map(this.stateManagers::get)
+                .orElse(this.defaultStateManger.get())
+                .getState();
     }
 
     public List<String> getDomainsForProvider(FeatureProvider provider) {

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ FeatureProvider getProvider() {`
`76`	`76`	`return delegate;`
`77`	`77`	`}`
`78`	`78`
`79`		`- public boolean hasSameProvider(FeatureProvider featureProvider){`
	`79`	`+ public boolean hasSameProvider(FeatureProvider featureProvider) {`
`80`	`80`	`return this.delegate.equals(featureProvider);`
`81`	`81`	`}`
`82`	`82`	`}`
Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,10 @@ public ProviderState getProviderState(FeatureProvider featureProvider) {`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`public ProviderState getProviderState(String domain) {`
`90`		`- return Optional.ofNullable(domain).map(this.stateManagers::get).orElse(this.defaultStateManger.get()).getState();`
	`90`	`+ return Optional.ofNullable(domain)`
	`91`	`+ .map(this.stateManagers::get)`
	`92`	`+ .orElse(this.defaultStateManger.get())`
	`93`	`+ .getState();`
`91`	`94`	`}`
`92`	`95`
`93`	`96`	`public List<String> getDomainsForProvider(FeatureProvider provider) {`