Add support for changing the timeout

Author: Miguel Angel

lib/code_climate/test_reporter/client.rb

@@ -80,8 +80,8 @@ def http_client(uri)
             http.ca_file = File.expand_path('../../../../config/cacert.pem', __FILE__)
             http.verify_depth = 5
           end
-          http.open_timeout = DEFAULT_TIMEOUT # in seconds
-          http.read_timeout = DEFAULT_TIMEOUT # in seconds
+          http.open_timeout = CodeClimate::TestReporter.configuration.timeout
+          http.read_timeout = CodeClimate::TestReporter.configuration.timeout
         end
       end
 

lib/code_climate/test_reporter/configuration.rb

@@ -21,7 +21,7 @@ def self.configuration
     class Configuration
       attr_accessor :branch, :path_prefix, :gzip_request, :git_dir
 
-      attr_writer :logger, :profile
+      attr_writer :logger, :profile, :timeout
 
       def initialize
         @gzip_request = true
@@ -39,6 +39,10 @@ def skip_token
         @skip_token ||= "nocov"
       end
 
+      def timeout
+        @timeout ||= Client::DEFAULT_TIMEOUT
+      end
+
       private
 
       def default_logger

spec/lib/client_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+module CodeClimate::TestReporter
+  describe Client do
+    it 'sets the http timeout per configuration' do
+      new_timeout = 969
+      CodeClimate::TestReporter.configure do |config|
+        config.timeout = new_timeout
+      end
+
+      response = double(:response, code: 200)
+      net_http = double(:net_http, request: response)
+      allow(Net::HTTP).to receive(:new).
+        and_return(net_http)
+
+      expect(net_http).to receive(:open_timeout=).
+        with(new_timeout)
+      expect(net_http).to receive(:read_timeout=).
+        with(new_timeout)
+
+      Client.new.post_results("")
+    end
+  end
+end

spec/lib/configuration_spec.rb

@@ -15,6 +15,7 @@ module CodeClimate::TestReporter
         expect(CodeClimate::TestReporter.configuration.profile).to eq('test_frameworks')
         expect(CodeClimate::TestReporter.configuration.path_prefix).to be_nil
         expect(CodeClimate::TestReporter.configuration.skip_token).to eq('nocov')
+        expect(CodeClimate::TestReporter.configuration.timeout).to eq(Client::DEFAULT_TIMEOUT)
       end
     end
 
@@ -60,7 +61,14 @@ module CodeClimate::TestReporter
         CodeClimate::TestReporter.configure do |config|
           config.path_prefix = nil
         end
+      end
+
+      it 'stores timeout' do
+        CodeClimate::TestReporter.configure do |config|
+          config.timeout = 666
+        end
 
+        expect(CodeClimate::TestReporter.configuration.timeout).to eq(666)
       end
     end
   end