Skip to content

Conversation

okainov
Copy link
Collaborator

@okainov okainov commented Dec 6, 2015

@TatianaKhanova @timonov @Ilyazykov @elenaPlankina @YusipovIgor @dmitryvodop, посмотрите, пожалуйста.

+Created setup in VM tests
In fact, real logger is just an extension of fake one
@okainov
Copy link
Collaborator Author

okainov commented Dec 6, 2015

Лабораторная сделана по документу из курса https://github.com/UNN-VMK-Software/agile-course-practice/blob/master/docs/lab-3-logger-legacy.md, но хотелось бы заметить, что в случае Питона использование мок-объектов намного удобнее. Т.е. вместо написания фиктивного логгера, достаточно в тесте "мокнуть" реальный логгер и проверить, что определенный метод логгера был вызван с определенными параметрами.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

неудачное название функции "log"

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет, стандартное название функции логгирования. См. документацию по стандартному logging

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

неопытному читателю не совсем понятно, что делает эта функция. Документация для искушенных.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Log - глагол,означающий "записывать"

@timonov
Copy link
Contributor

timonov commented Dec 13, 2015

Перед глаголом обычно ставится частица to: to_log()
👍

@okainov
Copy link
Collaborator Author

okainov commented Dec 13, 2015

@tkhanova

@okainov
Copy link
Collaborator Author

okainov commented Dec 15, 2015

@nikolaev94

@tkhanova
Copy link
Contributor

👍

@okainov
Copy link
Collaborator Author

okainov commented Dec 17, 2015

@AoD314

AoD314 added a commit that referenced this pull request Dec 17, 2015
Кайнов - Лабораторная работа #3
@AoD314 AoD314 merged commit 16da8d0 into master Dec 17, 2015
@okainov okainov deleted the kainov-oleg-lab3-logger branch January 27, 2016 06:57
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.

4 participants