sift 1.1.7.2 → 1.1.7.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    N2IzYmI3MWNkNWUyNTQyY2IyMGI5OGVjYjM2YmRhYjI2N2RjOGRlNA==
+    ZmNiNGJmZDg2OGU5ZTEzYjNmOWJhZjAzMGNiMzZkYTE5NTVmOGFkZg==
   data.tar.gz: !binary |-
-    NTFhYzU1MGQxYzBiNGQ1Y2ExYzFjZjEzMmM3MzRjZWE1MjUzZGJiNg==
+    NjJmY2FmYWQxMWEyNjFlY2MwMDdmOGZlN2ZiMWE0ODFiMGI1N2MyYg==
 SHA512:
   metadata.gz: !binary |-
-    ODk0NjQ2MzU5MTIzMGUzY2ZiMDMyYmM2MDc1N2U5N2ZjMmZkODMxMTg3NjBl
-    Y2Q3ZTc3YmY1OGI0MWE4ZDQxZGNiODkyMjU1Yjg4YmZkMTgwYjg2NmI0MGVk
-    ZDQ0M2M3Zjc3MjliZGE0NzUzNTU4YmI2OGFkNDlhYmRhNmFiYTQ=
+    M2FkOGQyZTM0MzdjM2FiMGVjNjU0ZDgxOTU0MjFjNjQ4NWE0NzM2ZGZkNGJi
+    ZjhkNmZlMzA5MGRiNGU1OWM3NGI4MzFjMDZkODE5NzQ3NDkyZGQwYmVhNzIy
+    NzI3ZmIzZjkzMzMyMWEwMTBmMzMxYTIwMjA0YTExN2UxZThhNDE=
   data.tar.gz: !binary |-
-    MGJmZmYxZDgwNjZmMjQ2OTg4ZWVlZjIwZDVjNDFiOWYyYjAzNmViZTYxYWEz
-    ZjQyNjhhZDViNzA4N2M1NzIxNmRjNGRiYjkwNzM1NTE2MDEyZGZiZGNiMzM3
-    Y2ZlMDE0NzhmN2Q5NTg3NzRjZTYzODAyMGNiNGIyYzU4NWU0ZTQ=
+    ZGQ0Zjk1YTRmM2YyYzBhMjkzYzcxNzZkOWIxNTIxNTY2MzcyZmUyY2U3ZDRi
+    YTk1OGExNzBlZDFjNjhlZjhlOTE4NDUzOWJjNjI0Y2MwOTFmYmEwMmY1MDI0
+    OWU1MDQ0YzBmN2EzMmE1ZmQyMDRmNTJkNTI5Y2IzNzAwNTlmN2M=

data/HISTORY

@@ -1,3 +1,6 @@
+=== 1.1.7.3 2015-08-18
+* Added support for return_action
+
 === 1.1.7.2 2015-04-13
 * Fixed backwards compatibility issue
 

data/lib/sift/client.rb

@@ -121,26 +121,34 @@ module Sift
     #   Overrides the default API path with a different URL.
     #
     # return_score (optional)
-    #   Whether the API response should include a score for this user (the score will
+    #   Whether the API response should include a score for this user. The score will
     #   be calculated using the submitted event.  This feature must be
     #   enabled for your account in order to use it.  Please contact
     #   support@siftscience.com if you are interested in using this feature.
     #
+    # return_action (optional)
+    #   Whether the API response should include an action triggered for this transaction.
+    #
     # == Returns:
     #   In the case of an HTTP error (timeout, broken connection, etc.), this
     #   method returns nil; otherwise, a Response object is returned and captures
     #   the status message and status code. In general, you can ignore the returned
     #   result, though.
     #
-    def track(event, properties = {}, timeout = nil, path = nil, return_score = false, api_key = @api_key)
+    def track(event, properties = {}, timeout = nil, path = nil, return_score = false, api_key = @api_key, return_action = false)
+      warn "[WARNING] api_key cannot be empty, fallback to default api_key." if api_key.to_s.empty?
+      api_key ||= @api_key
       raise("event must be a non-empty string") if (!event.is_a? String) || event.empty?
       raise("properties cannot be empty") if properties.empty?
       raise("Bad api_key parameter") if api_key.empty?
       path ||= @path
       timeout ||= @timeout
-      if return_score
-        path = path + "?return_score=true"
-      end
+
+      uri = URI.parse(API_ENDPOINT)      
+      uri.query = URI.encode_www_form(URI.decode_www_form(uri.query.to_s) << ["return_score", "true"]) if return_score
+      uri.query = URI.encode_www_form(URI.decode_www_form(uri.query.to_s) << ["return_action", "true"]) if return_action
+      path = path + "?" + uri.query if !uri.query.to_s.empty?
+
       options = {
         :body => MultiJson.dump(delete_nils(properties).merge({"$type" => event,
                                                                "$api_key" => api_key})),
@@ -207,7 +215,9 @@ module Sift
       raise("user_id must be a non-empty string") if (!user_id.is_a? String) || user_id.to_s.empty?
 
       path = Sift.current_users_label_api_path(user_id)
-      track("$label", delete_nils(properties), timeout, path, false, api_key)
+
+      # No return_action logic supported when using labels.
+      track("$label", delete_nils(properties), timeout, path, false, api_key, false)
     end
 
     # Unlabels a user.  This call is blocking.
@@ -237,6 +247,9 @@ module Sift
     end
 
     private
+    # def add_query_parameter(query_parameter)
+    #   uri = URI.parse(API_ENDPOINT)
+    # end
     def delete_nils(properties)
       properties.delete_if do |k, v|
         case v

data/lib/sift/version.rb

@@ -1,4 +1,4 @@
 module Sift
-  VERSION = "1.1.7.2"
+  VERSION = "1.1.7.3"
   API_VERSION = "203"
 end

data/spec/unit/client_spec.rb

@@ -42,6 +42,41 @@ describe Sift::Client do
     }
   end
 
+  def action_response_json
+    {
+      :user_id => "247019",
+      :score => 0.93,
+      :actions => [{
+                    :action_id => "1234567890abcdefghijklmn",
+                    :time => 1437421587052,
+                    :triggers => [{
+                      :triggerType => "FORMULA",
+                      :source => "synchronous_action",
+                      :trigger_id => "12345678900987654321abcd"
+                    }],
+                    :entity => {
+                      :type => "USER",
+                      :id => "23056"
+                    }
+                  },
+                  {
+                    :action_id => "12345678901234567890abcd",
+                    :time => 1437421587410,
+                    :triggers => [{
+                      :triggerType => "FORMULA",
+                      :source => "synchronous_action",
+                      :trigger_id => "abcd12345678901234567890"
+                    }],
+                    :entity => {
+                      :type => "ORDER",
+                      :id => "order_at_ 1437421587009"
+                    }
+                  }],
+      :status => 0,
+      :error_message => "OK"
+    }
+  end
+
   def fully_qualified_api_endpoint
     Sift::Client::API_ENDPOINT + Sift.current_rest_api_path
   end
@@ -58,17 +93,17 @@ describe Sift::Client do
   end
 
   it "Cannot instantiate client with nil, empty, non-string, or blank api key" do
-    expect(lambda { Sift::Client.new(nil) }).to raise_error
-    expect(lambda { Sift::Client.new("") }).to raise_error
-    expect(lambda { Sift::Client.new(123456) }).to raise_error
-    expect(lambda { Sift::Client.new() }).to raise_error
+    expect(lambda { Sift::Client.new(nil) }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new("") }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new(123456) }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new() }).to raise_error(StandardError)
   end
 
   it "Cannot instantiate client with nil, empty, non-string, or blank path" do
     api_key = "test_local_api_key"
-    expect(lambda { Sift::Client.new(api_key, nil) }).to raise_error
-    expect(lambda { Sift::Client.new(api_key, "") }).to raise_error
-    expect(lambda { Sift::Client.new(api_key, 123456) }).to raise_error
+    expect(lambda { Sift::Client.new(api_key, nil) }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new(api_key, "") }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new(api_key, 123456) }).to raise_error(StandardError)
   end
 
   it "Can instantiate client with non-default timeout" do
