chargify_api_ares 0.3.7 → 0.3.8

data/VERSION

@@ -1 +1 @@
-0.3.7
+0.3.8

data/chargify_api_ares.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{chargify_api_ares}
-  s.version = "0.3.7"
+  s.version = "0.3.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Michael Klett", "The Lab Rats @ Phase Two Labs", "Brian Rose", "Nathan Verni"]
-  s.date = %q{2010-11-11}
+  s.authors = ["Michael Klett", "Nathan Verni", "The Lab Rats @ Phase Two Labs", "Brian Rose"]
+  s.date = %q{2011-2-20}
   s.description = %q{}
   s.email = %q{mklett@grasshopper.com}
   s.extra_rdoc_files = [

data/lib/chargify_api_ares.rb

@@ -63,6 +63,7 @@ module Chargify
 
       Base.site                     = site
       Subscription::Component.site  = site + "/subscriptions/:subscription_id"
+      Subscription::Transaction.site  = site + "/subscriptions/:subscription_id"
     end
   end
   
@@ -78,8 +79,8 @@ module Chargify
   end
   
   class Site < Base
-    def self.clear_data!
-      post(:clear_data)
+    def self.clear_data!(params = {})
+      post(:clear_data, params)
     end
   end
   
@@ -134,15 +135,15 @@ module Chargify
     # 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, :charge => attrs
+      post :charges, {}, attrs.to_xml(:root => :charge)
     end
     
     def credit(attrs = {})
-      post :credits, :credit => attrs
+      post :credits, {}, attrs.to_xml(:root => :credit)
     end
     
     def refund(attrs = {})
-      post :refunds, :refund => attrs
+      post :refunds, {}, attrs.to_xml(:root => :refund)
     end
     
     def reactivate(params = {})
@@ -157,8 +158,9 @@ module Chargify
       post :migrations, :migration => attrs
     end
     
-    def transactions()
-      Transaction.find(:all, :params =>{:subscription_id => self.id})
+    def transactions(params = {})
+      params.merge!(:subscription_id => self.id)
+      Transaction.find(:all, :params => params)
     end
     
     class Component < Base
@@ -167,6 +169,22 @@ module Chargify
         self.component_id
       end
     end
+
+    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.prefix_options[:subscription_id]).refund(attrs)
+      end
+    end
   end
 
   class Product < Base
@@ -212,6 +230,19 @@ module Chargify
   end
   
   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
   
   class PaymentProfile < Base

data/spec/remote/remote_spec.rb

@@ -251,7 +251,7 @@ if run_remote_tests?
         lambda{
           @subscription.credit(:amount => 7, :memo => 'credit')
         }.should change{@subscription.reload.transactions.size}.by(1)
-        @subscription.transactions.first.amount_in_cents.should == 700
+        @subscription.transactions.first.amount_in_cents.should == -700
       end
     end
     
@@ -277,6 +277,50 @@ if run_remote_tests?
         @subscription.transactions.first.amount_in_cents.should == 700
         @subscription.transactions.first.transaction_type.should == 'refund'
       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
+      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
+      end
     end
     
     def already_cleared_site_data?
@@ -467,4 +511,4 @@ if run_remote_tests?
     end
 
   end
-end
+end

metadata

@@ -1,24 +1,24 @@
 --- !ruby/object:Gem::Specification 
 name: chargify_api_ares
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 3
   prerelease: false
   segments: 
   - 0
   - 3
-  - 7
-  version: 0.3.7
+  - 8
+  version: 0.3.8
 platform: ruby
 authors: 
 - Michael Klett
+- Nathan Verni
 - The Lab Rats @ Phase Two Labs
 - Brian Rose
-- Nathan Verni
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2010-11-11 00:00:00 -05:00
+date: 2011-02-20 00:00:00 -05:00
 default_executable: 
 dependencies: []