go-redis / redismock

Redis client Mock
https://redis.uptrace.dev
BSD 2-Clause "Simplified" License
280 stars 60 forks source link

Expect for Module Loadex #69

Closed SoulPancake closed 1 year ago

SoulPancake commented 1 year ago

https://github.com/redis/go-redis/pull/2490 We need to write mocks for this And not sure what can we use from this package for that purpose Can someone suggest on this? Or do we need to build that as well?

SoulPancake commented 1 year ago

@monkey92t Can I raise a PR for ExpectModuleLoadex?

monkey92t commented 1 year ago

The purpose of redismock is to test existing API, not to create API that don't exist.

ktsivkov commented 1 year ago

@monkey92t Does that mean, that in order to create a Pull Request for this project, we will first need to have the PR https://github.com/redis/go-redis/pull/2490 for redis merged?