feature: redis fixtures support #176
Conversation
README-ru.md
Outdated
| - `-db_dsn <...>` dsn для вашей тестовой SQL базы данных (бд будет очищена перед наполнением!), поддерживается только PostgreSQL | ||
| - `-aerospike_host <...>` при использовании Aerospike - URL для подключения к нему в формате `host:port/namespace` | ||
| - `-redis_addr <...>` при использовании Redis - адрес для подключения к Redis в формате `host:port` |
There was a problem hiding this comment.
я бы сделал наверное возможность полную ссылку писать (redis://...),
go-redis/redis умеет из ссылки парсить достаточное большое кол-во параметров, чтобы потом не возиться с паролями и прочим
There was a problem hiding this comment.
Как я понял с паролями в адресе он работать не может к сожалению. Пароль и пользователь передаются в отдельных параметрах.
There was a problem hiding this comment.
У go-redis есть функция ParseURL я бы заиспользовал ее. Да и вообще впринципе URL проще расширять при необходимости в обратно совместимом режиме, не добавляя новые параметры. В горедис достаточная функция, но вроде без поддержки кластера/сентинела, что впринципе для тестирования не нужно. В крайнем случае можно будет реализовать парсинг ссылки самостоятельно потом и сделать UniversalClient.
anemtarev
force-pushed
the
redis_fixtures
branch
2 times, most recently
from
c52c022
to
44824b9
Compare
anemtarev
force-pushed
the
redis_fixtures
branch
from
44824b9
to
e6f086d
Compare
|
🚀 PR was released in |
Add redis fixtures support.
All major redis types are supported:
Also implement the way to use custom fixture loader for gonkey library mode. That allows usage third-party fixture loaders for tests.