Don't depend on Redis directly

Author: ngan

Gemfile

@@ -9,4 +9,5 @@ gem "rake"
 gem "rspec"
 gem "standard"
 gem "fakeredis"
+gem "redis"
 gem "connection_pool"

Gemfile.lock

@@ -3,7 +3,6 @@ PATH
   specs:
     bigrails-redis (0.3.0)
       rails (>= 6)
-      redis (~> 4.0)
 
 GEM
   remote: https://rubygems.org/
@@ -206,6 +205,7 @@ DEPENDENCIES
   connection_pool
   fakeredis
   rake
+  redis
   rspec
   standard
 

bigrails-redis.gemspec

@@ -22,5 +22,4 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
 
   spec.add_dependency "rails", ">= 6"
-  spec.add_dependency "redis", "~> 4.0"
 end

lib/big_rails/redis/registry.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require "redis"
-
 module BigRails
   module Redis
     class Registry
@@ -61,10 +59,10 @@ def build_connection(name)
       end
 
       def build_wrapped_connection(connection)
-        if connection.is_a?(::Redis)
-          connection
-        else
+        if connection.is_a?(::ConnectionPool)
           ::ConnectionPool.wrap(pool: connection)
+        else
+          connection
         end
       end