feat: Complete cleanup of legacy files and remove Lombok from SDK tests

## Legacy File Cleanup
- Remove entire legacy `src/` directory (85+ source files, 100+ test files)
- Remove `pom.xml.backup` backup file
- Remove orphaned `test_noop_access.java` test file
- Cleanup eliminates duplicate code after successful module separation

## SDK Test Lombok Removal (17 files)
### Files with @Getter annotations:
- MockHook.java: Added manual getters (isBeforeCalled, isAfterCalled, etc.)
- ContextStoringProvider.java: Added manual getEvaluationContext() getter

### Files with @SneakyThrows annotations:
- Replaced with proper `throws Exception` declarations in 12 test files:
  EventProviderTest, TrackingSpecTest, FlagEvaluationSpecTest, EventsTest,
  FeatureProviderStateManagerTest, HookSpecTest, LoggingHookTest,
  InMemoryProviderTest, StepDefinitions, TestEventsProvider,
  ThreadLocalTransactionContextPropagatorTest
- DeveloperExperienceTest: Replaced with proper try-catch block

### Files with @UtilityClass annotations:
- ProviderFixture.java, TestFlagsUtils.java, ConditionStubber.java:
  Replaced with private constructors to prevent instantiation

## Results
- Project is now completely Lombok-free (both API and SDK modules)
- Clean multi-module structure without legacy duplicates
- All main source code compiles successfully
- Maintains same test functionality with manual implementations

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Signed-off-by: Simon  Schrottner <[email protected]>

Author: aepfli

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

@@ -28,7 +28,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -120,13 +119,16 @@ void providerLockedPerTransaction() {
         class MutatingHook implements Hook {
 
             @Override
-            @SneakyThrows
             // change the provider during a before hook - this should not impact the evaluation in progress
             public Optional before(HookContext ctx, Map hints) {
+                try {
 
-                api.setProviderAndWait(TestEventsProvider.newInitializedTestEventsProvider());
+                    api.setProviderAndWait(TestEventsProvider.newInitializedTestEventsProvider());
 
-                return Optional.empty();
+                    return Optional.empty();
+                } catch (Exception e) {
+                    throw new RuntimeException(e);
+                }
             }
         }
 

openfeature-sdk/src/test/java/dev/openfeature/sdk/EventProviderTest.java

@@ -17,7 +17,6 @@
 import dev.openfeature.sdk.internal.TriConsumer;
 import dev.openfeature.sdk.testutils.TestStackedEmitCallsProvider;
 import io.cucumber.java.AfterAll;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -30,8 +29,7 @@ class EventProviderTest {
     private TestEventProvider eventProvider;
 
     @BeforeEach
-    @SneakyThrows
-    void setup() {
+    void setup() throws Exception {
         eventProvider = new TestEventProvider();
         eventProvider.initialize(null);
     }
@@ -97,10 +95,9 @@ void doesNotThrowWhenOnEmitSame() {
     }
 
     @Test
-    @SneakyThrows
     @Timeout(value = 2, threadMode = Timeout.ThreadMode.SEPARATE_THREAD)
     @DisplayName("should not deadlock on emit called during emit")
-    void doesNotDeadlockOnEmitStackedCalls() {
+    void doesNotDeadlockOnEmitStackedCalls() throws Exception {
         TestStackedEmitCallsProvider provider = new TestStackedEmitCallsProvider();
         new DefaultOpenFeatureAPI().setProviderAndWait(provider);
     }

openfeature-sdk/src/test/java/dev/openfeature/sdk/EventsTest.java

@@ -23,7 +23,6 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.function.Consumer;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Nested;
@@ -687,8 +686,7 @@ class HandlerRemoval {
                 text = "The API and client MUST provide a function allowing the removal of event handlers.")
         @Test
         @DisplayName("should not run removed events")
-        @SneakyThrows
-        void removedEventsShouldNotRun() {
+        void removedEventsShouldNotRun() throws Exception {
             final String name = "removedEventsShouldNotRun";
             final Consumer<EventDetails> handler1 = mockHandler();
             final Consumer<EventDetails> handler2 = mockHandler();

openfeature-sdk/src/test/java/dev/openfeature/sdk/FeatureProviderStateManagerTest.java

@@ -15,7 +15,6 @@
 import dev.openfeature.api.exceptions.GeneralError;
 import java.util.concurrent.atomic.AtomicInteger;
 import javax.annotation.Nullable;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -30,17 +29,15 @@ public void setUp() {
         wrapper = new FeatureProviderStateManager(testDelegate);
     }
 
-    @SneakyThrows
     @Test
-    void shouldOnlyCallInitOnce() {
+    void shouldOnlyCallInitOnce() throws Exception {
         wrapper.initialize(null);
         wrapper.initialize(null);
         assertThat(testDelegate.initCalled.get()).isOne();
     }
 
-    @SneakyThrows
     @Test
-    void shouldCallInitTwiceWhenShutDownInTheMeantime() {
+    void shouldCallInitTwiceWhenShutDownInTheMeantime() throws Exception {
         wrapper.initialize(null);
         wrapper.shutdown();
         wrapper.initialize(null);
@@ -53,21 +50,19 @@ void shouldSetStateToNotReadyAfterConstruction() {
         assertThat(wrapper.getState()).isEqualTo(ProviderState.NOT_READY);
     }
 
-    @SneakyThrows
     @Test
     @Specification(
             number = "1.7.3",
             text =
                     "The client's provider status accessor MUST indicate READY if the initialize function of the associated provider terminates normally.")
-    void shouldSetStateToReadyAfterInit() {
+    void shouldSetStateToReadyAfterInit() throws Exception {
         assertThat(wrapper.getState()).isEqualTo(ProviderState.NOT_READY);
         wrapper.initialize(null);
         assertThat(wrapper.getState()).isEqualTo(ProviderState.READY);
     }
 
-    @SneakyThrows
     @Test
-    void shouldSetStateToNotReadyAfterShutdown() {
+    void shouldSetStateToNotReadyAfterShutdown() throws Exception {
         assertThat(wrapper.getState()).isEqualTo(ProviderState.NOT_READY);
         wrapper.initialize(null);
         assertThat(wrapper.getState()).isEqualTo(ProviderState.READY);

openfeature-sdk/src/test/java/dev/openfeature/sdk/FlagEvaluationSpecTest.java

@@ -38,7 +38,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
@@ -56,8 +55,7 @@ private Client _client() {
         return api.getClient();
     }
 
-    @SneakyThrows
-    private Client _initializedClient() {
+    private Client _initializedClient() throws Exception {
         TestEventsProvider provider = new TestEventsProvider();
         provider.initialize(null);
         api.setProviderAndWait(provider);
@@ -91,13 +89,12 @@ void provider() {
         assertThat(api.getProvider()).isEqualTo(mockProvider);
     }
 
-    @SneakyThrows
     @Specification(
             number = "1.1.8",
             text =
                     "The API SHOULD provide functions to set a provider and wait for the initialize function to return or throw.")
     @Test
-    void providerAndWait() {
+    void providerAndWait() throws Exception {
         FeatureProvider provider = new TestEventsProvider(500);
         api.setProviderAndWait(provider);
         Client client = api.getClient();
@@ -110,13 +107,12 @@ void providerAndWait() {
         assertThat(client2.getProviderState()).isEqualTo(ProviderState.READY);
     }
 
-    @SneakyThrows
     @Specification(
             number = "1.1.8",
             text =
                     "The API SHOULD provide functions to set a provider and wait for the initialize function to return or throw.")
     @Test
-    void providerAndWaitError() {
+    void providerAndWaitError() throws Exception {
         FeatureProvider provider1 = new TestEventsProvider(500, true, "fake error");
         assertThrows(GeneralError.class, () -> api.setProviderAndWait(provider1));
 
@@ -361,9 +357,8 @@ void detail_flags() {
             number = "1.5.1",
             text =
                     "The evaluation options structure's hooks field denotes an ordered collection of hooks that the client MUST execute for the respective flag evaluation, in addition to those already configured.")
-    @SneakyThrows
     @Test
-    void hooks() {
+    void hooks() throws Exception {
         Client c = _initializedClient();
         Hook<Boolean> clientHook = mockBooleanHook();
         Hook<Boolean> invocationHook = mockBooleanHook();

openfeature-sdk/src/test/java/dev/openfeature/sdk/HookSpecTest.java

@@ -42,7 +42,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.mockito.ArgumentCaptor;
@@ -443,9 +442,8 @@ void error_stops_before() {
             number = "4.4.6",
             text =
                     "If an error occurs during the evaluation of before or after hooks, any remaining hooks in the before or after stages MUST NOT be invoked.")
-    @SneakyThrows
     @Test
-    void error_stops_after() {
+    void error_stops_after() throws Exception {
         Hook<Boolean> h = mockBooleanHook();
         doThrow(RuntimeException.class).when(h).after(any(), any(), any());
         Hook<Boolean> h2 = mockBooleanHook();
@@ -468,9 +466,8 @@ void error_stops_after() {
     @Specification(number = "4.5.2", text = "hook hints MUST be passed to each hook.")
     @Specification(number = "4.2.2.1", text = "Condition: Hook hints MUST be immutable.")
     @Specification(number = "4.5.3", text = "The hook MUST NOT alter the hook hints structure.")
-    @SneakyThrows
     @Test
-    void hook_hints() {
+    void hook_hints() throws Exception {
         String hintKey = "My hint key";
         Client client = getClient(null);
         Hook<Boolean> mutatingHook = new BooleanHook() {
@@ -552,7 +549,7 @@ void flag_eval_hook_order() {
             number = "4.4.7",
             text = "If an error occurs in the before hooks, the default value MUST be returned.")
     @Test
-    void error_hooks__before() {
+    void error_hooks__before() throws Exception {
         Hook hook = mockBooleanHook();
         doThrow(RuntimeException.class).when(hook).before(any(), any());
         Client client = getClient(TestEventsProvider.newInitializedTestEventsProvider());
@@ -570,7 +567,7 @@ void error_hooks__before() {
             number = "4.4.5",
             text = "If an error occurs in the before or after hooks, the error hooks MUST be invoked.")
     @Test
-    void error_hooks__after() {
+    void error_hooks__after() throws Exception {
         Hook hook = mockBooleanHook();
         doThrow(RuntimeException.class).when(hook).after(any(), any(), any());
         Client client = getClient(TestEventsProvider.newInitializedTestEventsProvider());
@@ -584,7 +581,7 @@ void error_hooks__after() {
     }
 
     @Test
-    void erroneous_flagResolution_setsAppropriateFieldsInFlagEvaluationDetails() {
+    void erroneous_flagResolution_setsAppropriateFieldsInFlagEvaluationDetails() throws Exception {
         Hook hook = mockBooleanHook();
         doThrow(RuntimeException.class).when(hook).after(any(), any(), any());
         String flagKey = "test-flag-key";
@@ -630,7 +627,7 @@ void shortCircuit_flagResolution_runsHooksWithAllFields() {
     }
 
     @Test
-    void successful_flagResolution_setsAppropriateFieldsInFlagEvaluationDetails() {
+    void successful_flagResolution_setsAppropriateFieldsInFlagEvaluationDetails() throws Exception {
         Hook hook = mockBooleanHook();
         String flagKey = "test-flag-key";
         Client client = getClient(TestEventsProvider.newInitializedTestEventsProvider());
@@ -655,7 +652,7 @@ void successful_flagResolution_setsAppropriateFieldsInFlagEvaluationDetails() {
     }
 
     @Test
-    void multi_hooks_early_out__before() {
+    void multi_hooks_early_out__before() throws Exception {
         Hook<Boolean> hook = mockBooleanHook();
         Hook<Boolean> hook2 = mockBooleanHook();
         doThrow(RuntimeException.class).when(hook).before(any(), any());
@@ -681,7 +678,7 @@ void multi_hooks_early_out__before() {
             text =
                     "Any `evaluation context` returned from a `before` hook MUST be passed to subsequent `before` hooks (via `HookContext`).")
     @Test
-    void beforeContextUpdated() {
+    void beforeContextUpdated() throws Exception {
         String targetingKey = "test-key";
         EvaluationContext ctx = new ImmutableContext(targetingKey);
         Hook<Boolean> hook = mockBooleanHook();
@@ -749,7 +746,7 @@ void mergeHappensCorrectly() {
             text =
                     "If a finally hook abnormally terminates, evaluation MUST proceed, including the execution of any remaining finally hooks.")
     @Test
-    void first_finally_broken() {
+    void first_finally_broken() throws Exception {
         Hook hook = mockBooleanHook();
         doThrow(RuntimeException.class).when(hook).before(any(), any());
         doThrow(RuntimeException.class).when(hook).finallyAfter(any(), any(), any());
@@ -773,7 +770,7 @@ void first_finally_broken() {
             text =
                     "If an error hook abnormally terminates, evaluation MUST proceed, including the execution of any remaining error hooks.")
     @Test
-    void first_error_broken() {
+    void first_error_broken() throws Exception {
         Hook hook = mockBooleanHook();
         doThrow(RuntimeException.class).when(hook).before(any(), any());
         doThrow(RuntimeException.class).when(hook).error(any(), any(), any());
@@ -792,7 +789,7 @@ void first_error_broken() {
         order.verify(hook).error(any(), any(), any());
     }
 
-    private Client getClient(FeatureProvider provider) {
+    private Client getClient(FeatureProvider provider) throws Exception {
         if (provider == null) {
             api.setProviderAndWait(TestEventsProvider.newInitializedTestEventsProvider());
         } else {
@@ -806,9 +803,8 @@ private Client getClient(FeatureProvider provider) {
     void default_methods_so_impossible() {}
 
     @Specification(number = "4.3.9.1", text = "Instead of finally, finallyAfter SHOULD be used.")
-    @SneakyThrows
     @Test
-    void doesnt_use_finally() {
+    void doesnt_use_finally() throws Exception {
         assertThatCode(() -> Hook.class.getMethod("finally", HookContext.class, Map.class))
                 .as("Not possible. Finally is a reserved word.")
                 .isInstanceOf(NoSuchMethodException.class);

openfeature-sdk/src/test/java/dev/openfeature/sdk/ThreadLocalTransactionContextPropagatorTest.java

@@ -8,7 +8,6 @@
 import dev.openfeature.api.ImmutableContext;
 import java.util.concurrent.Callable;
 import java.util.concurrent.FutureTask;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.Test;
 
 public class ThreadLocalTransactionContextPropagatorTest {
@@ -32,9 +31,8 @@ public void emptyTransactionContext() {
         assertNull(result);
     }
 
-    @SneakyThrows
     @Test
-    public void setTransactionContextTwoThreads() {
+    public void setTransactionContextTwoThreads() throws Exception {
         EvaluationContext firstContext = new ImmutableContext();
         EvaluationContext secondContext = new ImmutableContext();
 

openfeature-sdk/src/test/java/dev/openfeature/sdk/TrackingSpecTest.java

@@ -28,7 +28,6 @@
 import dev.openfeature.sdk.fixtures.ProviderFixture;
 import java.util.HashMap;
 import java.util.Map;
-import lombok.SneakyThrows;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
@@ -54,8 +53,7 @@ void getApiInstance() {
                     + "particular action or application state, with parameters `tracking event name` (string, required) and "
                     + "`tracking event details` (optional), which returns nothing.")
     @Test
-    @SneakyThrows
-    void trackMethodFulfillsSpec() {
+    void trackMethodFulfillsSpec() throws Exception {
 
         ImmutableContext ctx = new ImmutableContext();
         MutableTrackingEventDetails details = new MutableTrackingEventDetails(0.0f);

openfeature-sdk/src/test/java/dev/openfeature/sdk/e2e/ContextStoringProvider.java

@@ -5,9 +5,7 @@
 import dev.openfeature.api.Metadata;
 import dev.openfeature.api.ProviderEvaluation;
 import dev.openfeature.api.Value;
-import lombok.Getter;
 
-@Getter
 public class ContextStoringProvider implements FeatureProvider {
     private EvaluationContext evaluationContext;
 
@@ -45,4 +43,8 @@ public ProviderEvaluation<Value> getObjectEvaluation(String key, Value defaultVa
         this.evaluationContext = ctx;
         return null;
     }
+
+    public EvaluationContext getEvaluationContext() {
+        return evaluationContext;
+    }
 }