Skip to content

Conversation

@lechefer
Copy link

No description provided.

@vitkarpenko
Copy link
Contributor

Привет, спасибо за PR! Можешь, пожалуйста, добавить небольшое описание по использованию в ридми? Примерно так https://github.com/lamoda/gonkey#aerospike

Denis Chervinskiy added 4 commits October 25, 2023 01:49
- Поправил замечания из PR
- Исправлены баги с $ операторами
@lechefer
Copy link
Author

В первой итерации не заметил, что фактически не реализовал поддержку $ операторов. Текущие тесты покрывают только парсинг тестов, но не их выполнение в бд. Как вариант делать как и с PostgreSql, мокать и тестить так, но думаю лучше будет сделать подъём баз и прогон тестов на них. Собственно, так и удалось выявить проблемы

@vitkarpenko
Copy link
Contributor

Давай, пожалуйста, пока что сделаем хотя бы минимальные тесты на моках, чтобы в целом понимать, что идут запросы в базу какие-то и они выглядят правильно. Про интеграционные тесты на фикстуры хорошая идея, заведу на это отдельный issue чтобы этот PR не раздувался.

P. S. Сорри за задержку в ответе

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants