feat: remove the deprecated setTargetingKey method in EvaluationContext. (open-feature#290)

remove the deprecated setTargetingKey method in EvaluationContext.

Signed-off-by: thiyagu06 <[email protected]>

Author: thiyagu06

src/main/java/dev/openfeature/sdk/EvaluationContext.java

@@ -7,12 +7,6 @@
 @SuppressWarnings("PMD.BeanMembersShouldSerialize")
 public interface EvaluationContext extends Structure {
     String getTargetingKey();
-    
-    /**
-     * Mutating targeting key is not supported in all implementations and will be removed.
-     */
-    @Deprecated
-    void setTargetingKey(String targetingKey);
 
     /**
      * Merges this EvaluationContext object with the second overriding the this in

src/main/java/dev/openfeature/sdk/ImmutableContext.java

@@ -58,15 +58,6 @@ public ImmutableContext(String targetingKey, Map<String, Value> attributes) {
         this.targetingKey = targetingKey;
     }
 
-    /**
-     * Mutating targeting key is not supported in ImmutableContext and will be removed.
-     */
-    @Override
-    @Deprecated
-    public void setTargetingKey(String targetingKey) {
-        throw new UnsupportedOperationException("changing of targeting key is not allowed");
-    }
-
     /**
      * Merges this EvaluationContext object with the passed EvaluationContext, overriding in case of conflict.
      *

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

@@ -164,8 +164,8 @@ public class EvalContextTest {
 
     @Test void merge_targeting_key() {
         String key1 = "key1";
-        EvaluationContext ctx1 = new MutableContext(key1);
-        EvaluationContext ctx2 = new MutableContext();
+        MutableContext ctx1 = new MutableContext(key1);
+        MutableContext ctx2 = new MutableContext();
 
         EvaluationContext ctxMerged = ctx1.merge(ctx2);
         assertEquals(key1, ctxMerged.getTargetingKey());

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

@@ -12,13 +12,6 @@
 
 class ImmutableContextTest {
 
-    @Test
-    @DisplayName("Mutating targeting key is not allowed on Immutable Context")
-    void shouldThrowUnsupportedExceptionWhenMutatingTargetingKey() {
-        EvaluationContext ctx = new ImmutableContext("targeting key", new HashMap<>());
-        assertThrows(UnsupportedOperationException.class, () -> ctx.setTargetingKey(""));
-    }
-
     @DisplayName("attributes mutation should not affect the immutable context")
     @Test
     void shouldCreateCopyOfAttributesForImmutableContext() {