openpay 1.0.1 → 1.0.2

data/Gemfile

@@ -1,10 +1,15 @@
 source 'https://rubygems.org'
 
+
 gem 'rspec'
-gem 'rest-client'
-gem 'factory_girl'
+gem 'rest-client',  '~>1.6.7'
+gem 'factory_girl' ,  '4.2.0'
 gem 'json_spec'
 
 
+
+
+
 # Specify your gem's dependencies in Openpay.gemspec
 gemspec
+

data/README.md

@@ -1,5 +1,4 @@
-# openpay-ruby
-
+# Openpay-Ruby
 
 ##Description
 
@@ -322,14 +321,6 @@ open_pay_resource.delete_all(customer_id=nil)
 
         customer_cards=cards.all(customer_id)
 
-- all merchant cards
-
-        merchant_cards=cards.all
-
-- all customer card
-
-        customer_cards=cards.all(customer_id)
-
 - delete merchant card
 
         cards.delete(card_id)
@@ -338,11 +329,11 @@ open_pay_resource.delete_all(customer_id=nil)
 
         cards.delete(card_id,customer_id)
 
- -delete all merchant cards
+ - delete all merchant cards
 
         cards.delete_all
 
- -delete all customer cards
+ - delete all customer cards
 
          cards.delete_all(customer_id)
 
