redis-rb / redis-client

Simple low level client for Redis 6+
MIT License
124 stars 60 forks source link

Add missing test for block in `ensure_connected`. #86

Closed shannoncole closed 1 year ago

shannoncole commented 1 year ago

Elsewhere in this method we test if a block was provided and only yield when necessary. There's one case missing, this change address that missing case.

casperisfine commented 1 year ago

Thank you. Would you mind adding a test to cover this branch?

byroot commented 1 year ago

Hum, actually, we never call the method with retriable: false and no block.