Utilisation de Redis.new plutôt que de la variable globale $redis dans les tests

Author: echarp

test/controllers/admin/banners_controller_test.rb

@@ -9,7 +9,7 @@ class Admin::BannersControllerTest < ActionDispatch::IntegrationTest
 
   teardown do
     # Reset
-    $redis.del('banners')
+    Redis.new.del('banners')
   end
 
   test 'should list banners' do

test/controllers/moderation/news_controller_test.rb

@@ -126,7 +126,7 @@ class Moderation::NewsControllerTest < ActionDispatch::IntegrationTest
     assert_redirected_to moderation_news_url news(:news)
 
     # Reset
-    $redis.del('nodes/ppp')
+    Redis.new.del('nodes/ppp')
   end
 
   test 'should get vote' do

test/controllers/moderation/polls_controller_test.rb

@@ -78,6 +78,6 @@ class Moderation::PollControllerTest < ActionDispatch::IntegrationTest
     assert_redirected_to root_url
 
     # Reset
-    $redis.del('nodes/ppp')
+    Redis.new.del('nodes/ppp')
   end
 end

test/controllers/polls_controller_test.rb

@@ -67,7 +67,7 @@ class PollsControllerTest < ActionDispatch::IntegrationTest
 
   test 'should vote' do
     # Remove eventual existing vote
-    $redis.del "polls/#{polls(:one).id}/127.0.0.1"
+    Redis.new.del "polls/#{polls(:one).id}/127.0.0.1"
     sign_in accounts :joe
 
     post vote_poll_url polls(:one), position: 0
@@ -80,7 +80,7 @@ class PollsControllerTest < ActionDispatch::IntegrationTest
 
   test 'should not vote' do
     # Remove eventual existing vote
-    $redis.del "polls/#{polls(:one).id}/127.0.0.1"
+    Redis.new.del "polls/#{polls(:one).id}/127.0.0.1"
     sign_in accounts :joe
 
     post vote_poll_url polls(:one)

test/controllers/relevances_controller_test.rb

@@ -5,7 +5,7 @@ class RelevancesControllerTest < ActionDispatch::IntegrationTest
 
   test 'should vote for' do
     sign_in accounts 'visitor_0'
-    $redis.del "comments/#{comments(:one).id}/votes/for", accounts('visitor_0').id
+    Redis.new.del "comments/#{comments(:one).id}/votes/for", accounts('visitor_0').id
 
     assert_difference 'comments(:one).nb_votes_for' do
       post relevance_for_node_comment_url nodes(:tracker_one), comments(:one), format: :json
@@ -18,7 +18,7 @@ class RelevancesControllerTest < ActionDispatch::IntegrationTest
 
   test 'should vote against' do
     sign_in accounts 'visitor_2'
-    $redis.del "comments/#{comments(:one).id}/votes/against", accounts('visitor_2').id
+    Redis.new.del "comments/#{comments(:one).id}/votes/against", accounts('visitor_2').id
 
     assert_difference 'comments(:one).nb_votes_against' do
       post relevance_against_node_comment_url nodes(:tracker_one), comments(:one), format: :json

test/controllers/votes_controller_test.rb

@@ -6,7 +6,7 @@ class VotesControllerTest < ActionDispatch::IntegrationTest
   test 'should vote for' do
     sign_in accounts 'admin_0'
 
-    $redis.del "nodes/#{nodes(:one).id}/votes/#{accounts('admin_0').id}"
+    Redis.new.del "nodes/#{nodes(:one).id}/votes/#{accounts('admin_0').id}"
     assert_difference 'nodes(:one).score' do
       post vote_for_node_url nodes(:one)
 
@@ -29,7 +29,7 @@ class VotesControllerTest < ActionDispatch::IntegrationTest
   test 'should vote against' do
     sign_in accounts 'admin_0'
 
-    $redis.del "nodes/#{nodes(:one).id}/votes/#{accounts('admin_0').id}"
+    Redis.new.del "nodes/#{nodes(:one).id}/votes/#{accounts('admin_0').id}"
     assert_difference 'nodes(:one).score', -1 do
       post vote_against_node_url nodes(:one)