update logs

Author: oakbani

src/Optimizely/Optimizely.php

@@ -931,7 +931,7 @@ private function getFeatureVariableValueFromVariation($featureFlagKey, $variable
         if ($variation === null) {
             $this->_logger->log(
                 Logger::INFO,
-                "User '{$userId}'is not in any variation, ".
+                "User '{$userId}' is not in experiment or rollout, ".
                 "returning default value '{$variableValue}'."
             );
         } else {
@@ -941,14 +941,13 @@ private function getFeatureVariableValueFromVariation($featureFlagKey, $variable
                     $variableValue = $variableUsage->getValue();
                     $this->_logger->log(
                         Logger::INFO,
-                        "Returning variable value '{$variableValue}' for variation '{$variation->getKey()}' ".
-                        "of feature flag '{$featureFlagKey}'"
+                        "Returning variable value '{$variableValue}' for variable key '{$variableKey}' ".
+                        "of feature flag '{$featureFlagKey}'."
                     );
                 } else {
                     $this->_logger->log(
                         Logger::INFO,
-                        "Variable '{$variableKey}' is not used in variation '{$variation->getKey()}', ".
-                        "returning default value '{$variableValue}'."
+                        "Variable value is not defined. Returning the default variable value '{$variableValue}'."
                     );
                 }
             } else {

tests/OptimizelyTest.php

@@ -3558,7 +3558,7 @@ public function testGetFeatureVariableValueForTypeGivenFeatureFlagIsNotEnabledFo
             ->method('log')
             ->with(
                 Logger::INFO,
-                "User 'user_id'is not in any variation, returning default value '14.99'."
+                "User 'user_id' is not in experiment or rollout, returning default value '14.99'."
             );
 
         $this->assertSame(
@@ -3595,8 +3595,8 @@ public function testGetFeatureVariableValueForTypeGivenFeatureFlagIsEnabledForUs
             ->method('log')
             ->with(
                 Logger::INFO,
-                "Returning variable value '42.42' for variation 'control' ".
-                    "of feature flag 'double_single_variable_feature'"
+                "Returning variable value '42.42' for variable key 'double_variable' ".
+                "of feature flag 'double_single_variable_feature'."
             );
 
         $this->assertSame(
@@ -3675,8 +3675,8 @@ public function testGetFeatureVariableValueForTypeWithRolloutRule()
             ->method('log')
             ->with(
                 Logger::INFO,
-                "Returning variable value 'true' for variation '177771' ".
-                "of feature flag 'boolean_single_variable_feature'"
+                "Returning variable value 'true' for variable key 'boolean_variable' ".
+                "of feature flag 'boolean_single_variable_feature'."
             );
 
         $this->assertTrue($this->optimizelyObject->getFeatureVariableBoolean('boolean_single_variable_feature', 'boolean_variable', 'user_id', []));
@@ -3751,7 +3751,7 @@ public function testGetFeatureVariableValueForTypeGivenFeatureFlagIsEnabledForUs
             ->method('log')
             ->with(
                 Logger::INFO,
-                "Variable 'double_variable' is not used in variation 'control', returning default value '14.99'."
+                "Variable value is not defined. Returning the default variable value '14.99'."
             );
 
         $this->assertSame(
@@ -3793,11 +3793,11 @@ public function testGetFeatureVariableValueForTypeWithEmptyUserID()
             ->method('log')
             ->with(
                 Logger::INFO,
-                "User ''is not in any variation, returning default value '14.99'."
+                "User 'test_user' is not in experiment or rollout, returning default value '14.99'."
             );
 
         $this->assertSame(
-            $this->optimizelyObject->getFeatureVariableValueForType('double_single_variable_feature', 'double_variable', '', [], 'double'),
+            $this->optimizelyObject->getFeatureVariableValueForType('double_single_variable_feature', 'double_variable', 'test_user', [], 'double'),
             14.99
         );
     }
@@ -4254,8 +4254,8 @@ public function testGetAllFeatureVariablesGivenFeatureFlagIsEnabledForUserAndVar
             ->method('log')
             ->with(
                 Logger::INFO,
-                "Returning variable value '42.42' for variation 'control' ".
-                "of feature flag 'double_single_variable_feature'"
+                "Returning variable value '42.42' for variable key 'double_variable'".
+                " of feature flag 'double_single_variable_feature'."
             );
 
         $this->assertSame(