418 / исправлено формирование представления теста для allure #443
Conversation
WalkthroughВ данном запросе на изменение внесены правки в функцию Changes
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 (
|
![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)
tests/src/CommonModules/ОМ_ЮТОтчетAllureСлужебный/Module.bsl (1)
84-98: Предлагаю расширить тестовое покрытие.Текущая реализация проверяет базовый сценарий, но рекомендуется добавить проверки для следующих случаев:
- Пустое имя теста
- Имя теста со спецсимволами
- Длинное имя теста
Пример дополнительного тестового метода:
Процедура ОписаниеТеста_ПредставлениеСпециальныеСлучаи() Экспорт ТестовыеСлучаи = Новый Массив; ТестовыеСлучаи.Добавить(""); ТестовыеСлучаи.Добавить("&""'<>"); ТестовыеСлучаи.Добавить("Очень длинное имя теста " + СтрПовтор(".", 100)); Для Каждого ИмяТеста Из ТестовыеСлучаи Цикл Модуль = ТестовыеДанные.ОписаниеТестовогоМодуля(); Набор = ТестовыеДанные.ОписаниеТестовогоНабора(Модуль); Тест = ТестовыеДанные.ОписаниеТеста(Модуль, Набор); Тест.Имя = ИмяТеста; ОписаниеТеста = ЮТОтчетAllureСлужебный.ОписаниеТеста(Тест, Набор, Модуль); ЮТест.ОжидаетЧто(ОписаниеТеста) .ИмеетТип("Структура") .Свойство("name").Равно(ИмяТеста) .Заполнено() ; КонецЦикла; КонецПроцедурыexts/yaxunit/src/CommonModules/ЮТОтчетAllureСлужебный/Module.bsl (1)
89-91: Добавьте проверку корректности имени тестаРекомендуется добавить проверку заполненности
РезультатТеста.Имядля предотвращения пустых имен тестов в отчете.- Описание.name = РезультатТеста.Имя; + Если НЕ ЗначениеЗаполнено(РезультатТеста.Имя) Тогда + ВызватьИсключение "Не заполнено имя теста"; + КонецЕсли; + Описание.name = РезультатТеста.Имя;
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (2)
exts/yaxunit/src/CommonModules/ЮТОтчетAllureСлужебный/Module.bsl(1 hunks)tests/src/CommonModules/ОМ_ЮТОтчетAllureСлужебный/Module.bsl(2 hunks)
🔇 Additional comments (2)
tests/src/CommonModules/ОМ_ЮТОтчетAllureСлужебный/Module.bsl (1)
27-27: Корректное добавление теста!
Новый тест добавлен в логичное место рядом с другими тестами описания.
exts/yaxunit/src/CommonModules/ЮТОтчетAllureСлужебный/Module.bsl (1)
90-90: Проверьте корректность отображения имен тестов в отчете Allure
Изменение способа формирования имени теста с РезультатТеста.Метод + ПредставлениеРежима на РезультатТеста.Имя может повлиять на читаемость отчетов. Убедитесь, что:
- Имена тестов остаются уникальными
- Режим выполнения теста отражается в отчете
✅ Verification successful
Изменение в отображении имени теста не повлияет на его идентификацию
Анализ кода показывает, что:
- Полное имя теста (
fullName) по-прежнему содержит информацию о режиме выполнения:ПолноеИмяМетода + ПредставлениеРежима - Режим выполнения теста добавляется как тег:
ДобавитьМетку(Описание, "tag", РезультатТеста.Режим) - Уникальность теста обеспечивается через
testCaseId, который формируется из полного идентификатора, включающего номер, метод и режим
Изменение корректно, так как:
- Сохраняется уникальная идентификация тестов
- Информация о режиме выполнения доступна в тегах и полном имени
- В отчёте JUnit также сохраняется контекст выполнения через атрибут
context
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check test name uniqueness and representation in test results
# Look for test name assignments in the codebase
rg -A 3 'РезультатТеста\.Имя\s*='
Length of output: 35
Script:
#!/bin/bash
# Let's try a broader search for test result properties and test name assignments
rg -i "результаттеста\." --type bsl -A 5
# Also search for the specific function where the change was made
ast-grep --pattern 'Функция ОписаниеТеста($$$) Экспорт
$$$
КонецФункции'
Length of output: 147
Script:
#!/bin/bash
# Search without file type restriction
rg -i "результаттеста\." -A 5
# Let's also look for the test name related code
rg -i "описание\.name\s*=" -A 2
Length of output: 9683
Описание изменений
Close #418
Summary by CodeRabbit
Новые функции
Исправления ошибок