vwo-sdk 1.22.0 → 1.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 84d01560f605a69e8ed011a19ba4e3aae0f67341fd653a562d817801d6a55073
-  data.tar.gz: b1355bef1b2b095774c702a6757c1a7141e2cf9758d0a1b8a0510111a8ee597a
+  metadata.gz: 3a74040a37074e3fb8ffc75990383654fec552c7007414409ad6563de6173fe9
+  data.tar.gz: bac8b215309d14868f7cc8022f9d8afe1924d0ad22df3ed0919e136d8025c8cb
 SHA512:
-  metadata.gz: cc2c8a411e3c0a674ab2bc00e540b3e49a8dd943ec50aa04fba04839ef6f3e255fc275529f872aa5ca73dfe893cb43e8853a0875cc3dda9330554b6491590947
-  data.tar.gz: 5482e79564a365ec8fe4012b25997096352129e915daca28774c790c416a528302ea1f0bb685ef559b9ee0846b0229957a8f6ab77008f4e48222a7cae5892699
+  metadata.gz: 985e03d85f42c33db7c65580abce890127707b11e479a001a372d75ed779aad2d8010edcf2cc2e3dccb8fef39d02be3a165300465beef0e82851ce221985ef5f
+  data.tar.gz: d90e9528ba974d9e9a29bb7ee4f2d3b3fcb56e14edb15f62b0aaa45e2c2353c94723084c667785de871a8b43bb5d105747d4413683e097c44dd9f7e68e4d6a55

data/lib/vwo/constants.rb

@@ -27,7 +27,7 @@ class VWO
     HTTP_PROTOCOL = 'http://'
     HTTPS_PROTOCOL = 'https://'
     URL_NAMESPACE = '6ba7b811-9dad-11d1-80b4-00c04fd430c8'
-    SDK_VERSION = '1.22.0'
+    SDK_VERSION = '1.23.0'
     SDK_NAME = 'ruby'
     VWO_DELIMITER = '_vwo_'
     MAX_EVENTS_PER_REQUEST = 5000

data/lib/vwo/core/variation_decider.rb

@@ -460,9 +460,9 @@ class VWO
 
       def evaluate_whitelisting(user_id, campaign, api_name, campaign_key, variation_targeting_variables = {}, disable_logs = false)
         if variation_targeting_variables.nil?
-          variation_targeting_variables = { '_vwo_user_id' => user_id }
+          variation_targeting_variables = { _vwo_user_id: user_id }
         else
-          variation_targeting_variables['_vwo_user_id'] = user_id
+          variation_targeting_variables[:_vwo_user_id] = user_id
         end
         targeted_variations = []
 
@@ -717,7 +717,7 @@ class VWO
 
         group_campaigns.each do |campaign|
           if called_campaign["id"] == campaign["id"] || check_presegmentation(campaign, user_id, custom_variables, '', true) && @bucketer.user_part_of_campaign?(user_id, campaign)
-            eligible_campaigns.append(campaign)
+            eligible_campaigns.push(campaign)
           end
         end
         return eligible_campaigns
@@ -748,7 +748,7 @@ class VWO
       def get_eligible_campaigns_key(eligible_campaigns)
         eligible_campaigns_key = []
         eligible_campaigns.each do |campaign|
-          eligible_campaigns_key.append(campaign["key"])
+          eligible_campaigns_key.push(campaign["key"])
         end
         eligible_campaigns_key
       end
@@ -763,7 +763,7 @@ class VWO
         non_eligible_campaigns_key = []
         group_campaigns.each do |campaign|
           unless eligible_campaigns.include? campaign
-            non_eligible_campaigns_key.append(campaign["key"])
+            non_eligible_campaigns_key.push(campaign["key"])
           end
         end
         non_eligible_campaigns_key

data/lib/vwo/services/operand_evaluator.rb

@@ -91,11 +91,10 @@ class VWO
       # @return [Boolean]
       def evaluate_custom_variable?(operand, custom_variables)
         # Extract custom_variable_key and custom_variables_value from operand
-
         operand_key, operand = get_key_value(operand)
 
         # Retrieve corresponding custom_variable value from custom_variables
-        custom_variables_value = custom_variables[operand_key]
+        custom_variables_value = custom_variables[operand_key.to_sym]
 
         # Pre process custom_variable value
         custom_variables_value = process_custom_variables_value(custom_variables_value)
@@ -113,7 +112,7 @@ class VWO
       def evaluate_user?(operand, custom_variables)
         users = operand.split(',')
         users.each do |user|
-          return true if user.strip == custom_variables['_vwo_user_id']
+          return true if user.strip == custom_variables[:_vwo_user_id]
         end
         false
       end

data/lib/vwo/utils/campaign.rb

@@ -299,7 +299,7 @@ class VWO
           group_campaign_ids.each do |campaign_id|
             settings_file["campaigns"].each do |campaign|
               if campaign["id"] == campaign_id && campaign["status"] == STATUS_RUNNING
-                group_campaigns.append(campaign)
+                group_campaigns.push(campaign)
               end
             end
           end

data/lib/vwo/utils/utility.rb

