Skip to content

Commit

Permalink
Merge pull request #944 from supercaracal/fix-flaky-test-for-synchrony
Browse files Browse the repository at this point in the history
Fix flaky test case for synchrony driver
  • Loading branch information
byroot committed Sep 8, 2020
2 parents cf44d71 + 0da604f commit af087c5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/sentinel_test.rb
Expand Up @@ -30,7 +30,7 @@ def test_the_client_can_connect_to_available_slaves
end
}
RedisMock.start(commands) do |port|
redis = build_slave_role_client(sentinels: [{ host: 'localhost', port: port }])
redis = build_slave_role_client(sentinels: [{ host: '127.0.0.1', port: port }])
assert_equal 'PONG', redis.ping
end
end
Expand All @@ -45,7 +45,7 @@ def test_the_client_raises_error_when_there_is_no_available_slaves
end
}
RedisMock.start(commands) do |port|
redis = build_slave_role_client(sentinels: [{ host: 'localhost', port: port }])
redis = build_slave_role_client(sentinels: [{ host: '127.0.0.1', port: port }])
assert_raises(Redis::CannotConnectError) { redis.ping }
end
end
Expand Down

0 comments on commit af087c5

Please sign in to comment.