lcoveralls 0.1.0 → 0.2.0

data/lib/lcoveralls/option_parser.rb

@@ -23,10 +23,12 @@ module Lcoveralls
 
     def parse!(args)
       options =  {
-        :color    => $stderr.isatty,
-        :git      => 'git',
-        :service  => File.basename($0),
-        :severity => Logger::INFO
+        :color          => $stderr.isatty,
+        :git            => 'git',
+        :retry_count    => 0,
+        :retry_interval => 10.0,
+        :service        => File.basename($0),
+        :severity       => Logger::INFO
       }
 
       if ENV.has_key? 'TRAVIS_JOB_NUMBER' then
@@ -54,6 +56,19 @@ module Lcoveralls
       o.on('-t', '--token TOKEN', 'Set coveralls repo token')      { |token| options[:token] = token }
       o.separator ''
 
+      o.separator 'Network options:'
+      o.on('--timeout DURATION',
+           'Timout, in seconds, for network open and read operations') do |duration|
+        options[:timeout] = duration.to_f
+      end
+      o.on('--retry-count COUNT', 'Number of times to retry sending to coveralls.io') do |count|
+        options[:retry_count] = count.to_i
+      end
+      o.on('--retry-interval DURATION', 'Time to wait between retries') do |count|
+        options[:retry_interval] = count.to_f;
+      end
+      o.separator ''
+
       o.separator 'Stderr output options:'
       o.on(      '--[no-]color', 'Colorize output')  { |color| options[:color] = color }
       o.on('-d', '--debug',      'Enable debugging') { options[:severity] = Logger::DEBUG }

data/lib/lcoveralls/runner.rb

@@ -159,6 +159,21 @@ module Lcoveralls
       end if Dir.exist?(root_dir)
     end
 
+    def should_retry
+      return false unless @options[:retry_count] > 0
+      @options[:retry_count] = @options[:retry_count] - 1;
+
+      if @options[:retry_interval] > 0 then
+        @log.info { "Sleeping for #{@options[:retry_interval]} seconds before retrying" }
+        begin
+          sleep @options[:retry_interval]
+        rescue Interrupt
+          return false
+        end
+      end
+      true
+    end
+
     def run
       # Find *.info tracefiles if none specified on the command line.
       Find.find('.') do |path|
@@ -197,13 +212,30 @@ module Lcoveralls
       # Send (if not in dryrun mode) the Coveralls API request.
       uri = URI('https://coveralls.io/api/v1/jobs')
       http = Net::HTTP.new(uri.host, uri.port)
+      http.open_timeout = @options[:timeout] if @options.has_key? :timeout
+      http.read_timeout = @options[:timeout] if @options.has_key? :timeout
+      http.ssl_timeout = @options[:timeout] if @options.has_key? :timeout
       http.use_ssl = true
       http.verify_mode = OpenSSL::SSL::VERIFY_PEER
 
       if !@options[:dryrun] then
-        @log.debug { "Sending #{request.body.size} bytes to coveralls.io" }
-        res = http.request(request)
-        puts res.body if res
+        begin
+          @log.debug { "Sending #{request.body.size} bytes to coveralls.io" }
+          response = http.request(request)
+          @log.debug { "HTTP response status: #{response.code} #{response.message}" }
+          raise response.code unless response.is_a? Net::HTTPSuccess
+          puts response.body
+        rescue RuntimeError => e
+          raise unless response
+          @log.error { "Received non-OK response: #{response.code} #{response.message}" }
+          puts response.body
+          retry if should_retry unless response.is_a? Net::HTTPClientError
+          exit!
+        rescue SocketError => error
+          @log.error { error }
+          retry if should_retry
+          exit!
+        end
       end
     end
 

data/lib/lcoveralls/version.rb

@@ -15,5 +15,5 @@
 #
 
 module Lcoveralls
-  VERSION = [ 0, 1, 0 ]
+  VERSION = [ 0, 2, 0 ]
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lcoveralls
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
   prerelease: 
 platform: ruby
 authors: