killbill-litle 1.3.0 → 1.4.0

data/Jarfile

@@ -1,5 +1,5 @@
 jar 'com.ning.billing:killbill-api', '0.3.0'
 jar 'com.ning.billing.plugin:killbill-plugin-api-notification', '0.2.4'
 jar 'com.ning.billing.plugin:killbill-plugin-api-payment', '0.2.4'
-jar 'com.ning.billing:killbill-util:tests', '0.2.6-SNAPSHOT'
+jar 'com.ning.billing:killbill-util:tests', '0.3.0'
 jar 'javax.servlet:javax.servlet-api', '3.0.1'

data/VERSION

@@ -1 +1 @@
-1.3.0
+1.4.0

data/killbill-litle.gemspec

@@ -22,7 +22,7 @@ Gem::Specification.new do |s|
 
   s.rdoc_options << '--exclude' << '.'
 
-  s.add_dependency 'killbill', '~> 1.1.2'
+  s.add_dependency 'killbill', '~> 1.2.0'
   s.add_dependency 'activemerchant', '~> 2.0.0'
   s.add_dependency 'activerecord', '~> 3.2.1'
   s.add_dependency 'sinatra', '~> 1.3.4'

data/lib/litle/api.rb

@@ -13,7 +13,9 @@ module Killbill::Litle
       ActiveRecord::Base.connection.close
     end
 
-    def process_payment(kb_account_id, kb_payment_id, kb_payment_method_id, amount_in_cents, currency, call_context = nil, options = {})
+    def process_payment(kb_account_id, kb_payment_id, kb_payment_method_id, amount, currency, call_context = nil, options = {})
+      amount_in_cents = (amount * 100).to_i
+
       # If the payment was already made, just return the status
       # TODO Should we set the Litle Id field to check for dups (https://www.litle.com/mc-secure/DupeChecking_V1.2.pdf)?
       litle_transaction = LitleTransaction.from_kb_payment_id(kb_payment_id) rescue nil
@@ -45,7 +47,9 @@ module Killbill::Litle
       litle_transaction.litle_response.to_payment_response
     end
 
-    def process_refund(kb_account_id, kb_payment_id, amount_in_cents, currency, call_context = nil, options = {})
+    def process_refund(kb_account_id, kb_payment_id, amount, currency, call_context = nil, options = {})
+      amount_in_cents = (amount * 100).to_i
+
       litle_transaction = LitleTransaction.find_candidate_transaction_for_refund(kb_payment_id, amount_in_cents)
 
       # Set a default report group

data/lib/litle/models/litle_response.rb

@@ -105,7 +105,7 @@ module Killbill::Litle
 
       if type == :payment
         p_info_plugin = Killbill::Plugin::Model::PaymentInfoPlugin.new
-        p_info_plugin.amount = amount_in_cents
+        p_info_plugin.amount = BigDecimal.new(amount_in_cents.to_s) / 100.0 if amount_in_cents
         p_info_plugin.created_date = created_date
         p_info_plugin.effective_date = effective_date
         p_info_plugin.status = (success ? :PROCESSED : :ERROR)
@@ -116,7 +116,7 @@ module Killbill::Litle
         p_info_plugin
       else
         r_info_plugin = Killbill::Plugin::Model::RefundInfoPlugin.new
-        r_info_plugin.amount = amount_in_cents
+        r_info_plugin.amount = BigDecimal.new(amount_in_cents.to_s) / 100.0 if amount_in_cents
         r_info_plugin.created_date = created_date
         r_info_plugin.effective_date = effective_date
         r_info_plugin.status = (success ? :PROCESSED : :ERROR)

data/lib/litle.rb

@@ -1,5 +1,6 @@
 require 'active_record'
 require 'activemerchant'
+require 'bigdecimal'
 require 'pathname'
 require 'sinatra'
 require 'singleton'

data/pom.xml

@@ -25,7 +25,7 @@
     <groupId>com.ning.killbill.ruby</groupId>
     <artifactId>litle-plugin</artifactId>
     <packaging>pom</packaging>
-    <version>1.3.0</version>
+    <version>1.4.0</version>
     <name>litle-plugin</name>
     <url>http://github.com/killbill/killbill-litle-plugin</url>
     <description>Plugin for accessing Litle as a payment gateway</description>

data/spec/litle/integration_spec.rb

@@ -1,5 +1,4 @@
 require 'spec_helper'
-require 'logger'
 
 ActiveMerchant::Billing::Base.mode = :test
 
@@ -54,12 +53,12 @@ describe Killbill::Litle::PaymentPlugin do
 
   it 'should be able to charge and refund' do
     pm = create_payment_method
-    amount_in_cents = 10000
+    amount = BigDecimal.new("100")
     currency = 'USD'
     kb_payment_id = SecureRandom.uuid
 
-    payment_response = @plugin.process_payment pm.kb_account_id, kb_payment_id, pm.kb_payment_method_id, amount_in_cents, currency
-    payment_response.amount.should == amount_in_cents
+    payment_response = @plugin.process_payment pm.kb_account_id, kb_payment_id, pm.kb_payment_method_id, amount, currency
+    payment_response.amount.should == amount
     payment_response.status.should == :PROCESSED
 
     # Verify our table directly
@@ -70,14 +69,14 @@ describe Killbill::Litle::PaymentPlugin do
     response.params_litleonelineresponse_saleresponse_order_id.should == Killbill::Litle::Utils.compact_uuid(kb_payment_id)
 
     payment_response = @plugin.get_payment_info pm.kb_account_id, kb_payment_id
-    payment_response.amount.should == amount_in_cents
+    payment_response.amount.should == amount
     payment_response.status.should == :PROCESSED
 
     # Check we cannot refund an amount greater than the original charge
-    lambda { @plugin.process_refund pm.kb_account_id, kb_payment_id, amount_in_cents + 1, currency }.should raise_error RuntimeError
+    lambda { @plugin.process_refund pm.kb_account_id, kb_payment_id, amount + 1, currency }.should raise_error RuntimeError
 
-    refund_response = @plugin.process_refund pm.kb_account_id, kb_payment_id, amount_in_cents, currency
-    refund_response.amount.should == amount_in_cents
+    refund_response = @plugin.process_refund pm.kb_account_id, kb_payment_id, amount, currency
+    refund_response.amount.should == amount
     refund_response.status.should == :PROCESSED
 
     # Verify our table directly
@@ -88,15 +87,15 @@ describe Killbill::Litle::PaymentPlugin do
     # Check we can retrieve the refund
     refund_response = @plugin.get_refund_info pm.kb_account_id, kb_payment_id
     # Apparently, Litle returns positive amounts for refunds
-    refund_response.amount.should == amount_in_cents
+    refund_response.amount.should == amount
     refund_response.status.should == :PROCESSED
 
     # Make sure we can charge again the same payment method
-    second_amount_in_cents = 29471
+    second_amount = BigDecimal.new("294.71")
     second_kb_payment_id = SecureRandom.uuid
 
-    payment_response = @plugin.process_payment pm.kb_account_id, second_kb_payment_id, pm.kb_payment_method_id, second_amount_in_cents, currency
-    payment_response.amount.should == second_amount_in_cents
+    payment_response = @plugin.process_payment pm.kb_account_id, second_kb_payment_id, pm.kb_payment_method_id, second_amount, currency
+    payment_response.amount.should == second_amount
     payment_response.status.should == :PROCESSED
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: killbill-litle
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.0
   prerelease:
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2013-06-21 00:00:00.000000000 Z
+date: 2013-06-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: killbill
@@ -17,13 +17,13 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.2
+        version: 1.2.0
     none: false
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.2
+        version: 1.2.0
     none: false
   prerelease: false
   type: :runtime