Полная информация об исключении при загрузке файлов с дымовыми тестами в случае конфигурации с поддержкой синхронности #1096
Conversation
There was a problem hiding this comment.
уже можно\нужно юзать простое СтрШаблон вместо КонтекстЯдра.СтрШаблон_
There was a problem hiding this comment.
Заменил. Может в методе СтрШаблон_ тогда заменить тело на вызов простого СтрШаблон и пометить метод как устаревший, чтобы было понятнее, что его уже использовать не нужно? Или я не до конца понимаю его назначение?
There was a problem hiding this comment.
аналогично заменить на СтрШаблон
There was a problem hiding this comment.
проверял в пакетном режиме?
имхо при вызове исключения здесь основной фреймворк тестирования может упасть,
а тогда сеанс 1С просто зависнет и останется в памяти, не завершившись.
а что происходит в интерактивном режиме?
There was a problem hiding this comment.
В пакетном режиме.
Выводится сообщение:
Ошибка загрузки и выполнения тестов в пакетном режиме
И дальше подробная информация об ошибке
Менеджер тестирования завершается, а клиент действительно остаётся открыт. Но так он может оставаться и из-за других исключений.
Срабатывает вот этот перехват исключения, возможно в нём стоит добавить закрытие клиентов тестирования так же, как вот здесь. Тогда ничего зависшего оставаться не будет.
В интерактивном режиме.
Фреймворк не закрывается, появляется два сообщения.
- Короткое:
Не удалось загрузить файл ИмяФайла.epf
Причина исключения
- И подробное описание ошибки со стеком вызовов.
Итого
По-моему поведение приемлемое. Но как я уже писал, исключение в этой строке хотелось бы убрать, заменив на установку статуса ошибки для тестов в целом.
There was a problem hiding this comment.
остается главный вопрос - почему исправление сделано в Загрузчике каталога, а не в загрузчике файла?
1 ведь основную работу выполняет именно загрузчик файла
2 и в коде обработчика из ПР используются те же данные, что передаются в загрузчик файла - полное имя файла
Предлагаю перенести код Попытки-Исключение в метод Загрузить загрузчика файлов
There was a problem hiding this comment.
@BarinovIN а в целом я с исправлением согласен, только нужно код перенести в загрузчик файла и тогда я смержу.
There was a problem hiding this comment.
Исправление сделал там, где было необработанное исключение, добавил его полноценную обработку.
Другой вариант исправления - убрать перехват исключения. Он же только сообщение об ошибке уточняет - это не обязательно.
Необходим ли перехват исключения в загрузчике файла?
Итого, предлагаю совсем убрать перехват исключения отсюда, но и в загрузчик файла не добавлять его.
There was a problem hiding this comment.
Перенёс перехват исключения в загрузчик файла.
artbear
changed the title
|
Уточнил заголовок, т.к. ПР решает проблему только для конфигураций с поддержкой синхронности для асинхронных конфигураций не используются методы Загрузить, а используются методы вида НачатьЗагрузку. а в них обработка сложнее, т.к. простого исключения недостаточно, остановится только текущая асинхронная функция, а не весь фреймворк (( |
Перехват исключения фактически нужен "на будущее". Для возможности пропустить незагруженный файл и обработать остальные, но со статусом ошибки для тестов в целом. Пока установки такого статуса нету, поэтому будет вызвано исключение и все тесты не выполнятся. Для этих же целей дерево тестов нужно сформировать заранее, чтобы функция всегда его возвращала, после замены вызова исключения на установку статуса ошибки. Сейчас перехват исключения поможет быстро узнать имя файла, который не удалось загруить. см: vanessa-opensource#1095
2 participants
частичная реализация #1095
сделанные изменения
@vanessa-opensource/Collaborators - просьба прокомментировать и проверить