exists? fails with undefined method >' for true:TrueClass`
#922
Comments
|
Could you provide the backtrace please? Otherwise it's hard to answer. |
|
Also what's the version of your Redis server? |
|
Sure, the backtrace: Redis-server: |
|
I'm afraid something must be wrong with your Redis client. We specifically test that code path, and I just tried it again manually it works just fine, and >> Redis.new._client.call([:exists, 'a'])
=> 1I also don't see anything wrong in I see it breaks in your test suite, aren't you using Are you using a custom connection implementation? Anyway, at this stage I can only assume the issue is with your app, and you will have to investigate the problem. |
|
We're indeed using fakeredis, at least it is listed in the gemfile. I do not see any Anyway, thank you very much for the fast replies! I'll update this issue with any news, maybe others have similar problems :) |
If it's in your Gemfile, it's required by default. |
|
Looks like someone opened a PR already: guilleiguaran/fakeredis#247 |
|
Yeah we have fakredis only in Again, thank you very much. I'll wait for the fix in fakeredis before updating redis-rb. |
|
Since the issue is determined to be 3rd party, I'll close. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After updating redis-rb gem to 4.2.1 our tests fail with
undefined method>' for true:TrueClass. I'm not sure if this is the fault of theredis-session-store` gem or not.It seems to come from #918 and this method:
In my examples
valueis true, which results in the error above. A fix could be checking ifkeys.count > 1and then doing thevalue > 0thing, and if not, just return value. But not sure about that. Any ideas? :)The text was updated successfully, but these errors were encountered: