Skip to content

Fix is_hidden patch error#63

Merged
grigoriev-semyon merged 1 commit intomainfrom
is-hidden-update-fix
Aug 11, 2024
Merged

Fix is_hidden patch error#63
grigoriev-semyon merged 1 commit intomainfrom
is-hidden-update-fix

Conversation

@DaymasS
Copy link
Copy Markdown
Contributor

@DaymasS DaymasS commented Aug 11, 2024

Изменения

Исправлена ошибка, из-за которой при попытке сделать кнопку снова видимой без изменения каких-либо других полей выдавалась ошибка 400 Empty schema

Детали реализации

Проверка на получение пустой схемы в patch button теперь производится через сравнивание каждого поля с None, а не через булевое значение каждого поля.

Также добавлен тест по данному изменению

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?

@DaymasS DaymasS self-assigned this Aug 11, 2024
@github-actions
Copy link
Copy Markdown

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
services_backend
   __main__.py440%1–7
   schemas.py8450%6–9
services_backend/models
   base.py13469%19–22
   database.py83396%81–82, 102
services_backend/routes
   button.py1965970%83, 113–148, 164–165, 169, 174, 180, 213, 218, 244, 250, 253, 255, 279, 281, 296–327
   category.py1191092%83, 88–89, 114–118, 148, 180–181
TOTAL4578482% 

Summary

Tests Skipped Failures Errors Time
38 0 💤 0 ❌ 0 🔥 1.786s ⏱️

@grigoriev-semyon grigoriev-semyon merged commit 9de68dc into main Aug 11, 2024
@grigoriev-semyon grigoriev-semyon deleted the is-hidden-update-fix branch August 11, 2024 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants