customerio 0.2.0 → 0.3.0

data/CHANGELOG.markdown

@@ -1,3 +1,8 @@
+## Customerio 0.3.0 - Dec 28, 2012 ##
+
+* Now raise an error if an API call doesn't respond with a 200 response code
+* Removed dependency on ActiveSupport
+
 ## Customerio 0.2.0 - Nov 21, 2012 ##
 
 * Allow raw hashes to be passed into `identify` and `track` methods rather than a customer object.

data/customerio.gemspec

@@ -16,7 +16,6 @@ Gem::Specification.new do |gem|
   gem.version       = Customerio::VERSION
 
   gem.add_dependency('httparty', '~> 0.9.0')
-  gem.add_dependency('activesupport')
 
   gem.add_development_dependency('rake')
   gem.add_development_dependency('rspec')

data/lib/customerio.rb

@@ -1,5 +1,5 @@
 require "customerio/version"
 
 module Customerio
-  autoload :Client, 'customerio/client'
+  require "customerio/client"
 end

data/lib/customerio/client.rb

@@ -1,32 +1,31 @@
 require 'httparty'
-require 'active_support/hash_with_indifferent_access'
 
 module Customerio
   class Client
-	  include HTTParty
-	  base_uri 'https://app.customer.io'
+    include HTTParty
+    base_uri 'https://app.customer.io'
 
     CustomerProxy = Struct.new("Customer", :id)
 
     class MissingIdAttributeError < RuntimeError; end
     class InvalidResponse < RuntimeError; end
 
-	  @@id_block = nil
+    @@id_block = nil
 
-	  def self.id(&block)
+    def self.id(&block)
       warn "[DEPRECATION] Customerio::Client.id customization is deprecated."
       @@id_block = block
-	  end
+    end
 
-	  def self.default_config
-	  	@@id_block = nil
-	  end
+    def self.default_config
+      @@id_block = nil
+    end
 
-	  def initialize(site_id, secret_key)
-	    @auth = { :username => site_id, :password => secret_key }
-	  end
+    def initialize(site_id, secret_key)
+      @auth = { :username => site_id, :password => secret_key }
+    end
 
-	  def identify(*args)
+    def identify(*args)
       attributes = extract_attributes(args)
 
       if args.any?
@@ -34,10 +33,10 @@ module Customerio
         attributes = attributes_from(customer).merge(attributes)
       end
 
-	    create_or_update(attributes)
-	  end
+      create_or_update(attributes)
+    end
 
-	  def track(*args)
+    def track(*args)
       attributes = extract_attributes(args)
 
       if args.length == 1
@@ -52,35 +51,35 @@ module Customerio
         identify(attributes_from(customer))
         create_customer_event(id_from(customer), event_name, attributes)
       end
-	  end
+    end
 
-	  private
+    private
 
-	  def create_or_update(attributes = {})
+    def create_or_update(attributes = {})
       raise MissingIdAttributeError.new("Must provide an customer id") unless attributes[:id]
 
       url = customer_path(attributes[:id])
       attributes[:id] = custom_id(attributes[:id])
 
-	    verify_response(self.class.put(url, options.merge(:body => attributes)))
-	  end
+      verify_response(self.class.put(url, options.merge(:body => attributes)))
+    end
 
-	  def create_customer_event(customer_id, event_name, attributes = {})
+    def create_customer_event(customer_id, event_name, attributes = {})
       create_event("#{customer_path(customer_id)}/events", event_name, attributes)
-	  end
+    end
 
     def create_anonymous_event(event_name, attributes = {})
       create_event("/api/v1/events", event_name, attributes)
     end
 
-	  def create_event(url, event_name, attributes = {})
-	  	body = { :name => event_name, :data => attributes }
-	    verify_response(self.class.post(url, options.merge(:body => body)))
-	  end
+    def create_event(url, event_name, attributes = {})
+      body = { :name => event_name, :data => attributes }
+      verify_response(self.class.post(url, options.merge(:body => body)))
+    end
 
-	  def customer_path(id)
-	    "/api/v1/customers/#{custom_id(id)}"
-	  end
+    def customer_path(id)
+      "/api/v1/customers/#{custom_id(id)}"
+    end
 
     def verify_response(response)
       if response.code >= 200 && response.code < 300
@@ -91,18 +90,19 @@ module Customerio
     end
 
     def extract_attributes(args)
-      HashWithIndifferentAccess.new(args.last.is_a?(Hash) ? args.pop : {})
+      hash = args.last.is_a?(Hash) ? args.pop : {}
+      hash.inject({}){ |hash, (k,v)| hash[k.to_sym] = v; hash }
     end
 
     def attributes_from(customer)
       if id?(customer)
-        HashWithIndifferentAccess.new(:id => customer)
+        { :id => customer }
       else
-        HashWithIndifferentAccess.new(
+        {
           :id => id_from(customer),
           :email => customer.email,
           :created_at => customer.created_at.to_i
-        )
+        }
       end
     end
 
@@ -115,16 +115,16 @@ module Customerio
       end
     end
 
-	  def custom_id(id)
-	  	@@id_block ? @@id_block.call(id) : id
-	  end
+    def custom_id(id)
+      @@id_block ? @@id_block.call(id) : id
+    end
 
     def id?(object)
       object.is_a?(Integer) || object.is_a?(String)
     end
 
