Hw4 #3
Open
Hw4 #3
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Code Coverage
|
edikgoose
suggested changes
Mar 18, 2024
edikgoose
left a comment
edikgoose
left a comment
There was a problem hiding this comment.
Почини пайплайн, пожалуйста) (-1 балл)
Остальное все хорошо. Не совсем понял твой вопрос по ссылки, можешь поподробнее плиз расписать?
| - postgresql:/var/lib/postgresql/data | ||
| networks: | ||
| - backend | ||
| liquibase-migrations: |
There was a problem hiding this comment.
Миграции лучше хранить в модуле, в котором используется эта бдшечка. В твоем случае это scrapper. В resources закидай
| CREATE TABLE IF NOT EXISTS link | ||
| ( | ||
| id BIGINT PRIMARY KEY, | ||
| url VARCHAR(255) UNIQUE NOT NULL, |
There was a problem hiding this comment.
У нас обычно TEXT используется. В целом, во всем удобнее
| @CsvSource({"SELECT EXISTS (SELECT FROM pg_tables WHERE tablename = 'chat');", | ||
| "SELECT EXISTS (SELECT FROM pg_tables WHERE tablename = 'link');", | ||
| "SELECT EXISTS (SELECT FROM pg_tables WHERE tablename = 'chat_link');"}) | ||
| public void tableExistsTest(String query) { |
There was a problem hiding this comment.
Обычно миграции не тестят, но пусть будет)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Привет! Прости, что пропала, был завал на учебе :с
На базе IntegrationTest добавила запуск миграций + сделала небольшой тест для проверки миграции базы данных и создания нужных таблиц. Не уверена, стоит ли их разделять на 2 разных теста.
Хотела уточнить насчет разделения ссылок по сервисам. На данный момент оно актуально при добавлении в бд(в таблице link есть поле service_name) и получении списка отслеживаемых ссылок(так как группируем их по сервису). Пока что у меня из идей только фильтровать их по сервису при вызове \list, но тогда есть сценарий, когда бот будет многократно выполнять одну и ту же операцию за короткий промежуток времени. Есть ли какой-то хороший способ организовать это разделение(возможно, кэшировать этот список)?