Kitura / Kitura-redis

Swift Redis library
Apache License 2.0
95 stars 26 forks source link

Added extra unit tests #38

Closed shmuelk closed 7 years ago

shmuelk commented 7 years ago

Description

Added extra tests for various APIs and error conditions that weren't tested. I also enabled some tests that were commented out due to Redis server version issues in the past. Lastly the Redis.info test now writes out the server version to the log.

Motivation and Context

Improve our test coverage

How Has This Been Tested?

Ran swift test on both Linux and macOS.

Checklist:

youming-lin commented 7 years ago

Looks good. Should all the tests have expectations, or just selected ones?

shmuelk commented 7 years ago

Just the ones that do. Eventually we need to go over the Kitura-redis tests and use the expectation paradigm all over the place.