Skip to content

Commit c324962

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

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

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

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,21 @@
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+
29+
}
2230

2331
@Test
2432
void simpleBooleanFlag() {
25-
OpenFeatureAPI api = new OpenFeatureAPI();
2633
api.setProviderAndWait(new TestEventsProvider());
2734
Client client = api.getClient();
2835
Boolean retval = client.getBooleanValue(flagKey, false);
@@ -33,7 +40,6 @@ void simpleBooleanFlag() {
3340
void clientHooks() {
3441
Hook<Boolean> exampleHook = mockBooleanHook();
3542

36-
OpenFeatureAPI api = new OpenFeatureAPI();
3743
api.setProviderAndWait(new TestEventsProvider());
3844
Client client = api.getClient();
3945
client.addHooks(exampleHook);
@@ -47,7 +53,6 @@ void evalHooks() {
4753
Hook<Boolean> clientHook = mockBooleanHook();
4854
Hook<Boolean> evalHook = mockBooleanHook();
4955

50-
OpenFeatureAPI api = new OpenFeatureAPI();
5156
api.setProviderAndWait(new TestEventsProvider());
5257
Client client = api.getClient();
5358
client.addHooks(clientHook);
@@ -68,7 +73,6 @@ void evalHooks() {
6873
@Test
6974
void providingContext() {
7075

71-
OpenFeatureAPI api = new OpenFeatureAPI();
7276
api.setProviderAndWait(new TestEventsProvider());
7377
Client client = api.getClient();
7478
Map<String, Value> attributes = new HashMap<>();
@@ -85,7 +89,6 @@ void providingContext() {
8589

8690
@Test
8791
void brokenProvider() {
88-
OpenFeatureAPI api = new OpenFeatureAPI();
8992
api.setProviderAndWait(new AlwaysBrokenProvider());
9093
Client client = api.getClient();
9194
FlagEvaluationDetails<Boolean> retval = client.getBooleanDetails(flagKey, false);
@@ -132,7 +135,6 @@ public Optional before(HookContext ctx, Map hints) {
132135
@Test
133136
void setProviderAndWaitShouldPutTheProviderInReadyState() {
134137
String domain = "domain";
135-
OpenFeatureAPI api = new OpenFeatureAPI();
136138
api.setProviderAndWait(domain, new TestEventsProvider());
137139
Client client = api.getClient(domain);
138140
assertThat(client.getProviderState()).isEqualTo(ProviderState.READY);
@@ -145,7 +147,6 @@ void setProviderAndWaitShouldPutTheProviderInReadyState() {
145147
@Test
146148
void shouldPutTheProviderInStateErrorAfterEmittingErrorEvent() {
147149
String domain = "domain";
148-
OpenFeatureAPI api = new OpenFeatureAPI();
149150
TestEventsProvider provider = new TestEventsProvider();
150151
api.setProviderAndWait(domain, provider);
151152
Client client = api.getClient(domain);
@@ -161,7 +162,6 @@ void shouldPutTheProviderInStateErrorAfterEmittingErrorEvent() {
161162
@Test
162163
void shouldPutTheProviderInStateStaleAfterEmittingStaleEvent() {
163164
String domain = "domain";
164-
OpenFeatureAPI api = new OpenFeatureAPI();
165165
TestEventsProvider provider = new TestEventsProvider();
166166
api.setProviderAndWait(domain, provider);
167167
Client client = api.getClient(domain);
@@ -177,7 +177,6 @@ void shouldPutTheProviderInStateStaleAfterEmittingStaleEvent() {
177177
@Test
178178
void shouldPutTheProviderInStateReadyAfterEmittingReadyEvent() {
179179
String domain = "domain";
180-
OpenFeatureAPI api = new OpenFeatureAPI();
181180
TestEventsProvider provider = new TestEventsProvider();
182181
api.setProviderAndWait(domain, provider);
183182
Client client = api.getClient(domain);

0 commit comments

Comments
 (0)