boletosimples 0.4.2 → 1.0.3

data/lib/boletosimples/middlewares/user_agent.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   module Middleware
     class UserAgent < Faraday::Middleware
       def call(env)
-        env[:request_headers]["User-Agent"] = BoletoSimples.configuration.user_agent
+        env[:request_headers]['User-Agent'] = BoletoSimples.configuration.user_agent
         @app.call(env)
       end
     end

data/lib/boletosimples/resources/bank_billet.rb

@@ -1,14 +1,8 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class BankBillet < BaseModel
-
-    def cancel
-      self.class.request(:_method => :put, :_path => self.class.build_request_path('bank_billets/:id/cancel', {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, :cancel
+    custom_post :duplicate
   end
-end
+end

data/lib/boletosimples/resources/bank_billet_account.rb

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

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

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class CustomerImport < BaseModel
-    collection_path "imports/customers"
-    resource_path "imports/customers/:id"
+    collection_path 'imports/customers'
+    resource_path 'imports/customers/:id'
   end
-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

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 module BoletoSimples
   class CustomerSubscriptionImport < BaseModel
-    collection_path "imports/customer_subscriptions"
-    resource_path "imports/customer_subscriptions/:id"
+    collection_path 'imports/customer_subscriptions'
+    resource_path 'imports/customer_subscriptions/:id'
   end
-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.2'
+  VERSION = '1.0.3'
 end

data/spec/boletosimples/configuration_spec.rb

@@ -1,100 +1,72 @@
-# encoding: UTF-8
-require 'spec_helper'
+# # frozen_string_literal: true
 
-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 }
-  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 }
-    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