Тимонова - Лабораторная работа #3 #18
Conversation
|
@GodfatherThe |
There was a problem hiding this comment.
Все-таки решила перейти на 2.7? :)
Но as tk можно было бы оставить, чтобы не менять кучу существующего кода.
There was a problem hiding this comment.
да, сменила версию, но поменять все-таки пришлось, так как согласно PEP-8 naming conventions CamelCase переменая на должна быть импортирована в нижнем регистре.
|
Еще не увидел отображения лога на форме. Вроде как в требованиях прописано, что такая возможность должна быть. |
There was a problem hiding this comment.
Я бы название файла изменил, убрав прочерк. А то получается как-то забавно.
-- Hey, I module
-- Hey! I code
-- Hello guys! I logger!
Обычно i не отделяется от названия классов и файлов.
There was a problem hiding this comment.
Мне кажется, достаточно четко написано, какое именно состояние тестируется - состояние после добавления сообщения в лог список.
Так вот, у вас у логера нет состояния. И в других тестах вы уже тестируете не состояния, а вполне конкретные факты, что можно добавить в лог несколько сообщений и получить последнюю запись. А в этом тесте непонятное "состояние". Более логичным было бы назвать test_can_log_one_message, например.
There was a problem hiding this comment.
это как с to log and log: кому-то понятно, кому-то нет.
|
@GodfatherThe |
|
👍 |
|
👍 |
|
@AoD314 |
Тимонова - Лабораторная работа #3
No description provided.