Fix/Ошибка при указании в extension-stages "bdd" #184
Conversation
…готовке расширений
ПрохождениеДобавлено новое поле Изменения
Оценка сложности кода🎯 1 (Тривиально) | ⏱️ ~3 минуты Изменения состоят из простого добавления одного поля с аннотацией и минорного обновления метода Стихотворение
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/ru/pulsar/jenkins/library/configuration/BddOptions.groovy (1)
32-33: Добавьте разделитель между полями в toString().В методе
toString()отсутствует разделитель между полямиvrunnerSettingsиvrunnerSteps, что затрудняет чтение вывода. Текущий формат создаст строку видаvrunnerSettings='...'vrunnerSteps=...без пробела или запятой.Примените этот diff для улучшения читаемости:
return "BddOptions{" + - "vrunnerSettings='" + vrunnerSettings + '\'' + + "vrunnerSettings='" + vrunnerSettings + '\'' + ", " + "vrunnerSteps=" + vrunnerSteps +
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
resources/globalConfiguration.jsonis excluded by!**/*.jsonresources/schema.jsonis excluded by!**/*.json
📒 Files selected for processing (1)
src/ru/pulsar/jenkins/library/configuration/BddOptions.groovy(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (1)
src/ru/pulsar/jenkins/library/configuration/BddOptions.groovy (1)
10-13: ПолеvrunnerSettingsдолжным образом интегрировано. Проверка пройдена.Проверка подтверждает, что поле
vrunnerSettingsактивно используется в коде:
- 7+ файлов этапов читают
options.vrunnerSettingsи передают его в команды vrunner с флагом--settings- Во всех местах использования правильно проверяется наличие файла перед применением
toString()корректно включает новое полеДублирование пути
"./tools/vrunner.json"междуvrunnerSettingsиvrunnerStepsявляется намеренным и приемлемым:vrunnerSteps— это статический массив примеров команд (документация), аvrunnerSettings— это настраиваемый параметр конфигурации, используемый всеми этапами пайплайна. Поле успешно устраняетMissingPropertyException.
|
@Kyrales дай, пожалуйста, полный стак-трейс ошибки |
@nixel2007 лови: Found unhandled hudson.remoting.ProxyException exception: |
3 participants
Не учтены настройки vrunnerSettings для bdd этапа, а они нужны для подготовки базы с нужными расширениями. Если ничего не указывать то отрабатывает корректно, но когда указываем bdd возникает ошибка:
groovy.lang.MissingPropertyException: No such property: vrunnerSettings for class: ru.pulsar.jenkins.library.configuration.BddOptions