Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Возобновил работу событий установки ПередУстановкой и ПриУстановке #139

Merged
merged 4 commits into from
Sep 19, 2018

Conversation

artbear
Copy link
Member

@artbear artbear commented Sep 18, 2018

Доработал фичу для проверки отработки событий установки пакета
@artbear artbear self-assigned this Sep 18, 2018
@artbear artbear changed the base branch from master to develop September 18, 2018 20:39
@@ -319,13 +324,34 @@
//

Функция ИзвлечьОбязательныйФайл(Знач Чтение, Знач ИмяФайла)
Лог.Отладка("Извлечение: " + ИмяФайла);
// Лог.Отладка("Извлечение: %1", ИмяФайла);
// Элемент = Чтение.Элементы.Найти(ИмяФайла);
Copy link
Member

Choose a reason for hiding this comment

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

Закомментированный код

features/install-file.feature Outdated Show resolved Hide resolved
Тогда Вывод команды "opm" содержит "ИНФОРМАЦИЯ - Установка завершена"
И я вижу в консоли вывод
# | Событие установки - ПередСборкой - рабочий каталог |
# | Событие установки - ПослеСборки - рабочий каталог |
Copy link
Member

Choose a reason for hiding this comment

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

каменты в фиче. Не нужны

Copy link
Member Author

Choose a reason for hiding this comment

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

это WIP, конечно, в финале будет убрано

ВременныйКаталог = БДД.ПолучитьИзКонтекста(ПутьКаталогаИсходников);
ИскомыйФайл = Новый Файл(ОбъединитьПути(ВременныйКаталог, ИмяФайла));
Ожидаем.Что(ИскомыйФайл.Существует(), "Файл должен был существовать").ЭтоИстина();

КонецПроцедуры

//я включаю полную отладку логов пакетов OneScript
Процедура ЯВключаюПолнуюОтладкуЛоговПакетовOnescript() Экспорт
УстановитьПеременнуюСреды("LOGOS_LEVEL", "DEBUG");
Copy link
Member

Choose a reason for hiding this comment

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

Странный шаг сценария поведения системы

Copy link
Member Author

Choose a reason for hiding this comment

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

как я писал, это WIP-версия,
в финале этот шаг будет уведен в 1bdd для проверки поведения продуктов и убран из текущей фичи вообще

шаг полезен для решения проблем при запуске кода через фичи 1bdd
без необходимости использования отладчика

@artbear
Copy link
Member Author

artbear commented Sep 19, 2018

Странно, мне казалось, что я WIP-отметку ставил :)
поэтому пока и много чего осталось в коде

@artbear artbear changed the title Возобновил работу событий установки ПередУстановкой и ПриУстановке WIP:Возобновил работу событий установки ПередУстановкой и ПриУстановке Sep 19, 2018
@artbear artbear changed the title WIP:Возобновил работу событий установки ПередУстановкой и ПриУстановке Возобновил работу событий установки ПередУстановкой и ПриУстановке Sep 19, 2018
@artbear
Copy link
Member Author

artbear commented Sep 19, 2018

Работа закончена.
У меня все тесты выполняются, а вот на тревисе какие-то странные проблемы, не относящиеся к PR

@artbear artbear merged commit 4d9ae31 into develop Sep 19, 2018
@artbear artbear deleted the feature/install-event branch September 19, 2018 14:56
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.

2 participants