rails-brotli-cache 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 44c290253b45a487daecc85b45026bd7d3f140644095c7f15cfb67dcee1299b8
-  data.tar.gz: d37323f26dda367401403714eb094dfd6a0e3274b7d88cb81eb2d5380ce91979
+  metadata.gz: ed8813c2a9849358eb4d1756cd39eda4acf03846ea166880b956d9c630a5f3ee
+  data.tar.gz: cdff030d0d6bdd7e2b7790c5aea0441ecc89e184165c38e899dc5abdc7f5c900
 SHA512:
-  metadata.gz: 537fe80385fbc5f8d339b56f789aaeefc45e2589456e2fb522afdcb3656af6254951654a0524b5136ad3cb25fb23c092ea6bc027377c39ed9f52645886a94fc0
-  data.tar.gz: a6057e3a65505003ee849e9f0a38a12963a78468c32275ca0e1d70ead490ddba7663d2a1d65b4359ca7fb626b27e15fc8552d77d9d35c353bbdf77f933fdb277
+  metadata.gz: 5bf99730b2b31ed9591157fe75764fb15c17cf84cfd664d148a339e8552ed27dc62512cdff39f84552bc151aa4105d8720546eef38aa9e2da0552427d05ab86f
+  data.tar.gz: 7aa77827f595e73d55dc838ac8c6159a3f0f9138ed5bcdf57b2a0876060e01b66704dfd93f2eb8dc6502d109a0933f825bfc9d422858cbb786b03a75569f32d6

data/README.md

@@ -101,12 +101,13 @@ config.cache_store = RailsBrotliCache::Store.new(
 )
 ```
 
-Gem appends `br-` to the cache key names to prevent conflicts with previously saved cache entries. You can disable this behaviour by adding the following initializer file:
-
-`app/config/initializers/rails-brotli-cache.rb`
+Gem appends `br-` to the cache key names to prevent conflicts with previously saved cache entries. You can disable this behaviour by passing `{ prefix: nil }` during initialization:
 
 ```ruby
-Rails.cache.disable_prefix!
+config.cache_store = RailsBrotliCache::Store.new(
+  ActiveSupport::Cache::MemoryStore.new,
+  { prefix: nil }
+)
 ```
 
 ## Testing

data/lib/rails-brotli-cache/store.rb

@@ -9,9 +9,15 @@ module RailsBrotliCache
     COMPRESS_QUALITY = ENV.fetch("BR_CACHE_COMPRESS_QUALITY", 5).to_i
     MARK_BR_COMPRESSED = "\x02".b
 
+    attr_reader :core_store
+
     def initialize(core_store, options = {})
       @core_store = core_store
-      @prefix = "br-"
+      @prefix = if options.key?(:prefix)
+        options.fetch(:prefix)
+      else
+        "br-"
+      end
     end
 
     def fetch(name, options = nil, &block)
@@ -74,10 +80,6 @@ module RailsBrotliCache
       @core_store.clear
     end
 
-    def disable_prefix!
-      @prefix = nil
-    end
-
     private
 
     def cache_key(name)

data/lib/rails-brotli-cache/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsBrotliCache
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

data/spec/rails-brotli-cache/redis_spec.rb

@@ -5,9 +5,14 @@ require 'spec_helper'
 return unless ENV['RAILS_CACHE_STORE'] == 'redis_cache_store'
 
 describe RailsBrotliCache do
+  let(:options) do
+    {}
+  end
+
   subject(:cache_store) do
     RailsBrotliCache::Store.new(
-      ActiveSupport::Cache::RedisCacheStore.new(redis: $redis)
+      ActiveSupport::Cache::RedisCacheStore.new(redis: $redis),
+      options
     )
   end
 
@@ -28,16 +33,25 @@ describe RailsBrotliCache do
     expect($redis.get("gz-test-key").size > $redis.get("br-test-key").size).to eq true
   end
 
-  describe "disable_prefix!" do
-    it "saves brotli cache entries without `br-` prefix" do
-      cache_store.fetch("test-key") { 123 }
-      expect($redis.get("test-key")).to eq nil
-      expect($redis.get("br-test-key")).to be_present
-      cache_store.disable_prefix!
-      cache_store.fetch("test-key-2") { 123 }
-      expect($redis.get("br-test-key-2")).to eq nil
-      expect($redis.get("test-key-2")).to be_present
-      cache_store.instance_variable_set(:@prefix, "br-")
+  describe "disable_prefix" do
+    context "default prefix" do
+      it "appends 'br-' prefix" do
+        cache_store.fetch("test-key") { 123 }
+        expect($redis.get("test-key")).to eq nil
+        expect($redis.get("br-test-key")).to be_present
+      end
+    end
+
+    context "no prefix" do
+      let(:options) do
+        { prefix: nil }
+      end
+
+      it "saves brotli cache entries without `br-` prefix" do
+        cache_store.fetch("test-key") { 123 }
+        expect($redis.get("br-test-key")).to eq nil
+        expect($redis.get("test-key")).to be_present
+      end
     end
   end
 end

data/spec/rails-brotli-cache/store_spec.rb

@@ -40,6 +40,12 @@ describe RailsBrotliCache do
     end
   end
 
+  describe "#core_store" do
+    it "exposes the underlying data store" do
+      expect(cache_store.core_store.class).to eq ActiveSupport::Cache::MemoryStore
+    end
+  end
+
   describe "#read and #write" do
     it "reads values stored in Rails cache with a prefix" do
       expect(cache_store.read("test-key")).to eq nil
@@ -67,12 +73,4 @@ describe RailsBrotliCache do
       expect(cache_store.read("test-key")).to eq nil
     end
   end
-
-  describe "disable_prefix!" do
-    it "saves brotli cache entries without `br-` prefix" do
-      cache_store.disable_prefix!
-      cache_store.fetch("test-key") { 123 }
-      cache_store.instance_variable_set(:@prefix, "br-")
-    end
-  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-brotli-cache
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - pawurb