mennanov / limiters

Golang rate limiters for distributed applications
https://godoc.org/github.com/mennanov/limiters
MIT License
458 stars 49 forks source link

add redis cluster client tests #49

Closed leeym closed 5 months ago

leeym commented 5 months ago

I should have done this in https://github.com/mennanov/limiters/pull/48, but I think it is not too late to do it now.

The existing test environment only has single node Redis and there is no Redis cluster. Given that the Redis and Redis cluster may behave differently, I think we should test them separately.

Also run it in two separate builds:

leeym commented 5 months ago

Not sure why there are only two checks and no tests executed. Close this PR and recreate one.