Skip to content

Commit 0218a3b

Browse files
committed
chore: add featureId as property on BaseVariable
1 parent e66412c commit 0218a3b

File tree

5 files changed

+16
-4
lines changed

5 files changed

+16
-4
lines changed

src/main/java/com/devcycle/sdk/server/common/model/BaseVariable.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,8 @@ public class BaseVariable {
3131

3232
@Schema(description = "Evaluation reason")
3333
private EvalReason eval;
34+
35+
@Schema(description = "Feature ID")
36+
@JsonProperty("_feature")
37+
private String featureId;
3438
}

src/main/java/com/devcycle/sdk/server/common/model/EvalReason.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,7 @@ private EvalReason(String reason, String details) {
4040
}
4141

4242
public static EvalReason defaultReason(DefaultReasonDetailsEnum details) {
43-
String defaultDetails = details.getValue();
44-
System.out.println("Setting default reason to " + defaultDetails);
45-
return new EvalReason("DEFAULT", defaultDetails);
43+
return new EvalReason("DEFAULT", details.getValue());
4644
}
4745

4846
public String getReason() {

src/test/java/com/devcycle/sdk/server/cloud/DevCycleCloudClientTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ public void getVariableValueByKeyTest() {
161161
}
162162

163163
@Test
164-
public void getVariablesTest() throws DevCycleException {
164+
public void allVariablesTest() throws DevCycleException {
165165
DevCycleUser user = DevCycleUser.builder()
166166
.userId("j_test")
167167
.build();
@@ -177,6 +177,7 @@ public void getVariablesTest() throws DevCycleException {
177177
BaseVariable testNumber = variables.get("test-number");
178178
Assert.assertNotNull(testNumber);
179179
Assert.assertEquals(100, (int) testNumber.getValue());
180+
Assert.assertEquals("62fbf6566f1ba302829f9e32", testNumber.getFeatureId());
180181
Assert.assertEquals(Variable.TypeEnum.NUMBER, testNumber.getType());
181182
Assert.assertEquals("SPLIT", testNumber.getEval().getReason());
182183
Assert.assertEquals("Random Distribution | User Id", testNumber.getEval().getDetails());

src/test/java/com/devcycle/sdk/server/helpers/TestResponse.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,27 +58,31 @@ public static Call<Map<String, BaseVariable>> getVariables() {
5858
.value(false)
5959
.type(Variable.TypeEnum.BOOLEAN)
6060
.eval(new EvalReason("SPLIT", "Random Distribution | User Id", "test_cloud_target_id"))
61+
.featureId("62fbf6566f1ba302829f9e32")
6162
.build());
6263
variables.put("test-true", BaseVariable.builder()
6364
.id(UUID.randomUUID().toString())
6465
.key("test-true")
6566
.value(true)
6667
.type(Variable.TypeEnum.BOOLEAN)
6768
.eval(new EvalReason("SPLIT", "Random Distribution | User Id", "test_cloud_target_id"))
69+
.featureId("62fbf6566f1ba302829f9e32")
6870
.build());
6971
variables.put("test-number", BaseVariable.builder()
7072
.id(UUID.randomUUID().toString())
7173
.key("test-number")
7274
.value(100)
7375
.type(Variable.TypeEnum.NUMBER)
7476
.eval(new EvalReason("SPLIT", "Random Distribution | User Id", "test_cloud_target_id"))
77+
.featureId("62fbf6566f1ba302829f9e32")
7578
.build());
7679
variables.put("test-json", BaseVariable.builder()
7780
.id(UUID.randomUUID().toString())
7881
.key("test-json")
7982
.value("{'some':'json''}")
8083
.type(Variable.TypeEnum.JSON)
8184
.eval(new EvalReason("SPLIT", "Random Distribution | User Id", "test_cloud_target_id"))
85+
.featureId("62fbf6566f1ba302829f9e32")
8286
.build());
8387

8488
return Calls.response(variables);

src/test/java/com/devcycle/sdk/server/local/DevCycleLocalClientTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -377,6 +377,11 @@ public void allVariablesTest() {
377377
Assert.assertEquals(variables.get("num-var").getId(), "65272363125123fca69d3a7d");
378378
Assert.assertEquals(variables.get("json-var").getId(), "64372363125123fca69d3f7b");
379379
Assert.assertEquals(variables.size(), 4);
380+
381+
Assert.assertEquals(variables.get("string-var").getFeatureId(), "62fbf6566f1ba302829f9e32");
382+
Assert.assertEquals(variables.get("a-cool-new-feature").getFeatureId(), "62fbf6566f1ba302829f9e32");
383+
Assert.assertEquals(variables.get("num-var").getFeatureId(), "62fbf6566f1ba302829f9e32");
384+
Assert.assertEquals(variables.get("json-var").getFeatureId(), "62fbf6566f1ba302829f9e32");
380385
}
381386

382387
@Test

0 commit comments

Comments
 (0)