Addressing Mike's feedback

Author: aliabbasrizvi

src/Optimizely/Entity/Experiment.php

@@ -279,7 +279,6 @@ public function isExperimentRunning()
      * Determine if user is in forced variation of experiment.
      *
      * @param $userId string ID of the user.
-     *
      * @return boolean True if user is in forced variation of experiment. False otherwise.
      */
     public function isUserInForcedVariation($userId)

src/Optimizely/Optimizely.php

@@ -119,7 +119,7 @@ private function validateInputs($datafile, $skipJsonValidation)
      */
     private function validatePreconditions($experiment, $userId, $attributes)
     {
-        // TODO(ali): Insert attributes validation
+        //@TODO(ali): Insert attributes validation
 
         if (!$experiment->isExperimentRunning()) {
             return false;
@@ -129,7 +129,7 @@ private function validatePreconditions($experiment, $userId, $attributes)
             return true;
         }
 
-        // TODO(ali): Insert audience check
+        //@TODO(ali): Insert audience check
 
         return true;
     }

tests/OptimizelyTest.php

@@ -31,14 +31,14 @@ class OptimizelyTest extends \PHPUnit_Framework_TestCase
 {
     private $datafile;
     private $eventBuilderMock;
-    private $optObj;
+    private $optimizelyObject;
     private $projectConfig;
 
     public function setUp()
     {
         $this->datafile = DATAFILE;
         $this->projectConfig = new ProjectConfig($this->datafile);
-        $this->optObj = new Optimizely($this->datafile);
+        $this->optimizelyObject = new Optimizely($this->datafile);
 
         // Mock EventBuilder
         $this->eventBuilderMock = $this->getMockBuilder(EventBuilder::class)
@@ -152,7 +152,7 @@ public function testValidatePreconditionsExperimentNotRunning()
 
         $this->assertFalse(
             $validatePreconditions->invoke(
-                $this->optObj,
+                $this->optimizelyObject,
                 $this->projectConfig->getExperimentFromKey('paused_experiment'),
                 'test_user',
                 [])
@@ -166,7 +166,7 @@ public function testValidatePreconditionsExperimentRunning()
 
         $this->assertTrue(
             $validatePreconditions->invoke(
-                $this->optObj,
+                $this->optimizelyObject,
                 $this->projectConfig->getExperimentFromKey('test_experiment'),
                 'test_user',
                 [])
@@ -180,7 +180,7 @@ public function testValidatePreconditionsUserInForcedVariationNotInExperiment()
 
         $this->assertTrue(
             $validatePreconditions->invoke(
-                $this->optObj,
+                $this->optimizelyObject,
                 $this->projectConfig->getExperimentFromKey('test_experiment'),
                 'user_1',
                 [])
@@ -194,7 +194,7 @@ public function testValidatePreconditionsUserInForcedVariationInExperiment()
 
         $this->assertTrue(
             $validatePreconditions->invoke(
-                $this->optObj,
+                $this->optimizelyObject,
                 $this->projectConfig->getExperimentFromKey('test_experiment'),
                 'user1',
                 [])
@@ -209,7 +209,7 @@ public function testValidatePreconditionsUserNotInForcedVariationNotInExperiment
         // Will get updated when we have audience evaluation and user does not meet conditions
         $this->assertTrue(
             $validatePreconditions->invoke(
-                $this->optObj,
+                $this->optimizelyObject,
                 $this->projectConfig->getExperimentFromKey('test_experiment'),
                 'test_user',
                 [])
@@ -224,7 +224,7 @@ public function testValidatePreconditionsUserNotInForcedVariationInExperiment()
         // Will get updated when we have audience evaluation and user does meets conditions
         $this->assertTrue(
             $validatePreconditions->invoke(
-                $this->optObj,
+                $this->optimizelyObject,
                 $this->projectConfig->getExperimentFromKey('test_experiment'),
                 'test_user',
                 [])
@@ -296,12 +296,12 @@ public function testActivateExperimentNotRunning()
 
     public function testGetVariation()
     {
-        $this->assertEquals('control', $this->optObj->getVariation('test_experiment', 'test_user'));
+        $this->assertEquals('control', $this->optimizelyObject->getVariation('test_experiment', 'test_user'));
     }
 
     public function testGetVariationExperimentNotRunning()
     {
-        $this->assertNull($this->optObj->getVariation('paused_experiment', 'test_user'));
+        $this->assertNull($this->optimizelyObject->getVariation('paused_experiment', 'test_user'));
     }
 
     public function testTrackNoAttributesNoEventValue()