Skip to content

Commit d38d345

Browse files
aliabbasrizvialiabbasrizvi
committed
Adding validations
1 parent 3133189 commit d38d345

File tree

6 files changed

+140
-3
lines changed

6 files changed

+140
-3
lines changed

‎src/Optimizely/Bucketer.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ public function bucket(ProjectConfig $config, Experiment $experiment, $userId)
113113

114114
// Check if user is whitelisted for a variation.
115115
$forcedVariations = $experiment->getForcedVariations();
116-
if (!is_null($forcedVariations) and isset($forcedVariations[$userId])) {
116+
if (!is_null($forcedVariations) && isset($forcedVariations[$userId])) {
117117
$variationKey = $forcedVariations[$userId];
118118
$variation = $config->getVariationFromKey($experiment->getKey(), $variationKey);
119119
return $variation;

‎src/Optimizely/Entity/Experiment.php‎

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -264,4 +264,27 @@ public function isInMutexGroup()
264264
{
265265
return !is_null($this->_groupPolicy) && $this->_groupPolicy == 'random';
266266
}
267+
268+
/**
269+
* Determine if experiment is running or not.
270+
*
271+
* @return boolean True if experiment has status "Running". False otherwise.
272+
*/
273+
public function isExperimentRunning()
274+
{
275+
return !is_null($this->_status) && $this->_status == 'Running';
276+
}
277+
278+
/**
279+
* Determine if user is in forced variation of experiment.
280+
*
281+
* @param $userId string ID of the user.
282+
*
283+
* @return boolean True if user is in forced variation of experiment. False otherwise.
284+
*/
285+
public function isUserInForcedVariation($userId)
286+
{
287+
$forcedVariations = $this->getForcedVariations();
288+
return !is_null($forcedVariations) && isset($forcedVariations[$userId]);
289+
}
267290
}

‎src/Optimizely/Optimizely.php‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,18 @@ private function validateInputs($datafile, $skipJsonValidation)
119119
*/
120120
private function validatePreconditions($experiment, $userId, $attributes)
121121
{
122+
// TODO(ali): Insert attributes validation
123+
124+
if (!$experiment->isExperimentRunning()) {
125+
return false;
126+
}
127+
128+
if ($experiment->isUserInForcedVariation($userId)) {
129+
return true;
130+
}
131+
132+
// TODO(ali): Insert audience check
133+
122134
return true;
123135
}
124136

‎tests/OptimizelyTest.php‎

Lines changed: 88 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,12 +31,14 @@ class OptimizelyTest extends \PHPUnit_Framework_TestCase
3131
{
3232
private $datafile;
3333
private $eventBuilderMock;
34+
private $optObj;
3435
private $projectConfig;
3536

3637
public function setUp()
3738
{
3839
$this->datafile = DATAFILE;
3940
$this->projectConfig = new ProjectConfig($this->datafile);
41+
$this->optObj = new Optimizely($this->datafile);
4042

4143
// Mock EventBuilder
4244
$this->eventBuilderMock = $this->getMockBuilder(EventBuilder::class)
@@ -143,6 +145,92 @@ public function testValidateInputsInvalidFileJsonValidationSkipped()
143145
);
144146
}
145147

148+
public function testValidatePreconditionsExperimentNotRunning()
149+
{
150+
$validatePreconditions = new \ReflectionMethod('Optimizely\Optimizely', 'validatePreconditions');
151+
$validatePreconditions->setAccessible(true);
152+
153+
$this->assertFalse(
154+
$validatePreconditions->invoke(
155+
$this->optObj,
156+
$this->projectConfig->getExperimentFromKey('paused_experiment'),
157+
'test_user',
158+
[])
159+
);
160+
}
161+
162+
public function testValidatePreconditionsExperimentRunning()
163+
{
164+
$validatePreconditions = new \ReflectionMethod('Optimizely\Optimizely', 'validatePreconditions');
165+
$validatePreconditions->setAccessible(true);
166+
167+
$this->assertTrue(
168+
$validatePreconditions->invoke(
169+
$this->optObj,
170+
$this->projectConfig->getExperimentFromKey('test_experiment'),
171+
'test_user',
172+
[])
173+
);
174+
}
175+
176+
public function testValidatePreconditionsUserInForcedVariationNotInExperiment()
177+
{
178+
$validatePreconditions = new \ReflectionMethod('Optimizely\Optimizely', 'validatePreconditions');
179+
$validatePreconditions->setAccessible(true);
180+
181+
$this->assertTrue(
182+
$validatePreconditions->invoke(
183+
$this->optObj,
184+
$this->projectConfig->getExperimentFromKey('test_experiment'),
185+
'user_1',
186+
[])
187+
);
188+
}
189+
190+
public function testValidatePreconditionsUserInForcedVariationInExperiment()
191+
{
192+
$validatePreconditions = new \ReflectionMethod('Optimizely\Optimizely', 'validatePreconditions');
193+
$validatePreconditions->setAccessible(true);
194+
195+
$this->assertTrue(
196+
$validatePreconditions->invoke(
197+
$this->optObj,
198+
$this->projectConfig->getExperimentFromKey('test_experiment'),
199+
'user1',
200+
[])
201+
);
202+
}
203+
204+
public function testValidatePreconditionsUserNotInForcedVariationNotInExperiment()
205+
{
206+
$validatePreconditions = new \ReflectionMethod('Optimizely\Optimizely', 'validatePreconditions');
207+
$validatePreconditions->setAccessible(true);
208+
209+
// Will get updated when we have audience evaluation and user does not meet conditions
210+
$this->assertTrue(
211+
$validatePreconditions->invoke(
212+
$this->optObj,
213+
$this->projectConfig->getExperimentFromKey('test_experiment'),
214+
'test_user',
215+
[])
216+
);
217+
}
218+
219+
public function testValidatePreconditionsUserNotInForcedVariationInExperiment()
220+
{
221+
$validatePreconditions = new \ReflectionMethod('Optimizely\Optimizely', 'validatePreconditions');
222+
$validatePreconditions->setAccessible(true);
223+
224+
// Will get updated when we have audience evaluation and user does meets conditions
225+
$this->assertTrue(
226+
$validatePreconditions->invoke(
227+
$this->optObj,
228+
$this->projectConfig->getExperimentFromKey('test_experiment'),
229+
'test_user',
230+
[])
231+
);
232+
}
233+
146234
public function testActivateNoAttributes()
147235
{
148236
$this->eventBuilderMock->expects($this->once())

‎tests/ProjectConfigTest.php‎

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ public function testInit()
6969
$experimentKeyMap->setAccessible(true);
7070
$this->assertEquals([
7171
'test_experiment' => $this->config->getExperimentFromKey('test_experiment'),
72+
'paused_experiment' => $this->config->getExperimentFromKey('paused_experiment'),
7273
'group_experiment_1' => $this->config->getExperimentFromKey('group_experiment_1'),
7374
'group_experiment_2' => $this->config->getExperimentFromKey('group_experiment_2')
7475
], $experimentKeyMap->getValue($this->config));
@@ -79,7 +80,8 @@ public function testInit()
7980
$this->assertEquals([
8081
'7716830082' => $this->config->getExperimentFromId('7716830082'),
8182
'7723330021' => $this->config->getExperimentFromId('7723330021'),
82-
'7718750065' => $this->config->getExperimentFromId('7718750065')
83+
'7718750065' => $this->config->getExperimentFromId('7718750065'),
84+
'7716830585' => $this->config->getExperimentFromId('7716830585')
8385
], $experimentIdMap->getValue($this->config));
8486

8587
// Check event key map
@@ -112,6 +114,10 @@ public function testInit()
112114
'control' => $this->config->getVariationFromKey('test_experiment', 'control'),
113115
'variation' => $this->config->getVariationFromKey('test_experiment', 'variation')
114116
],
117+
'paused_experiment' => [
118+
'control' => $this->config->getVariationFromKey('paused_experiment', 'control'),
119+
'variation' => $this->config->getVariationFromKey('paused_experiment', 'variation')
120+
],
115121
'group_experiment_1' => [
116122
'group_exp_1_var_1' => $this->config->getVariationFromKey('group_experiment_1', 'group_exp_1_var_1'),
117123
'group_exp_1_var_2' => $this->config->getVariationFromKey('group_experiment_1', 'group_exp_1_var_2')
@@ -130,6 +136,10 @@ public function testInit()
130136
'7722370027' => $this->config->getVariationFromId('test_experiment', '7722370027'),
131137
'7721010009' => $this->config->getVariationFromId('test_experiment', '7721010009')
132138
],
139+
'paused_experiment' => [
140+
'7722370427' => $this->config->getVariationFromId('paused_experiment', '7722370427'),
141+
'7721010509' => $this->config->getVariationFromId('paused_experiment', '7721010509')
142+
],
133143
'group_experiment_1' => [
134144
'7722260071' => $this->config->getVariationFromId('group_experiment_1', '7722260071'),
135145
'7722360022' => $this->config->getVariationFromId('group_experiment_1', '7722360022')

‎tests/TestData.php‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,11 @@
2626
"trafficAllocation": [{"entityId": "7722370027", "endOfRange": 4000},
2727
{"entityId": "7721010009", "endOfRange": 8000}], "audienceIds": ["7718080042"],
2828
"variations": [{"id": "7722370027", "key": "control"}, {"id": "7721010009", "key": "variation"}],
29-
"forcedVariations": {"user1": "control"}, "id": "7716830082"}], "version": "2",
29+
"forcedVariations": {"user1": "control"}, "id": "7716830082"}, {"status": "Paused", "key": "paused_experiment", "layerId": "7719779139",
30+
"trafficAllocation": [{"entityId": "7722370427", "endOfRange": 5000},
31+
{"entityId": "7721010509", "endOfRange": 8000}], "audienceIds": [],
32+
"variations": [{"id": "7722370427", "key": "control"}, {"id": "7721010509", "key": "variation"}],
33+
"forcedVariations": {}, "id": "7716830585"}], "version": "2",
3034
"audiences": [{"conditions": "[\"and\", [\"or\", [\"or\", {\"name\": \"device_type\", \"type\": \"custom_attribute\", \"value\": \"iPhone\"}]], [\"or\", [\"or\", {\"name\": \"location\", \"type\": \"custom_attribute\", \"value\": \"San Francisco\"}]]]", "id": "7718080042", "name": "iPhone users in San Francisco"}],
3135
"groups": [{"policy": "random", "trafficAllocation": [{"entityId": "7723330021", "endOfRange": 2000}, {"entityId": "7718750065", "endOfRange": 6000}], "experiments": [{"status": "Running", "key": "group_experiment_1", "layerId": "7721010011", "trafficAllocation": [{"entityId": "7722260071", "endOfRange": 5000}, {"entityId": "7722360022", "endOfRange": 10000}], "audienceIds": [], "variations": [{"id": "7722260071", "key": "group_exp_1_var_1"}, {"id": "7722360022", "key": "group_exp_1_var_2"}], "forcedVariations": {"user1": "group_exp_1_var_1"}, "id": "7723330021"}, {"status": "Running", "key": "group_experiment_2", "layerId": "7721020020", "trafficAllocation": [{"entityId": "7713030086", "endOfRange": 5000}, {"entityId": "7725250007", "endOfRange": 10000}], "audienceIds": [],
3236
"variations": [{"id": "7713030086", "key": "group_exp_2_var_1"}, {"id": "7725250007", "key": "group_exp_2_var_2"}], "forcedVariations": {}, "id": "7718750065"}], "id": "7722400015"}],

0 commit comments

Comments
 (0)