flipper-active_support_cache_store 1.0.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4af9c24a757e7b8a54a0aeb46b2ab060e7800c8d8c38c90daff37b046a0c3a13
-  data.tar.gz: dadc9bba7b9be524803b436548dd4cb1e67dac3ee3b8ea1b6f3bc53c58ebd980
+  metadata.gz: ac9ee92d7812cb26cb22b17a3927bfdd9be5ee24525333520f59fc244c04df71
+  data.tar.gz: 6fd4095421cc78c9141a694958a59eb0b1f89efcb0226768219290fb0bf6b1ac
 SHA512:
-  metadata.gz: 05f3321be57b5faf88b94bdde97c04d6716524b87947661a989d51c840f1437f8ba75e68ba86828bd582d66de4c3bf612e18dbaef72a7baef37cb92a2e0edcbd
-  data.tar.gz: 22e35550741299a38bba9867baec0915b293d5b93f78a4dba1696c374d8c87390c5a71ebb2b3f558dba2a72bf7c30b31c39125ae23f28e6dee23bdb7af94ff42
+  metadata.gz: a8b1984561fc79e70e3dc38615903014ee327a81614c97a6d7dfef4ffadc9d19c7477c66a8b44b029042a5345acca3126d77324941fee446ba60e3cbc2124239
+  data.tar.gz: '088af52428c6ec6c44f6bc5a2bd1d3f10772c859587fd8ca158a740fc14b56bb0765dcc7d3680dbf3271ec9f318340f55852ac1df64b2e41d9120e80d24e17c9'

data/lib/flipper/adapters/active_support_cache_store.rb

@@ -9,30 +9,21 @@ module Flipper
     class ActiveSupportCacheStore
       include ::Flipper::Adapter
 
-      Version = 'v1'.freeze
-      Namespace = "flipper/#{Version}".freeze
-      FeaturesKey = "#{Namespace}/features".freeze
-      GetAllKey = "#{Namespace}/get_all".freeze
-
-      # Private
-      def self.key_for(key)
-        "#{Namespace}/feature/#{key}"
-      end
-
       # Internal
       attr_reader :cache
 
-      # Public: The name of the adapter.
-      attr_reader :name
-
       # Public
       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
+
+        @cache_version = 'v1'.freeze
+        @namespace = "flipper/#{@cache_version}".freeze
+        @features_key = "#{@namespace}/features".freeze
+        @get_all_key = "#{@namespace}/get_all".freeze
       end
 
       # Public
@@ -43,14 +34,14 @@ module Flipper
       # Public
       def add(feature)
         result = @adapter.add(feature)
-        @cache.delete(FeaturesKey)
+        @cache.delete(@features_key)
         result
       end
 
       ## Public
       def remove(feature)
         result = @adapter.remove(feature)
-        @cache.delete(FeaturesKey)
+        @cache.delete(@features_key)
 
         if @write_through
           @cache.write(key_for(feature.key), default_config, @write_options)
@@ -80,12 +71,12 @@ module Flipper
       end
 
       def get_all
-        if @cache.write(GetAllKey, Time.now.to_i, @write_options.merge(unless_exist: true))
+        if @cache.write(@get_all_key, Time.now.to_i, @write_options.merge(unless_exist: true))
           response = @adapter.get_all
           response.each do |key, value|
             @cache.write(key_for(key), value, @write_options)
           end
-          @cache.write(FeaturesKey, response.keys.to_set, @write_options)
+          @cache.write(@features_key, response.keys.to_set, @write_options)
           response
         else
           features = read_feature_keys.map { |key| Flipper::Feature.new(key, self) }
@@ -122,12 +113,12 @@ module Flipper
       private
 
       def key_for(key)
-        self.class.key_for(key)
+        "#{@namespace}/feature/#{key}"
       end
 
       # Internal: Returns an array of the known feature keys.
       def read_feature_keys
-        @cache.fetch(FeaturesKey, @write_options) { @adapter.features }
+        @cache.fetch(@features_key, @write_options) { @adapter.features }
       end
 
       # Internal: Given an array of features, attempts to read through cache in

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '1.0.0'.freeze
+  VERSION = '1.1.1'.freeze
 end

data/spec/flipper/adapters/active_support_cache_store_spec.rb

@@ -28,8 +28,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
     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(cache.read("flipper/v1/feature/#{feature.key}")).to be_nil
+      expect(cache.exist?("flipper/v1/feature/#{feature.key}")).to be(false)
       expect(feature).not_to be_enabled
     end
 
@@ -37,8 +37,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore 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(cache.read("flipper/v1/feature/#{feature.key}")).to eq(adapter.default_config)
+        expect(cache.exist?("flipper/v1/feature/#{feature.key}")).to be(true)
         expect(feature).not_to be_enabled
       end
     end
@@ -52,8 +52,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
     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(cache.read("flipper/v1/feature/#{feature.key}")).to be_nil
+      expect(cache.exist?("flipper/v1/feature/#{feature.key}")).to be(false)
       expect(feature).to be_enabled
     end
 
@@ -61,8 +61,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore 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(cache.exist?("flipper/v1/feature/#{feature.key}")).to be(true)
+        expect(cache.read("flipper/v1/feature/#{feature.key}")).to include(boolean: 'true')
         expect(feature).to be_enabled
       end
     end
@@ -76,8 +76,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
     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(cache.read("flipper/v1/feature/#{feature.key}")).to be_nil
+      expect(cache.exist?("flipper/v1/feature/#{feature.key}")).to be(false)
       expect(feature).not_to be_enabled
     end
 
@@ -85,8 +85,8 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore 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(cache.exist?("flipper/v1/feature/#{feature.key}")).to be(true)
+        expect(cache.read("flipper/v1/feature/#{feature.key}")).to include(boolean: nil)
         expect(feature).not_to be_enabled
       end
     end
@@ -103,13 +103,13 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
       memory_adapter.reset
 
       adapter.get(stats)
-      expect(cache.read(described_class.key_for(search))).to be(nil)
-      expect(cache.read(described_class.key_for(other))).to be(nil)
+      expect(cache.read("flipper/v1/feature/#{search.key}")).to be(nil)
+      expect(cache.read("flipper/v1/feature/#{other.key}")).to be(nil)
 
       adapter.get_multi([stats, search, other])
 
-      expect(cache.read(described_class.key_for(search))[:boolean]).to eq('true')
-      expect(cache.read(described_class.key_for(other))[:boolean]).to be(nil)
+      expect(cache.read("flipper/v1/feature/#{search.key}")[:boolean]).to eq('true')
+      expect(cache.read("flipper/v1/feature/#{other.key}")[:boolean]).to be(nil)
 
       adapter.get_multi([stats, search, other])
       adapter.get_multi([stats, search, other])
@@ -128,9 +128,9 @@ RSpec.describe Flipper::Adapters::ActiveSupportCacheStore do
 
     it 'warms all features' do
       adapter.get_all
-      expect(cache.read(described_class.key_for(stats))[:boolean]).to eq('true')
-      expect(cache.read(described_class.key_for(search))[:boolean]).to be(nil)
-      expect(cache.read(described_class::GetAllKey)).to be_within(2).of(Time.now.to_i)
+      expect(cache.read("flipper/v1/feature/#{stats.key}")[:boolean]).to eq('true')
+      expect(cache.read("flipper/v1/feature/#{search.key}")[:boolean]).to be(nil)
+      expect(cache.read("flipper/v1/get_all")).to be_within(2).of(Time.now.to_i)
     end
 
     it 'returns same result when already cached' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-active_support_cache_store
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.1
 platform: ruby
 authors:
 - John Nunemaker
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-23 00:00:00.000000000 Z
+date: 2023-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 1.1.1
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement