chargify_api_ares 0.4.4 → 0.5.0

data/lib/chargify_api_ares/config.rb

@@ -0,0 +1,16 @@
+module Chargify
+  class << self
+    attr_accessor :subdomain, :api_key, :site, :format, :timeout
+
+    def configure
+      yield self
+
+      Base.user      = api_key
+      Base.password  = 'X'
+      Base.timeout   = timeout unless (timeout.blank?)
+      
+      self.site ||= "https://#{subdomain}.chargify.com"
+      Base.site = site
+    end
+  end
+end

data/lib/chargify_api_ares/resources/base.rb

@@ -0,0 +1,14 @@
+module Chargify
+  class Base < ActiveResource::Base
+    self.format = :xml
+
+    def self.element_name
+      name.split(/::/).last.underscore
+    end
+
+    def to_xml(options = {})
+      options.merge!(:dasherize => false)
+      super
+    end
+  end
+end

data/lib/chargify_api_ares/resources/component.rb

@@ -0,0 +1,4 @@
+module Chargify
+  class Component < Base
+  end
+end

data/lib/chargify_api_ares/resources/coupon.rb

@@ -0,0 +1,19 @@
+module Chargify
+  class Coupon < Base
+    def self.find_all_by_product_family_id(product_family_id)
+      Coupon.find(:all, :params => { :product_family_id => product_family_id })
+    end
+    
+    def self.find_by_product_family_id_and_code(product_family_id, code)
+      Coupon.new get(:lookup, :product_family_id => product_family_id, :code => code)
+    end
+    
+    def usage
+      get :usage
+    end
+
+    def archive
+      self.destroy
+    end
+  end
+end

data/lib/chargify_api_ares/resources/customer.rb

@@ -0,0 +1,21 @@
+module Chargify
+  class Customer < Base
+    def self.find_by_reference(reference)
+      Customer.new get(:lookup, :reference => reference)
+    end
+
+    class Subscription < Base
+      self.prefix = "/customers/:customer_id/"
+    end
+
+    def subscriptions(params = {})
+      params.merge!({:customer_id => self.id})
+      Subscription.find(:all, :params => params)
+    end
+
+    def payment_profiles(params = {})
+      params.merge!({:customer_id => self.id})
+      PaymentProfile.find(:all, :params => params)
+    end
+  end
+end

data/lib/chargify_api_ares/resources/payment_profile.rb

@@ -0,0 +1,4 @@
+module Chargify
+  class PaymentProfile < Base
+  end
+end

data/lib/chargify_api_ares/resources/product.rb

@@ -0,0 +1,25 @@
+module Chargify
+  class Product < Base
+    def self.find_by_handle(handle)
+      Product.new get(:lookup, :handle => handle)
+    end
+    
+    protected
+
+    # Products are created in the scope of a ProductFamily, i.e. /product_families/nnn/products
+    #
+    # This alters the collection path such that it uses the product_family_id that is set on the
+    # attributes.
+    def create
+      pfid = begin
+        self.product_family_id
+      rescue NoMethodError
+        0
+      end
+      connection.post("/product_families/#{pfid}/products.#{self.class.format.extension}", encode, self.class.headers).tap do |response|
+        self.id = id_from_response(response)
+        load_attributes_from_response(response)
+      end
+    end
+  end
+end

data/lib/chargify_api_ares/resources/product_family.rb

@@ -0,0 +1,34 @@
+module Chargify
+  class ProductFamily < Base
+    def self.find_by_handle(handle, attributes = {})
+      ProductFamily.find(:one, :from => :lookup, :params => { :handle => handle })
+    end
+    
+    class Product < Base
+      self.prefix = "/product_families/:product_family_id/"
+    end
+    
+    class Component < Base
+      self.prefix = "/product_families/:product_family_id/"
+    end
+    
+    class Coupon < Base
+      self.prefix = "/product_families/:product_family_id/"
+    end
+    
+    def products(params = {})
+      params.merge!(:product_family_id => self.id)
+      ::Chargify::ProductFamily::Product.find(:all, :params => params)
+    end
+    
+    def components(params = {})
+      params.merge!({:product_family_id => self.id})
+      ::Chargify::ProductFamily::Component.find(:all, :params => params)
+    end
+
+    def coupons(params = {})
+      params.merge!(:product_family_id => self.id)
+      ::Chargify::ProductFamily::Coupon.find(:all, :params => params)
+    end
+  end
+end

data/lib/chargify_api_ares/resources/site.rb

@@ -0,0 +1,7 @@
+module Chargify
+  class Site < Base
+    def self.clear_data!(params = {})
+      post(:clear_data, params)
+    end
+  end
+end

data/lib/chargify_api_ares/resources/statement.rb

@@ -0,0 +1,4 @@
+module Chargify
+  class Statement < Base
+  end
+end

data/lib/chargify_api_ares/resources/subscription.rb

@@ -0,0 +1,123 @@
+module Chargify
+  class Subscription < Base
+    def self.find_by_customer_reference(reference)
+      customer = Customer.find_by_reference(reference)
+      find(:first, :params => {:customer_id => customer.id})
+    end
+
+    # Strip off nested attributes of associations before saving, or type-mismatch errors will occur
+    def save
+      self.attributes.delete('customer')
+      self.attributes.delete('product')
+      self.attributes.delete('credit_card')
+      super
+    end
+
+    def cancel
+      destroy
+    end
+
+    def component(id)
+      Component.find(id, :params => {:subscription_id => self.id})
+    end
+
+    def components(params = {})
+      params.merge!({:subscription_id => self.id})
+      Component.find(:all, :params => params)
+    end
+
+    def payment_profile
+      self.respond_to?('credit_card') ? credit_card : nil
+    end
+
+    # Perform a one-time charge on an existing subscription.
+    # For more information, please see the one-time charge API docs available
+    # at: http://support.chargify.com/faqs/api/api-charges
+    def charge(attrs = {})
+      post :charges, {}, attrs.to_xml(:root => :charge)
+    end
+
+    def credit(attrs = {})
+      post :credits, {}, attrs.to_xml(:root => :credit)
+    end
+
+    def refund(attrs = {})
+      post :refunds, {}, attrs.to_xml(:root => :refund)
+    end
+
+    def reactivate(params = {})
+      put :reactivate, params
+    end
+
+    def reset_balance
+      put :reset_balance
+    end
+
+    def migrate(attrs = {})
+      post :migrations, :migration => attrs
+    end
+
+    def statement(id)
+      statement = Chargify::Statement.find(id)
+      raise ActiveResource::ResourceNotFound.new(nil) if (statement.subscription_id != self.id)
+      statement
+    end
+
+    def statements(params = {})
+      params.merge!(:subscription_id => self.id)
+      Statement.find(:all, :params => params)
+    end
+
+    def transactions(params = {})
+      params.merge!(:subscription_id => self.id)
+      Transaction.find(:all, :params => params)
+    end
+
+    def adjustment(attrs = {})
+      post :adjustments, {}, attrs.to_xml(:root => :adjustment)
+    end
+
+    def add_coupon(code)
+      post :add_coupon, :code => code
+    end
+
+    def remove_coupon(code=nil)
+      if code.nil?
+        delete :remove_coupon
+      else
+        delete :remove_coupon, :code => code
+      end
+    end
+
+    class Component < Base
+      self.prefix = "/subscriptions/:subscription_id/"
+
+      # All Subscription Components are considered already existing records, but the id isn't used
+      def id
+        self.component_id
+      end
+    end
+
+    class Statement < Base
+      self.prefix = "/subscriptions/:subscription_id/"
+    end
+
+    class Transaction < Base
+      self.prefix = "/subscriptions/:subscription_id/"
+
+      def full_refund(attrs = {})
+        return false if self.transaction_type != 'payment'
+
+        attrs.merge!(:amount_in_cents => self.amount_in_cents)
+        self.refund(attrs)
+      end
+
+      def refund(attrs = {})
+        return false if self.transaction_type != 'payment'
+
+        attrs.merge!(:payment_id => self.id)
+        Subscription.find(self.prefix_options[:subscription_id]).refund(attrs)
+      end
+    end
+  end
+end

data/lib/chargify_api_ares/resources/transaction.rb

@@ -0,0 +1,17 @@
+module Chargify
+  class Transaction < Base
+    def full_refund(attrs = {})
+      return false if self.transaction_type != 'payment'
+
+      attrs.merge!(:amount_in_cents => self.amount_in_cents)
+      self.refund(attrs)
+    end
+
+    def refund(attrs = {})
+      return false if self.transaction_type != 'payment'
+
+      attrs.merge!(:payment_id => self.id)
+      Subscription.find(self.subscription_id).refund(attrs)
+    end
+  end
+end

data/lib/chargify_api_ares/resources/usage.rb

@@ -0,0 +1,11 @@
+module Chargify
+  class Usage < Base
+    def subscription_id=(i)
+      self.prefix_options[:subscription_id] = i
+    end
+    
+    def component_id=(i)
+      self.prefix_options[:component_id] = i
+    end
+  end
+end

data/spec/factories.rb

@@ -2,46 +2,50 @@ FactoryGirl.define do
   sequence :email do |n|
     "customer#{n}@example.com"
   end
-
+  
+  sequence :product_name do |n|
+    "Product #{n}"
+  end
+  
   sequence :customer_id do |n|
     n
   end
-
-  factory :customer, :class => Chargify::Customer do |c|
-    c.first_name { Faker::Name.first_name }
-    c.last_name { Faker::Name.last_name }
-    c.email { FactoryGirl.generate(:email) }
-    c.organization { Faker::Company.name }
-    c.created_at { 2.days.ago }
-    c.updated_at { 1.hour.ago }
+  
+  sequence :subscription_id do |n|
+    n
   end
-
-
+  
   sequence :product_id do |n|
     n
   end
 
-  sequence :product_name do |n|
-    "Product #{n}"
+  factory :customer, :class => Chargify::Customer do |f|
+    f.first_name { Faker::Name.first_name }
+    f.last_name { Faker::Name.last_name }
+    f.email { FactoryGirl.generate(:email) }
+    f.organization { Faker::Company.name }
+    f.created_at { 2.days.ago }
+    f.updated_at { 1.hour.ago }
   end
 
-  factory :product, :class => Chargify::Product do |p|
-    p.name { FactoryGirl.generate(:product_name) }
+  factory :product, :class => Chargify::Product do |f|
+    f.name { FactoryGirl.generate(:product_name) }
   end
 
-  sequence :subscription_id do |n|
-    n
+  factory :product_family, :class => Chargify::ProductFamily do |f|
+    f.name { Faker::Name.name }
+    f.handle 'mining'
   end
 
-  factory :subscription, :class => Chargify::Subscription do |s|
-    s.balance_in_cents 500
-    s.current_period_ends_at 3.days.from_now
+  factory :subscription, :class => Chargify::Subscription do |f|
+    f.balance_in_cents 500
+    f.current_period_ends_at 3.days.from_now
   end
 
-  factory :subscription_with_extra_attrs, :parent => :subscription do |swea|
-    swea.customer Chargify::Customer.new
-    swea.product Chargify::Product.new
-    swea.credit_card "CREDIT CARD"
+  factory :subscription_with_extra_attrs, :parent => :subscription do |f|
+    f.customer Chargify::Customer.new
+    f.product Chargify::Product.new
+    f.credit_card "CREDIT CARD"
   end
 
   factory :component, :class => Chargify::Component do |f|

data/spec/remote/remote.example.yml

@@ -0,0 +1,6 @@
+# Copy this file to spec/remote/remote.yml and make adjustments as necessary.
+#
+# Note: Remote tests will only work when configured to run on a test site that uses the Bogus gateway.
+# Warning: All data in the site specified will be cleared and replaced with test data.
+api_key: your_api_key
+site: your_site

data/spec/remote/remote_spec.rb

@@ -1,514 +1,409 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
-if run_remote_tests?
-  describe "Remote" do
-    before(:all) do
-      clear_site_data
-      setup_plans
-      setup_customer
-    end
-    
-    describe "creating a new subscription to a product with a trial" do
-      context "when providing valid attributes for the customer and the payment profile" do
-        before(:each) do
-          @subscription = create_once(:subscription) do
-            Chargify::Subscription.create(
-              :product_handle => @@basic_plan.handle,
-              :customer_attributes => valid_customer_attributes,
-              :payment_profile_attributes => good_payment_profile_attributes
-            )
-          end
-        end
-        
-        it "successfully creates the subscription" do
-          @subscription.should be_a(Chargify::Subscription)
-        end
-        
-        it "sets the current_period_started_at attribute to now" do
-          @subscription.current_period_started_at.utc.should be_close(now.utc, approximately)
-        end
-        
-        it "sets the current_period_ends_at attribute to 1 month from now" do
-          @subscription.current_period_ends_at.utc.should be_close(one_month_from_now.utc, approximately)
-        end
-        
-        it "is in the trialing state" do
-          @subscription.state.should == 'trialing'
-        end
+describe "Remote" do
+
+  let(:acme_projects) { Chargify::ProductFamily.create(:name => "Acme Projects") }
+
+  let(:basic_plan) do
+    Chargify::Product.create(
+      :product_family_id => acme_projects.id,
+      :name => "Basic Plan",
+      :handle => "basic",
+      :price_in_cents => 1000,
+      :interval => 1,
+      :interval_unit => 'month',
+      :trial_interval => 1,
+      :trial_interval_unit => 'month',
+      :trial_price_in_cents => 0)
+  end
+
+  let(:pro_plan) do
+    Chargify::Product.create(
+      :product_family_id => acme_projects.id,
+      :name => "Pro Plan",
+      :handle => "pro",
+      :price_in_cents => 5000,
+      :interval => 1,
+      :interval_unit => 'month')
+  end
+
+  let(:johnadoe) do
+    Chargify::Customer.create(
+      :first_name => "John",
+      :last_name => "Doe",
+      :email => "john.doe@example.com",
+      :reference => "johndoe")
+  end
+  
+  let(:johnadoes_credit_card) { Chargify::PaymentProfile.create(good_payment_profile_attributes.merge(:customer_id => johnadoe.id)) }
+  
+  before(:all) do
+    # Make sure the test site data is set up correctly
+    clear_site_data; acme_projects; basic_plan; pro_plan; johnadoe; johnadoes_credit_card
+  end
+
+  describe "creating a new subscription to a product with a trial" do
+    context "when providing valid attributes for the customer and the payment profile" do
+      before(:all) do
+        @subscription = Chargify::Subscription.create(
+          :product_handle => basic_plan.handle, 
+          :customer_attributes => {
+            :first_name => "Rick",
+            :last_name => "James",
+            :email => "rick@example.com",
+            :reference => "rickjames"
+          }, 
+          :payment_profile_attributes => good_payment_profile_attributes) 
       end
-    end
-    
-    describe "creating a new subscription to a product without a trial" do
-      context "when providing an existing customer reference and valid payment profile attributes" do
-        before(:each) do
-          @subscription = create_once(:subscription) do
-            Chargify::Subscription.create(
-              :product_handle => @@pro_plan.handle,
-              :customer_reference => @@johnadoe.reference,
-              :payment_profile_attributes => good_payment_profile_attributes
-            )
-          end
-        end
-        
-        it "successfully creates the subscription" do
-          @subscription.should be_a(Chargify::Subscription)
-        end
-        
-        it "sets the current_period_started_at attribute to now" do
-          @subscription.current_period_started_at.utc.should be_close(now.utc, approximately)
-        end
-        
-        it "sets the current_period_ends_at attribute to 1 month from now" do
-          @subscription.current_period_ends_at.utc.should be_close(one_month_from_now.utc, approximately)
-        end
-        
-        it "is in the active state" do
-          @subscription.state.should == 'active'
-        end
-        
-        it "belongs to the existing customer" do
-          @subscription.customer.should == @@johnadoe
-        end
+      
+      it "sets the current_period_started_at attribute to now" do
+        @subscription.current_period_started_at.utc.should be_within(60).of(now.utc)
       end
 
-      context "when providing an existing customer reference and an existing payment profile" do
-        before(:each) do
-          @subscription = create_once(:subscription) do
-            Chargify::Subscription.create(
-              :product_handle => @@pro_plan.handle,
-              :customer_reference => @@johnadoe.reference,
-              :payment_profile_id => @@johnadoes_credit_card.id.to_s
-            )
-          end
-        end
-        
-        it "successfully creates the subscription" do
-          @subscription.should be_a(Chargify::Subscription)
-        end
-        
-        it "is in the active state" do
-          @subscription.state.should == 'active'
-        end
-        
-        it "belongs to the existing customer" do
-          @subscription.customer.should == @@johnadoe
-        end
-        
-        it "uses the provided credit card" do
-          expected_card = Chargify::PaymentProfile.find(@@johnadoes_credit_card.id)
-          @subscription.payment_profile.id.should == @@johnadoes_credit_card.id
-          @subscription.payment_profile.attributes.should == expected_card.attributes
-        end
+      it "sets the current_period_ends_at attribute to 1 month from now" do
+        @subscription.current_period_ends_at.utc.should be_within(60).of(one_month_from_now.utc)
       end
 
-      context "when providing valid attributes for the customer and attributes for a credit card that cannot be stored" do
-        before(:each) do
-          @customer_attributes = valid_customer_attributes.dup
-          @subscription = create_once(:subscription) do
-            Chargify::Subscription.create(
-              :product_handle => @@basic_plan.handle,
-              :customer_attributes => @customer_attributes,
-              :payment_profile_attributes => unstorable_payment_profile_attributes
-            )
-          end
-        end
-        
-        it "does not create the subscription" do
-          @subscription.should_not be_valid
-        end
-      
-        it "does not create the customer" do
-          lambda {
-            Chargify::Customer.find_by_reference(@customer_attributes[:reference])
-          }.should raise_error(ActiveResource::ResourceNotFound)
-        end
+      it "is in the trialing state" do
+        @subscription.state.should == 'trialing'
       end
-
     end
-    
-    describe "importing a subscription to a product with a trial and a next billing date 10 days from now" do
-      context "when giving valid attributes for the customer and the payment profile" do
-        before(:each) do
-          @subscription = create_once(:subscription) do
-            Chargify::Subscription.create(
-              :product_handle => @@basic_plan.handle,
-              :customer_attributes => valid_customer_attributes,
-              :payment_profile_attributes => pretokenized_card_attributes,
-              :next_billing_at => ten_days_from_now.utc
-            )
-          end
-        end
-        
-        it "successfully creates the subscription" do
-          @subscription.should be_a(Chargify::Subscription)
-        end
-        
-        it "sets the current_period_started_at attribute to now" do
-          @subscription.current_period_started_at.utc.should be_close(now.utc, approximately)
-        end
-    
-        it "sets the current_period_ends_at attribute to 1 month from now" do
-          @subscription.current_period_ends_at.utc.should be_close(ten_days_from_now.utc, approximately)
-        end
-        
-        it "is in the active state" do
-          @subscription.state.should == 'active'
-        end
+  end
+
+  describe "creating a new subscription to a product without a trial" do
+    context "when providing an existing customer reference and valid payment profile attributes" do
+      before(:all) do
+        @subscription = Chargify::Subscription.create(
+          :product_handle => pro_plan.handle,
+          :customer_reference => johnadoe.reference,
+          :payment_profile_attributes => good_payment_profile_attributes)
       end
-    end
 
-    describe "creating failed subscriptions" do
-      context "due to providing payment profile attribtues for a card that will be declined" do
-        before(:each) do
-          @customer_attributes = valid_customer_attributes.dup
-          @subscription = create_once(:subscription) do
-            Chargify::Subscription.create(
-              :product_handle => @@pro_plan.handle,
-              :customer_attributes => @customer_attributes,
-              :payment_profile_attributes => declined_payment_profile_attributes
-            )
-          end
-        end
-      
-        it "does not create the subscription" do
-          @subscription.should_not be_valid
-        end
-      
-        it "does not create the customer" do
-          lambda {
-            Chargify::Customer.find_by_reference(@customer_attributes[:reference])
-          }.should raise_error(ActiveResource::ResourceNotFound)
-        end
+      it "sets the current_period_started_at attribute to now" do
+        @subscription.current_period_started_at.utc.should be_within(60).of(now.utc)
       end
-    end
-    
-    describe "cancelling a subscription" do
-      before(:each) do
-        @subscription = create_once(:subscription) do
-          Chargify::Subscription.create(
-            :product_handle => @@pro_plan.handle,
-            :customer_reference => @@johnadoe.reference,
-            :payment_profile_attributes => good_payment_profile_attributes
-          )
-        end
-        @subscription.cancel
+
+      it "sets the current_period_ends_at attribute to 1 month from now" do
+        @subscription.current_period_ends_at.utc.should be_within(60).of(one_month_from_now.utc)
       end
-      
-      it "is in the canceled state" do
-        Chargify::Subscription.find(@subscription.id).state.should == 'canceled'
+
+      it "is in the active state" do
+        @subscription.state.should == 'active'
+      end
+
+      it "belongs to the existing customer" do
+        @subscription.customer.should == johnadoe
       end
     end
-    
-    describe "reactivating a subscriptions" do
-      before(:each) do
-        @subscription = create_once(:subscription) do
-          Chargify::Subscription.create(
-            :product_handle => @@pro_plan.handle,
-            :customer_reference => @@johnadoe.reference,
-            :payment_profile_attributes => good_payment_profile_attributes
-          )
-        end
-        @subscription.cancel
-        @subscription.reload.state.should == 'canceled'
-        @subscription.reactivate
+
+    context "when providing an existing customer reference and an existing payment profile" do
+      before(:all) do
+        @subscription = Chargify::Subscription.create(
+          :product_handle => pro_plan.handle,
+          :customer_reference => johnadoe.reference,
+          :payment_profile_id => johnadoes_credit_card.id.to_s)
       end
-      
-      it "puts it in the active state" do
-        @subscription.reload.state.should == 'active'
+
+      it "is in the active state" do
+        @subscription.state.should == 'active'
       end
-    end
-    
-    describe "adding a one time charge" do
-      before(:each) do
-        @subscription = create_once(:subscription) do
-          Chargify::Subscription.create(
-            :product_handle => @@pro_plan.handle,
-            :customer_reference => @@johnadoe.reference,
-            :payment_profile_attributes => good_payment_profile_attributes
-          )
-        end
+
+      it "belongs to the existing customer" do
+        @subscription.customer.should == johnadoe
       end
-      it "creates a charge and payment" do
-        lambda{
-          @subscription.charge(:amount => 7, :memo => 'One Time Charge')
-        }.should change{@subscription.reload.transactions.size}.by(2)
-        @subscription.transactions.first.amount_in_cents.should == 700
+
+      it "uses the provided credit card" do
+        expected_card = Chargify::PaymentProfile.find(johnadoes_credit_card.id)
+        @subscription.payment_profile.id.should == johnadoes_credit_card.id
+        @subscription.payment_profile.attributes.should == expected_card.attributes
       end
     end
-    
-    describe "adding a credit" do
-      before(:each) do
-        @subscription = create_once(:subscription) do
-          Chargify::Subscription.create(
-            :product_handle => @@pro_plan.handle,
-            :customer_reference => @@johnadoe.reference,
-            :payment_profile_attributes => good_payment_profile_attributes
-          )
-        end
+
+    context "when providing valid attributes for the customer and attributes for a credit card that cannot be stored" do
+      before(:all) do
+        @subscription = Chargify::Subscription.create(
+          :product_handle => basic_plan.handle,
+          :customer_attributes => {
+            :first_name => "Ziggy",
+            :last_name => "Marley",
+            :email => "ziggy@example.com",
+            :reference => "ziggy"
+          }, 
+          :payment_profile_attributes => unstorable_payment_profile_attributes)
       end
-      
-      it "creates a credit" do
-        lambda{
-          @subscription.credit(:amount => 7, :memo => 'credit')
-        }.should change{@subscription.reload.transactions.size}.by(1)
-        @subscription.transactions.first.amount_in_cents.should == -700
+
+      it "does not create the subscription" do
+        @subscription.should_not be_valid
       end
-    end
-    
-    describe "adding a refund" do
-      before(:each) do
-        @subscription = create_once(:subscription) do
-          Chargify::Subscription.create(
-            :product_handle => @@pro_plan.handle,
-            :customer_reference => @@johnadoe.reference,
-            :payment_profile_attributes => good_payment_profile_attributes
-          )
-        end
 
-        @subscription.charge(:amount => 7, :memo => 'One Time Charge')
-        @subscription.reload
+      it "does not create the customer" do
+        lambda {
+          Chargify::Customer.find_by_reference("ziggy")
+        }.should raise_error(ActiveResource::ResourceNotFound)
       end
+    end
 
-      it "creates a refund" do
-        lambda{
-          @subscription.refund :payment_id => @subscription.transactions[0].id, :amount => 7, 
-            :memo => 'Refunding One Time Charge'
-        }.should change{@subscription.reload.transactions.size}.by(1)
-        @subscription.transactions.first.amount_in_cents.should == 700
-        @subscription.transactions.first.transaction_type.should == 'refund'
+  end
+
+  describe "importing a subscription to a product with a trial and a next billing date 10 days from now" do
+    context "when giving valid attributes for the customer and the payment profile" do
+      before(:all) do
+        @subscription = Chargify::Subscription.create(
+          :product_handle => basic_plan.handle,
+          :customer_attributes => {
+            :first_name => "John",
+            :last_name => "Denver",
+            :email => "john.denver@example.com",
+            :reference => "johndenver"
+          }, 
+          :payment_profile_attributes => pretokenized_card_attributes,
+          :next_billing_at => ten_days_from_now.utc)
+        @subscription.should be_a(Chargify::Subscription)
       end
 
-      context "via subscription payment (Chargify::Subscription::Transaction)" do
-        before :each do
-          @payment = @subscription.transactions.first
-        end
-
-        it "creates a refund" do
-          lambda{
-            @payment.refund :amount => 7, :memo => 'Refunding One Time Charge'
-          }.should change{@subscription.reload.transactions.size}.by(1)
-          @subscription.transactions.first.amount_in_cents.should == 700
-          @subscription.transactions.first.transaction_type.should == 'refund'
-        end
-
-        it "creates a full refund" do
-          lambda{
-            @payment.full_refund :memo => 'Refunding One Time Charge'
-          }.should change{@subscription.reload.transactions.size}.by(1)
-          @subscription.transactions.first.amount_in_cents.should == 700
-          @subscription.transactions.first.transaction_type.should == 'refund'
-        end
+      it "sets the current_period_started_at attribute to now" do
+        @subscription.current_period_started_at.utc.should be_within(60).of(now.utc)
       end
 
-      context "via site payment (Chargify::Transaction)" do
-        before :each do
-          @site_payment = Chargify::Transaction.find(:first)
-        end
-
-        it "creates a refund" do
-          lambda{
-            @site_payment.refund :amount => 7, :memo => 'Refunding One Time Charge'
-          }.should change{@subscription.reload.transactions.size}.by(1)
-          @subscription.transactions.first.amount_in_cents.should == 700
-          @subscription.transactions.first.transaction_type.should == 'refund'
-        end
-
-        it "creates a full refund" do
-          lambda{
-            @site_payment.full_refund :memo => 'Refunding One Time Charge'
-          }.should change{@subscription.reload.transactions.size}.by(1)
-          @subscription.transactions.first.amount_in_cents.should == 700
-          @subscription.transactions.first.transaction_type.should == 'refund'
-        end
+      it "sets the current_period_ends_at attribute to 1 month from now" do
+        @subscription.current_period_ends_at.utc.should be_within(60).of(ten_days_from_now.utc)
       end
-    end
-    
-    def already_cleared_site_data?
-      @@already_cleared_site_data ||= nil
-      @@already_cleared_site_data == true
-    end
-    
-    def cleared_site_data!
-      @@already_cleared_site_data = true
-    end
-    
-    def clear_site_data
-      return if already_cleared_site_data?
-      begin
-        Chargify::Site.clear_data!
-        cleared_site_data!
-      rescue ActiveResource::ForbiddenAccess
-        raise StandardError.new("Remote specs may only be run against a site in test mode")
+
+      it "is in the active state" do
+        @subscription.state.should == 'active'
       end
     end
+  end
 
-    # Create Basic and Pro products in the Acme Projects family
-    def setup_plans
-      begin
-        @@acme_projects ||= Chargify::ProductFamily.find_by_handle('acme-projects')
-      rescue ActiveResource::ResourceNotFound
-        @@acme_projects = Chargify::ProductFamily.new(
-          :name => "Acme Projects"
-        )
-        result = @@acme_projects.save
-        result.should be_true
-      end
-      
-      begin
-        @@basic_plan ||= Chargify::Product.find_by_handle('basic')
-      rescue ActiveResource::ResourceNotFound
-        @@basic_plan = Chargify::Product.new(
-          :product_family_id => @@acme_projects.id,
-          :name => "Basic Plan",
-          :handle => "basic",
-          :price_in_cents => 1000,
-          :interval => 1,
-          :interval_unit => 'month',
-          :trial_interval => 1,
-          :trial_interval_unit => 'month',
-          :trial_price_in_cents => 0
-        )
-        result = @@basic_plan.save
-        result.should be_true
+  describe "creating failed subscriptions" do
+    context "due to providing payment profile attribtues for a card that will be declined" do
+      before(:all) do
+        @subscription = Chargify::Subscription.create(
+          :product_handle => pro_plan.handle,
+          :customer_attributes => {
+            :first_name => "Frank",
+            :last_name => "Sinatra",
+            :email => "frank.sinatra@example.com",
+            :reference => "franksinatra"
+          }, 
+          :payment_profile_attributes => declined_payment_profile_attributes)
       end
 
