correios_sigep 0.4.0 → 0.5.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZTkyNGM5YWJjODE5MmQwZjZjZGFmMzkwMWZhZDY4NzVjYzI3OWQ0OQ==
+    ZGE2NDlkNWI5OWMzYjkxOGE5NGUwNTA3ZjEyNjViYjUwOWQ1OTJlYQ==
   data.tar.gz: !binary |-
-    Yjc4YjIzN2U4MTIxZTBjMDIwZTUwNWY0M2YwY2QwY2NmYmVhOGJjNA==
+    ZDBkNjZkNjY0OWZhODE2NzRiM2Q3YjllZGE4ZmRjZTJlMWRhZWQ4Mw==
 SHA512:
   metadata.gz: !binary |-
-    YmIzOTYzYzljYjBhZWUxMmY2YTczNGY4OTJmOTMzNmY1YmRlZWRjZjdhNTQ0
-    YjZkZmExYjZlY2QyZDQ3ZThjMzU2M2Q3M2ZjZmYxYjVmMTdhNmIzNTRiY2Uz
-    NDhiMWI5MzhjZjc1NzYxMWIzYzU0NDI4M2M5MDU0ZDEwODNkY2Y=
+    NGE0NTZiZjhlYWY0MGRjYmEyZDNjNzYwMDdhODE2ZWQ2NTAwNWJjZjVmZmIy
+    OWM3MDRlNzJiY2JlMTFhYmJjMjQ1YWZlNDIzOWM2ODA5M2ZjNzE0ZDMzNjI3
+    MjZjNDdlYmMyYmU4NGIxOTE2MWE4YTU1MmUwMGVmN2EyMDY3NzU=
   data.tar.gz: !binary |-
-    ZmYwOTM1MGY2ZTE4YTYyZGFmZjBiODE5NTBjYWRkMGM5ZGNkZDQyMzZlODlj
-    NzU4MDBkMzI0MTlkOGJiNmUxMmNhMGJjOGFjMmY2ZmE3YTViYjI0MjcyMjE2
-    MzFjM2M4ZDBiMzliYjhmZDI3MmM1NTdkOTIzYTBiNTg0YjQ5ZWQ=
+    MjkzZTEwNGFiNDI2NzNkYTRhNTgwNTJkY2NiNjI0NThiNzA5OTc1ZmNhMTU4
+    OThkMTI0ZjIwZTA0MTg4ZjllNTBiNDBmMmZmM2M5NGI2OWMwN2U5ZTQ1ZmE5
+    ZmE0ODE4M2Q3OTdiM2JlOTQxMmYxODE0NWNjNTE3MjNkNjlkZTg=

data/.travis.yml

@@ -6,7 +6,6 @@ rvm:
 - 2.1.1
 - 2.2.2
 - 2.3.0
-- jruby-19mode
 bundler_args: --without debug
 deploy:
   provider: rubygems

data/correios_sigep.gemspec

@@ -19,6 +19,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_dependency 'net-http-persistent', '~> 2.9.4'
+  spec.add_dependency 'rack', '~> 1.6', '>= 1.6.4'
   spec.add_dependency 'savon', '~> 2.0'
 
   spec.add_development_dependency 'bundler', '~> 1.7'

data/lib/correios_sigep/configuration.rb

@@ -1,7 +1,7 @@
 module CorreiosSigep
   class Configuration
     attr_accessor :administrative_code, :card, :contract, :password,
-                  :service_code, :user, :wsdl_base_url, :proxy
+                  :service_code, :user, :wsdl_base_url, :proxy, :timeout
 
     def administrative_fields
       @administrative_fields ||=

data/lib/correios_sigep/logistic_reverse/base_client.rb

@@ -1,8 +1,18 @@
 module CorreiosSigep
   module LogisticReverse
     class BaseClient
+      DEFAULT_TIMEOUT = 30
+
       def initialize
-        options = { adapter: :net_http_persistent, proxy: CorreiosSigep.configuration.proxy, wsdl: wsdl }
+        timeout = CorreiosSigep.configuration.timeout || DEFAULT_TIMEOUT
+
+        options = {
+          adapter: :net_http_persistent,
+          proxy: CorreiosSigep.configuration.proxy,
+          wsdl: wsdl,
+          open_timeout: timeout,
+          read_timeout: timeout
+        }
         options.delete(:proxy) unless options[:proxy]
 
         options.merge!({ headers: { 'SOAPAction' => '' }}) if ENV['GEM_ENV'] == 'test'

data/lib/correios_sigep/version.rb

@@ -1,3 +1,3 @@
 module CorreiosSigep
-  VERSION = '0.4.0'
+  VERSION = '0.5.0'
 end

data/spec/correios_sigep/configuration_spec.rb

@@ -3,52 +3,52 @@ module CorreiosSigep
     subject { described_class.new }
 
     describe '#administrative_code' do
-      it 'return nil when unset' do
-        expect(subject.administrative_code).to_not be
+      it 'returns nil when unset' do
+        expect(subject.administrative_code).to be_nil
       end
     end
 
     describe '#administrative_code=' do
-      it 'can set the administrative code in configuration' do
+      it 'sets the administrative code in configuration' do
         subject.administrative_code = '12345'
         expect(subject.administrative_code).to eq '12345'
       end
     end
 
     describe '#card' do
-      it 'return nil when unset' do
-        expect(subject.card).to_not be
+      it 'returns nil when unset' do
+        expect(subject.card).to be_nil
       end
     end
 
     describe '#card=' do
-      it 'can set the card in configuration' do
+      it 'sets the card in configuration' do
         subject.card = 'card'
         expect(subject.card).to eq 'card'
       end
     end
 
     describe '#contract' do
-      it 'return nil when unset' do
-        expect(subject.contract).to_not be
+      it 'returns nil when unset' do
+        expect(subject.contract).to be_nil
       end
     end
 
     describe '#contract=' do
-      it 'can set the contract in configuration' do
+      it 'sets the contract in configuration' do
         subject.contract = 'contract'
         expect(subject.contract).to eq 'contract'
       end
     end
 
     describe '#password' do
-      it 'return nil when unset' do
-        expect(subject.password).to_not be
+      it 'returns nil when unset' do
+        expect(subject.password).to be_nil
       end
     end
 
     describe '#password=' do
-      it 'can set the password in configuration' do
+      it 'sets the password in configuration' do
         subject.password = 'password'
         expect(subject.password).to eq 'password'
       end
@@ -56,29 +56,42 @@ module CorreiosSigep
 
 
     describe '#service_code' do
-      it 'return nil when unset' do
-        expect(subject.service_code).to_not be
+      it 'returns nil when unset' do
+        expect(subject.service_code).to be_nil
       end
     end
 
     describe '#service_code=' do
-      it 'can set the password in configuration' do
+      it 'sets the password in configuration' do
         subject.service_code = 'service_code'
         expect(subject.service_code).to eq 'service_code'
       end
     end
 
     describe '#user' do
-      it 'return nil when unset' do
-        expect(subject.user).to_not be
+      it 'returns nil when unset' do
+        expect(subject.user).to be_nil
       end
     end
 
     describe '#user=' do
-      it 'can set the user in configuration' do
+      it 'sets the user in configuration' do
         subject.user = 'user'
         expect(subject.user).to eq 'user'
       end
     end
+
+    describe '#timeout' do
+      it 'returns nil when unset' do
+        expect(subject.timeout).to be_nil
+      end
+    end
+
+    describe '#timeout=' do
+      it 'sets the timeout in configuration' do
+        subject.timeout = 15
+        expect(subject.timeout).to eq 15
+      end
+    end
   end
 end

data/spec/correios_sigep/logistic_reverse/base_client_spec.rb

@@ -13,6 +13,8 @@ module CorreiosSigep
               adapter: :net_http_persistent,
               proxy: CorreiosSigep.configuration.proxy,
               wsdl:  described_class.new.wsdl,
+              open_timeout: CorreiosSigep::LogisticReverse::BaseClient::DEFAULT_TIMEOUT,
+              read_timeout: CorreiosSigep::LogisticReverse::BaseClient::DEFAULT_TIMEOUT,
               headers: { 'SOAPAction' => '' }
             }
           end
@@ -29,6 +31,8 @@ module CorreiosSigep
             {
               adapter: :net_http_persistent,
               wsdl: described_class.new.wsdl,
+              open_timeout: CorreiosSigep::LogisticReverse::BaseClient::DEFAULT_TIMEOUT,
+              read_timeout: CorreiosSigep::LogisticReverse::BaseClient::DEFAULT_TIMEOUT,
               headers: { 'SOAPAction' => '' }
             }
           end
@@ -58,6 +62,26 @@ module CorreiosSigep
         end
 
       end
+
+      context 'setting a timeout' do
+        subject { described_class.new }
+
+        before { CorreiosSigep.configuration.timeout = 15 }
+        let(:params) do
+          {
+            adapter: :net_http_persistent,
+            wsdl:  described_class.new.wsdl,
+            open_timeout: CorreiosSigep.configuration.timeout,
+            read_timeout: CorreiosSigep.configuration.timeout,
+            headers: { 'SOAPAction' => '' }
+          }
+        end
+
+        it 'initializes @client with informed timeout' do
+          expect(Savon).to receive(:client).with(params) { true }
+          subject
+        end
+      end
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: correios_sigep
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Carlos Ribeiro
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-10 00:00:00.000000000 Z
+date: 2016-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: net-http-persistent
@@ -25,6 +25,26 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 2.9.4
+- !ruby/object:Gem::Dependency
+  name: rack
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.6'
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 1.6.4
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.6'
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 1.6.4
 - !ruby/object:Gem::Dependency
   name: savon
   requirement: !ruby/object:Gem::Requirement