Доработки по проверке проекта EDT #259
Conversation
Stepa86
commented
Sep 21, 2018
Stepa86
commented
- В отчете Аллюр2 формируется ссылка на строчку кода. Необходимо передать путь к проекту.
- Добавлен параметр --EDTversion для указания конкретной версии EDT
- Больше не падает, если необязательные файлы не существуют. Например, когда EDT не нашел ошибок.
- Небольшие правки по замечаниям
…essa-runner into feature/validate_EDT
…runner into develop # Conflicts: # src/Классы/КомандаПроверкаПроектаEDT.os
|
|
||
| // Отсутствие этого файла не критично, поэтому обойдемся без исключений | ||
|
|
||
| Лог.Ошибка( "Файл пропускаемых ошибок (--exception-file) %1 не найден.", ИмяФайлаПропускаемыхОшибок ); |
There was a problem hiding this comment.
Отсутствие этого файла не критично
а зачем тогда Лог.Ошибка выводить?
Предлагаю исправить на Лог.Предупреждение
There was a problem hiding this comment.
Заменил на Предупреждение
|
|
||
| Если компоненты.Количество() > 2 Тогда | ||
|
|
||
| Если ВРег( компоненты[2] ) = "МОДУЛЬОБЪЕКТА" Тогда |
There was a problem hiding this comment.
Копипаст
ВРег( компоненты[2] )- и 4 дублирующихся блока кода
Если ВРег( компоненты[2] ) = "ХХХ" Тогда
компонентыСсылки.Добавить( "YYY.bsl" );
|
|
||
| // Имя объекта | ||
|
|
||
| Если компоненты.Количество() < 1 Тогда |
There was a problem hiding this comment.
условие лучше перенести выше, все равно возвращается ""
Если компоненты.Количество() < 1 Тогда
Возврат "";
КонецЕсли;
|
|
||
| Если ЗначениеЗаполнено( ПутьКФайламПроекта ) Тогда | ||
|
|
||
| кэшСоответствиеМетаданныхКаталогам = СоответствиеМетаданнымКаталогам(); |
There was a problem hiding this comment.
Переменная кэшСоответствиеМетаданныхКаталогам не всегда заполняется :(
а в коде ниже ты обращаешься к ней безусловно :(
каталог = кэшСоответствиеМетаданныхКаталогам[компоненты[0]];
тут точно нужно исправить код.
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.
Переделал на инициализацию по месту использования
| ИмяФайлаРезультата = ПолучитьИмяВременногоФайла("out"); | ||
| УдалятьФайлРезультата = Истина; | ||
| Лог.Отладка(" Файл результата не задан (--validation-result). Будет использован временный файл."); | ||
|
|
||
There was a problem hiding this comment.
Много пустых строк, в которых есть только пробелы или табы, добавились :(
видимо, в другом редакторе правил?
There was a problem hiding this comment.
Как раз в VSC все и правил. Причем он сам мне изменений не показывал таких.
|
ненужные пробелы в пустых строках убери через VSC, запушь и я замержу. |
|
@artbear VSC больше ничего не форматирует у меня |
|
в VSC выполни команду Trim Trailing Whitespaces |
|
@artbear А зачем их так убирать? Стандартное форматирование Alt+Shift+F их обратно возвращает |
|
Мержим? |
* wip Обработка умеет выгружать проверку EDT в файлы аллюр * wip Обработка умеет выгружать проверку EDT в файлы аллюр * wip Проверка проекта с помощью EDT * Код проверки проекта EDT завершен * Вернул имя --exception-file, для однотипности с синтаксис-проверкой * Создание ссылок на строку с проблемой в Аллюр-отчете * мерж * Правки по замечаниям * Форматирование, чтоб пробелов не было лишних * Форматирование