-	  def options
-	    { :basic_auth => @auth }
-	  end
+    def options
+      { :basic_auth => @auth }
+    end
   end
 end

data/lib/customerio/version.rb

@@ -1,3 +1,3 @@
 module Customerio
-  VERSION = "0.2.0"
+  VERSION = "0.3.0"
 end

data/spec/client_spec.rb

@@ -3,7 +3,7 @@ require 'spec_helper'
 describe Customerio::Client do
 	let(:client)   { Customerio::Client.new("SITE_ID", "API_KEY") }
 	let(:customer) { mock("Customer", :id => 5, :email => "customer@example.com", :created_at => Time.now) }
-  let(:response) { mock("Response", code: 200) }
+  let(:response) { mock("Response", :code => 200) }
 
   before do
     # Dont call out to customer.io
@@ -24,7 +24,7 @@ describe Customerio::Client do
     end
 
     it "raises an error if PUT doesn't return a 2xx response code" do
-      Customerio::Client.should_receive(:put).and_return(mock("Response", code: 500))
+      Customerio::Client.should_receive(:put).and_return(mock("Response", :code => 500))
       lambda { client.identify(:id => 5) }.should raise_error(Customerio::Client::InvalidResponse)
     end
 
@@ -46,7 +46,7 @@ describe Customerio::Client do
           :created_at => Time.now.to_i,
           :first_name => "Bob",
           :plan => "basic"
-        }.stringify_keys
+        }
       }).and_return(response)
 
       client.identify(:id => 5, :email => "customer@example.com", :created_at => Time.now.to_i, :first_name => "Bob", :plan => "basic")
@@ -64,7 +64,7 @@ describe Customerio::Client do
             :id => 5,
             :email => "customer@example.com",
             :created_at => Time.now.to_i
-          }.stringify_keys
+          }
         }).and_return(response)
 
         client.identify(customer)
@@ -79,11 +79,26 @@ describe Customerio::Client do
             :created_at => Time.now.to_i,
             :first_name => "Bob",
             :plan => "basic"
-          }.stringify_keys
+          }
         }).and_return(response)
 
         client.identify(customer, :first_name => "Bob", :plan => "basic")
       end
+
+      it "allows customer object attributes to be overriden" do
+        Customerio::Client.should_receive(:put).with("/api/v1/customers/5", {
+          :basic_auth => anything(),
+          :body => {
+            :id => 5,
+            :email => "customer2@example.com",
+            :created_at => Time.now.to_i,
+            :first_name => "Bob",
+            :plan => "basic"
+          }
+        }).and_return(response)
+
+        client.identify(customer, "email" => "customer2@example.com", :first_name => "Bob", :plan => "basic")
+      end
     end
 
     context "client has customized identities" do
@@ -100,7 +115,7 @@ describe Customerio::Client do
             :id => "production_5",
             :email => "customer@example.com",
             :created_at => Time.now.to_i
-          }.stringify_keys
+          }
         }).and_return(response)
 
         client.identify(customer)
@@ -113,7 +128,7 @@ describe Customerio::Client do
             :id => "production_5",
             :email => "customer@example.com",
             :created_at => Time.now.to_i
-          }.stringify_keys
+          }
         }).and_return(response)
 
         client.identify(:id => 5, :email => "customer@example.com", :created_at => Time.now.to_i)
@@ -128,12 +143,12 @@ describe Customerio::Client do
   	end
 
     it "raises an error if POST doesn't return a 2xx response code" do
-      Customerio::Client.should_receive(:post).and_return(mock("Response", code: 500))
+      Customerio::Client.should_receive(:post).and_return(mock("Response", :code => 500))
       lambda { client.track(customer, "purchase") }.should raise_error(Customerio::Client::InvalidResponse)
     end
 
   	it "calls identify with the user's attributes to ensure they've been properly identified" do
-  		client.should_receive(:identify).with({ :id => 5, :email => "customer@example.com", :created_at => Time.now.to_i }.stringify_keys).and_return(response)
+  		client.should_receive(:identify).with({ :id => 5, :email => "customer@example.com", :created_at => Time.now.to_i }).and_return(response)
   		client.track(customer, "purchase")
   	end
 
@@ -160,7 +175,7 @@ describe Customerio::Client do
   			:basic_auth => anything(),
   			:body => {
   				:name => "purchase",
-  			  :data => { :type => "socks", :price => "13.99" }.stringify_keys
+  			  :data => { :type => "socks", :price => "13.99" }
   			}
   		}).and_return(response)
 
@@ -172,7 +187,7 @@ describe Customerio::Client do
   			:basic_auth => anything(),
   			:body => {
   				:name => "purchase",
-  			  :data => { :type => "socks", :price => "13.99" }.stringify_keys
+  			  :data => { :type => "socks", :price => "13.99" }
   			}
   		}).and_return(response)
 
@@ -234,7 +249,7 @@ describe Customerio::Client do
           :basic_auth => anything(),
           :body => {
             :name => "purchase",
-            :data => { :type => "socks", :price => "13.99" }.stringify_keys
+            :data => { :type => "socks", :price => "13.99" }
           }
         }).and_return(response)
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: customerio
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-22 00:00:00.000000000 Z
+date: 2012-12-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -27,22 +27,6 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.9.0
-- !ruby/object:Gem::Dependency
-  name: activesupport
-  requirement: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement