Properly prefix redis snapshot key

Author: vincentvanbush

lib/lit/adapters/hybrid_storage.rb

@@ -35,7 +35,7 @@ def hash_snapshot= (timestamp)
   end
 
   def redis_snapshot
-    timestamp = Lit.redis.get('lit:_snapshot')
+    timestamp = Lit.redis.get(Lit.prefix + '_snapshot')
     if timestamp.nil?
       timestamp = Time.current.to_s
       Lit.redis_snapshot = timestamp
@@ -44,13 +44,21 @@ def redis_snapshot
   end
 
   def redis_snapshot= (timestamp)
-    Lit.redis.set('lit:_snapshot', timestamp)
+    Lit.redis.set(Lit.prefix + '_snapshot', timestamp)
   end
 
   def determine_redis_provider
     ENV[ENV['REDIS_PROVIDER'] || 'REDIS_URL']
   end
 
+  def prefix
+    pfx = 'lit:'
+    if Lit.storage_options.is_a?(Hash)
+      pfx += "#{Lit.storage_options[:prefix]}:" if Lit.storage_options.key?(:prefix)
+    end
+    pfx
+  end
+
   class HybridStorage
     def initialize
       Lit.redis
@@ -131,11 +139,7 @@ def sort
     private
 
     def _prefix
-      prefix = 'lit:'
-      if Lit.storage_options.is_a?(Hash)
-        prefix += "#{Lit.storage_options[:prefix]}:" if Lit.storage_options.key?(:prefix)
-      end
-      prefix
+      Lit.prefix
     end
 
     def _prefixed_key(key = '')

test/support/clear_snapshots.rb

@@ -1,7 +1,6 @@
 require 'lit/adapters/hybrid_storage'
 def clear_snapshots
-  trace_var :$_hash, proc { |h| print 'hash is now', v }
   Lit.reset_hash if defined?($_hash)
   Lit.hash_snapshot = nil if defined?($_hash_snapshot)
-  Lit.redis.del('lit:_snapshot') if defined?($redis)
+  Lit.redis.del(Lit.prefix + '_snapshot') if defined?($redis)
 end