Fix crash in tests: Use explicit RedisCommand instead of tuple by fabianfett · Pull Request #52 · swift-server/RediStack
Navigation Menu
{{ message }}
swift-server / RediStack Public
- Notifications You must be signed in to change notification settings
- Fork 30
Merged
Fix crash in tests: Use explicit RedisCommand instead of tuple#52
fabianfett merged 2 commits intomainfrom
Fix crash in tests: Use explicit RedisCommand instead of tuple#52
fabianfett merged 2 commits intomainfrom
Conversation
Copy link Copy Markdown
Member
fabianfett
commented
Jun 16, 2023
fabianfett
commented
Motivation
Currently the RediStack tests crash for our new main branch. We should have working tests.
Changes
- Use explicit
RedisCommandinstead of tuple
fabianfett
changed the title
Fix crash in 1.x tests
Fix crash in tests: Use explicit RedisCommand instead of tuple
fabianfett
mentioned this pull request
Merged
fabianfett
added this to the
Transfer milestone
Joannis approved these changes Jun 16, 2023
Respect `REDIS_URL` in all tests
fabianfett
force-pushed
the
ff-fix-1.x-tests
branch
from
863b9c5 to
9185279
Compare
fabianfett
merged commit
35e0d67
into
main
fabianfett
deleted the
ff-fix-1.x-tests
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment