firstBitMarksistskaya / jenkins-lib

Jenkins shared library для 1С:Предприятие 8
MIT License
158 stars 67 forks source link

Yaxunit #94

Closed kuzja086 closed 4 months ago

kuzja086 commented 1 year ago

89 Пока черновой вариант, все топорно и костыльно. Хочу сверить концепцию и уточнить некоторые моменты.

Пока остановился на том, что тесты запускаются и отображаются в панельке junit в jenkins на моей машине=) . Дальше буду рефакторить.

1) Сейчас захардкорил скачивание расширения yaxunit. Было несколько идей таких как:

nixel2007 commented 1 year ago

Спасибо большое, погляжу

kuzja086 commented 1 year ago

@nixel2007 Не получилось посмотреть? 😊

nixel2007 commented 1 year ago

Только из отпуска вышел. Обязательно доберусь на днях.

ovcharenko-di commented 1 year ago

@kuzja086 планируешь доделать? я бы воспользовался

По п.2 лично я поддерживаю твое решение грузить расширение прямо в стейдже, потому что сейчас подключенное расширение YAXUnit может приводить к ошибкам при "обычной" работе, соответственно, оно может мешать bdd и дымовым тестам.

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

Такой набор параметров не покрывает только тот случай, когда тесты хранятся в отдельном репозитории, но мне кажется, что это крайне неудобно и так никто не делает.

@nixel2007 есть мысли?

kuzja086 commented 1 year ago

@ovcharenko-di Да, планирую закончить. Сейчас жду обратной связи от @nixel2007 😊 Идея с параметрами стейджа мне нравится, например если cfe не заполнен, то качаем из оф. Репо на GH и testPath относительный путь к исходникам расширения с тестами. Если не заполнен, то значит предполагаем, что тесты в ядре.

Есть идеи как быть с путем к отчету junit? На мой взгляд лучше доработать Yaxunit, чтобы через параметры запуска можно было переопределять его. Может еще варианты есть ?

ovcharenko-di commented 1 year ago

@kuzja086 по поводу пути к отчету junit: я считаю, что в библиотеке путь к отчету junit можно просто захардкодить, как это сделано сейчас для шагов bdd и smokeTest. Тогда ожидается, что пользователь библиотеки сам предоставит конфиг, в котором будет указан именно этот путь.

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

ovcharenko-di commented 1 year ago

@kuzja086 я в твоем форке создал PR!

kuzja086 commented 1 year ago

@nixel2007 мы готовы))

ovcharenko-di commented 1 year ago

@nixel2007 вот теперь точно готовы)

ovcharenko-di commented 1 year ago

@nixel2007 посмотри, плз

вроде все отработал, получилось даже избавиться от одного параметра в конфиге

nixel2007 commented 4 months ago

Superseded by #114