boa_vista 0.0.10 → 0.0.11

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTY3NWZiM2IxM2I5MzJlYjNhMGE3NTM4NWVjMGZlNTdjZjQ0OTAzYQ==
+    ZjA4OTMwMzkxMDcwNzdmNjE5MTAxOWYxYWMyODgyM2ExMGJkOGNmMg==
   data.tar.gz: !binary |-
-    N2E1M2ViZDIzNmMwYTQ5ZDVjNDEzNWVhMGQ1NWEyOTRhZTE3ZDIyNg==
+    YWY2NjM3MDlmZTdhOWU4OGIxNzJiODAxYmE1MGRlYTI3YTU4ZTQ5YQ==
 SHA512:
   metadata.gz: !binary |-
-    MmRlMjAzNmZmOTUwZmZkMmIzNzIyYWRjMzFjMjNjOTUzYzgxYWRlYTc1ZDFh
-    OWY2NjkwYjhmNjg3YmZjNjBjZjgyNGQ5NjRhY2JiZTkyNTgwNzFiMDA4YzY1
-    MWJiYzUxYWEzMDgxNDUzOGNlN2UwY2JhOWIzZGQxOTQyMWQ1Mjc=
+    NjA0NTY2ZTNlZTJmN2Q0NzQ1YzY2ZTI1MTE3Mjc0MDg4ODgzOTEwMTYzMmU3
+    MGJmOTU4ZjhiY2U2NDRiOTk4ZWJmYTYzY2NiYjk1MzU4Zjc4Zjc4ZWE0MDZh
+    MzU2ZjQxNDg5NGE3YWIyMzNhMjFjNDNkNTlmN2JmNjIyZjhkMWQ=
   data.tar.gz: !binary |-
-    YzM3MDg0NWNhZWQxZTk3Nzg5ZGVhZmEzOWEzYjdhY2Q5YTRiNjk5YjQ2ZDA0
-    NzAwZDZkMTg3OTYxYmU5Y2Q4NGY3ZThlMTIyMDk0MDYzMDAwY2Q1ZDUzNjI2
-    NGVkOWIyYmNmMjUxZmM3ZjUzNGJhNDFkN2FiZGRhNjc0YjkxYTA=
+    MDVmOTY4MjVlZWE0ZGE2ZmY5MTY5MmQ4ZTNkOTk1ODUzNDkxNjQ1MGJjYTgx
+    ODAxOThhYjEwNmYzODVjZGU1NGI5Njc1MjVhNjhmZGFiYjc5YzIwZmFiZGJh
+    M2VjYTY1NWVjMTJlMzM5N2E5YWYwYzFjMjc3YjZlNDk2NzI0NWQ=

data/lib/boa_vista/client.rb

@@ -14,10 +14,11 @@ module BoaVista
       begin
         response = nil
         execute_with_timeout do
-          response = BoaVista::Http.new(uri(create_request)).call.body
+          response = BoaVista::Http.new(uri(create_request)).call
+          fail Errors::ResponseStatusInvalid.new("Response status is: #{response.status}") if response.status != 200
           BoaVista.log(:debug, "Response of request: #{response}")
         end
-        @response.new(parsed_response(response)).read
+        @response.new(parsed_response(response.body)).read
       rescue Timeout::Error
         BoaVista.log(:debug, "Request is Timeout")
         fail Errors::Timeout.new('timed out')

data/lib/boa_vista/configuration.rb

@@ -1,6 +1,6 @@
 module BoaVista
   class Configuration
-    attr_accessor :code, :password, :url, :timeout, :proxy_url, :proxy_user, :proxy_pass
+    attr_accessor :code, :password, :url, :timeout, :proxy_url
 
     def has_proxy?
       !self.proxy_url.nil?

data/lib/boa_vista/errors.rb

@@ -4,5 +4,6 @@ module BoaVista
     class InvalidLayout < StandardError; end
     class FileNotExists < StandardError; end
     class InvalidDocumentType < StandardError; end
+    class ResponseStatusInvalid < StandardError; end
   end
 end

data/lib/boa_vista/http.rb

@@ -1,5 +1,4 @@
-require 'timeout'
-require 'net/http'
+require 'faraday'
 
 module BoaVista
   class Http
@@ -15,30 +14,16 @@ module BoaVista
     private
 
     def execute_request_using_proxy
-      BoaVista.log(:debug, "Executing request with proxy #{proxy_uri.host}")
-      request = Net::HTTP::Get.new(@uri.path)
-
-      proxy.start(@uri.path, @uri.port) do |http|
-        http.request(request)
-      end
+      BoaVista.log(:debug, "Executing with proxy: #{BoaVista.configuration.proxy_url}")
+      BoaVista.log(:debug, "Executing request for uri: #{@uri}")
+      ::Faraday.new(proxy: BoaVista.configuration.proxy_url, url: @uri).get
     end
 
     def execute_request
       BoaVista.log(:debug, "Executing request for uri: #{@uri}")
-      Net::HTTP.get_response(@uri)
-    end
-
-    def proxy
-      @proxy ||= Net::HTTP::Proxy(proxy_uri.host,
-                                  proxy_uri.port,
-                                  BoaVista.configuration.proxy_user,
-                                  BoaVista.configuration.proxy_pass)
-    end
-
-    def proxy_uri
-      @proxy_uri ||= URI(BoaVista.configuration.proxy_url)
+      ::Faraday.new(url: @uri).get
     end
- end
+  end
 end
 
 

data/lib/boa_vista/version.rb

@@ -1,3 +1,3 @@
 module BoaVista
-  VERSION = '0.0.10'
+  VERSION = '0.0.11'
 end

data/spec/lib/boa_vista/client_spec.rb

@@ -35,8 +35,14 @@ RSpec.describe BoaVista::Client do
     end
 
     context 'when timeout error' do
+      let(:faraday_connection) { instance_double(Faraday::Connection) }
+
+      before do
+        allow(Faraday).to receive(:new).and_return(faraday_connection)
+        allow(faraday_connection).to receive(:get) { raise(Timeout::Error) }
+      end
+
       it 'raise BoaVista::Errors::Timeout' do
-        allow(Net::HTTP).to receive(:get_response).with(anything) { raise(Timeout::Error) }
         expect { subject }.to raise_error(BoaVista::Errors::Timeout)
       end
     end

data/spec/lib/boa_vista/version_spec.rb

@@ -1,3 +1,3 @@
 RSpec.describe BoaVista::VERSION do
-  specify { expect(BoaVista::VERSION).to eq '0.0.10' }
+  specify { expect(BoaVista::VERSION).to eq '0.0.11' }
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: boa_vista
 version: !ruby/object:Gem::Version
-  version: 0.0.10
+  version: 0.0.11
 platform: ruby
 authors:
 - Gabriel Souza
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-21 00:00:00.000000000 Z
+date: 2016-03-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -25,6 +25,20 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: faraday
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement