381 / Изменен подход запуска тестов из формы отчета #456
Conversation
WalkthroughВ данном пулл-реквесте добавлены новые процедуры и функции для улучшения управления тестовыми модулями в рамках тестового фреймворка. Введена процедура Changes
Assessment against linked issues
Possibly related PRs
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
SonarQube Quality Gate failed! 
No data Coverage
| Процедура ПередВыполнениемТестов(Модули, Отказ) | ||
|
|
||
| Если Модули.Количество() = 0 Тогда | ||
| ПоказатьПредупреждение( , "Нет тестов для запуска"); |
There was a problem hiding this comment.
🔗Проверьте корректность выполнения кода после асинхронного метода <ПоказатьПредупреждение>
suspicious Why is this an issue?
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
exts/yaxunit/src/DataProcessors/ЮТЮнитТесты/Forms/Основная/Module.bsl (1)
176-178: Проверьте необходимость экспорта процедурыПослеВыполненияВыделенныхМодулейПроцедура
ПослеВыполненияВыделенныхМодулейпомечена ключевым словомЭкспорт. Убедитесь, что экспорт действительно необходим и процедура вызывается из внешних контекстов.exts/yaxunit/src/CommonModules/ЮТИсполнительСлужебныйКлиент/Module.bsl (1)
62-75: Реализация корректна и соответствует архитектуре фреймворка.Процедура правильно интегрируется с существующей цепочкой обработчиков и следует установленным паттернам асинхронного выполнения.
Предлагаю добавить документацию к процедуре:
+// Выполняет тестовые модули с заданными параметрами. +// +// Параметры: +// ПараметрыЗапуска - Структура - Параметры запуска тестов +// ИсполняемыеМодули - Массив - Список модулей для выполнения +// ОбработчикЗавершения - ОписаниеОповещения - Обработчик завершения выполнения +// Процедура ВыполнитьТестовыеМодули(ПараметрыЗапуска, ИсполняемыеМодули, ОбработчикЗавершения) Экспорт
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (5)
exts/yaxunit/src/CommonModules/ЮТИсполнительСлужебныйКлиент/Module.bsl(1 hunks)exts/yaxunit/src/CommonModules/ЮТПараметрыЗапускаСлужебный/Module.bsl(1 hunks)exts/yaxunit/src/DataProcessors/ЮТЮнитТесты/Forms/Основная/Module.bsl(4 hunks)exts/yaxunit/src/DataProcessors/ЮТЮнитТесты/Forms/СозданиеНастройки/Module.bsl(3 hunks)tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl(1 hunks)
🔇 Additional comments (11)
exts/yaxunit/src/DataProcessors/ЮТЮнитТесты/Forms/Основная/Module.bsl (6)
162-173: Проверьте корректность обработки переменной Отказ
В процедуре ЗапуститьВыделенныеТесты добавлена обработка переменной Отказ и вызов процедур ПередВыполнениемТестов и ПослеВыполненияВыделенныхМодулей. Убедитесь, что переменная Отказ правильно устанавливается и используется для контроля дальнейшего выполнения тестов.
171-171: Убедитесь в корректности передачи параметров в метод ВыполнитьТестовыеМодули
При вызове ЮТИсполнительСлужебныйКлиент.ВыполнитьТестовыеМодули передаются параметры ПараметрыЗапускаТестирования и Модули. Проверьте, что эти параметры содержат необходимые данные для успешного выполнения тестов.
558-558: Инициализация параметров запуска тестирования по умолчанию
В процедуре ЗагрузитьТесты добавлено использование метода ПараметрыТестированияПоУмолчанию. Это обеспечивает получение стандартных настроек для запуска тестов.
621-636: Добавлена новая процедура ПередВыполнениемТестов
Процедура ПередВыполнениемТестов проверяет наличие тестов для запуска и сбрасывает состояние тестирования модулей. Это улучшает предварительную подготовку перед запуском тестов.
638-667: Обновление результатов после выполнения тестов
В процедуре ПослеВыполненияТестов происходит обработка результатов тестирования и обновление интерфейса. Убедитесь, что все результаты корректно отображаются и статистика обновляется правильно.
871-874: Проверка переинициализации параметров запуска
В функции ПараметрыЗапуска происходит переинициализация параметров запуска тестирования. Убедитесь, что это не приводит к потере пользовательских настроек и соответствует логике приложения.
exts/yaxunit/src/CommonModules/ЮТПараметрыЗапускаСлужебный/Module.bsl (1)
50-57: Добавлена функция для получения параметров тестирования по умолчанию
Новая функция ПараметрыТестированияПоУмолчанию возвращает стандартные параметры запуска тестирования и устанавливает их по умолчанию через УстановитьПараметрыЗапускаПоУмолчанию. Это способствует единообразию настроек в приложении.
tests/src/CommonModules/ОМ_ЮТКонструкторВариантов/Module.bsl (1)
188-193: Улучшено сообщение об ошибке при некорректном типе параметра
В процедуре ДобавитьКомбинации_ПроверкаТипаПараметра добавлено подробное сообщение об исключении, указывающее ожидаемый и фактический тип параметра. Это повышает информативность ошибок и упрощает отладку.
exts/yaxunit/src/DataProcessors/ЮТЮнитТесты/Forms/СозданиеНастройки/Module.bsl (3)
45-46: Инициализация параметров по умолчанию при открытии формы
В процедуре ПриОткрытии добавлен вызов УстановитьПараметрыПоУмолчанию, что обеспечивает установку стандартных настроек перед началом работы с формой.
Line range hint 440-451: Использование параметров по умолчанию при сохранении конфигурации
В процедуре СохранитьКонфигурациюЗапуска теперь используются параметры по умолчанию из ПараметрыТестированияПоУмолчанию. Убедитесь, что это не конфликтует с пользовательскими настройками и сохраняет необходимые параметры.
576-588: Добавлена процедура УстановитьПараметрыПоУмолчанию для инициализации настроек
Новая процедура устанавливает параметры формы на основе значений по умолчанию, полученных из ПараметрыТестированияПоУмолчанию. Это повышает модульность кода и упрощает его поддержку.
Описание изменений
Close #381
Close #444
Summary by CodeRabbit
Новые функции
ВыполнитьТестовыеМодулидля выполнения тестовых модулей.ПараметрыТестированияПоУмолчаниюдля получения параметров тестирования по умолчанию.ПередВыполнениемТестовиПослеВыполненияТестовдля управления выполнением тестов.УстановитьПараметрыПоУмолчаниюдля инициализации конфигурационных переменных.Исправления ошибок
Документация