flipper-active_support_cache_store 0.21.0.rc1 → 0.22.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69675aef1db8485e9f3cb21398969b3031a3d3040b37356861c963d28acc4dff
-  data.tar.gz: aefece2a0e08906eedec01aae748c3abcb7b1e6b5ff1dd8366a6c569068dd7d7
+  metadata.gz: 869bf3e6899bd729e79ff1d388e9d16d3e4794729cfadf8957661bd9333d41b6
+  data.tar.gz: bc352e5202e188faae59bb17dd34ac7723f5237d4d1d6965f76d1d9c582446e1
 SHA512:
-  metadata.gz: 4a4d65cb32750bd93c37dce86702e7b11e3d945a4cfe1c385c452f5ee8306ff3be2f29163d704f94a19d3462518f4a00f7db77ed8a0cb84acb4130208595ee23
-  data.tar.gz: fa9d79f5c02b4a053242fdbc7dec9c63630aa905c1beca9f3319d4520598fe46887438162104b008620ef92c8c8feb666948868966153ead918ad9b43b5f7a32
+  metadata.gz: 5d3b7f373dbbe26df81d34b6b1a8cf09649acd611ee39f94658c553d9f989aae4a2551ac754e20a3a5881a5ce03b4f6b53a21cb6cb74170709dcdf4426a33f6d
+  data.tar.gz: 4d72face3fe6025d5ca86a86656d7ef1ce409ad2158e5bcc8366c3533b4f54e54856d869d9e69b7c9059761b950ba0a11ebfe8df00376640d42679f7eb85db6d

data/docs/active_support_cache_store/README.md

@@ -22,11 +22,17 @@ Or install it yourself with:
 require 'active_support/cache'
 require 'flipper/adapters/active_support_cache_store'
 
-memory_adapter = Flipper::Adapters::Memory.new
-cache = ActiveSupport::Cache::MemoryStore.new
-adapter = Flipper::Adapters::ActiveSupportCacheStore.new(memory_adapter, cache, expires_in: 5.minutes)
-flipper = Flipper.new(adapter)
+Flipper.configure do |config|
+  config.adapter do
+    Flipper::Adapters::ActiveSupportCacheStore.new(
+      Flipper::Adapters::Memory.new,
+      ActiveSupport::Cache::MemoryStore.new, # Or Rails.cache
+      expires_in: 5.minutes
+    )
+  end
+end
 ```
+
 Setting `expires_in` is optional and will set an expiration time on Flipper cache keys.  If specified, all flipper keys will use this `expires_in` over the `expires_in` passed to your ActiveSupport cache constructor.
 
 ## Internals

data/flipper-active_support_cache_store.gemspec

@@ -21,5 +21,5 @@ Gem::Specification.new do |gem|
   gem.metadata      = Flipper::METADATA
 
   gem.add_dependency 'flipper', "~> #{Flipper::VERSION}"
-  gem.add_dependency 'activesupport', '>= 5.0', '< 7'
+  gem.add_dependency 'activesupport', '>= 4.2', '< 7'
 end

data/lib/flipper/adapters/active_support_cache_store.rb

@@ -25,12 +25,13 @@ module Flipper
       attr_reader :name
 
       # Public
-      def initialize(adapter, cache, expires_in: nil)
+      def initialize(adapter, cache, expires_in: nil, write_through: false)
         @adapter = adapter
         @name = :active_support_cache_store
         @cache = cache
         @write_options = {}
         @write_options[:expires_in] = expires_in if expires_in
+        @write_through = write_through
       end
 
       # Public
@@ -49,7 +50,13 @@ module Flipper
       def remove(feature)
         result = @adapter.remove(feature)
         @cache.delete(FeaturesKey)
-        @cache.delete(key_for(feature.key))
+
+        if @write_through
+          @cache.write(key_for(feature.key), default_config, @write_options)
+        else
+          @cache.delete(key_for(feature.key))
+        end
+
         result
       end
 
@@ -88,14 +95,26 @@ module Flipper
       ## Public
       def enable(feature, gate, thing)
         result = @adapter.enable(feature, gate, thing)
-        @cache.delete(key_for(feature.key))
+
+        if @write_through
+          @cache.write(key_for(feature.key), @adapter.get(feature), @write_options)
+        else
+          @cache.delete(key_for(feature.key))
+        end
+
         result
       end
 
       ## Public
       def disable(feature, gate, thing)
         result = @adapter.disable(feature, gate, thing)
-        @cache.delete(key_for(feature.key))
+
+        if @write_through
+          @cache.write(key_for(feature.key), @adapter.get(feature), @write_options)
+        else
+          @cache.delete(key_for(feature.key))
+        end
+
         result
       end
 

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '0.21.0.rc1'.freeze
+  VERSION = '0.22.1'.freeze
 end

data/spec/flipper/adapters/active_support_cache_store_spec.rb

@@ -10,7 +10,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
     Flipper::Adapters::OperationLogger.new(Flipper::Adapters::Memory.new)
   end
   let(:cache) { ActiveSupport::Cache::MemoryStore.new }
-  let(:adapter) { described_class.new(memory_adapter, cache, expires_in: 10.seconds) }
+  let(:write_through) { false }
+  let(:adapter) { described_class.new(memory_adapter, cache, expires_in: 10.seconds, write_through: write_through) }
   let(:flipper) { Flipper.new(adapter) }
 
   subject { adapter }
@@ -22,11 +23,75 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
   it_should_behave_like 'a flipper adapter'
 
   describe '#remove' do
-    it 'expires feature' do
-      feature = flipper[:stats]
+    let(:feature) { flipper[:stats] }
+
+    before do
       adapter.get(feature)
       adapter.remove(feature)
-      expect(cache.read(described_class.key_for(feature))).to be(nil)
+    end
+
+    it 'expires feature and deletes the cache' do
+      expect(cache.read(described_class.key_for(feature))).to be_nil
+      expect(cache.exist?(described_class.key_for(feature))).to be(false)
+      expect(feature).not_to be_enabled
+    end
+
+    context 'with write-through caching' do
+      let(:write_through) { true }
+
+      it 'expires feature and writes an empty value to the cache' do
+        expect(cache.read(described_class.key_for(feature))).to eq(adapter.default_config)
+        expect(cache.exist?(described_class.key_for(feature))).to be(true)
+        expect(feature).not_to be_enabled
+      end
+    end
+  end
+
+  describe '#enable' do
+    let(:feature) { flipper[:stats] }
+
+    before do
+      adapter.enable(feature, feature.gate(:boolean), flipper.boolean)
+    end
+
+    it 'enables feature and deletes the cache' do
+      expect(cache.read(described_class.key_for(feature))).to be_nil
+      expect(cache.exist?(described_class.key_for(feature))).to be(false)
+      expect(feature).to be_enabled
+    end
+
+    context 'with write-through caching' do
+      let(:write_through) { true }
+
+      it 'expires feature and writes to the cache' do
+        expect(cache.exist?(described_class.key_for(feature))).to be(true)
+        expect(cache.read(described_class.key_for(feature))).to include(boolean: 'true')
+        expect(feature).to be_enabled
+      end
+    end
+  end
+
+  describe '#disable' do
+    let(:feature) { flipper[:stats] }
+
+    before do
+      adapter.disable(feature, feature.gate(:boolean), flipper.boolean)
+    end
+
+    it 'disables feature and deletes the cache' do
+      expect(cache.read(described_class.key_for(feature))).to be_nil
+      expect(cache.exist?(described_class.key_for(feature))).to be(false)
+      expect(feature).not_to be_enabled
+    end
+
+    context 'with write-through caching' do
+      let(:write_through) { true }
+
+      it 'expires feature and writes to the cache' do
+        expect(cache.exist?(described_class.key_for(feature))).to be(true)
+        expect(cache.read(described_class.key_for(feature))).to include(boolean: nil)
+        expect(feature).not_to be_enabled
+      end
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-active_support_cache_store
 version: !ruby/object:Gem::Version
-  version: 0.21.0.rc1
+  version: 0.22.1
 platform: ruby
 authors:
 - John Nunemaker
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-05-01 00:00:00.000000000 Z
+date: 2021-08-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,21 +16,21 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.21.0.rc1
+        version: 0.22.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.21.0.rc1
+        version: 0.22.1
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7'
@@ -40,7 +40,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
         version: '7'
@@ -73,9 +73,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.0.3
 signing_key: