boletosimples 0.4.1 → 1.0.2

data/lib/boletosimples/resources/bank_billet_discharge.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class BankBilletDischarge < BaseModel
+  end
+end

data/lib/boletosimples/resources/bank_billet_payment.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class BankBilletPayment < BaseModel
+  end
+end

data/lib/boletosimples/resources/bank_billet_remittance.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class BankBilletRemittance < BaseModel
+  end
+end

data/lib/boletosimples/resources/base_model.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class BaseModel
     include Her::Model
@@ -5,4 +7,4 @@ module BoletoSimples
     parse_root_in_json true
     include_root_in_json true
   end
-end
+end

data/lib/boletosimples/resources/customer.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Customer < BaseModel
   end
-end
+end

data/lib/boletosimples/resources/customer_import.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class CustomerImport < BaseModel
+    collection_path 'imports/customers'
+    resource_path 'imports/customers/:id'
+  end
+end

data/lib/boletosimples/resources/customer_subscription.rb

@@ -1,14 +1,7 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class CustomerSubscription < BaseModel
-
-    def next_charge
-      self.class.request(:_method => :post, :_path => self.class.build_request_path('customer_subscriptions/:id/next_charge', {self.class.primary_key => id})) do |parsed_data, response|
-        assign_attributes(self.class.parse(parsed_data[:data])) if parsed_data[:data].any?
-        @metadata = parsed_data[:metadata]
-        @response_errors = parsed_data[:errors]
-        @response = response
-      end
-      return @response.success?
-    end
+    custom_post :next_charge
   end
 end

data/lib/boletosimples/resources/customer_subscription_import.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class CustomerSubscriptionImport < BaseModel
+    collection_path 'imports/customer_subscriptions'
+    resource_path 'imports/customer_subscriptions/:id'
+  end
+end

data/lib/boletosimples/resources/discharge.rb

@@ -1,14 +1,7 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Discharge < BaseModel
-    
-    def pay_off
-      self.class.request(:_method => :put, :_path => self.class.build_request_path('discharges/:id/pay_off', {self.class.primary_key => id})) do |parsed_data, response|
-        assign_attributes(self.class.parse(parsed_data[:data])) if parsed_data[:data].any?
-        @metadata = parsed_data[:metadata]
-        @response_errors = parsed_data[:errors]
-        @response = response
-      end
-      return @response.success?
-    end
+    custom_put :pay_off
   end
-end
+end

data/lib/boletosimples/resources/email_delivery.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class EmailDelivery < BaseModel
+  end
+end

data/lib/boletosimples/resources/event.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Event < BaseModel
   end
-end
+end

data/lib/boletosimples/resources/installment.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Installment < BaseModel
   end
-end
+end

data/lib/boletosimples/resources/remittance.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Remittance < BaseModel
   end
-end
+end

data/lib/boletosimples/resources/sms_delivery.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module BoletoSimples
+  class SmsDelivery < BaseModel
+  end
+end

data/lib/boletosimples/resources/transaction.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Transaction < BaseModel
   end
-end
+end

data/lib/boletosimples/resources/webhook.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class Webhook < BaseModel
   end
-end
+end

data/lib/boletosimples/resources/webhook_delivery.rb

@@ -1,4 +1,6 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class WebhookDelivery < BaseModel
   end
-end
+end

data/lib/boletosimples/response_error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   # BoletoSimples::ResponseError
   # Exception that gets raised if the response is an error (4xx or 5xx)
@@ -29,15 +31,18 @@ module BoletoSimples
       @status = response[:status].to_i
       @method = response[:method].to_s.upcase
       @url = response[:url]
-      @error_message = @body[:error]
+
+      errors = response[:body][:errors]
+      @error_message = errors.first[:title] unless errors.blank?
 
       super
     end
 
     def to_s
-      msg = "#{status} #{method} #{url}"
+      msg = ''
+      msg += "#{status} #{method} #{url}"
       msg << " (#{error_message})" unless error_message.blank?
       msg
     end
   end
-end
+end

data/lib/boletosimples/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module BoletoSimples
-  VERSION = '0.4.1'
+  VERSION = '1.0.2'
 end

data/spec/boletosimples/configuration_spec.rb

@@ -1,84 +1,72 @@
-# encoding: UTF-8
-require 'spec_helper'
+# # frozen_string_literal: true
 
-RSpec.describe BoletoSimples::Configuration do
-  describe 'defaults' do
-    before {
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_ENV').and_return(nil)
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_APP_ID').and_return(nil)
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_APP_SECRET').and_return(nil)
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_ACCESS_TOKEN').and_return(nil)
-    }
-    subject { BoletoSimples::Configuration.new }
-    it { expect(subject.environment).to eq(:sandbox) }
-    it { expect(subject.base_uri).to eq('https://sandbox.boletosimples.com.br/api/v1') }
-    it { expect(subject.application_id).to be_nil }
-    it { expect(subject.application_secret).to be_nil }
-    it { expect(subject.access_token).to be_nil }
-    it { expect(subject.cache).to be_nil }
-    it { expect(subject).not_to be_access_token }
-  end
-  describe 'environment variables' do
-    before {
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_ENV').and_return('production')
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_APP_ID').and_return('app-id')
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_APP_SECRET').and_return('app-secret')
-      allow(ENV).to receive(:[]).with('BOLETOSIMPLES_ACCESS_TOKEN').and_return('access-token')
-    }
-    subject { BoletoSimples::Configuration.new }
-    it { expect(subject.environment).to eq(:production) }
-    it { expect(subject.base_uri).to eq('https://boletosimples.com.br/api/v1') }
-    it { expect(subject.application_id).to eq('app-id') }
-    it { expect(subject.application_secret).to eq('app-secret') }
-    it { expect(subject.access_token).to eq('access-token') }
-    it { expect(subject).to be_access_token }
-    describe 'cache' do
-      it { expect(subject.cache).to be_nil }
-      it { expect(Her::API.default_api.connection.builder.handlers).not_to include(Faraday::HttpCache) }
-    end
-  end
-  describe 'configuration' do
-    let(:cache_object) { double('Dalli') }
-    before {
-      BoletoSimples.configure do |c|
-        c.environment = :production
-        c.application_id = 'app-id'
-        c.application_secret = 'app-secret'
-        c.access_token = 'access-token'
-        c.cache = cache_object
-      end
-    }
-    subject { BoletoSimples.configuration }
-    it { expect(subject.environment).to eq(:production) }
-    it { expect(subject.user_agent).to eq("BoletoSimples Ruby Client v#{BoletoSimples::VERSION} (contato@boletosimples.com.br)") }
-    it { expect(subject.base_uri).to eq('https://boletosimples.com.br/api/v1') }
-    it { expect(subject.application_id).to eq('app-id') }
-    it { expect(subject.application_secret).to eq('app-secret') }
-    it { expect(subject.access_token).to eq('access-token') }
-    describe 'cache' do
-      it { expect(subject.cache).to eq(cache_object) }
-      it { expect(Her::API.default_api.connection.builder.handlers).to include(Faraday::HttpCache) }
-    end
-    describe 'client credentials' do
-      context 'invalid credentials', vcr: { cassette_name: 'configuration/client_credentials/invalid'} do
-        before {
-          BoletoSimples.configure do |c|
-            c.application_id = 'app-id'
-            c.application_secret = 'app-secret'
-            c.access_token = nil
-          end
-        }
-        it { expect{subject.client_credentials}.to raise_error(BoletoSimples::ResponseError, "401 POST https://sandbox.boletosimples.com.br/api/v1/oauth2/token (invalid_client)") }
-      end
-      context 'valid credentials', vcr: { cassette_name: 'configuration/client_credentials/valid'} do
-        # Before running this spec again, you need to set environment variable BOLETOSIMPLES_APP_ID and BOLETOSIMPLES_APP_SECRET
-        before {
-          BoletoSimples.configure do |c|
-            c.access_token = nil
-          end
-        }
-        it { expect(subject.client_credentials).to include(:access_token) }
-      end
-    end
-  end
-end
+# require 'spec_helper'
+
+# RSpec.describe BoletoSimples::Configuration do
+#   describe 'defaults' do
+#     subject { described_class.new }
+
+#     before do
+#       stub_env('BOLETOSIMPLES_ENV', nil)
+#       stub_env('BOLETOSIMPLES_USER_AGENT', nil)
+#       subject.setup_her
+#     end
+
+#     it do
+#       expect(subject.environment).to eq(:sandbox)
+#       expect(subject.base_uri).to eq('https://sandbox.boletosimples.com.br/api/v1')
+#       expect(subject.cache).to be_nil
+#       expect(subject.user_agent).to be_nil
+#     end
+#   end
+
+#   describe 'environment variables' do
+#     subject { BoletoSimples.configuration }
+
+#     before do
+#       stub_env('BOLETOSIMPLES_ENV', 'production')
+#       stub_env('BOLETOSIMPLES_USER_AGENT', 'email@minhaempresa.com.br')
+#       BoletoSimples.configure
+#     end
+
+#     it do
+#       expect(subject.environment).to eq(:production)
+#       expect(subject.base_uri).to eq('https://boletosimples.com.br/api/v1')
+#       expect(subject.user_agent).to eq('email@minhaempresa.com.br')
+#     end
+
+#     describe 'cache' do
+#       it do
+#         expect(subject.cache).to be_nil
+#         expect(Her::API.default_api.connection.builder.handlers).not_to include(Faraday::HttpCache)
+#       end
+#     end
+#   end
+
+#   describe 'configuration' do
+#     subject { BoletoSimples.configuration }
+
+#     let(:cache_object) { double('Dalli') }
+
+#     before do
+#       BoletoSimples.configure do |c|
+#         c.environment = :production
+#         c.cache = cache_object
+#         c.user_agent = 'Meu agent'
+#       end
+#     end
+
+#     it do
+#       expect(subject.environment).to eq(:production)
+#       expect(subject.user_agent).to eq('Meu agent')
+#       expect(subject.base_uri).to eq('https://boletosimples.com.br/api/v1')
+#     end
+
+#     describe 'cache' do
+#       it do
+#         expect(subject.cache).to eq(cache_object)
+#         expect(Her::API.default_api.connection.builder.handlers).to include(Faraday::HttpCache)
+#       end
+#     end
+#   end
+# end

data/spec/boletosimples/last_request_spec.rb

@@ -1,34 +1,22 @@
-# encoding: UTF-8
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe BoletoSimples::LastRequest do
-  before {
-    BoletoSimples.configure do |c|
-      c.application_id = nil
-      c.application_secret = nil
-    end
-  }
-  describe 'bank_billets', vcr: { cassette_name: 'last_request/bank_billets'} do
-    before { BoletoSimples::BankBillet.all.size }
-    subject { BoletoSimples.last_request }
-    it { expect(subject).to be_kind_of(BoletoSimples::LastRequest) }
-    it { expect(subject.body).to be_kind_of(Array) }
-    it { expect(subject.body.first).to be_kind_of(Hash) }
-    it { expect(subject.response_headers).to be_kind_of(Hash) }
-    it { expect(subject.total).to be_kind_of(Integer) }
-    it { expect(subject.ratelimit_limit).to be_kind_of(Integer) }
-    it { expect(subject.ratelimit_remaining).to be_kind_of(Integer) }
-    it { expect(subject.links.keys).to match_array([:next, :last]) }
-  end
-  describe 'user_info', vcr: { cassette_name: 'last_request/userinfo'} do
-    before { BoletoSimples::Extra.userinfo }
+  describe 'bank_billets', vcr: { cassette_name: 'last_request/bank_billets' } do
     subject { BoletoSimples.last_request }
-    it { expect(subject).to be_kind_of(BoletoSimples::LastRequest) }
-    it { expect(subject.body).to be_kind_of(Hash) }
-    it { expect(subject.response_headers).to be_kind_of(Hash) }
-    it { expect(subject.total).to eq(0) }
-    it { expect(subject.ratelimit_limit).to be_kind_of(Integer) }
-    it { expect(subject.ratelimit_remaining).to be_kind_of(Integer) }
-    it { expect(subject.links).to eq({}) }
+
+    before { BoletoSimples::BankBillet.all(page: 2).size }
+
+    it do
+      expect(subject).to be_kind_of(described_class)
+      expect(subject.body).to be_kind_of(Array)
+      expect(subject.body.first).to be_kind_of(Hash)
+      expect(subject.response_headers).to be_kind_of(Hash)
+      expect(subject.total).to be_kind_of(Integer)
+      expect(subject.ratelimit_limit).to be_kind_of(Integer)
+      expect(subject.ratelimit_remaining).to be_kind_of(Integer)
+      expect(subject.links.keys).to match_array(%i[first next prev])
+    end
   end
 end

data/spec/boletosimples/resources/bank_billet_account_spec.rb

@@ -1,62 +1,86 @@
-# encoding: UTF-8
+# frozen_string_literal: true
 
 require 'spec_helper'
 
-# Before running this spec again, you need to set environment variable BOLETOSIMPLES_ACCESS_TOKEN
+# Before running this spec again, you need to set environment variable BOLETOSIMPLES_API_TOKEN
 RSpec.describe BoletoSimples::BankBilletAccount do
-  before {
-    BoletoSimples.configure do |c|
-      c.application_id = nil
-      c.application_secret = nil
-    end
-  }
   describe 'methods' do
-    before {
+    before do
       VCR.use_cassette('resources/bank_billet_account/create/valid') do
-        @bank_billet_account = BoletoSimples::BankBilletAccount.create({
-            bank_contract_slug: 'sicoob-02',
-            next_our_number: '1',
-            agency_number: '4327',
-            agency_digit: '3',
-            account_number: '3666',
-            account_digit: '8',
-            extra1_length: '1234567'
-          })
+        @bank_billet_account = described_class.create(
+          bank_contract_slug: 'sicoob-02',
+          next_our_number: '1',
+          agency_number: '4327',
+          agency_digit: '3',
+          account_number: '3666',
+          account_digit: '8',
+          extra1: '1234567',
+          beneficiary_name: 'XPTO S.A.',
+          beneficiary_cnpj_cpf: '22.622.867/0001-11',
+          beneficiary_address_street: 'Rua S',
+          beneficiary_address_city: 'São Paulo',
+          beneficiary_address_state: 'SP',
+          beneficiary_address_neighborhood: 'Moema',
+          beneficiary_address_zipcode: '04524030'
+        )
       end
-    }
+    end
+
     describe 'create' do
       context 'valid parameters' do
         subject { @bank_billet_account }
-        it { expect(subject).to be_a_kind_of(BoletoSimples::BankBilletAccount) }
-        it { expect(subject.response_errors).to eq({}) }
-        it { expect(subject.attributes.keys).to match_array(["account_digit", "account_number", "agency_digit", "agency_number", "bank_contract_slug", "extra1", "extra1_digit", "extra1_length", "extra2", "extra2_digit", "id", "next_our_number"]) }
+
+        it do
+          expect(subject).to be_a_kind_of(described_class)
+          expect(subject.response_errors).to eq({})
+        end
       end
+
       context 'invalid parameters' do
         context 'empty bank_billet' do
-          subject {
+          subject do
             VCR.use_cassette('resources/bank_billet_account/create/invalid_root') do
-              BoletoSimples::BankBilletAccount.create({})
+              described_class.create({})
             end
+          end
+
+          it {
+            expect(subject.response_errors).to eq([{ code: 422, status: 422,
+                                                     title: 'bank_billet_account não pode ficar em branco' }])
           }
-          it { expect(subject.response_errors).to eq({ bank_billet_account: ["não pode ficar em branco"] }) }
         end
+
         context 'invalid params' do
-          subject {
+          subject do
             VCR.use_cassette('resources/bank_billet_account/create/invalid_params') do
-              BoletoSimples::BankBilletAccount.create({ bank_contract_slug: '' })
+              described_class.create({ bank_contract_slug: '' })
             end
+          end
+
+          it {
+            expect(subject.response_errors).to eq({ bank_contract_slug: ['não pode ficar em branco'],
+                                                    beneficiary_name: ['não pode ficar em branco'],
+                                                    beneficiary_cnpj_cpf: ['não pode ficar em branco'],
+                                                    beneficiary_address_street: ['não pode ficar em branco'],
+                                                    beneficiary_address_city: ['não pode ficar em branco'],
+                                                    beneficiary_address_neighborhood: ['não pode ficar em branco'],
+                                                    beneficiary_address_state: ['não pode ficar em branco'],
+                                                    beneficiary_address_zipcode: ['não pode ficar em branco'] })
           }
-          it { expect(subject.response_errors).to eq({ :agency_number => ["não pode ficar em branco"], :account_number => ["não pode ficar em branco"], :bank_contract_slug => ["não pode ficar em branco"] }) }
         end
       end
     end
+
     describe 'find', vcr: { cassette_name: 'resources/bank_billet_account/find' } do
-      subject { BoletoSimples::BankBilletAccount.find(@bank_billet_account.id) }
-      it { expect(subject).to be_a_kind_of(BoletoSimples::BankBilletAccount) }
+      subject { described_class.find(@bank_billet_account.id) }
+
+      it { expect(subject).to be_a_kind_of(described_class) }
     end
+
     describe 'all', vcr: { cassette_name: 'resources/bank_billet_account/all' } do
-      subject { BoletoSimples::BankBilletAccount.all }
-      it { expect(subject.first).to be_a_kind_of(BoletoSimples::BankBilletAccount) }
+      subject { described_class.all }
+
+      it { expect(subject.first).to be_a_kind_of(described_class) }
     end
   end
-end
+end