7
98%
main: 98%

Ran 21 Aug 2025 12:51AM UTC

Files 27

Run time 1s

Badge

Embed ▾

Committed 21 Aug 2025 12:50AM UTC coverage: 35.665%. First build

Job # 17114020698.7

Build Type

push

github

Committed by

web-flow

Commit Message

Fix `Database.call` with expire methods (#334)

# What?
* It fixes calls to `expire*` methods when the calling happens through
the `.call` method of MockRedis::Database
* It fixes a test that will inconsistently fail when we run tests one
after another, within a few seconds of each run

# Why?
`expire` values are not respected when we set these via the Database
object. So tests that depend on testing that we set the ttl correctly
will fail. This happens because the `.call` method fails to call
successfully `public_send` due to the need to parse out the kwarg
arguments.

# How?
The fix was to identify if we have an `expire` call in the command. If
that is the case, we call a `send_expire` method that will correctly
transform the call so that we can call those methods with the correct
format.

For the test, I noticed that adding a slight wait for 1 millisecond was
enough to consistently get that test passed

# Notes

I didn't add a version number because I didn't want to be be arrogant
and assume that the PR will be passed. I am happy to modify things or
try some other solution if you prefer that.

Thanks so much for such a useful gem

Run Details

678 of 1901 relevant lines covered (35.67%)

8.65 hits per line

sds / mock_redis / 17114020698 / 7
98%
main: 98%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job ruby3.3-7.0 - 17114020698.7

sds / mock_redis / 17114020698 / 7 98% main: 98%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job ruby3.3-7.0 - 17114020698.7

sds / mock_redis / 17114020698 / 7
98%
main: 98%

README BADGES
x