[FSSDK-11177] unnecessary log removal

Author: junaed-optimizely

OptimizelySDK.Tests/CmabTests/DecisionServiceCmabTest.cs

@@ -103,11 +103,6 @@ public void TestGetVariationWithCmabExperimentReturnsVariation()
             Assert.AreEqual(VARIATION_A_ID, result.ResultObject.Variation.Id);
             Assert.AreEqual(TEST_CMAB_UUID, result.ResultObject.CmabUuid);
 
-            var reasons = result.DecisionReasons.ToReport(true);
-            var expectedMessage =
-                $"CMAB decision fetched for user {TEST_USER_ID} in experiment {TEST_EXPERIMENT_KEY}.";
-            Assert.Contains(expectedMessage, reasons);
-
             _cmabServiceMock.Verify(c => c.GetDecision(
                 It.IsAny<ProjectConfig>(),
                 It.IsAny<OptimizelyUserContext>(),
@@ -309,11 +304,6 @@ public void TestGetVariationWithCmabExperimentCacheHit()
                     It.IsAny<string>(),
                     It.IsAny<TimeSpan?>()),
                 Times.Once);
-
-            var reasons = result2.DecisionReasons.ToReport(true);
-            var expectedMessage =
-                $"CMAB decision fetched for user {TEST_USER_ID} in experiment {TEST_EXPERIMENT_KEY}.";
-            Assert.Contains(expectedMessage, reasons);
         }
 
         /// <summary>

OptimizelySDK.Tests/CmabTests/OptimizelyUserContextCmabTest.cs

@@ -382,10 +382,6 @@ public void TestDecideWithCmabExperimentIncludeReasons()
 
             Assert.IsNotNull(decision);
             Assert.IsNotNull(decision.Reasons);
-            var expectedMessage = string.Format(CmabConstants.CMAB_DECISION_FETCHED, TEST_USER_ID,
-                TEST_EXPERIMENT_KEY);
-            Assert.IsTrue(decision.Reasons.Any(r => r.Contains(expectedMessage)),
-                "Decision reasons should include CMAB fetch success message.");
             Assert.AreEqual(1, _cmabService.CallCount);
         }
 

OptimizelySDK/Bucketing/DecisionService.cs

@@ -331,10 +331,6 @@ OptimizelyDecideOption[] options
                         new VariationDecisionResult(null), reasons);
                 }
 
-                var successMessage = string.Format(CmabConstants.CMAB_DECISION_FETCHED, userId,
-                    experiment.Key);
-                Logger.Log(LogLevel.INFO, reasons.AddInfo(successMessage));
-
                 return Result<VariationDecisionResult>.NewResult(
                     new VariationDecisionResult(variation, cmabDecision.CmabUuid), reasons);
             }

OptimizelySDK/Cmab/CmabConstants.cs

@@ -34,9 +34,6 @@ internal static class CmabConstants
         public const string CMAB_FETCH_FAILED =
             "Failed to fetch CMAB data for experiment {0}.";
 
-        public const string CMAB_DECISION_FETCHED =
-            "CMAB decision fetched for user {0} in experiment {1}.";
-
         public static readonly TimeSpan MAX_TIMEOUT = TimeSpan.FromSeconds(10);
         public static readonly TimeSpan DEFAULT_CACHE_TTL = TimeSpan.FromMinutes(10);