Merge pull request #6 from wallarm/fix-s3-download

Author: DmitryBochkarev

lib/cloud_storage/objects/s3.rb

@@ -38,7 +38,7 @@ def delete!
       private
 
       def internal_download(local_file)
-        @resource.bucket(bucket_name).object(key).download_file(local_file)
+        @client.get_object(response_target: local_file, bucket: bucket_name, key: key)
 
         local_file
       end

lib/cloud_storage/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CloudStorage
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

spec/cloud_storage/objects/s3_spec.rb

@@ -62,6 +62,16 @@
 
         expect(content).to eq("This is a test upload\n")
       end
+
+      context 'when 6mb file' do
+        let(:file) { File.open('spec/fixtures/6mb.bin', 'rb') }
+
+        it do
+          expect { content }.not_to(change { opened_tmp_files_count })
+
+          expect(content.size).to eq(6_000_000)
+        end
+      end
     end
 
     context 'when download to a custom tmp' do
@@ -83,6 +93,20 @@
           .to("This is a test upload\n")
           .and change { opened_tmp_files_count }.by(0)
       end
+
+      context 'when 6mb file' do
+        let(:file) { File.open('spec/fixtures/6mb.bin', 'rb') }
+
+        it do
+          tmp
+
+          expect { content }
+            .to change(tmp, :size)
+            .from(0)
+            .to(6_000_000)
+            .and change { opened_tmp_files_count }.by(0)
+        end
+      end
     end
   end
 end