Merge pull request #3146 from betagouv/dev

2018-12-07-03

lib/active_storage/service/cellar_service.rb

@@ -4,7 +4,7 @@ module ActiveStorage
       @adapter = Cellar::CellarAdapter.new(access_key_id, secret_access_key, bucket)
     end
 
-    def upload(key, io, checksum: nil)
+    def upload(key, io, checksum: nil, **)
       instrument :upload, key: key, checksum: checksum do
         @adapter.session { |s| s.upload(key, io, checksum) }
       end

lib/cellar/cellar_adapter.rb

@@ -34,7 +34,7 @@ module Cellar
         @signer = signer
       end
 
-      def upload(key, io, checksum: nil, **)
+      def upload(key, io, checksum)
         with_io_length(io) do |io, length|
           request = Net::HTTP::Put.new("/#{key}")
           request.content_type = 'application/octet-stream'

lib/tasks/task_helper.rb

@@ -45,7 +45,9 @@ class ProgressReport
       @count = count
       @total = [@count, @total].max
     end
-    @per_10_000 = 10_000 * @count / @total
+    if @total&.nonzero?
+      @per_10_000 = 10_000 * @count / @total
+    end
   end
 
   def print_progress
@@ -53,10 +55,14 @@ class ProgressReport
     percent = sprintf('%5.1f%%', @per_10_000 / 100.0)
     total = @total.to_s
     count = @count.to_s.rjust(total.length)
-    rake_print("\r#{percent} (#{count}/#{total}) [#{format_duration(elapsed)}/#{format_duration(elapsed * 10_000 / @per_10_000)}]")
+    rake_print("\r#{percent} (#{count}/#{total}) [#{format_duration(elapsed)}/#{format_duration(elapsed * 10_000.0 / @per_10_000)}]")
   end
 
   def format_duration(seconds)
-    Time.at(seconds).utc.strftime('%H:%M:%S')
+    if seconds.finite?
+      Time.at(seconds).utc.strftime('%H:%M:%S')
+    else
+      '--:--:--'
+    end
   end
 end