openpay 1.0.7 → 2.0.1

data/test/spec/bankaccounts_spec.rb

@@ -10,6 +10,8 @@ describe Bankaccounts do
     @merchant_id='mywvupjjs9xdnryxtplq'
     @private_key='sk_92b25d3baec149e6b428d81abfe37006'
 
+    #LOG.level=Logger::DEBUG
+
     @openpay=OpenpayApi.new(@merchant_id, @private_key)
     @bank_accounts=@openpay.create(:bankaccounts)
     @customers=@openpay.create(:customers)
@@ -21,18 +23,43 @@ describe Bankaccounts do
       expect(@bank_accounts).to respond_to(meth)
     end
   end
+  describe '.list' do
+
+    it 'list the bank accounts using a creation_gte filter' do
+
+      customer_hash= FactoryBot.build(:customer)
+      customer=@customers.create(customer_hash)
+      expect(@bank_accounts.all(customer['id']).size).to be 0
+
+      account_hash=FactoryBot.build(:bank_account)
+      bank=@bank_accounts.create(account_hash, customer['id'])
+      expect(@bank_accounts.all(customer['id']).size).to be 1
+
+      search_params = OpenpayUtils::SearchParams.new
+      search_params.limit = 1
+      search_params.creation_gte = '2000-01-01'
 
+      expect(@bank_accounts.all(customer['id']).size).to eq 1
+      expect(@bank_accounts.list(search_params , customer['id']).size).to eq 1
+
+      @bank_accounts.delete(customer['id'], bank['id'])
+      @customers.delete(customer['id'])
+
+    end
+  end
+=begin
   describe '.create' do
 
     it 'creates  a customer bank account' do
-      customer_hash= FactoryGirl.build(:customer)
+      
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
-
-      account_hash=FactoryGirl.build(:bank_account)
+       
+      account_hash=FactoryBot.build(:bank_account)
       bank=@bank_accounts.create(account_hash, customer['id'])
 
       bank_account=@bank_accounts.get(customer['id'], bank['id'])
-      expect(bank_account['alias']).to match 'Cuenta principal'
+      expect(bank_account['alias']).to match "Cuenta principal"
 
       @bank_accounts.delete(customer['id'], bank['id'])
       @customers.delete(customer['id'])
@@ -40,15 +67,14 @@ describe Bankaccounts do
     end
 
   end
-
   describe '.get' do
 
     it 'get a given bank account for a given customer' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
-      account_hash=FactoryGirl.build(:bank_account)
+      account_hash=FactoryBot.build(:bank_account)
       bank=@bank_accounts.create(account_hash, customer['id'])
 
       bank_account=@bank_accounts.get(customer['id'], bank['id'])
@@ -64,10 +90,10 @@ describe Bankaccounts do
 
     it 'iterator for  all given customer bank accounts' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
-      account_hash=FactoryGirl.build(:bank_account)
+      account_hash=FactoryBot.build(:bank_account)
       bank=@bank_accounts.create(account_hash, customer['id'])
 
       @bank_accounts.each(customer['id']) do |bank_account|
@@ -85,11 +111,11 @@ describe Bankaccounts do
 
     it 'list the bank accounts using a given filter' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
       expect(@bank_accounts.all(customer['id']).size).to be 0
 
-      account_hash=FactoryGirl.build(:bank_account)
+      account_hash=FactoryBot.build(:bank_account)
       bank=@bank_accounts.create(account_hash, customer['id'])
       expect(@bank_accounts.all(customer['id']).size).to be 1
 
@@ -109,11 +135,11 @@ describe Bankaccounts do
 
     it 'all bank accounts for a given customer' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
       expect(@bank_accounts.all(customer['id']).size).to be 0
 
-      account_hash=FactoryGirl.build(:bank_account)
+      account_hash=FactoryBot.build(:bank_account)
       bank=@bank_accounts.create(account_hash, customer['id'])
       expect(@bank_accounts.all(customer['id']).size).to be 1
 
@@ -132,11 +158,11 @@ describe Bankaccounts do
 
     it 'deletes a given bank account' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
       expect(@bank_accounts.all(customer['id']).size).to be 0
 
-      account_hash=FactoryGirl.build(:bank_account)
+      account_hash=FactoryBot.build(:bank_account)
       bank=@bank_accounts.create(account_hash, customer['id'])
       expect(@bank_accounts.all(customer['id']).size).to be 1
 
@@ -155,10 +181,10 @@ describe Bankaccounts do
 
     it 'deletes all bank accounts' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
-      account_hash=FactoryGirl.build(:bank_account)
+      account_hash=FactoryBot.build(:bank_account)
 
       @bank_accounts.create(account_hash, customer['id'])
       expect(@bank_accounts.all(customer['id']).size).to be 1
@@ -177,4 +203,5 @@ describe Bankaccounts do
     end
 
   end
+=end
 end

data/test/spec/cards_spec.rb

@@ -10,6 +10,8 @@ describe Cards do
 
     @merchant_id='mywvupjjs9xdnryxtplq'
     @private_key='sk_92b25d3baec149e6b428d81abfe37006'
+    
+    #LOG.level=Logger::DEBUG
 
     @openpay=OpenpayApi.new(@merchant_id, @private_key)
     @cards=@openpay.create(:cards)
@@ -34,7 +36,7 @@ describe Cards do
     it 'creates merchant card' do
 
       #creates merchant card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       cards=@cards.create(card_hash)
       expect(cards).to be_a(Hash)
 
@@ -54,9 +56,9 @@ describe Cards do
     it 'creates a customer card' do
 
       #creates a customer
-      card_hash = FactoryGirl.build(:valid_card, holder_name: 'Pepe')
+      card_hash = FactoryBot.build(:valid_card, holder_name: 'Pepe')
       customers=@openpay.create(:customers)
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer=customers.create(customer_hash)
 
       #creates a customer card
@@ -80,35 +82,33 @@ describe Cards do
 
     end
 
-    it 'fails when trying to create an existing merchant card' do
+    it 'create an existing merchant card' do
 
       #creates merchant card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
+      
+      card=@cards.create(card_hash)
+      #cleanup
+      @cards.delete(card['id'])
+      
       card=@cards.create(card_hash)
-
-      #perform check
-      expect { @cards.create(card_hash) }.to raise_error(OpenpayTransactionException)
-
       #cleanup
       @cards.delete(card['id'])
 
     end
 
-    it 'fails when trying to create an existing customer card' do
+    it 'trying to create an existing customer card' do
 
       #creates a customer
-      card_hash = FactoryGirl.build(:valid_card, holder_name: 'Pepe')
+      card_hash = FactoryBot.build(:valid_card, holder_name: 'Pepe')
       customers=@openpay.create(:customers)
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer=customers.create(customer_hash)
 
       #creates a customer card
       card=@cards.create(card_hash, customer['id'])
       expect(card).to be_a(Hash)
 
-      #performs check
-      expect { @cards.create(card_hash, customer['id']) }.to raise_error(OpenpayTransactionException)
-
       #cleanup
       @cards.delete(card['id'], customer['id'])
       @customers.delete(customer['id'])
@@ -116,7 +116,7 @@ describe Cards do
     end
 
     it 'fails when using an expired card' do
-      card_hash = FactoryGirl.build(:expired_card)
+      card_hash = FactoryBot.build(:expired_card)
       #check
       expect { @cards.create(card_hash) }.to raise_error(OpenpayTransactionException)
       #extended check
@@ -130,7 +130,7 @@ describe Cards do
     end
 
     it 'fails when using a stolen card' do
-      card_json = FactoryGirl.build(:valid_card, card_number: '4000000000000119')
+      card_json = FactoryBot.build(:valid_card, card_number: '4000000000000119')
       expect { @cards.create(card_json) }.to raise_error(OpenpayTransactionException)
     end
 
@@ -140,16 +140,16 @@ describe Cards do
 
     it 'iterates over all existing merchant cards' do
       @cards.each do |card|
-        expect(card['expiration_year']).to match '14'
+        expect(card['expiration_year']).to match '20'
       end
     end
 
     it 'iterates over all existing customer cards' do
 
       #creates a customer
-      card_hash = FactoryGirl.build(:valid_card, holder_name: 'Pepe')
+      card_hash = FactoryBot.build(:valid_card, holder_name: 'Pepe')
       customers=@openpay.create(:customers)
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer=customers.create(customer_hash)
 
       #creates a customer card
@@ -157,7 +157,7 @@ describe Cards do
       expect(card).to be_a(Hash)
 
       @cards.each(customer['id']) do |c|
-        expect(c['expiration_year']).to match '14'
+        expect(c['expiration_year']).to match '25'
       end
 
       #cleanup
@@ -173,7 +173,7 @@ describe Cards do
     it 'deletes a merchant card' do
 
       #creates merchant card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       cards=@cards.create(card_hash)
       expect(cards).to be_a(Hash)
 
@@ -194,11 +194,11 @@ describe Cards do
 
       #create customer
       customers=@openpay.create(:customers)
-      customer_hash = FactoryGirl.build(:customer, name: 'Juan', last_name: 'Paez')
+      customer_hash = FactoryBot.build(:customer, name: 'Juan', last_name: 'Paez')
       customer=customers.create(customer_hash)
 
       #create customer card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card=@cards.create(card_hash, customer['id'])
 
       #delete card
@@ -221,7 +221,7 @@ describe Cards do
 
       #create merchant card
       bank_name='Banamex'
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
 
       card=@cards.create(card_hash)
       id=card['id']
@@ -243,12 +243,12 @@ describe Cards do
     it ' gets an existing customer card' do
 
       #create customer
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #creates card
       bank_name='Banamex'
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card=@cards.create(card_hash, customer['id'])
       id=card['id']
 
@@ -278,7 +278,7 @@ describe Cards do
       expect(@cards.all.size).to be 0
 
       #create merchant card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card=@cards.create(card_hash)
       id=card['id']
 
@@ -293,14 +293,14 @@ describe Cards do
     it 'all customer cards' do
 
       #create customer
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #check initial state
       expect(@cards.all(customer['id']).size).to be 0
 
       #create customer card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card=@cards.create(card_hash, customer['id'])
       id=card['id']
 
@@ -323,10 +323,10 @@ describe Cards do
     it 'list the merchant cards using a filter' do
 
       #create merchant card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card1 = @cards.create(card_hash)
 
-      card_hash = FactoryGirl.build(:valid_card2)
+      card_hash = FactoryBot.build(:valid_card2)
       card2 = @cards.create(card_hash)
 
       search_params = OpenpayUtils::SearchParams.new
@@ -338,18 +338,18 @@ describe Cards do
     it 'list the customer cards using a filter' do
 
       #create customer
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer = @customers.create(customer_hash)
 
       #creates card
       bank_name ='Banamex'
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card = @cards.create(card_hash, customer['id'])
       id = card['id']
 
       #creates card 2
       bank_name = 'Bancomer'
-      card_hash = FactoryGirl.build(:valid_card2)
+      card_hash = FactoryBot.build(:valid_card2)
       card = @cards.create(card_hash, customer['id'])
       id = card['id']
 
@@ -363,6 +363,19 @@ describe Cards do
 
     end
 
+    it 'list the merchant cards using a filter creation and amount' do
+      #create merchant card
+      card_hash = FactoryBot.build(:valid_card)
+      card1 = @cards.create(card_hash)
+      card_hash = FactoryBot.build(:valid_card2)
+      card2 = @cards.create(card_hash)
+      search_params = OpenpayUtils::SearchParams.new
+      search_params.limit = 1
+      creation_date = "2013-11-01"
+      search_params.creation_gte = creation_date
+      expect(@cards.list(search_params).size).to eq 1
+      @cards.delete_all
+    end
   end
 
 
@@ -379,11 +392,11 @@ describe Cards do
     it 'deletes all existing  merchant cards' do
 
       #create merchant card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       @cards.create(card_hash)
 
       #using json just for fun ...and test
-      card2_json = FactoryGirl.build(:valid_card2).to_json
+      card2_json = FactoryBot.build(:valid_card2).to_json
       @cards.create(card2_json)
 
       #perform check
@@ -400,14 +413,14 @@ describe Cards do
     it 'deletes all existing cards for a given customer' do
 
       #creates customer
-      customer_hash = FactoryGirl.build(:customer)
+      customer_hash = FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #check initial state
       expect(@cards.all(customer['id']).size).to be 0
 
       #create card
-      card_hash = FactoryGirl.build(:valid_card)
+      card_hash = FactoryBot.build(:valid_card)
       card=@cards.create(card_hash, customer['id'])
 
       #perform check

data/test/spec/charges_spec.rb

@@ -9,6 +9,8 @@ describe Charges do
 
     @openpay=OpenpayApi.new(@merchant_id,@private_key)
     @customers=@openpay.create(:customers)
+    
+    #LOG.level=Logger::DEBUG
 
     @charges=@openpay.create(:charges)
     @cards=@openpay.create(:cards)
@@ -23,17 +25,17 @@ describe Charges do
       expect(@charges).to respond_to(meth)
     end
   end
-
+  
   describe '.create' do
 
     it 'creates a new merchant charge using the card method using a pre-stored card' do
 
       #create card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash)
 
       #create charge attached to prev created card
-      charge_hash=FactoryGirl.build(:card_charge, source_id: card['id'], order_id: card['id'],amount: 101)
+      charge_hash=FactoryBot.build(:card_charge, source_id: card['id'], order_id: card['id'],amount: 101)
       charge=@charges.create(charge_hash)
 
       #perform check
@@ -48,15 +50,15 @@ describe Charges do
     it 'creates a new customer charge using the card method using a pre-stored card' do
 
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
 
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'])
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'])
       charge=@charges.create(charge_hash,customer['id'])
 
       #perform check
@@ -69,30 +71,6 @@ describe Charges do
 
     end
 
-    it 'creates a new customer charge using the bank_account method' do
-
-      #create new customer
-      customer_hash= FactoryGirl.build(:customer)
-      customer=@customers.create(customer_hash)
-
-      #create bank account
-      account_hash=FactoryGirl.build(:bank_account)
-      account=@bank_accounts.create(account_hash,customer['id'])
-
-      #create charge
-      charge_hash=FactoryGirl.build(:bank_charge, order_id: account['id'])
-      charge=@charges.create(charge_hash,customer['id'])
-
-      #perform check
-      stored_charge=@charges.get(charge['id'],customer['id'])
-      expect(stored_charge['amount']).to be_within(0.1).of(10000)
-
-      #clean up
-      @bank_accounts.delete(customer['id'],account['id'])
-      @customers.delete(customer['id'])
-
-    end
-
   end
 
   describe '.get' do
@@ -100,11 +78,11 @@ describe Charges do
     it 'gets a merchant charge'  do
 
       #create card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash)
 
       #create charge attached to prev created card
-      charge_hash=FactoryGirl.build(:card_charge, source_id: card['id'], order_id: card['id'],amount: 505)
+      charge_hash=FactoryBot.build(:card_charge, source_id: card['id'], order_id: card['id'],amount: 505)
       charge=@charges.create(charge_hash)
 
       #perform check
@@ -119,15 +97,15 @@ describe Charges do
     it 'gets a customer charge' do
 
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
 
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000)
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000)
       charge=@charges.create(charge_hash,customer['id'])
 
       #perform check
@@ -151,7 +129,7 @@ describe Charges do
 
     it 'all customer charges' do
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       expect(@charges.all(customer['id']).size).to be 0
@@ -166,15 +144,15 @@ describe Charges do
     it 'captures a merchant card charge'  do
 
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create merchant card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash)
 
       #create merchant charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000, capture:'false')
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000, capture:'false')
       charge=@charges.create(charge_hash)
 
       #capture merchant charge
@@ -188,15 +166,15 @@ describe Charges do
 
     it 'captures a customer card charge' do
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
 
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000,capture:'false')
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000,capture:'false')
       charge=@charges.create(charge_hash,customer['id'])
 
       #capture customer charge
@@ -215,15 +193,15 @@ describe Charges do
     it 'confirms a capture on a merchant charge' do
 
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create merchant card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash)
 
       #create merchant charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000, capture:'false')
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000, capture:'false')
       charge=@charges.create(charge_hash)
 
       confirm_capture_options = {  transaction_id: charge['id'], amount: 100  }
@@ -240,15 +218,15 @@ describe Charges do
 
     it 'confirms a capture on a customer charge' do
 
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
 
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000,capture:'false')
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000,capture:'false')
       charge=@charges.create(charge_hash,customer['id'])
 
       confirm_capture_options = { customer_id: customer['id'], transaction_id: charge['id'], amount: 100  }
@@ -270,15 +248,15 @@ describe Charges do
     #Refunds apply only for card charges
     it 'refunds  an existing merchant charge' do
       #create card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash)
 
       #create charge attached to prev created card
-      charge_hash=FactoryGirl.build(:card_charge, source_id: card['id'], order_id: card['id'],amount: 505)
+      charge_hash=FactoryBot.build(:card_charge, source_id: card['id'], order_id: card['id'],amount: 505)
       charge=@charges.create(charge_hash)
 
       #creates refund_description
-      refund_description=FactoryGirl.build(:refund_description)
+      refund_description=FactoryBot.build(:refund_description)
       expect(@charges.get(charge['id'])['refund']).to be nil
 
       @charges.refund(charge['id'],refund_description)
@@ -288,35 +266,36 @@ describe Charges do
       @cards.delete(card['id'])
 
     end
-
+    
     it 'refunds an existing customer charge' do
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
 
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4000)
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 100)
       charge=@charges.create(charge_hash,customer['id'])
 
       #creates refund_description
-      refund_description=FactoryGirl.build(:refund_description)
-
+      refund_description=FactoryBot.build(:refund_description)
 
       #perform check
       expect(@charges.get(charge['id'],customer['id'])['refund']).to be nil
+      sleep(50)
       @charges.refund(charge['id'],refund_description,customer['id'])
-      expect(@charges.get(charge['id'],customer['id'])['refund']['amount'] ).to be_within(0.1).of(4000)
+      
+      expect(@charges.get(charge['id'],customer['id'])['refund']['amount'] ).to be_within(0.1).of(100)
 
       #clean up
       @cards.delete(card['id'],customer['id'])
       @customers.delete(customer['id'])
 
     end
-
+    
   end
 
   describe '.list' do
@@ -324,25 +303,23 @@ describe Charges do
     it 'list customer charges' do
 
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
-
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
-
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'])
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'])
       charge=@charges.create(charge_hash,customer['id'])
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id']+"1")
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id']+"1")
       charge2=@charges.create(charge_hash,customer['id'])
-
       #perform check
       search_params = OpenpayUtils::SearchParams.new
       search_params.limit = 1
+      search_params.creation_gte = '2000-01-01'
+      search_params.amount_gte = 1
       expect(@charges.all(customer['id']).size).to eq 2
       expect(@charges.list(search_params,customer['id']).size).to eq 1
-
       #clean up
       @cards.delete(card['id'],customer['id'])
       @customers.delete(customer['id'])
@@ -363,17 +340,17 @@ describe Charges do
     it 'iterate over customer charges' do
 
       #create new customer
-      customer_hash= FactoryGirl.build(:customer)
+      customer_hash= FactoryBot.build(:customer)
       customer=@customers.create(customer_hash)
 
       #create customer card
-      card_hash=FactoryGirl.build(:valid_card)
+      card_hash=FactoryBot.build(:valid_card)
       card=@cards.create(card_hash,customer['id'])
 
       #create charge
-      charge_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4)
+      charge_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'],amount: 4)
       @charges.create(charge_hash,customer['id'])
-      charge2_hash=FactoryGirl.build(:card_charge, source_id:card['id'],order_id: card['id'+"2"],amount: 4)
+      charge2_hash=FactoryBot.build(:card_charge, source_id:card['id'],order_id: card['id'+"2"],amount: 4)
 
       @charges.create(charge2_hash,customer['id'])