@@ -662,7 +653,6 @@ rescue OpenpayApiTransactionError => e
 end
 ```
 
-
 ###These exceptions have the following attributes:
 
 - *category*
@@ -686,12 +676,13 @@ In the Openpay dashboard you are able to see every request and its corresponding
 - bank accounts for merchant cannot be created using the api. It should be done through the dashboard.
 - Is recommended to reset your account using the dashboard when running serious testing (assure clean state)
 - check openpay_ap.rb for Logger configuration
-- travis  https://travis-ci.org/open-pay/openpay-ruby
+- travis  https://travis-ci.org/open-pay/openpay-ruby , if a test fails it will leave leave some records, it may affect posterior tests.
+   it is recommended to reset the console/account to assure a clean state after a failure occurs.
 
 ## More information
 For more use cases take a look at the *test/spec* folder
 
-1.  http://docs.openpay.mx/
+  1.  http://docs.openpay.mx/
 
 
 

data/Rakefile

@@ -1,12 +1,15 @@
-require 'bundler/gem_tasks'
-require 'rspec/core/rake_task'
+ require 'rspec'
+ require 'rspec/core/rake_task'
 
+ task :default => [:spec]
 
+ desc 'run specifications'
+ RSpec::Core::RakeTask.new do |t|
+   t.pattern = 'test/spec/*'
+ end
 
-task :default => [:spec]
 
-desc 'run specifications'
-RSpec::Core::RakeTask.new do |t|
-  t.pattern = 'test/spec/*'
-end
+ RSpec::Core::RakeTask.new('bankaccounts') do |t|
+   t.pattern = 'test/spec/bankaccounts*'
+ end
 

data/lib/OpenPay/Cards.rb

@@ -2,7 +2,6 @@ require 'open_pay_resource'
 
 class Cards < OpenPayResource
 
-
   def list(creation,before,after,offset=0,limit=10)
 
   end
@@ -73,4 +72,4 @@ class Cards < OpenPayResource
   end
 
 
-end
+end

data/lib/OpenPay/Customers.rb

@@ -34,7 +34,6 @@ class Customers < OpenPayResource
       raise OpenpayException.new('This method is not supported on PRODUCTION',false)
     end
     each_bank_account(customer) do |account|
-      warn "deleting bank_account: #{account['id']}"
       delete("#{customer}/bankaccounts/#{account['id']}")
     end
   end

data/lib/OpenPay/Plans.rb

@@ -2,13 +2,10 @@ require 'open_pay_resource'
 
 class Plans  < OpenPayResource
 
-
   def update(plan,plan_id)
     put(plan, "#{plan_id}")
   end
 
-
-
   def each_subscription(plan_id)
     get("#{plan_id}/subscriptions")
   end
@@ -23,6 +20,4 @@ class Plans  < OpenPayResource
   end
 
 
-
-
 end

data/lib/OpenPay/bankaccounts.rb

@@ -1,8 +1,6 @@
 require 'open_pay_resource'
 
 
-
-#TODO change name
 class Bankaccounts < OpenPayResource
 
 
@@ -58,9 +56,4 @@ class Bankaccounts < OpenPayResource
 
 
 
-
-
-
-
-
 end

data/lib/OpenPay/errors/openpay_exception_factory.rb

@@ -5,6 +5,8 @@ class OpenpayExceptionFactory
   def   OpenpayExceptionFactory::create(exception)
 
     LOG.warn("An exception has been raised (original exception class: #{exception.class})")
+    LOG.warn("An exception has been raised (original exception class: #{exception.message })")
+
 
     case exception
 

data/lib/OpenPay/open_pay_resource.rb

@@ -52,7 +52,6 @@ class OpenPayResource
     end
 
     each do |res|
-      warn "deleting #{res}"
       self.delete(res['id'])
     end
 
@@ -136,7 +135,8 @@ class OpenPayResource
 
     LOG.debug("#{self.class.name.downcase}:")
     LOG.debug "   POST URL:#{url(args)}"
-    LOG.debug "   json: #{json}"
+    #For security reasons we keep it hide
+    #LOG.debug "   json: #{json}"
 
     begin
 
@@ -185,7 +185,7 @@ class OpenPayResource
 
 
     LOG.info "PUT URL:#{url}"
-    LOG.info "   json: #{json}"
+    #LOG.info "   json: #{json}"
 
     begin
       res= RestClient::Request.new(

data/lib/OpenPay/openpay_api.rb

@@ -1,26 +1,23 @@
 require 'logger'
-require 'open_pay_resource_factory'
 require 'base64'
 require 'rest-client'
 require 'uri'
-require 'openpay/errors/openpay_exception'
 
+require 'openpay/open_pay_resource_factory'
+require 'errors/openpay_exception'
 
 LOG= Logger.new(STDOUT)
-
 #change to Logger::DEBUG if need trace information
 #due the nature of the information, we recommend to never use a log file when in debug
-LOG.level=Logger::INFO
+LOG.level=Logger::ERROR
 
 
 class OpenpayApi
 
-
   #API Endpoints
   API_DEV='https://sandbox-api.openpay.mx/v1/'
   API_PROD='https://api.openpay.mx'
 
-
   #by default the testing environment is used
   def initialize(merchant_id, private_key,production=false)
     @merchant_id=merchant_id

data/lib/openpay/bankaccounts.rb

@@ -0,0 +1,59 @@
+require 'open_pay_resource'
+
+
+class Bankaccounts < OpenPayResource
+
+
+  def create(bank_account,customer_id)
+    customers=@api_hook.create(:customers)
+    customers.create_bank_account(customer_id,bank_account)
+  end
+
+
+  def get(customer_id='',bank_account=nil)
+    customers=@api_hook.create(:customers)
+
+    if bank_account
+      customers.get_bank_account(customer_id,bank_account)
+    else
+      customers.get_bank_account(customer_id)
+    end
+
+  end
+
+
+  def delete(customer_id,bank_account)
+    customers=@api_hook.create(:customers)
+    customers.delete_bank_account(customer_id,bank_account)
+  end
+
+
+  def each(customer_id)
+    customers=@api_hook.create(:customers)
+     customers.each_bank_account(customer_id)  do |acc|
+       yield acc
+     end
+
+  end
+
+
+  def all(customer_id)
+      customers=@api_hook.create(:customers)
+      customers.all_bank_accounts(customer_id)
+  end
+
+
+  def delete_all(customer_id)
+
+    if env == :production
+      raise OpenpayException.new('This method is not supported on PRODUCTION',false)
+    end
+
+    customers=@api_hook.create(:customers)
+    customers.delete_all_bank_accounts(customer_id)
+
+  end
+
+
+
+end

data/lib/openpay/cards.rb

@@ -0,0 +1,75 @@
+require 'open_pay_resource'
+
+class Cards < OpenPayResource
+
+  def list(creation,before,after,offset=0,limit=10)
+
+  end
+
+
+  def get(card='',customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.get_card(customer_id,card)
+    else
+      super card
+    end
+  end
+
+
+  def create(card,customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.create_card(customer_id,card)
+    else
+      super card
+    end
+  end
+
+
+  def delete(card_id,customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.delete_card(customer_id,card_id)
+    else
+      super card_id
+    end
+  end
+
+
+  def delete_all(customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.delete_all_cards(customer_id)
+    else
+      each do |card|
+        delete(card['id'])
+      end
+    end
+  end
+
+
+  def each(customer_id=nil)
+    if customer_id
+      all(customer_id).each do |card|
+        yield card
+      end
+    else
+        all.each do |card|
+          yield card
+        end
+    end
+  end
+
+
+  def all(customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.all_cards(customer_id)
+    else
+      super   ''
+    end
+  end
+
+
+end

data/lib/openpay/charges.rb

@@ -0,0 +1,77 @@
+require 'open_pay_resource'
+
+class Charges < OpenPayResource
+
+  def all(customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.all_charges(customer_id)
+    else
+      super ''
+    end
+  end
+
+
+  def cancel(transaction_id,customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.cancel_charge(customer_id, transaction_id)
+    else
+         post('', transaction_id+'/cancel')
+     end
+  end
+
+
+  def refund(transaction_id,description,customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.refund_charge(customer_id,transaction_id,description)
+    else
+      post(description, transaction_id+'/refund')
+    end
+  end
+
+
+  def capture(transaction_id,customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.capture_charge(customer_id,transaction_id )
+    else
+      post( '',transaction_id+'/capture')
+    end
+  end
+
+
+  def each(customer_id=nil)
+    if customer_id
+      all(customer_id).each do |card|
+        yield card
+      end
+    else
+      all.each do |card|
+        yield card
+      end
+    end
+  end
+
+
+
+  def get(charge='', customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.get_charge(customer_id, charge)
+    else
+      super charge
+    end
+  end
+
+  def create(charge, customer_id=nil)
+    if customer_id
+      customers=@api_hook.create(:customers)
+      customers.create_charge(customer_id, charge)
+    else
+      super charge
+    end
+  end
+
+end

data/lib/openpay/customers.rb

@@ -0,0 +1,194 @@
+require 'open_pay_resource'
+
+
+
+class Customers < OpenPayResource
+
+
+  #Bankaccount
+  def create_bank_account(customer,bank_account)
+    create(bank_account,"#{customer}/bankaccounts")
+  end
+
+  def get_bank_account(customer,bank_id)
+    get("#{customer}/bankaccounts/#{bank_id}")
+  end
+
+  def all_bank_accounts(customer)
+    get("#{customer}/bankaccounts/")
+  end
+
+  def each_bank_account(customer)
+    get("#{customer}/bankaccounts/").each do |account|
+       yield account
+    end
+  end
+
+
+  def delete_bank_account(customer,bank_id)
+    delete("#{customer}/bankaccounts/#{bank_id}")
+  end
+
+  def delete_all_bank_accounts(customer)
+    if env == :production
+      raise OpenpayException.new('This method is not supported on PRODUCTION',false)
+    end
+    each_bank_account(customer) do |account|
+      delete("#{customer}/bankaccounts/#{account['id']}")
+    end
+  end
+
+
+ #Charges
+  # customers.create_charge(customer_id,charge)
+  def create_charge(customer_id,charge)
+      create(charge,"#{customer_id}/charges")
+  end
+
+  #gets a charge_id for a given customer_id
+  def get_charge(customer_id,charge_id)
+    get("#{customer_id}/charges/#{charge_id}")
+  end
+
+  #return all charges for the given customer_id
+  def all_charges(customer_id)
+    get("#{customer_id}/charges/")
+  end
+
+  def cancel_charge(customer_id,charge_id)
+    post(charge_id,"#{customer_id}/charges/#{charge_id}/cancel")
+  end
+
+  def refund_charge(customer_id,charge_id,description)
+    post(description,"#{customer_id}/charges/#{charge_id}/refund")
+  end
+
+  def capture_charge(customer_id,charge_id)
+      post('',"#{customer_id}/charges/#{charge_id}/capture")
+  end
+
+
+
+  #Payouts
+  def create_payout(customer_id,payout)
+    post(payout,"#{customer_id}/payouts")
+  end
+
+  def all_payouts(customer_id)
+   get("#{customer_id}/payouts")
+  end
+
+  def get_payout(customer_id,payout_id)
+    get("#{customer_id}/payouts/#{payout_id}")
+  end
+
+  def each_payout(customer_id)
+     all_payouts(customer_id).each do |pay|
+       yield pay
+     end
+  end
+
+
+
+
+  #Transfers
+  def create_transfer(customer_id,transfer)
+    post(transfer,"#{customer_id}/transfers")
+  end
+
+  def all_transfers(customer_id)
+    get("#{customer_id}/transfers/")
+  end
+
+  def get_transfer(customer_id,transfer_id)
+    get("#{customer_id}/transfers/#{transfer_id}")
+  end
+
+  def each_transfer(customer_id)
+    all_transfers(customer_id).each do |trans|
+      yield trans
+    end
+  end
+
+
+
+
+  #Subscriptions
+  def create_subscription(subscription, plan_id)
+    post(subscription,"#{plan_id}/subscriptions")
+  end
+
+
+  def delete_subscription(customer_id,subscription_id)
+    delete("#{customer_id}/subscriptions/#{subscription_id}")
+  end
+
+
+  def get_subscription(customer_id,subscription_id)
+    get("#{customer_id}/subscriptions/#{subscription_id}")
+  end
+
+
+  def all_subscriptions(customer_id)
+    get("#{customer_id}/subscriptions/")
+  end
+
+  def each_subscription(customer_id)
+    all_subscriptions(customer_id).each do |cust|
+        yield cust
+    end
+  end
+
+
+  def delete_all_subscriptions(customer_id)
+    if env == :production
+      raise OpenPayError ('This method is not supported on PRODUCTION')
+    end
+    all_subscriptions(customer_id).each do |sub|
+      delete_subscription(customer_id,sub['id'])
+    end
+  end
+
+
+
+
+  #Card
+  def create_card(customer,card)
+       create(card,"#{customer}/cards")
+  end
+
+
+  def get_card(customer,card_id)
+    get("#{customer}/cards/#{card_id}")
+  end
+
+
+  def delete_card(customer,card_id)
+      delete("#{customer}/cards/#{card_id}")
+  end
+
+
+  def delete_all_cards(customer_id)
+    if env == :production
+      raise OpenPayError ('This method is not supported on PRODUCTION')
+    end
+    each_card(customer_id) do |card|
+      delete_card(customer_id,card['id'])
+    end
+  end
+
+
+  def each_card(customer)
+       get("#{customer}/cards").each do |card|
+           yield card
+       end
+  end
+
+
+  def all_cards(customer)
+    get("#{customer}/cards")
+  end
+
+
+
+end

data/lib/openpay/errors/openpay_connection_exception.rb

@@ -0,0 +1,3 @@
+require 'openpay/errors/openpay_exception'
+class OpenpayConnectionException < OpenpayException
+end

data/lib/openpay/errors/openpay_exception.rb

@@ -0,0 +1,29 @@
+class OpenpayException < StandardError
+
+  attr_reader :description
+  attr_reader :http_code
+  attr_reader :http_body
+  attr_reader :json_body
+  attr_reader :error_code
+  attr_reader :category
+
+  def initialize(message=nil,json_message=true)
+  #openpay errors  have a json error string
+   if json_message
+      json= JSON.parse message
+      @description = json['description']
+      @http_code =   json['http_code']
+      @error_code = json['error_code']
+      @json_body = message
+     @category   = json['category']
+   #other errors may or not have a json error string, so this case is for them
+   else
+     @description = message
+     @http_code  =   ''
+     @http_body  =  ''
+     @json_body  =  ''
+     @error_code = ''
+    end
+  end
+
+end