New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Доработки по проверке проекта EDT #259

Merged
merged 13 commits into from Sep 21, 2018

Conversation

Projects
None yet
3 participants
@Stepa86
Copy link
Contributor

Stepa86 commented Sep 21, 2018

  1. В отчете Аллюр2 формируется ссылка на строчку кода. Необходимо передать путь к проекту.
  2. Добавлен параметр --EDTversion для указания конкретной версии EDT
  3. Больше не падает, если необязательные файлы не существуют. Например, когда EDT не нашел ошибок.
  4. Небольшие правки по замечаниям
@artbear
Copy link
Collaborator

artbear left a comment

Небольшие замечания


// Отсутствие этого файла не критично, поэтому обойдемся без исключений

Лог.Ошибка( "Файл пропускаемых ошибок (--exception-file) %1 не найден.", ИмяФайлаПропускаемыхОшибок );

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

Отсутствие этого файла не критично

а зачем тогда Лог.Ошибка выводить?
Предлагаю исправить на Лог.Предупреждение

This comment has been minimized.

@Stepa86

Stepa86 Sep 21, 2018

Contributor

Заменил на Предупреждение


Если компоненты.Количество() > 2 Тогда

Если ВРег( компоненты[2] ) = "МОДУЛЬОБЪЕКТА" Тогда

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

Копипаст

  • ВРег( компоненты[2] )
  • и 4 дублирующихся блока кода
Если ВРег( компоненты[2] ) = "ХХХ" Тогда
			компонентыСсылки.Добавить( "YYY.bsl" );

This comment has been minimized.

@Stepa86

Stepa86 Sep 21, 2018

Contributor

Поправил


// Имя объекта

Если компоненты.Количество() < 1 Тогда

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

условие лучше перенести выше, все равно возвращается ""

Если компоненты.Количество() < 1 Тогда
		
		Возврат "";
		
	КонецЕсли;

Если ЗначениеЗаполнено( ПутьКФайламПроекта ) Тогда

кэшСоответствиеМетаданныхКаталогам = СоответствиеМетаданнымКаталогам();

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

Переменная кэшСоответствиеМетаданныхКаталогам не всегда заполняется :(

а в коде ниже ты обращаешься к ней безусловно :(

каталог = кэшСоответствиеМетаданныхКаталогам[компоненты[0]];

тут точно нужно исправить код.

This comment has been minimized.

@Stepa86

Stepa86 Sep 21, 2018

Contributor

В начале функции стоит как раз обратная проверка

Если Не ЗначениеЗаполнено( ПутьКФайламПроекта ) Тогда
		
	Возврат "";
		
КонецЕсли;

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

Проверка стоит - это хорошо.
но вот такая неявная инициализация переменной - это плохо, такой код же читать и сопровождать сложно.
может быть, проверку в начале функции изменить на проверку кэшСоответствиеМетаданныхКаталогам ?

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

или еще лучше - переменную кэшСоответствиеМетаданныхКаталогам заполнять как раз в этой функции, все равно переменная юзается только внутри функции ??

This comment has been minimized.

@Stepa86

Stepa86 Sep 21, 2018

Contributor

Переделал на инициализацию по месту использования

ИмяФайлаРезультата = ПолучитьИмяВременногоФайла("out");
УдалятьФайлРезультата = Истина;
Лог.Отладка(" Файл результата не задан (--validation-result). Будет использован временный файл.");

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

Много пустых строк, в которых есть только пробелы или табы, добавились :(
видимо, в другом редакторе правил?

This comment has been minimized.

@Stepa86

Stepa86 Sep 21, 2018

Contributor

Как раз в VSC все и правил. Причем он сам мне изменений не показывал таких.

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Sep 21, 2018

ненужные пробелы в пустых строках убери через VSC, запушь и я замержу.

@artbear artbear self-assigned this Sep 21, 2018

@artbear artbear added this to the 1.6.0 milestone Sep 21, 2018

@Stepa86

This comment has been minimized.

Copy link
Contributor

Stepa86 commented Sep 21, 2018

@artbear VSC больше ничего не форматирует у меня

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Sep 21, 2018

в VSC выполни команду Trim Trailing Whitespaces
по русски точное название не знаю - "удалить пробелы"?

@Stepa86

This comment has been minimized.

Copy link
Contributor

Stepa86 commented Sep 21, 2018

@artbear А зачем их так убирать? Стандартное форматирование Alt+Shift+F их обратно возвращает

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Sep 21, 2018

Мержим?

@artbear artbear merged commit 031277b into silverbulleters:develop Sep 21, 2018

1 check failed

continuous-integration/jenkins/pr-head This commit cannot be built
Details

@Stepa86 Stepa86 deleted the Stepa86:feature/validate_EDT2 branch Sep 21, 2018

VladFrost added a commit to VladFrost/vanessa-runner that referenced this pull request Sep 24, 2018

Доработки по проверке проекта EDT (silverbulleters#259)
* wip Обработка умеет выгружать проверку EDT в файлы аллюр

* wip Обработка умеет выгружать проверку EDT в файлы аллюр

* wip Проверка проекта с помощью EDT

* Код проверки проекта EDT завершен

* Вернул имя --exception-file, для однотипности с синтаксис-проверкой

* Создание ссылок на строку с проблемой в Аллюр-отчете

* мерж

* Правки по замечаниям

* Форматирование, чтоб пробелов не было лишних

* Форматирование
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment