use global redis variable for Redis instance

config/coverband.rb

@@ -2,13 +2,8 @@
 
 # config/coverband.rb NOT in the initializers
 Coverband.configure do |config|
-  redis = if Rails.env.test?
-    require 'mock_redis'
-    MockRedis.new(url: Settings.redis.app_data.url)
-  else
-    Redis.new(url: Settings.redis.app_data.url)
-  end
-  config.store = Coverband::Adapters::RedisStore.new(redis)
+
+  config.store = Coverband::Adapters::RedisStore.new($redis)
   config.logger = Rails.logger
 
   # config options false, true. (defaults to false)

config/initializers/breakers.rb

@@ -40,15 +40,7 @@
 
 Rails.application.reloader.to_prepare do
 
-  redis = if Rails.env.test?
-    require 'mock_redis'
-    MockRedis.new(url: REDIS_CONFIG[:redis][:url])
-  else
-    Redis.new(REDIS_CONFIG[:redis].to_h)
-  end
-
-  redis_namespace = Redis::Namespace.new('breakers', redis: redis)
-
+  redis_namespace = Redis::Namespace.new('breakers', redis: $redis)
 
   services = [
     DebtManagementCenter::DebtsConfiguration.instance.breakers_service,

config/initializers/rack_attack.rb

@@ -8,14 +8,7 @@ def remote_ip
     end
   end
 
-  redis = if Rails.env.test?
-    require 'mock_redis'
-    MockRedis.new(url: REDIS_CONFIG[:redis][:url])
-  else
-    Redis.new(REDIS_CONFIG[:redis].to_h)
-  end
-
-  Rack::Attack.cache.store = Rack::Attack::StoreProxy::RedisStoreProxy.new(redis)
+  Rack::Attack.cache.store = Rack::Attack::StoreProxy::RedisStoreProxy.new($redis)
 
   throttle('example/ip', limit: 1, period: 5.minutes) do |req|
     req.ip if req.path == '/v0/limited'

modules/meb_api/config/initializers/breakers.rb

@@ -11,14 +11,7 @@
 
 Rails.application.reloader.to_prepare do
 
-  redis = if Rails.env.test?
-    require 'mock_redis'
-    MockRedis.new(url: REDIS_CONFIG[:redis][:url])
-  else
-    Redis.new(REDIS_CONFIG[:redis].to_h)
-  end
-
-  redis_namespace = Redis::Namespace.new('breakers', redis: redis)
+  redis_namespace = Redis::Namespace.new('breakers', redis: $redis)
 
   services = [
     MebApi::DGI::Configuration.instance.breakers_service,

spec/requests/breakers_integration_spec.rb

@@ -30,7 +30,6 @@
     # Not clearing the breakers would cause subsequent RX calls to fail after the breaker is
     # triggered in this group.
 
-    # fakeredis/rspec has a `before` callback, but it's for the suite, not each example. Oops.
     Breakers.client.redis_connection.redis.flushdb
   end
 

spec/spec_helper.rb

@@ -195,9 +195,7 @@
   end
 
   config.before(:each) do
-    mock_redis = MockRedis.new(url: REDIS_CONFIG[:redis][:url])
-    allow(Redis).to receive(:new).and_return(mock_redis)
-    mock_redis.flushdb
+    $redis.flushdb
   end
 
 end