-      begin
-        @@pro_plan ||= Chargify::Product.find_by_handle('pro')
-      rescue ActiveResource::ResourceNotFound
-        @@pro_plan = Chargify::Product.new(
-          :product_family_id => @@acme_projects.id,
-          :name => "Pro Plan",
-          :handle => "pro",
-          :price_in_cents => 5000,
-          :interval => 1,
-          :interval_unit => 'month'
-        )
-        result = @@pro_plan.save
-        result.should be_true
+      it "does not create the subscription" do
+        @subscription.should_not be_valid
       end
-    end
 
-    def setup_customer
-      # Create a customer
-      begin
-        @@johnadoe ||= Chargify::Customer.find_by_reference('a')
-      rescue ActiveResource::ResourceNotFound
-        @@johnadoe = Chargify::Customer.new(valid_customer_attributes)
-        result = @@johnadoe.save
-        result.should be_true
-        
-        @@johnadoes_credit_card = Chargify::PaymentProfile.new(
-          good_payment_profile_attributes.merge(:customer_id => @@johnadoe.id)
-        )
-        result = @@johnadoes_credit_card.save
-        result.should be_true
+      it "does not create the customer" do
+        lambda {
+          Chargify::Customer.find_by_reference("franksinatra")
+        }.should raise_error(ActiveResource::ResourceNotFound)
       end
     end
-    
-    def now
-      Time.now
-    end
-  
-    def one_month_from_now
-      Time.now + 1.month
+  end
+
+  describe "cancelling a subscription" do
+    before(:all) do
+      @subscription = Chargify::Subscription.create(
+        :product_handle => pro_plan.handle,
+        :customer_reference => johnadoe.reference,
+        :payment_profile_attributes => good_payment_profile_attributes)
     end
-  
-    def ten_days_from_now
-      10.days.from_now
+
+    it "is in the canceled state" do
+      @subscription.cancel
+      Chargify::Subscription.find(@subscription.id).state.should == 'canceled'
     end
-  
-    # Gives a reasonable range for time comparisons
-    def approximately
-      @approximately ||= 5.minutes
+  end
+
+  describe "reactivating a subscriptions" do
+    before(:all) do
+      @subscription = Chargify::Subscription.create(
+        :product_handle => pro_plan.handle,
+        :customer_reference => johnadoe.reference,
+        :payment_profile_attributes => good_payment_profile_attributes)
     end
-  
-    def valid_customer_attributes
-      initial = next_customer_initial
-      {
-        :first_name => "John #{initial.upcase}",
-        :last_name => "Doe",
-        :email => "john.#{initial}.doe@example.com",
-        :reference => initial
-      }
+
+    it "puts it in the active state" do
+      @subscription.cancel
+      @subscription.reload.state.should == 'canceled'
+      @subscription.reactivate
+      @subscription.reload.state.should == 'active'
     end
-  
-    def good_payment_profile_attributes
-      {
-        :full_number => '1',
-        :expiration_month => '12',
-        :expiration_year => Time.now.year + 1
-      }
+  end
+
+  describe "adding a one time charge" do
+    before(:all) do
+      @subscription = Chargify::Subscription.create(
+        :product_handle => pro_plan.handle,
+        :customer_reference => johnadoe.reference,
+        :payment_profile_attributes => good_payment_profile_attributes)
     end
 
-    def declined_payment_profile_attributes
-      {
-        :full_number => '2',
-        :expiration_month => '12',
-        :expiration_year => Time.now.year + 1
-      }
+    it "creates a charge and payment" do
+      lambda{
+        @subscription.charge(:amount => 7, :memo => 'One Time Charge')
+      }.should change{@subscription.reload.transactions.size}.by(2)
+      most_recent_transaction(@subscription).amount_in_cents.should == 700
     end
+  end
 
-    def unstorable_payment_profile_attributes
-      {
-        :full_number => '3',
-        :expiration_month => '12',
-        :expiration_year => Time.now.year + 1
-      }
+  describe "adding a credit" do
+    before(:all) do
+      @subscription = Chargify::Subscription.create(
+        :product_handle => pro_plan.handle,
+        :customer_reference => johnadoe.reference,
+        :payment_profile_attributes => good_payment_profile_attributes)
     end
 
-    def expired_payment_profile_attributes
-      {
-        :full_number => '1',
-        :expiration_month => '12',
-        :expiration_year => Time.now.year - 1
-      }
+    it "creates a credit" do
+      lambda{
+        @subscription.credit(:amount => 7, :memo => 'credit')
+      }.should change{@subscription.reload.transactions.size}.by(1)
+      most_recent_transaction(@subscription).amount_in_cents.should == -700
     end
-  
-    def pretokenized_card_attributes
-      {
-        :vault_token => '1',
-        :current_vault => 'bogus',
-        :expiration_month => '12',
-        :expiration_year => Time.now.year + 1,
-        :last_four => '1234',
-        :card_type => 'visa'
-      }
+  end
+
+  describe "adding a refund" do
+    before(:all) do
+      @subscription = Chargify::Subscription.create(
+        :product_handle => pro_plan.handle,
+        :customer_reference => johnadoe.reference,
+        :payment_profile_attributes => good_payment_profile_attributes)
     end
-  
-    # Don't create more than 26 customers until we add more initials :)
-    def next_customer_initial
-      @@customer_initial_index ||= 0
-      initial = customer_initials[@@customer_initial_index]
-      @@customer_initial_index += 1
-      initial
+
+    before(:each) do
+      @subscription.charge(:amount => 7, :memo => 'One Time Charge')
+      @subscription.reload
+      @payment = most_recent_transaction(@subscription)
+      @payment.transaction_type.should == 'payment'
     end
-  
-    # An array of intials
-    def customer_initials
-      @customer_initials ||= ('a'..'z').to_a
+
+    context "via Chargify::Subscription#refund" do
+      it "creates a refund" do
+        lambda{
+          @subscription.refund :payment_id => @payment.id, :amount => 7, 
+            :memo => 'Refunding One Time Charge'
+        }.should change{@subscription.reload.transactions.size}.by(1)
+
+        tx = most_recent_transaction(@subscription)
+
+        tx.amount_in_cents.should == 700
+        tx.transaction_type.should == 'refund'
+      end
     end
 
-    # Allows us to create remote resources once per spec hierarchy
-    def create_once(type, &block)
-      hierarchy = example_group_hierarchy.collect(&:object_id).to_s
+    context "via Chargify::Transaction#refund" do
+      it "creates a refund" do
+        lambda{
+          @payment.refund :amount => 7, :memo => 'Refunding One Time Charge'
+        }.should change{@subscription.reload.transactions.size}.by(1)
+        tx = most_recent_transaction(@subscription)
+
+        tx.amount_in_cents.should == 700
+        tx.transaction_type.should == 'refund'
+      end
+
+      it "creates a full refund" do
+        lambda{
+          @payment.full_refund :memo => 'Refunding One Time Charge'
+        }.should change{@subscription.reload.transactions.size}.by(1)
+        tx = most_recent_transaction(@subscription)
 
-      unless resource(type, hierarchy)
-        register_resource(type, hierarchy, block.call)
+        tx.amount_in_cents.should == 700
+        tx.transaction_type.should == 'refund'
       end
-      resource(type, hierarchy)
     end
-  
-    def resource(type, hierarchy)
-      @@resources ||= {}
-      @@resources[type] ||= {}
-      @@resources[type][hierarchy]
+  end
+
+  def clear_site_data
+    begin
+      Chargify::Site.clear_data!
+      Chargify::Product.find(:all).count.should == 0
+      Chargify::Customer.find(:all).count.should == 0
+      Chargify::ProductFamily.find(:all).count.should == 0
+    rescue ActiveResource::ForbiddenAccess
+      raise StandardError.new("Remote specs may only be run against a site in test mode")
     end
-  
-    def register_resource(type, hierarchy, resource)
-      @@resources ||= {}
-      @@resources[type] ||= {}
-      @@resources[type][hierarchy] = resource
+  end
+
+  def now
+    Time.now
+  end
+
+  def one_month_from_now
+    Time.now + 1.month
+  end
+
+  def ten_days_from_now
+    10.days.from_now
+  end
+
+  def good_payment_profile_attributes
+    {
+      :full_number => '1',
+      :expiration_month => '12',
+      :expiration_year => Time.now.year + 1
+    }
+  end
+
+  def declined_payment_profile_attributes
+    {
+      :full_number => '2',
+      :expiration_month => '12',
+      :expiration_year => Time.now.year + 1
+    }
+  end
+
+  def unstorable_payment_profile_attributes
+    {
+      :full_number => '3',
+      :expiration_month => '12',
+      :expiration_year => Time.now.year + 1
+    }
+  end
+
+  def expired_payment_profile_attributes
+    {
+      :full_number => '1',
+      :expiration_month => '12',
+      :expiration_year => Time.now.year - 1
+    }
+  end
+
+  def pretokenized_card_attributes
+    {
+      :vault_token => '1',
+      :current_vault => 'bogus',
+      :expiration_month => '12',
+      :expiration_year => Time.now.year + 1,
+      :last_four => '1234',
+      :card_type => 'visa'
+    }
+  end
+
+  # Allows us to create remote resources once per spec hierarchy
+  def create_once(type, &block)
+    hierarchy = example_group_hierarchy.collect(&:object_id).to_s
+
+    unless resource(type, hierarchy)
+      register_resource(type, hierarchy, block.call)
     end
+    resource(type, hierarchy)
+  end
+
+  def resource(type, hierarchy)
+    @resources ||= {}
+    @resources[type] ||= {}
+    @resources[type][hierarchy]
+  end
+
+  def register_resource(type, hierarchy, resource)
+    @resources ||= {}
+    @resources[type] ||= {}
+    @resources[type][hierarchy] = resource
+  end
 
+  def most_recent_transaction(scope)
+    scope.transactions.sort_by(&:id).last
   end
 end