From 87894b3019af41a736aeba541fba2197776433ec Mon Sep 17 00:00:00 2001 From: Zeeshan Ashraf Date: Tue, 22 Dec 2020 14:53:17 -0800 Subject: [PATCH] fixed a log message --- lib/optimizely/decision_service.rb | 2 +- spec/decision_service_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/optimizely/decision_service.rb b/lib/optimizely/decision_service.rb index 799543d7..ecb5c534 100644 --- a/lib/optimizely/decision_service.rb +++ b/lib/optimizely/decision_service.rb @@ -425,7 +425,7 @@ def get_saved_variation_id(project_config, experiment_id, user_profile) variation_id = decision[:variation_id] return variation_id, nil if project_config.variation_id_exists?(experiment_id, variation_id) - message = "User '#{user_profile['user_id']}' was previously bucketed into variation ID '#{variation_id}' for experiment '#{experiment_id}', but no matching variation was found. Re-bucketing user." + message = "User '#{user_profile[:user_id]}' was previously bucketed into variation ID '#{variation_id}' for experiment '#{experiment_id}', but no matching variation was found. Re-bucketing user." @logger.log(Logger::INFO, message) [nil, message] diff --git a/spec/decision_service_spec.rb b/spec/decision_service_spec.rb index 13eb8269..2f149b06 100644 --- a/spec/decision_service_spec.rb +++ b/spec/decision_service_spec.rb @@ -380,7 +380,7 @@ variation_received, reasons = decision_service.get_variation(config, 'test_experiment', 'test_user') expect(variation_received).to eq('111128') expect(reasons).to eq([ - "User '' was previously bucketed into variation ID '111111' for experiment '111127', but no matching variation was found. Re-bucketing user.", + "User 'test_user' was previously bucketed into variation ID '111111' for experiment '111127', but no matching variation was found. Re-bucketing user.", "Audiences for experiment 'test_experiment' collectively evaluated to TRUE.", "User 'test_user' is in variation 'control' of experiment 'test_experiment'." ])