@@ -0,0 +1,40 @@
+# Copyright 2019-2021 Wingify Software Pvt. Ltd.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+require 'json'
+require_relative './validations'
+
+# Generic utility module
+class VWO
+    module Utils
+        module Utility
+            include Validations
+
+            # converting hash with keys as strings into hash with keys as strings
+            # @param[Hash]
+            # @return[Hash]
+            def convert_to_symbol_hash(hashObject)
+                convertedHash = {}
+                hashObject.each do |key, value|
+                    if valid_hash?(value)
+                        convertedHash[key.to_sym] = convert_to_symbol_hash(value)
+                    else
+                        convertedHash[key.to_sym] = value
+                    end
+                end
+                convertedHash
+            end
+        end
+    end
+end

data/lib/vwo.rb

@@ -23,6 +23,7 @@ require_relative 'vwo/utils/campaign'
 require_relative 'vwo/utils/impression'
 require_relative 'vwo/utils/feature'
 require_relative 'vwo/utils/custom_dimensions'
+require_relative 'vwo/utils/utility'
 require_relative 'vwo/constants'
 require_relative 'vwo/core/variation_decider'
 require_relative 'vwo/services/batch_events_dispatcher'
@@ -39,6 +40,7 @@ class VWO
   include Utils::CustomDimensions
   include Utils::Campaign
   include Utils::Impression
+  include Utils::Utility
   include CONSTANTS
 
   FILE = FileNameEnum::VWO
@@ -62,6 +64,7 @@ class VWO
     settings_file = nil,
     options = {}
   )
+    options = convert_to_symbol_hash(options)
     @account_id = account_id
     @sdk_key = sdk_key
     @user_storage = user_storage
@@ -295,9 +298,10 @@ class VWO
       return
     end
 
+    options = convert_to_symbol_hash(options)
     # Retrieve custom variables
-    custom_variables = options['custom_variables'] || options[:custom_variables]
-    variation_targeting_variables = options['variation_targeting_variables'] || options[:variation_targeting_variables]
+    custom_variables = options[:custom_variables]
+    variation_targeting_variables = options[:variation_targeting_variables]
 
     should_track_returning_user = get_should_track_returning_user(options)
     # Validate input parameters
@@ -468,9 +472,10 @@ class VWO
       )
       return
     end
+    options = convert_to_symbol_hash(options)
     # Retrieve custom variables
-    custom_variables = options['custom_variables'] || options[:custom_variables]
-    variation_targeting_variables = options['variation_targeting_variables'] || options[:variation_targeting_variables]
+    custom_variables = options[:custom_variables]
+    variation_targeting_variables = options[:variation_targeting_variables]
 
     # Validate input parameters
     unless valid_string?(campaign_key) && valid_string?(user_id) && (custom_variables.nil? || valid_hash?(custom_variables)) &&
@@ -579,9 +584,10 @@ class VWO
       return false
     end
 
-    revenue_value = options['revenue_value'] || options[:revenue_value]
-    custom_variables = options['custom_variables'] || options[:custom_variables]
-    variation_targeting_variables = options['variation_targeting_variables'] || options[:variation_targeting_variables]
+    options = convert_to_symbol_hash(options)
+    revenue_value = options[:revenue_value]
+    custom_variables = options[:custom_variables]
+    variation_targeting_variables = options[:variation_targeting_variables]
     should_track_returning_user = get_should_track_returning_user(options)
     goal_type_to_track = get_goal_type_to_track(options)
 
@@ -799,9 +805,10 @@ class VWO
       return false
     end
 
+    options = convert_to_symbol_hash(options)
     # Retrieve custom variables
-    custom_variables = options['custom_variables'] || options[:custom_variables]
-    variation_targeting_variables = options['variation_targeting_variables'] || options[:variation_targeting_variables]
+    custom_variables = options[:custom_variables]
+    variation_targeting_variables = options[:variation_targeting_variables]
     should_track_returning_user = get_should_track_returning_user(options)
     @logger.log(
       LogLevelEnum::INFO,
@@ -993,9 +1000,10 @@ class VWO
       return
     end
 
+    options = convert_to_symbol_hash(options)
     # Retrieve custom variables
-    custom_variables = options['custom_variables'] || options[:custom_variables]
-    variation_targeting_variables = options['variation_targeting_variables'] || options[:variation_targeting_variables]
+    custom_variables = options[:custom_variables]
+    variation_targeting_variables = options[:variation_targeting_variables]
 
     unless valid_string?(campaign_key) && valid_string?(variable_key) && valid_string?(user_id) &&
       (custom_variables.nil? || valid_hash?(custom_variables)) && (variation_targeting_variables.nil? || valid_hash?(variation_targeting_variables))

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vwo-sdk
 version: !ruby/object:Gem::Version
-  version: 1.22.0
+  version: 1.23.0
 platform: ruby
 authors:
 - VWO
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-02 00:00:00.000000000 Z
+date: 2021-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: codecov
@@ -87,7 +87,6 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/abc.txt
 - lib/vwo.rb
 - lib/vwo/constants.rb
 - lib/vwo/core/bucketer.rb
@@ -112,6 +111,7 @@ files:
 - lib/vwo/utils/impression.rb
 - lib/vwo/utils/request.rb
 - lib/vwo/utils/segment.rb
+- lib/vwo/utils/utility.rb
 - lib/vwo/utils/uuid.rb
 - lib/vwo/utils/validations.rb
 homepage: https://vwo.com/fullstack/server-side-testing/

data/lib/abc.txt

@@ -1,28 +0,0 @@
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS
-{"user_id"=>"Ashley", "variation_name"=>"Control", "campaign_key"=>"DEV_TEST_162"}
-GOT_ELIGIBLE_CAMPAIGNS