Skip to content

Рефакторинг кода#3

Open
MariaKhodorova wants to merge 4 commits intomainfrom
refactor/style-guidelines
Open

Рефакторинг кода#3
MariaKhodorova wants to merge 4 commits intomainfrom
refactor/style-guidelines

Conversation

@MariaKhodorova
Copy link
Copy Markdown
Owner

@MariaKhodorova MariaKhodorova commented May 6, 2025

issue: #4

@MariaKhodorova
Copy link
Copy Markdown
Owner Author

В рефакторинге тестов были внесены следующие изменения:

  1. Использование методов assert из unittest: Все стандартные assert заменены на методы self.assertIsNotNone, self.assertGreater, self.assertIn, self.assertEqual, что улучшает читаемость и дает более информативные сообщения об ошибках.
  2. Добавление сообщений об ошибках: Все утверждения сопровождаются сообщениями, что помогает легко находить причины неудачи.
  3. Использование контекстных менеджеров: Для проверки исключений применён with self.assertRaises(), что является лучшей практикой.
  4. Переименование переменных: Переменные были переименованы для соблюдения соглашений о наименованиях и улучшения читаемости.
  5. Упрощение логики и улучшение структуры: Логика тестов была упрощена, удалены лишние print() вызовы, добавлены комментарии для лучшего понимания тестов.

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.

1 participant