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 #258

Merged
merged 6 commits into from Sep 20, 2018

Conversation

Projects
None yet
3 participants
@Stepa86
Copy link
Contributor

Stepa86 commented Sep 20, 2018

#248

  1. Если передать рабочую область и проект, то выполнит проверку.
  2. Если передать имя файла результата, то запишет результат проверки в него.
  3. Если передать имя файла предыдущего результата, то результат будет записан как разность новых ошибок и старых.
    Ошибки и предупреждения, которые есть в предыдущем файле, но которых нет в новом - будут помечены как passed (Исправлено).
    Ошибки и предупреждения, которые есть только в новом файле результатов - будут помечены как failed (Ошибки) и broken (Предупреждения).
    Все остальные ошибки и предупреждения, которые есть в обоих файлах, будут помечены как skipped (Пропущено).
  4. Если передать путь к папке для Аллюр, то сформирует там файлы в формате Аллюр2
@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Sep 20, 2018

Отличный PR

@Stepa86 У меня только небольшой вопрос/комментарий:
сейчас у тебя при поиске исключений строка ошибка ищется в каждой строке исключений.

В итоге мы лишаемся возможности указать некий шаблон исключения
и должны явно и с точностью до символа прописывать текст в файле исключений

Это может быть не очень удобно.

Правда, в синтакс-проверке у нас точно такой же код :(

Может быть, есть смысл сравнивать наоборот - искать строку исключения в строке ошибки из ЕДТ или синтакс-контроля?

@artbear artbear self-assigned this Sep 20, 2018

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

@Stepa86

This comment has been minimized.

Copy link
Contributor

Stepa86 commented Sep 20, 2018

@artbear И тут и в синтаксис-проверке как раз и ищется строка исключения в строке ошибки. Например, строка исключения ".ир" выключает почти все инструменты разработчика из проверки (помечает пропущенными).

Предлагаю менять/добавлять механизм под конкретные кейсы. Я пока сам не понимаю как этим пользоваться. И после выхода новой ЕДТ может все поехать.

Хочу добавить формирование ссылки на строку с ошибкой на гитсервере в карточке ошибки у Аллюра. Это лучше в отдельный PR или в этот?

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Sep 20, 2018

Лучше отдельно, этот я принимаю

@artbear artbear merged commit 2ad7dab into silverbulleters:develop Sep 20, 2018

1 check failed

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

This comment has been minimized.

Copy link
Contributor

VladFrost commented Sep 21, 2018

@Stepa86 Супер, очень круто! Я прям на код засмотрелся как всё продумано сделано.
Но есть один нюанс, если на компе установлено несколько версий edt, то нужно указывать конкретную версию при запуске: ring edt@1.9.1 workspace validate
2018-09-21 08_46_38
2018-09-21 08_48_01


Команда = Новый Команда;
Команда.УстановитьСтрокуЗапуска( "ring edt workspace validate" );
Команда.УстановитьКодировкуВывода(КодировкаТекста.ANSI);

This comment has been minimized.

@VladFrost

VladFrost Sep 21, 2018

Contributor

Ы-ы-ы! Я тоже на это нарвался. Буду приставать к разрабочтикам EDT, почему ring не работает при chcp 65001

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

Ага, мне также показалось очень странным такое решение с кодировкой

This comment has been minimized.

@VladFrost

VladFrost Sep 24, 2018

Contributor

@Stepa86 @artbear Мне подсказали, что можно в переменную окружения RING_OPTS передать параметры set RING_OPTS=-Dfile.encoding=UTF-8 -Dosgi.nl=ru и тогда всё срастётся:
2018-09-24 11_12_53


КонецФункции

Функция РазностнаяТаблицаРезультатов( Знач пТЗ_пред, Знач пТЗ_нов )

This comment has been minimized.

@VladFrost

VladFrost Sep 21, 2018

Contributor

Вот это офигенно!


категории = Новый Массив;

массивСтатусов = Новый Массив;

This comment has been minimized.

@VladFrost

VladFrost Sep 21, 2018

Contributor

А здесь копипаста. Я бы вынес наполнение категорий в отдельный метод и его бы четыре раза вызвал с параметрами.

This comment has been minimized.

@artbear

artbear Sep 21, 2018

Collaborator

да, так было бы проще и нагляднее.

но я не стал тормозить PR таким небольшим замечанием

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

Проверка проекта EDT (silverbulleters#258)
* 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