Skip to content
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

Добавление автопоказа отчета allure #233 #245

Merged
merged 4 commits into from
Jul 18, 2018
Merged

Добавление автопоказа отчета allure #233 #245

merged 4 commits into from
Jul 18, 2018

Conversation

thedemoncat
Copy link
Contributor

@thedemoncat thedemoncat commented Jul 17, 2018

Проблематика: Для просмотра отчета Allure приходится выполнить множество лишних действий: открыть командную строку, написать команду генерации отчета и отображения в браузере

Задача: Упростить отображение, предоставив кнопку и опцию автопоказа

Реализация::

  • На вкладке "Запуск сценариев", в меню "Внешние инструменты" добавлена кнопка "Отобразить отчет Allure в браузере"

image

  • На вкладке "Сервис" - "Отчет о запуске сценариев" - добавленна опция "Отображать отчет Allure в браузере", при ее установке отчет будет показан после прогона тестов

  • Настройка папки скриншотов перенесена на вкладку "Сервис" - "Отчет о запуске сценариев", так как там ей самое место

image

@artbear - просьба прокомментировать и проверить

Реализация #233

@artbear artbear added this to the 5.5.0 milestone Jul 17, 2018
@thedemoncat thedemoncat changed the title Добавление автопоказ отчета allure #233 Добавление автопоказа отчета allure #233 Jul 17, 2018
Copy link
Collaborator

@artbear artbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличное дополнение.

Нужны мелкие правки.
Поправишь ?


КонецЕсли;

КомандаГерерации = "cmd /c allure generate --clean "+ Объект.КаталогOutputAllure +" && allure open";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

КомандаГерерации опечатка

Если Не ЗначениеЗаполнено(Объект.ДелатьОтчетВФорматеАллюр) Тогда

Сообщить("Заполните каталог исходников отчета Allure");
//Возврат;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

закомментированный код удаляем

// <продолжение описания параметра>
//
&НаКлиенте
Процедура СоздатьОткрытьОтчетAllureПродолжение(КодВозврата, ДополнительныеПараметры)Экспорт
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем нужен пустой метод СоздатьОткрытьОтчетAllureПродолжение??

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вроде бы можно создать пустой Новый ОписаниеОповещения(, ЭтотОбъект); ?

Copy link
Contributor Author

@thedemoncat thedemoncat Jul 17, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Новый ОписаниеОповещения(, ЭтотОбъект) - Не получилось. параметр стал не обязательным только в 8.3.12(

{ВнешняяОбработка.bddRunner.Форма.УправляемаяФорма.Форма(19695)}: Ошибка при вызове конструктора (ОписаниеОповещения)
	ОписаниеОповещения = Новый ОписаниеОповещения(, ЭтотОбъект);
по причине:
Недопустимое значение параметра (параметр номер '1')

@thedemoncat
Copy link
Contributor Author

thedemoncat commented Jul 17, 2018

Update:

  • При снятии крыжика у "Формировать данные отчета Allure" снимается и настройка "Отображать отчет allure"
  • Исправление ошибок

@artbear artbear merged commit 75703b6 into vanessa-opensource:develop Jul 18, 2018
Возврат;
КонецЕсли;

КомандаГенерации = "cmd /c allure generate --clean "+ Объект.КаталогOutputAllure +" && allure open";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

на linux не работает.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pumbaEO Эх, поздновато :(

А какой вариант для linux будет верным?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно revert сделать, права должні біть. sh -c и команді для запуска генерации.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"sh -c allure generate --clean "+ Объект.КаталогOutputAllure +" && allure open" - так?
В bddRunner есть функционал проверки на используемую ось, могу добавить.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По нормальному никак не получится, т.к. настройками безопасности по умолчанию отключено открытие браузера и команда только сгенирит отчет, но не откроет браузер со случайным портом. Поэтому лучше сделать проверку ЭтоLinux и вывести сообщением необходимую команду, чтоб пользователь вручную смогу скопировать и запустить.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pumbaEO Добавишь такой код? или мне добавить?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pumbaEO Я добавил код для линукса, посмотришь? 3978e72

@thedemoncat thedemoncat deleted the feature/issue-233 branch July 20, 2018 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants