Add failing test for EXPIRE and fix it too

vapor / redis

Vapor provider for RediStack

MIT License

459 stars 58 forks source link

Add failing test for EXPIRE and fix it too #122

Closed vzsg closed 6 years ago

vzsg commented 6 years ago

Calling expire in the current tagged version throws an error: ⚠️ Redis Error: ERR Protocol error: expected '$', got ':', which means that the expiration time is expected as a bulk string, not as an integer literal on the wire.

This PR adds a test and fixes the issue.

penny-coin commented 6 years ago

Hey @vzsg, you just merged a pull request, have a coin!

You now have 1503 coins.