optimizely-sdk 1.0.2 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0245df6d1e29df95c54b1e1ffff564bf8271697a
-  data.tar.gz: 5d083b0a9b67ae6785bb7c39f52f60eca8c50701
+  metadata.gz: 6b62ea44bca6fdbbd5c1086e88505cc27968fdff
+  data.tar.gz: 95a11cfbea360da196f6ba1eb9f243a5d8edfbd8
 SHA512:
-  metadata.gz: 1ea777db826ad4a880336a6ffc804d016fd8c156107e8405c2598c12f96030bc896463b9b4418ec18332e0bb632f4e81b5c5e632527873cd1e22ed06b8e95f3b
-  data.tar.gz: a3ac225efa1189c7b2b11b9d25711fd6eec3f7fc2b35ccd3e51795fcbdcc34005842d6ea79a95e93645e1a91317e7f8e0b4cf1e72eea4a84dd4d537bc31ebf09
+  metadata.gz: 615a93c3c0c3993d4f719bf97f967b3ac6af4e1cc75e3beb6ab968be6f30d3cbe5844aaaea31b71a78ee3b096564194af6a782d8a671510c965ecc4558caba52
+  data.tar.gz: 5df1b06c97e9454e149764ff30db4e78a59c5f9265062ffa337af26da7405fa534f33ae5c8777b4f45536e6ad27045129ce1ac9ac6fb7e055005087e76624dc1

data/lib/optimizely.rb

@@ -162,13 +162,13 @@ module Optimizely
       @config.get_variation_key_from_id(experiment_key, variation_id)
     end
 
-    def track(event_key, user_id, attributes = nil, event_value = nil)
+    def track(event_key, user_id, attributes = nil, event_tags = nil)
       # Send conversion event to Optimizely.
       #
       # event_key - Goal key representing the event which needs to be recorded.
       # user_id - String ID for user.
       # attributes - Hash representing visitor attributes and values which need to be recorded.
-      # event_value - Value associated with the event. Can be used to represent revenue in cents.
+      # event_tags - Hash representing metadata associated with the event.
 
       unless @is_valid
         logger = SimpleLogger.new
@@ -176,7 +176,15 @@ module Optimizely
         return nil
       end
 
+      if event_tags and event_tags.is_a? Numeric
+        event_tags = {
+          'revenue' => event_tags
+        }
+        @logger.log(Logger::WARN, 'Event value is deprecated in track call. Use event tags to pass in revenue value instead.')
+      end
+
       return nil if attributes && !attributes_valid?(attributes)
+      return nil if event_tags && !event_tags_valid?(event_tags)
 
       experiment_ids = @config.get_experiment_ids_for_goal(event_key)
       if experiment_ids.empty?
@@ -202,7 +210,7 @@ module Optimizely
       end
 
       conversion_event = @event_builder.create_conversion_event(event_key, user_id, attributes,
-                                                                event_value, valid_experiment_keys)
+                                                                event_tags, valid_experiment_keys)
       @logger.log(Logger::INFO,
                   'Dispatching conversion event to URL %s with params %s.' % [conversion_event.url,
                                                                               conversion_event.params])
@@ -251,6 +259,15 @@ module Optimizely
       true
     end
 
+    def event_tags_valid?(event_tags)
+      unless Helpers::Validator.event_tags_valid?(event_tags)
+        @logger.log(Logger::ERROR, 'Provided event tags are in an invalid format.')
+        @error_handler.handle_error(InvalidEventTagFormatError)
+        return false
+      end
+      true
+    end
+
     def validate_inputs(datafile, skip_json_validation)
       unless skip_json_validation
         raise InvalidInputError.new('datafile') unless Helpers::Validator.datafile_valid?(datafile)

data/lib/optimizely/event_builder.rb

@@ -16,6 +16,7 @@
 require_relative './audience'
 require_relative './params'
 require_relative './version'
+require_relative '../optimizely/helpers/event_tag_utils'
 
 module Optimizely
   class Event
@@ -89,20 +90,21 @@ module Optimizely
       Event.new(:post, IMPRESSION_EVENT_ENDPOINT, @params, POST_HEADERS)
     end
 
-    def create_conversion_event(event_key, user_id, attributes, event_value, experiment_keys)
+    def create_conversion_event(event_key, user_id, attributes, event_tags, experiment_keys)
       # Create conversion Event to be sent to the logging endpoint.
       #
       # event_key - Event key representing the event which needs to be recorded.
       # user_id - ID for user.
       # attributes - Hash representing user attributes and values which need to be recorded.
-      # event_value - Value associated with the event. Can be used to represent revenue in cents.
+      # event_tags - Hash representing metadata associated with the event.
       # experiment_keys - Array of valid experiment keys for the event
       #
       # Returns event hash encapsulating the conversion event.
 
       @params = {}
       add_common_params(user_id, attributes)
-      add_conversion_event(event_key, event_value)
+      add_conversion_event(event_key)
+      add_event_tags(event_tags)
       add_layer_states(user_id, experiment_keys)
       Event.new(:post, CONVERSION_EVENT_ENDPOINT, @params, POST_HEADERS)
     end
@@ -161,26 +163,47 @@ module Optimizely
       }
     end
 
-    def add_conversion_event(event_key, event_value)
+    def add_event_tags(event_tags)
+      @params['eventFeatures'] ||= []
+      @params['eventMetrics'] ||= []
+
+      return if event_tags.nil?
+
+      event_tags.each_pair do |event_tag_key, event_tag_value|
+        next if event_tag_value.nil?
+
+        event_feature = {
+          'id' => event_tag_key,
+          'type' => 'custom',
+          'value' => event_tag_value,
+          'shouldIndex' => false,
+        }
+        @params['eventFeatures'].push(event_feature)
+
+      end
+
+      event_value = Helpers::EventTagUtils.get_revenue_value(event_tags)
+
+      if event_value
+        event_metric = {
+          'name' => 'revenue',
+          'value' => event_value
+        }
+        @params['eventMetrics'].push(event_metric)
+      end
+
+    end
+
+    def add_conversion_event(event_key)
       # Add conversion event information to the event.
       #
       # event_key - Event key representing the event which needs to be recorded.
-      # event_value - Value associated with the event. Can be used to represent revenue in cents.
 
       event_id = @config.event_key_map[event_key]['id']
       event_name = @config.event_key_map[event_key]['key']
 
       @params['eventEntityId'] = event_id
-      @params['eventFeatures'] = []
       @params['eventName'] = event_name
-      @params['eventMetrics'] = []
-
-      if event_value
-        @params['eventMetrics'].push({
-          'name' => 'revenue',
-          'value' => event_value,
-        })
-      end
     end
 
     def add_layer_states(user_id, experiment_keys)
@@ -241,18 +264,21 @@ module Optimizely
       Event.new(:get, sprintf(OFFLINE_API_PATH, project_id: @params[Params::PROJECT_ID]), @params, {})
     end
 
-    def create_conversion_event(event_key, user_id, attributes, event_value, experiment_keys)
+    def create_conversion_event(event_key, user_id, attributes, event_tags, experiment_keys)
       # Create conversion Event to be sent to the logging endpoint.
       #
       # event_key - Goal key representing the event which needs to be recorded.
       # user_id - ID for user.
       # attributes - Hash representing user attributes and values which need to be recorded.
-      # event_value - Value associated with the event. Can be used to represent revenue in cents.
+      # event_tags - Hash representing metadata associated with the event.
       # experiment_keys - Array of valid experiment keys for the goal
       #
       # Returns event hash encapsulating the conversion event.
 
       @params = {}
+
+      event_value = Helpers::EventTagUtils.get_revenue_value(event_tags)
+
       add_common_params(user_id, attributes)
       add_conversion_goal(event_key, event_value)
       add_experiment_variation_params(user_id, experiment_keys)

data/lib/optimizely/exceptions.rb

@@ -40,6 +40,14 @@ module Optimizely
     end
   end
 
+  class InvalidEventTagFormatError < Error
+    # Raised when attributes are provided in an invalid format (e.g. not a Hash)
+
+    def initialize(msg = 'Event tags provided are in an invalid format.')
+      super
+    end
+  end
+
   class InvalidExperimentError < Error
     # Raised when an invalid experiment key is provided
 

data/lib/optimizely/helpers/event_tag_utils.rb

@@ -0,0 +1,57 @@
+#
+#    Copyright 2017, Optimizely and contributors
+#
+#    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 'optimizely/logger'
+require 'optimizely/helpers/validator'
+
+module Optimizely
+  module Helpers
+    module EventTagUtils
+      module_function
+
+      def get_revenue_value(event_tags)
+        # Grab the revenue value from the event tags. "revenue" is a reserved keyword.
+        #
+        # event_tags - Hash representing metadata associated with the event.
+        # Returns revenue value as an integer number
+        # Returns nil if revenue can't be retrieved from the event tags.
+
+        if event_tags.nil? or !Helpers::Validator.attributes_valid?(event_tags)
+          return nil
+        end
+
+        unless event_tags.has_key?('revenue')
+          return nil
+        end
+
+        logger = SimpleLogger.new
+        raw_value = event_tags['revenue']
+
+        unless raw_value.is_a? Numeric
+          logger.log(Logger::WARN, "Failed to parse revenue value #{raw_value} from event tags.")
+          return nil
+        end
+
+        if raw_value.is_a? Float
+          logger.log(Logger::WARN, "Failed to parse revenue value #{raw_value} from event tags.")
+          return nil
+        end
+
+        logger.log(Logger::INFO, "Parsed revenue value #{raw_value} from event tags.")
+        raw_value
+      end
+    end
+  end
+end

data/lib/optimizely/helpers/validator.rb

@@ -32,6 +32,16 @@ module Optimizely
         attributes.is_a?(Hash)
       end
 
+      def event_tags_valid?(event_tags)
+        # Determines if provided event tags are valid.
+        #
+        # event_tags - Event tags to be validated.
+        #
+        # Returns boolean depending on validity of event tags.
+
+        event_tags.is_a?(Hash)
+      end
+
       def datafile_valid?(datafile)
         # Determines if a given datafile is valid.
         #

data/lib/optimizely/version.rb

@@ -1,5 +1,5 @@
 #
-#    Copyright 2016, Optimizely and contributors
+#    Copyright 2016-2017, Optimizely and contributors
 #
 #    Licensed under the Apache License, Version 2.0 (the "License");
 #    you may not use this file except in compliance with the License.
@@ -14,5 +14,5 @@
 #    limitations under the License.
 #
 module Optimizely
-  VERSION = '1.0.2'.freeze
+  VERSION = '1.1.0'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: optimizely-sdk
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.1.0
 platform: ruby
 authors:
 - Andrew Delikat
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-15 00:00:00.000000000 Z
+date: 2017-03-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -126,6 +126,7 @@ files:
 - lib/optimizely/event_dispatcher.rb
 - lib/optimizely/exceptions.rb
 - lib/optimizely/helpers/constants.rb
+- lib/optimizely/helpers/event_tag_utils.rb
 - lib/optimizely/helpers/group.rb
 - lib/optimizely/helpers/validator.rb
 - lib/optimizely/logger.rb
@@ -152,7 +153,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 2.5.2
 signing_key: 
 specification_version: 4
 summary: Ruby SDK for Optimizely's testing framework