From c01d305a124dde71679f518b22650a394846fa14 Mon Sep 17 00:00:00 2001
From: Sohail Hussain <mirza.sohailhussain@gmail.com>
Date: Mon, 13 Jul 2020 12:17:53 -0700
Subject: [PATCH 1/2] rolloutrule should be non-zero.

---
 OptimizelySDK.Tests/DecisionServiceTest.cs    | 18 ++++++++
 OptimizelySDK.Tests/EmptyRolloutRule.json     | 44 +++++++++++++++++++
 .../OptimizelySDK.Tests.csproj                |  1 +
 OptimizelySDK.Tests/Utils/TestData.cs         |  7 +++
 OptimizelySDK/Bucketing/DecisionService.cs    |  4 ++
 5 files changed, 74 insertions(+)
 create mode 100644 OptimizelySDK.Tests/EmptyRolloutRule.json

diff --git a/OptimizelySDK.Tests/DecisionServiceTest.cs b/OptimizelySDK.Tests/DecisionServiceTest.cs
index acb475ad..f90e8507 100644
--- a/OptimizelySDK.Tests/DecisionServiceTest.cs
+++ b/OptimizelySDK.Tests/DecisionServiceTest.cs
@@ -569,6 +569,24 @@ public void TestGetVariationForFeatureExperimentGivenMutexGroupAndUserNotBuckete
         #region GetVariationForFeatureRollout Tests
 
         // Should return null when rollout doesn't exist for the feature.
+        [Test]
+        public void TestGetVariationForFeatureRolloutWhenNoRuleInRollouts()
+        {
+            var projectConfig = DatafileProjectConfig.Create(TestData.EmptyRolloutDatafile, new NoOpLogger(), new NoOpErrorHandler());
+
+            Assert.IsNotNull(projectConfig);
+            var featureFlag = projectConfig.FeatureKeyMap["empty_rollout"];
+            var rollout = projectConfig.GetRolloutFromId(featureFlag.RolloutId);
+
+            Assert.AreEqual(rollout.Experiments.Count, 0);
+
+            var decisionService = new DecisionService(new Bucketer(new NoOpLogger()), new NoOpErrorHandler(), null, new NoOpLogger());
+
+            var variation = decisionService.GetVariationForFeatureRollout(featureFlag, "userId1", null, projectConfig);
+
+            Assert.IsNull(variation);
+        }
+
         [Test]
         public void TestGetVariationForFeatureRolloutWhenRolloutIsNotInDataFile()
         {
diff --git a/OptimizelySDK.Tests/EmptyRolloutRule.json b/OptimizelySDK.Tests/EmptyRolloutRule.json
new file mode 100644
index 00000000..3e51cc5d
--- /dev/null
+++ b/OptimizelySDK.Tests/EmptyRolloutRule.json
@@ -0,0 +1,44 @@
+{
+  "version": "4",
+  "rollouts": [
+    {
+      "experiments": [
+      ],
+      "id": "18309384009"
+    }
+  ],
+  "typedAudiences": [],
+  "anonymizeIP": true,
+  "projectId": "18326250003",
+  "variables": [],
+  "featureFlags": [
+    {
+      "experimentIds": [],
+      "rolloutId": "18309384009",
+      "variables": [
+        {
+          "defaultValue": "",
+          "type": "string",
+          "id": "18323951833",
+          "key": "var_1"
+        }
+      ],
+      "id": "18244658520",
+      "key": "empty_rollout"
+    }
+  ],
+  "experiments": [],
+  "audiences": [
+    {
+      "conditions": "[\"or\", {\"match\": \"exact\", \"name\": \"$opt_dummy_attribute\", \"type\": \"custom_attribute\", \"value\": \"$opt_dummy_value\"}]",
+      "id": "$opt_dummy_audience",
+      "name": "Optimizely-Generated Audience for Backwards Compatibility"
+    }
+  ],
+  "groups": [],
+  "attributes": [],
+  "botFiltering": false,
+  "accountId": "8272261422",
+  "events": [],
+  "revision": "2"
+}
\ No newline at end of file
diff --git a/OptimizelySDK.Tests/OptimizelySDK.Tests.csproj b/OptimizelySDK.Tests/OptimizelySDK.Tests.csproj
index d25a2da7..eb22549c 100644
--- a/OptimizelySDK.Tests/OptimizelySDK.Tests.csproj
+++ b/OptimizelySDK.Tests/OptimizelySDK.Tests.csproj
@@ -119,6 +119,7 @@
   <ItemGroup>
     <EmbeddedResource Include="TestData.json" />
     <EmbeddedResource Include="simple_ab_experiments.json" />
+    <EmbeddedResource Include="EmptyRolloutRule.json" />
   </ItemGroup>
   <ItemGroup>
     <ProjectReference Include="..\OptimizelySDK\OptimizelySDK.csproj">
diff --git a/OptimizelySDK.Tests/Utils/TestData.cs b/OptimizelySDK.Tests/Utils/TestData.cs
index 1a8d0a63..32688d8b 100644
--- a/OptimizelySDK.Tests/Utils/TestData.cs
+++ b/OptimizelySDK.Tests/Utils/TestData.cs
@@ -27,6 +27,7 @@ public class TestData
         private static string simpleABExperimentsDatafile = null;
         private static string unsupportedVersionDatafile = null;
         private static string typedAudienceDatafile = null;
+        private static string emptyRolloutDatafile = null;
 
         public static string Datafile
         {
@@ -52,6 +53,12 @@ public static string UnsupportedVersionDatafile
             }
         }
 
+        public static string EmptyRolloutDatafile {
+            get {
+                return emptyRolloutDatafile ?? (emptyRolloutDatafile = LoadJsonData("EmptyRolloutRule.json"));
+            }
+        }
+
         public static string TypedAudienceDatafile
         {
             get
diff --git a/OptimizelySDK/Bucketing/DecisionService.cs b/OptimizelySDK/Bucketing/DecisionService.cs
index f5f45c89..0abb7341 100644
--- a/OptimizelySDK/Bucketing/DecisionService.cs
+++ b/OptimizelySDK/Bucketing/DecisionService.cs
@@ -389,6 +389,10 @@ public virtual FeatureDecision GetVariationForFeatureRollout(FeatureFlag feature
                 return null;
             }
 
+            if (rollout.Experiments == null ||  rollout.Experiments.Count == 0) {
+                return null;
+            }
+
             Variation variation = null;
             var rolloutRulesLength = rollout.Experiments.Count;
 

From 96d724241e9d669263ab9a058669d772efd93de6 Mon Sep 17 00:00:00 2001
From: msohailhussain <mirza.sohailhussain@gmail.com>
Date: Mon, 13 Jul 2020 21:39:00 -0700
Subject: [PATCH 2/2] Update EmptyRolloutRule.json

---
 OptimizelySDK.Tests/EmptyRolloutRule.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/OptimizelySDK.Tests/EmptyRolloutRule.json b/OptimizelySDK.Tests/EmptyRolloutRule.json
index 3e51cc5d..70913612 100644
--- a/OptimizelySDK.Tests/EmptyRolloutRule.json
+++ b/OptimizelySDK.Tests/EmptyRolloutRule.json
@@ -41,4 +41,4 @@
   "accountId": "8272261422",
   "events": [],
   "revision": "2"
-}
\ No newline at end of file
+}