boletosimples 0.6.0 → 2.1.0

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)
@@ -7,14 +9,14 @@ module BoletoSimples
   # Examples:
   #
   #   BoletoSimples::BankBillet.all
-  #   BoletoSimples::ResponseError: 401 POST https://sandbox.boletosimples.com.br/api/v1/bank_billets
+  #   BoletoSimples::ResponseError: 401 POST https://api-sandbox.kobana.com.br/v1/bank_billets
   #
   #   begin
   #     BoletoSimples::BankBillet.all
   #   rescue BoletoSimples::ResponseError => response
   #     response.status # => 401
   #     response.method # => "GET"
-  #     response.url # => "https://sandbox.boletosimples.com.br/api/v1/bank_billets"
+  #     response.url # => "https://api-sandbox.kobana.com.br/v1/bank_billets"
   #     response.body # => "{"error":"Você precisa se logar ou registrar antes de prosseguir."}"
   #     response.error_message # => "Você precisa se logar ou registrar antes de prosseguir."
   #   end
@@ -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.6.0'
+  VERSION = '2.1.0'
 end

data/lib/boletosimples.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'her'
 require 'faraday_middleware'
 require 'faraday-http-cache'
@@ -5,7 +7,6 @@ require 'boletosimples/version'
 
 module BoletoSimples
   autoload :Configuration, 'boletosimples/configuration'
-  autoload :Extra, 'boletosimples/extra'
   autoload :ResponseError, 'boletosimples/response_error'
   autoload :LastRequest, 'boletosimples/last_request'
 
@@ -16,6 +17,7 @@ module BoletoSimples
   autoload :BankBilletRemittance, 'boletosimples/resources/bank_billet_remittance'
   autoload :Event, 'boletosimples/resources/event'
   autoload :EmailDelivery, 'boletosimples/resources/email_delivery'
+  autoload :SmsDelivery, 'boletosimples/resources/sms_delivery'
   autoload :Customer, 'boletosimples/resources/customer'
   autoload :CustomerImport, 'boletosimples/resources/customer_import'
   autoload :CustomerSubscription, 'boletosimples/resources/customer_subscription'
@@ -28,14 +30,13 @@ module BoletoSimples
   autoload :Webhook, 'boletosimples/resources/webhook'
   autoload :WebhookDelivery, 'boletosimples/resources/webhook_delivery'
 
-  module Partner
-    autoload :User, 'boletosimples/resources/partner/user'
-  end
-
   module Middleware
     autoload :UserAgent, 'boletosimples/middlewares/user_agent'
+    autoload :CustomHeaders, 'boletosimples/middlewares/custom_headers'
     autoload :RaiseError, 'boletosimples/middlewares/raise_error'
     autoload :LastRequest, 'boletosimples/middlewares/last_request'
+    autoload :Debug, 'boletosimples/middlewares/debug'
+    autoload :Bearer, 'boletosimples/middlewares/bearer'
   end
 
   class << self
@@ -47,7 +48,6 @@ module BoletoSimples
       configuration.setup_her # after changing configuration gem her should be configured
     end
   end
-
 end
 
 BoletoSimples.configure

data/spec/boletosimples/configuration_spec.rb

@@ -1,101 +1,88 @@
-# encoding: UTF-8
+# # frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe BoletoSimples::Configuration do
-  before {
-    @last_env = {}
-    @last_env['BOLETOSIMPLES_ENV'] = ENV['BOLETOSIMPLES_ENV']
-    @last_env['BOLETOSIMPLES_APP_ID'] = ENV['BOLETOSIMPLES_APP_ID']
-    @last_env['BOLETOSIMPLES_APP_SECRET'] = ENV['BOLETOSIMPLES_APP_SECRET']
-    @last_env['BOLETOSIMPLES_ACCESS_TOKEN'] = ENV['BOLETOSIMPLES_ACCESS_TOKEN']
-  }
-  after {
-    ENV['BOLETOSIMPLES_ENV'] = @last_env['BOLETOSIMPLES_ENV']
-    ENV['BOLETOSIMPLES_APP_ID'] = @last_env['BOLETOSIMPLES_APP_ID']
-    ENV['BOLETOSIMPLES_APP_SECRET'] = @last_env['BOLETOSIMPLES_APP_SECRET']
-    ENV['BOLETOSIMPLES_ACCESS_TOKEN'] = @last_env['BOLETOSIMPLES_ACCESS_TOKEN']
-  }
   describe 'defaults' do
-    before {
-      ENV['BOLETOSIMPLES_ENV'] = nil
-      ENV['BOLETOSIMPLES_APP_ID'] = nil
-      ENV['BOLETOSIMPLES_APP_SECRET'] = nil
-      ENV['BOLETOSIMPLES_ACCESS_TOKEN'] = nil
-    }
-    subject { BoletoSimples::Configuration.new }
-    before { subject.setup_her}
-    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 }
+    subject { described_class.new }
+
+    before do
+      stub_env('BOLETOSIMPLES_ENV', nil)
+      stub_env('BOLETOSIMPLES_USER_AGENT', nil)
+      subject.setup_her
+    end
+
+    after do
+      BoletoSimples.configure
+    end
+
+    it do
+      expect(subject.environment).to eq(:sandbox)
+      expect(subject.base_uri).to eq('https://api-sandbox.kobana.com.br/v1')
+      expect(subject.cache).to be_nil
+      expect(subject.user_agent).to be_nil
+      expect(subject.custom_headers).to eq({})
+    end
   end
+
   describe 'environment variables' do
-    before {
-      ENV['BOLETOSIMPLES_ENV'] = 'production'
-      ENV['BOLETOSIMPLES_APP_ID'] = 'app-id'
-      ENV['BOLETOSIMPLES_APP_SECRET'] = 'app-secret'
-      ENV['BOLETOSIMPLES_ACCESS_TOKEN'] = 'access-token'
-    }
-    before  { BoletoSimples.configure }
     subject { BoletoSimples.configuration }
-    # 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 }
-    it { expect(subject.user_agent).to eq("BoletoSimples Ruby Client v#{BoletoSimples::VERSION} (contato@boletosimples.com.br)") }
+
+    before do
+      stub_env('BOLETOSIMPLES_ENV', 'production')
+      stub_env('BOLETOSIMPLES_USER_AGENT', 'email@minhaempresa.com.br')
+      BoletoSimples.configure
+    end
+
+    after do
+      BoletoSimples.configure
+    end
+
+    it do
+      expect(subject.environment).to eq(:production)
+      expect(subject.base_uri).to eq('https://api.kobana.com.br/v1')
+      expect(subject.user_agent).to eq('email@minhaempresa.com.br')
+    end
+
     describe 'cache' do
-      it { expect(subject.cache).to be_nil }
-      it { expect(Her::API.default_api.connection.builder.handlers).not_to include(Faraday::HttpCache) }
+      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 {
+
+    before do
       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
         c.user_agent = 'Meu agent'
-      end
-    }
-    subject { BoletoSimples.configuration }
-    it { expect(subject.environment).to eq(:production) }
-    it { expect(subject.user_agent).to eq("Meu agent") }
-    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
+        c.custom_headers = {
+          'X-CUSTOM' => 'CONTENT'
         }
-        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
+
+    after do
+      BoletoSimples.configure
+    end
+
+    it do
+      expect(subject.environment).to eq(:production)
+      expect(subject.user_agent).to eq('Meu agent')
+      expect(subject.custom_headers).to eq({ 'X-CUSTOM' => 'CONTENT' })
+      expect(subject.base_uri).to eq('https://api.kobana.com.br/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

data/spec/boletosimples/last_request_spec.rb

@@ -1,34 +1,23 @@
-# 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.request_headers).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/middlewares/custom_headers_spec.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe BoletoSimples::Middleware::CustomHeaders do
+  describe 'bank_billets', vcr: { cassette_name: 'custom_headers' } do
+    subject { BoletoSimples.last_request }
+
+    before do
+      BoletoSimples.configure do |c|
+        c.custom_headers = {
+          'X-CUSTOM' => 'CONTENT'
+        }
+      end
+      BoletoSimples::BankBillet.all(page: 2).size
+    end
+
+    after do
+      BoletoSimples.configure
+    end
+
+    it do
+      expect(subject.request_headers).to include({ 'X-CUSTOM' => 'CONTENT' })
+    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

data/spec/boletosimples/resources/bank_billet_discharge_spec.rb

@@ -1,17 +1,12 @@
-# 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::BankBilletDischarge do
-  before {
-    BoletoSimples.configure do |c|
-      c.application_id = nil
-      c.application_secret = nil
-    end
-  }
-  describe 'all', vcr: { cassette_name: 'resources/bank_billet_discharge/all'} do
-    subject { BoletoSimples::BankBilletDischarge.all }
-    it { expect(subject.first).to be_a_kind_of(BoletoSimples::BankBilletDischarge) }
+  describe 'all', vcr: { cassette_name: 'resources/bank_billet_discharge/all' } do
+    subject { described_class.all }
+
+    it { expect(subject.first).to be_a_kind_of(described_class) }
   end
-end
+end

data/spec/boletosimples/resources/bank_billet_payment_spec.rb

@@ -1,17 +1,12 @@
-# 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::BankBilletPayment do
-  before {
-    BoletoSimples.configure do |c|
-      c.application_id = nil
-      c.application_secret = nil
-    end
-  }
-  describe 'all', vcr: { cassette_name: 'resources/bank_billet_payment/all'} do
-    subject { BoletoSimples::BankBilletPayment.all }
-    it { expect(subject.first).to be_a_kind_of(BoletoSimples::BankBilletPayment) }
+  describe 'all', vcr: { cassette_name: 'resources/bank_billet_payment/all' } do
+    subject { described_class.all }
+
+    it { expect(subject.first).to be_a_kind_of(described_class) }
   end
-end
+end

data/spec/boletosimples/resources/bank_billet_remittance_spec.rb

@@ -1,17 +1,12 @@
-# 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::BankBilletRemittance do
-  before {
-    BoletoSimples.configure do |c|
-      c.application_id = nil
-      c.application_secret = nil
-    end
-  }
-  describe 'all', vcr: { cassette_name: 'resources/bank_billet_remittance/all'} do
-    subject { BoletoSimples::BankBilletRemittance.all }
-    it { expect(subject.first).to be_a_kind_of(BoletoSimples::BankBilletRemittance) }
+  describe 'all', vcr: { cassette_name: 'resources/bank_billet_remittance/all' } do
+    subject { described_class.all }
+
+    it { expect(subject.first).to be_a_kind_of(described_class) }
   end
-end
+end