Add some tests for NamespacedEnvCache

Author: ArtOfCode-

lib/namespaced_env_cache.rb

@@ -85,7 +85,7 @@ def write_collection(name, value, **opts)
         raise TypeError, "Can't cache more than one type of object via write_collection"
       end
 
-      data = [types[0], ...value.map(&:id)]
+      data = [types[0].to_s, ...value.map(&:id)]
       namespaced = construct_ns_key(name, include_community: include_community(opts))
       @underlying.write(namespaced, data, **opts)
     end
@@ -99,6 +99,7 @@ def write_collection(name, value, **opts)
     def read_collection(name, **opts)
       namespaced = construct_ns_key(name, include_community: include_community(opts))
       data = @underlying.read(namespaced, **opts)
+      return nil if data.nil?
       type = data.slice!(0)
       type.constantize.where(id: data)
     end

test/lib/namespaced_env_cache_test.rb

@@ -0,0 +1,34 @@
+require 'test_helper'
+
+class NamespacedEnvCacheTest < ActiveSupport::TestCase
+  setup :new_cache_store
+
+  test 'write_collection/read_collection' do
+    collection = Post.where(user: users(:standard_user))
+    assert_nothing_raised do
+      @cache.write_collection('test_posts_cache', collection)
+    end
+    data = @cache.read_collection('test_posts_cache')
+    assert_not_nil data
+    original_ids = collection.map(&:id)
+    cached_ids = data.map(&:id)
+    cached_ids.each do |cached|
+      assert_includes original_ids, cached
+    end
+  end
+
+  test 'fetch_collection' do
+    data = assert_nothing_raised do
+      @cache.fetch_collection('test_posts_cache') do
+        Post.where(user: users(:standard_user))
+      end
+    end
+    assert_not_nil data
+  end
+
+  private
+
+  def new_cache_store
+    @cache = QPixel::NamespacedEnvCache.new(ActiveSupport::Cache::MemoryStore.new)
+  end
+end