Support for legacy in-memory storage behavior. In memory storage - persists encoding for upload / download

CHANGELOG.md

@@ -1,3 +1,8 @@
+v0.6.1
+------
+- Support for legacy in-memory storage behavior
+   * In memory storage - persists encoding for upload / download
+
 v0.6.0
 ------
 - [BREAKING] Remove support for Ruby 2.6

lib/bucket_store/in_memory.rb

@@ -26,7 +26,10 @@ def reset!
 
     def upload!(bucket:, key:, file:)
       file.tap do |f|
-        @buckets[bucket][key] = f.read
+        @buckets[bucket][key] = {
+          data: f.read,
+          encoding: file.external_encoding,
+        }
       end
 
       {
@@ -37,7 +40,10 @@ def upload!(bucket:, key:, file:)
 
     def download(bucket:, key:, file:)
       file.tap do |f|
-        f.write(@buckets[bucket].fetch(key))
+        f.set_encoding(
+          @buckets[bucket].fetch(key).fetch(:encoding),
+        )
+        f.write(@buckets[bucket].fetch(key).fetch(:data))
         f.rewind
       end
     end

lib/bucket_store/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BucketStore
-  VERSION = "0.6.0"
+  VERSION = "0.6.1"
 end

spec/bucket_store_integration_spec.rb

@@ -182,4 +182,27 @@
   include_examples "adapter integration", "disk://bucket"
   include_examples "adapter integration", "s3://bucket"
   include_examples "adapter integration", "gs://bucket"
+
+  context "inmemory storage" do
+    let(:bucket_store) { described_class.for("#{base_bucket_uri}/#{temp_filename}") }
+    let(:base_bucket_uri) { "inmemory://bucket" }
+    let(:temp_filename) { "temp-file" }
+
+    shared_examples "encoding persistance" do |encoding|
+      it "persists encoding between upload/download for #{encoding}", :aggregate_failures do
+        upload_data = String.new("some string", encoding: encoding)
+        bucket_store.upload!(upload_data)
+        download = bucket_store.download
+        download_data = download.fetch(:content)
+
+        expect(upload_data.encoding).to eq(encoding)
+        expect(download_data.encoding).to eq(encoding)
+        expect(upload_data).to eq(download_data)
+        expect(upload_data.object_id).to_not eq(download_data.object_id)
+      end
+    end
+
+    include_examples "encoding persistance", Encoding::UTF_8
+    include_examples "encoding persistance", Encoding::ISO_8859_1
+  end
 end