[FSSDK-11837] legacy api adjustment with key null support in decide (#388)

Author: junaed-optimizely

OptimizelySDK.Tests/OptimizelyUserContextTest.cs

@@ -405,6 +405,23 @@ public void DecideInvalidFlagKey()
             Assert.IsTrue(TestData.CompareObjects(decision, decisionExpected));
         }
 
+        [Test]
+        public void DecideNullFlagKey()
+        {
+            var user = Optimizely.CreateUserContext(UserID);
+            user.SetAttribute("browser_type", "chrome");
+
+            var decisionExpected = OptimizelyDecision.NewErrorDecision(
+                null,
+                user,
+                DecisionMessage.Reason(DecisionMessage.FLAG_KEY_INVALID, "null"),
+                ErrorHandlerMock.Object,
+                LoggerMock.Object);
+            var decision = user.Decide(null);
+
+            Assert.IsTrue(TestData.CompareObjects(decision, decisionExpected));
+        }
+
         [Test]
         public void DecideWhenConfigIsNull()
         {

OptimizelySDK/Optimizely.cs

@@ -871,6 +871,13 @@ OptimizelyDecideOption[] options
                     ErrorHandler, Logger);
             }
 
+            if (key == null)
+            {
+                return OptimizelyDecision.NewErrorDecision(key, user,
+                    DecisionMessage.Reason(DecisionMessage.FLAG_KEY_INVALID, "null"),
+                    ErrorHandler, Logger);
+            }
+
             var allOptions = GetAllOptions(options).
                 Where(opt => opt != OptimizelyDecideOption.ENABLED_FLAGS_ONLY).
                 ToArray();