#32 Формирование отчета в формате Generic Execution (Sonar)#33
Conversation
|
@arkuznetsov Очень интересно, спасибо! Пока некогда смотреть :( Очень постараюсь посмотреть и выдать обратную связь |
artbear
left a comment
artbear
left a comment
There was a problem hiding this comment.
Отличное дополнение.
Но есть замечания.
@arkuznetsov Исправишь?
затем я приму.
ЗЫ иначе irac на штатном 1testrunner выполнить нельзя :(
| ИЛИ Найти(текстОшибки, "Object method not found (") > 0) | ||
| И (Найти(текстОшибки, ИмяМетода) > 0) Тогда | ||
| Если Найти(текстОшибки, "Метод объекта не обнаружен ("+имяМетода+")") > 0 | ||
| ИЛИ Найти(текстОшибки, "Object method not found ("+имяМетода+")") > 0 Тогда |
There was a problem hiding this comment.
Откати, плиз, эти строки
Если Найти(текстОшибки, "Метод объекта не обнаружен ("+имяМетода+")") > 0
+ ИЛИ Найти(текстОшибки, "Object method not found ("+имяМетода+")") > 0 Тогдат.к. этот старый вариант неверно работает в 20 релизе :(
There was a problem hiding this comment.
Сорри, не понимаю, куда откатить? Я подлил к себе из твоего актуального девелопа там все на месте.
| Процедура Инициализация() | ||
| Лог = Логирование.ПолучитьЛог(ИмяЛога()); | ||
|
|
||
| ФорматЛогФайла = "JU"; |
There was a problem hiding this comment.
- Значение
"JU"нигде в коде не используется :(- Есть проверка на
GE - Зачем
"JU"вообще тогда нужно?
- Есть проверка на
- Предлагаю вместо магических строк (
GEиJU) заюзать фиксированную структуру и обращаться к ее свойствам- Например,
ВидыФорматаЛогФайла.JUnitилиВидыФорматаЛогФайла.GenericSonar
- Например,
There was a problem hiding this comment.
Примеры есть в 1тестраннере.
Например, ЗначенияСостоянияТестов
There was a problem hiding this comment.
Добавил структуру с форматами.
В структуре оставил значение JUnit, а также назначение значения по умолчанию для дальнейшего развития.
| ЗаписьXML = Новый ЗаписьXML; | ||
| ЗаписьXML.ОткрытьФайл(ПутьОтчетаВФорматеJUnitxml.ПолноеИмя); | ||
| ЗаписьXML.ЗаписатьБезОбработки(СтрокаХМЛ); | ||
| ЗаписьXML.Закрыть(); |
There was a problem hiding this comment.
Код дублируется с аналогичным методом в формировании отчета Junit.
Копипаст = зло :)
ЗаписьXML = Новый ЗаписьXML;
+ ЗаписьXML.ОткрытьФайл(ПутьОтчетаВФорматеJUnitxml.ПолноеИмя);
+ ЗаписьXML.ЗаписатьБезОбработки(СтрокаХМЛ);
+ ЗаписьXML.Закрыть();There was a problem hiding this comment.
Предлагаю выделить отдельный метод для этих строк
|
|
||
| Процедура ЗавершитьЗаписьВФайлОтчетаТестированияВФорматеGExecXML(ЗаписьXML, ДатаНачала) | ||
|
|
||
| Утверждения.ПроверитьНеРавенство(НаборТестов.Количество(), 0); |
There was a problem hiding this comment.
Предлагаю расшифровать утверждение, чтобы было понятно, в каком месте кода утверждение упало.
Например,
Утверждения.ПроверитьНеРавенство(НаборТестов.Количество(), 0, "Количество тестов равно нуля, что неверно. Метод ЗавершитьЗаписьВФайлОтчетаТестированияВФорматеGExecXML ");Добавлена структура с перечислением доступных форматов файла отчета; Запись файла отчета вынесена в отдельную процедуру; Мелкие исправления оформления кода и коментариев
|
@artbear А по что у тебя сборка на трависе поломана? |
| ИЛИ Найти(текстОшибки, "Object method not found (") > 0) | ||
| И (Найти(текстОшибки, ИмяМетода) > 0) Тогда | ||
| Если Найти(текстОшибки, "Метод объекта не обнаружен (" + имяМетода + ")") > 0 | ||
| ИЛИ Найти(текстОшибки, "Object method not found (" + имяМетода + ")") > 0 Тогда |
There was a problem hiding this comment.
Как я уже писал, откати эти строки, оставь, как у меня сейчас написано.
There was a problem hiding this comment.
Иначе на новых релизах движка будут ошибки
There was a problem hiding this comment.
Так вот я не пойму как у тебя сейчас :-(. Я начинающий волшебник :-).
У меня в строках 777-778 вообще что-то другое, в 901-902 вроде так и есть.
И у меня на 20 крутится.
There was a problem hiding this comment.
Так, вроде въехал. Не привык я еще различия нормально читать.
2 participants
No description provided.