Merge branch 'development' into SDKS-7676

Author: nmayorsplit

client/src/main/java/io/split/client/SplitClientImpl.java

@@ -23,6 +23,7 @@
 import org.slf4j.LoggerFactory;
 
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
@@ -226,7 +227,7 @@ private SplitResult getTreatmentWithConfigInternal(String matchingKey, String bu
                                                        Object> attributes, MethodEnum methodEnum) {
         long initTime = System.currentTimeMillis();
         try {
-            checkSDKReady(methodEnum);
+            checkSDKReady(methodEnum, Arrays.asList(featureFlag));
 
             if (_container.isDestroyed()) {
                 _log.error("Client has already been destroyed - no calls possible");
@@ -290,7 +291,7 @@ private Map<String, SplitResult> getTreatmentsWithConfigInternal(String matching
             return new HashMap<>();
         }
         try {
-            checkSDKReady(methodEnum);
+            checkSDKReady(methodEnum, featureFlagNames);
             if (_container.isDestroyed()) {
                 _log.error("Client has already been destroyed - no calls possible");
                 return createMapControl(featureFlagNames);
@@ -360,10 +361,11 @@ private Event createEvent(String key, String trafficType, String eventType) {
         return event;
     }
 
-    private void checkSDKReady(MethodEnum methodEnum) {
+    private void checkSDKReady(MethodEnum methodEnum, List<String> featureFlagNames) {
+        String toPrint =  featureFlagNames.size() == 1 ? featureFlagNames.get(0): String.join(",", featureFlagNames);
         if (!_gates.isSDKReady()) {
-            _log.warn(String.format("%s: the SDK is not ready, results may be incorrect. Make sure to wait for SDK readiness " +
-                            "before using this method", methodEnum.getMethod()));
+            _log.warn(String.format("%s: the SDK is not ready, results may be incorrect for feature flag %s. Make sure to wait for " +
+                            "SDK readiness before using this method", methodEnum.getMethod(), toPrint));
             _telemetryConfigProducer.recordNonReadyUsage();
         }
     }

client/src/main/java/io/split/client/SplitManagerImpl.java

@@ -58,7 +58,8 @@ public List<SplitView> splits() {
     @Override
     public SplitView split(String featureFlagName) {
         if (!_gates.isSDKReady()) { {
-            _log.warn("split: the SDK is not ready, results may be incorrect. Make sure to wait for SDK readiness before using this method");
+            _log.warn(String.format("the SDK is not ready, results may be incorrect for feature flag %s. Make sure to wait " +
+                    "for SDK readiness before using this method", featureFlagName));
             _telemetryConfigProducer.recordNonReadyUsage();
         }}
         Optional<String> result = SplitNameValidator.isValid(featureFlagName, "split");

client/src/main/java/io/split/client/api/SplitView.java

@@ -24,13 +24,15 @@ public class SplitView {
     public List<String> treatments;
     public long changeNumber;
     public Map<String, String> configs;
+    public String defaultTreatment;
 
     public static SplitView fromParsedSplit(ParsedSplit parsedSplit) {
         SplitView splitView = new SplitView();
         splitView.name = parsedSplit.feature();
         splitView.trafficType = parsedSplit.trafficTypeName();
         splitView.killed = parsedSplit.killed();
         splitView.changeNumber = parsedSplit.changeNumber();
+        splitView.defaultTreatment = parsedSplit.defaultTreatment();
 
         Set<String> treatments = new HashSet<String>();
         for (ParsedCondition condition : parsedSplit.parsedConditions()) {

client/src/test/java/io/split/client/SplitManagerImplTest.java

@@ -67,6 +67,7 @@ public void splitCallWithExistentSplit() {
         Assert.assertEquals(1, theOne.treatments.size());
         Assert.assertEquals("off", theOne.treatments.get(0));
         Assert.assertEquals(0, theOne.configs.size());
+        Assert.assertEquals("off", theOne.defaultTreatment);
     }
 
     @Test