2019-04-04 Требуется Проверять Наличие Новой Функции В Конце Работы
Постановка задачи: требовалось сделать резервный файл выгрузки, чтобы у клиентов не происходило проблем из-за старого файла выгрузки. При этом, в некоторых случаях происходила замена файла резервным.
В чем была проблема: Было сделано изменение, но у клиентов в новой версии функционально ничего не изменилось.
Краткая хронология: создавалась новая функция, её работа была проверена в во время кодирования, но она не была проверена в конце работы.
Изначально функция выглядела так:
.... download_from_carbon(){ if [ "$(time_of_day)" = 'NIGHT' ] && [ -f "$FILE_RESERVE" ]; then cp -a "$FILE_RESERVE" "$FILE" ....
Потом провели рефакторинг, перенесли логику из bash в утилиту для работы с файлом, но полностью функциональность перенести забыли, не хватало команд, потому что решено было передавать директорию для файла через env:
-DUMP = os.path.join(RKN_DIR, OUTPUT) +DUMP = os.getenv('DUMP', os.path.join(RKN_DIR, OUTPUT))
Ошибки:
- Не было полностью перенесена функциональность во время рефакторинга
- Не было проведено достаточное тестирование именно той функции, ради которой вносились изменения после всех изменений
Как надо делать: После всех изменений требуется проверять наличие именно той функциональности, ради которой создавалось изменение, это предохраняет от ошибок во время рефакторинга подтверждается, что было сделано именно то, что задумывали изначально.
~~OWNERAPPROVE~~
Прочитал правила разработки как не надо делать 2019-04-04 требуется проверять наличие новой функции в конце работы |
Обсуждение