**Постановка задачи**: требовалось сделать резервный файл выгрузки, чтобы у клиентов не происходило проблем из-за старого файла выгрузки. При этом, в некоторых случаях происходила замена файла резервным.
**В чем была проблема**: Было сделано изменение, но у клиентов в новой версии функционально ничего не изменилось.
**Краткая хронология**: создавалась новая функция, её работа была проверена в во время кодирования, но она не была проверена в конце работы.
Изначально функция выглядела так:
....
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~~
/*Не удаляйте эту строку и ниже!*/
{(rater>id=1|name=Прочитал_правила_разработки:как_не_надо_делать:2019-04-04_требуется_проверять_наличие_новой_функции_в_конце_работы|type=vote|trace=user|tracedetails=1)}