From 811c2d92f08d4dbf1cd22521679a6dc43fac1f5d Mon Sep 17 00:00:00 2001
From: Owais <oakbani@folio3.com>
Date: Fri, 4 Jan 2019 17:22:48 +0500
Subject: [PATCH 1/2] fix(audience evaluation): evaluate with unknown audience
 Ids

---
 src/Optimizely/ProjectConfig.php   |  7 ++++---
 src/Optimizely/Utils/Validator.php |  7 ++++++-
 tests/ProjectConfigTest.php        |  4 ++--
 tests/UtilsTests/ValidatorTest.php | 20 +++++++++++++++++++-
 4 files changed, 31 insertions(+), 7 deletions(-)

diff --git a/src/Optimizely/ProjectConfig.php b/src/Optimizely/ProjectConfig.php
index fd172299..4ae69616 100644
--- a/src/Optimizely/ProjectConfig.php
+++ b/src/Optimizely/ProjectConfig.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016, 2018, Optimizely
+ * Copyright 2016, 2018-2019 Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -455,7 +455,7 @@ public function getEvent($eventKey)
      * @param $audienceId string ID of the audience.
      *
      * @return Audience Entity corresponding to the ID.
-     *         Dummy entity is returned if ID is invalid.
+     *         Null is returned if ID is invalid.
      */
     public function getAudience($audienceId)
     {
@@ -465,7 +465,8 @@ public function getAudience($audienceId)
 
         $this->_logger->log(Logger::ERROR, sprintf('Audience ID "%s" is not in datafile.', $audienceId));
         $this->_errorHandler->handleError(new InvalidAudienceException('Provided audience is not in datafile.'));
-        return new Audience();
+
+        return null;
     }
 
     /**
diff --git a/src/Optimizely/Utils/Validator.php b/src/Optimizely/Utils/Validator.php
index 8800a07c..dd6f8eeb 100644
--- a/src/Optimizely/Utils/Validator.php
+++ b/src/Optimizely/Utils/Validator.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016-2018, Optimizely
+ * Copyright 2016-2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -160,7 +160,12 @@ public static function isUserInExperiment($config, $experiment, $userAttributes)
 
         $evaluateAudience = function($audienceId) use ($config, $evaluateCustomAttr) {
             $conditionTreeEvaluator = new ConditionTreeEvaluator();
+
             $audience = $config->getAudience($audienceId);
+            if ($audience === null) {
+                return null;
+            }
+
             return $conditionTreeEvaluator->evaluate($audience->getConditionsList(), $evaluateCustomAttr);
         };
 
diff --git a/tests/ProjectConfigTest.php b/tests/ProjectConfigTest.php
index b19c7526..82de415f 100644
--- a/tests/ProjectConfigTest.php
+++ b/tests/ProjectConfigTest.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016-2018, Optimizely
+ * Copyright 2016-2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -546,7 +546,7 @@ public function testGetAudienceInvalidKey()
             ->method('handleError')
             ->with(new InvalidAudienceException('Provided audience is not in datafile.'));
 
-        $this->assertEquals(new Audience(), $this->config->getAudience('invalid_id'));
+        $this->assertNull($this->config->getAudience('invalid_id'));
     }
 
     public function testGetAttributeValidKey()
diff --git a/tests/UtilsTests/ValidatorTest.php b/tests/UtilsTests/ValidatorTest.php
index b0909a14..459b886a 100644
--- a/tests/UtilsTests/ValidatorTest.php
+++ b/tests/UtilsTests/ValidatorTest.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * Copyright 2016-2018, Optimizely
+ * Copyright 2016-2019, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -346,6 +346,24 @@ public function testIsUserInExperimentWithAudienceConditionsSetToAudienceIdStrin
         );
     }
 
+    public function testIsUserInExperimentWithUnknownAudienceId()
+    {
+        $config = new ProjectConfig(DATAFILE, $this->loggerMock, new NoOpErrorHandler());
+        $experiment = $config->getExperimentFromKey('test_experiment');
+
+        // Both audience Ids and audience conditions exist. Audience Ids is ignored.
+        $experiment->setAudienceIds([]);
+        $experiment->setAudienceConditions(["or", "unknown_audience_id", "7718080042"]);
+        
+        $this->assertTrue(
+            Validator::isUserInExperiment(
+                $config,
+                $experiment,
+                ['device_type' => 'iPhone', 'location' => 'San Francisco']
+            )
+        );
+    }
+
     // test that isUserInExperiment evaluates simple audience.
     public function testIsUserInExperimentWithSimpleAudience()
     {

From 9dc0544e108410305506e22233cbefa3a6f3fb95 Mon Sep 17 00:00:00 2001
From: Owais <oakbani@folio3.com>
Date: Wed, 9 Jan 2019 17:21:20 +0500
Subject: [PATCH 2/2] add comment

---
 tests/UtilsTests/ValidatorTest.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/UtilsTests/ValidatorTest.php b/tests/UtilsTests/ValidatorTest.php
index 459b886a..be1e9769 100644
--- a/tests/UtilsTests/ValidatorTest.php
+++ b/tests/UtilsTests/ValidatorTest.php
@@ -355,6 +355,7 @@ public function testIsUserInExperimentWithUnknownAudienceId()
         $experiment->setAudienceIds([]);
         $experiment->setAudienceConditions(["or", "unknown_audience_id", "7718080042"]);
         
+        // User qualifies for audience with ID "7718080042".
         $this->assertTrue(
             Validator::isUserInExperiment(
                 $config,