Skip to content

Commit 739204d

Browse files
committed
fixup: test improvements
Signed-off-by: Simon Schrottner <simon.schrottner@dynatrace.com>
1 parent 54112fd commit 739204d

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

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

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,20 @@
1515
import java.util.Map;
1616
import java.util.Optional;
1717
import lombok.SneakyThrows;
18+
import org.junit.jupiter.api.BeforeEach;
1819
import org.junit.jupiter.api.Test;
1920

2021
class DeveloperExperienceTest implements HookFixtures {
2122
transient String flagKey = "mykey";
23+
private OpenFeatureAPI api;
24+
25+
@BeforeEach
26+
public void setUp() throws Exception {
27+
api = new OpenFeatureAPI();
28+
}
2229

2330
@Test
2431
void simpleBooleanFlag() {
25-
OpenFeatureAPI api = new OpenFeatureAPI();
2632
api.setProviderAndWait(new TestEventsProvider());
2733
Client client = api.getClient();
2834
Boolean retval = client.getBooleanValue(flagKey, false);
@@ -33,7 +39,6 @@ void simpleBooleanFlag() {
3339
void clientHooks() {
3440
Hook<Boolean> exampleHook = mockBooleanHook();
3541

36-
OpenFeatureAPI api = new OpenFeatureAPI();
3742
api.setProviderAndWait(new TestEventsProvider());
3843
Client client = api.getClient();
3944
client.addHooks(exampleHook);
@@ -47,7 +52,6 @@ void evalHooks() {
4752
Hook<Boolean> clientHook = mockBooleanHook();
4853
Hook<Boolean> evalHook = mockBooleanHook();
4954

50-
OpenFeatureAPI api = new OpenFeatureAPI();
5155
api.setProviderAndWait(new TestEventsProvider());
5256
Client client = api.getClient();
5357
client.addHooks(clientHook);
@@ -68,7 +72,6 @@ void evalHooks() {
6872
@Test
6973
void providingContext() {
7074

71-
OpenFeatureAPI api = new OpenFeatureAPI();
7275
api.setProviderAndWait(new TestEventsProvider());
7376
Client client = api.getClient();
7477
Map<String, Value> attributes = new HashMap<>();
@@ -85,7 +88,6 @@ void providingContext() {
8588

8689
@Test
8790
void brokenProvider() {
88-
OpenFeatureAPI api = new OpenFeatureAPI();
8991
api.setProviderAndWait(new AlwaysBrokenProvider());
9092
Client client = api.getClient();
9193
FlagEvaluationDetails<Boolean> retval = client.getBooleanDetails(flagKey, false);
@@ -132,7 +134,6 @@ public Optional before(HookContext ctx, Map hints) {
132134
@Test
133135
void setProviderAndWaitShouldPutTheProviderInReadyState() {
134136
String domain = "domain";
135-
OpenFeatureAPI api = new OpenFeatureAPI();
136137
api.setProviderAndWait(domain, new TestEventsProvider());
137138
Client client = api.getClient(domain);
138139
assertThat(client.getProviderState()).isEqualTo(ProviderState.READY);
@@ -145,7 +146,6 @@ void setProviderAndWaitShouldPutTheProviderInReadyState() {
145146
@Test
146147
void shouldPutTheProviderInStateErrorAfterEmittingErrorEvent() {
147148
String domain = "domain";
148-
OpenFeatureAPI api = new OpenFeatureAPI();
149149
TestEventsProvider provider = new TestEventsProvider();
150150
api.setProviderAndWait(domain, provider);
151151
Client client = api.getClient(domain);
@@ -161,7 +161,6 @@ void shouldPutTheProviderInStateErrorAfterEmittingErrorEvent() {
161161
@Test
162162
void shouldPutTheProviderInStateStaleAfterEmittingStaleEvent() {
163163
String domain = "domain";
164-
OpenFeatureAPI api = new OpenFeatureAPI();
165164
TestEventsProvider provider = new TestEventsProvider();
166165
api.setProviderAndWait(domain, provider);
167166
Client client = api.getClient(domain);
@@ -177,7 +176,6 @@ void shouldPutTheProviderInStateStaleAfterEmittingStaleEvent() {
177176
@Test
178177
void shouldPutTheProviderInStateReadyAfterEmittingReadyEvent() {
179178
String domain = "domain";
180-
OpenFeatureAPI api = new OpenFeatureAPI();
181179
TestEventsProvider provider = new TestEventsProvider();
182180
api.setProviderAndWait(domain, provider);
183181
Client client = api.getClient(domain);

0 commit comments

Comments
 (0)