@@ -76,23 +111,23 @@ describe Sift::Client do
   end
 
   it "Track call must specify an event name" do
-    expect(lambda { Sift::Client.new("foo").track(nil) }).to raise_error
-    expect(lambda { Sift::Client.new("foo").track("") }).to raise_error
+    expect(lambda { Sift::Client.new("foo").track(nil) }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new("foo").track("") }).to raise_error(StandardError)
   end
 
   it "Must specify an event name" do
-    expect(lambda { Sift::Client.new("foo").track(nil) }).to raise_error
-    expect(lambda { Sift::Client.new("foo").track("") }).to raise_error
+    expect(lambda { Sift::Client.new("foo").track(nil) }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new("foo").track("") }).to raise_error(StandardError)
   end
 
   it "Must specify properties" do
     event = "custom_event_name"
-    expect(lambda { Sift::Client.new("foo").track(event) }).to raise_error
+    expect(lambda { Sift::Client.new("foo").track(event) }).to raise_error(StandardError)
   end
 
   it "Score call must specify a user_id" do
-    expect(lambda { Sift::Client.new("foo").score(nil) }).to raise_error
-    expect(lambda { Sift::Client.new("foo").score("") }).to raise_error
+    expect(lambda { Sift::Client.new("foo").score(nil) }).to raise_error(StandardError)
+    expect(lambda { Sift::Client.new("foo").score("") }).to raise_error(StandardError)
   end
 
   it "Doesn't raise an exception on Net/HTTP errors" do
@@ -141,7 +176,7 @@ describe Sift::Client do
     expect(response.api_error_message).to eq("OK")
   end
 
-  it "Successfully submits event with overriden key" do
+  it "Successfully submits event with overridden key" do
     response_json = { :status => 0, :error_message => "OK"}
     stub_request(:post, "https://api.siftscience.com/v203/events").
       with { | request|
@@ -154,7 +189,7 @@ describe Sift::Client do
     event = "$transaction"
     properties = valid_transaction_properties
 
-    response = Sift::Client.new(api_key).track(event, properties, nil, nil, false, "overridden")
+    response = Sift::Client.new(api_key).track(event, properties, nil, nil, false, "overridden", false)
     expect(response.ok?).to eq(true)
     expect(response.api_status).to eq(0)
     expect(response.api_error_message).to eq("OK")
@@ -233,13 +268,34 @@ describe Sift::Client do
 
     event = "$transaction"
     properties = valid_transaction_properties
-    response = Sift::Client.new(api_key).track(event, properties, nil, nil, true)
+    response = Sift::Client.new(api_key).track(event, properties, nil, nil, true, nil, nil)
     expect(response.ok?).to eq(true)
     expect(response.api_status).to eq(0)
     expect(response.api_error_message).to eq("OK")
     expect(response.body["score_response"]["score"]).to eq(0.93)
   end
 
+  it "Successfuly make a sync action request" do
+
+    api_key = "foobar"
+    response_json = {
+      :status => 0,
+      :error_message => "OK",
+      :score_response => action_response_json
+    }
+
+    stub_request(:post, "https://api.siftscience.com/v203/events?return_action=true").
+      to_return(:status => 200, :body => MultiJson.dump(response_json), :headers => {"content-type"=>"application/json; charset=UTF-8","content-length"=> "74"})
+
+    event = "$transaction"
+    properties = valid_transaction_properties
+    response = Sift::Client.new(api_key).track(event, properties, nil, nil, nil, nil, true)
+    expect(response.ok?).to eq(true)
+    expect(response.api_status).to eq(0)
+    expect(response.api_error_message).to eq("OK")
+    expect(response.body["score_response"]["actions"].first["entity"]["type"]).to eq("USER")
+  end
+
 
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sift
 version: !ruby/object:Gem::Version
-  version: 1.1.7.2
+  version: 1.1.7.3
 platform: ruby
 authors:
 - Fred Sadaghiani
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-13 00:00:00.000000000 Z
+date: 2015-08-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec