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

Не все настройки ВА обрабатывают параметр workspaceRoot, переданный через runner run #559

Closed
MironovDmitry opened this issue Nov 13, 2019 · 6 comments
Labels
bug Something isn't working

Comments

@MironovDmitry
Copy link

Заполните шаблон обсуждения GIT

Функционал: При запуске тестов через runner run
   Как фичапис
   Я хочу передать параметр wokrspaceRoot, относительно которого должны строиться все относительные пути в настройках ВА

Контекст:
  Дано: команда запуска тестов runner run --v8version 8.3.12 --execute "./vanessa-automation/vanessa-automation.epf" --command "StartFeaturePlayer;workspaceRoot=D:\projectRootFolder;VBParams=D:\projectRootFolder\tools\JSON\VBParams.json"
 и пустая конфигурация запускаемая в режиме тестменеджера

Сценарий: Все настройки ВА, в параметрах которых в файле VBParams.json указаны пути пипа "$workspaceRoot/folder" корректно обработаны ВА
  Допустим: В фале VBParams.json параметр указан как "КаталогВыгрузкиAllure": "$workspaceRoot/build/out/allure"
  Когда: запускаю команду runner run ....
  Тогда: В настройках ВА параметр КаталогВыгрузкиAllure равен "D:\projectRootFolder\build\out\allure"

также не работает для параметров:
"КаталогВыгрузкиJUnit"
"КаталогВыгрузкиCucumberJson"
"КаталогВыгрузкиСкриншотов"
"КаталогПроекта": "$workspaceRoot"

Прикладывайте файлы скриншотов
image
image

@Pr-Mex Pr-Mex added the bug Something isn't working label Nov 14, 2019
@fenixnow
Copy link
Contributor

Вместо $workspaceRoot нужно использовать ./
поведение изменилось в 29 версии.
@Pr-Mex

@Pr-Mex
Copy link
Owner

Pr-Mex commented Jan 12, 2020

@fenixnow
Нужно вернуть совместимость со старым поведением.

@vl-lit
Copy link
Contributor

vl-lit commented Feb 10, 2020

@fenixnow @Pr-Mex
Коллеги, хотелось бы обратить внимание, что сейчас в разных методах модуля управляемой формы VA точка в начали путей и шаблон $workspaceRoot интерпретируется по разному.

Это приводит не только к несовместимости со старым поведением (до релиза 1.0.28 включительно), но и к тому, что нельзя задавать пути в файле настроек единообразно.

В методе "ЗагрузитьНастройкиНаСервере" вызывается не общий метод "ПреобразоватьПутьСТочкамиКНомальномуПути" , а происходит обработка точки в начале пути по собственному алгоритму:
image

В итоге при обработке каталогов библиотек в версии 1.0.29 точка преобразуется к каталогу проекта (в версии 1.028 преобразовывалась к каталогу VA - каталогу инструментов).

В то же время в методе "ПреобразоватьПутьСТочкамиКНомальномуПути" указано, что ради сохранения совместимости точка по прежнему воспринимается как каталог инструментов (каталог, в которой находится обработка VA):

image

Также при указании путей к библиотекам корректно воспринимается как шаблон $workspaceRoot так и шаблон $instrumentsRoot. А при указании пути к каталогу фич $workspaceRoot более не работает.

Поэтому сейчас можно использовать такую настройку:

image

Имеет смысл не только вернуть возможность указывать $workspaceRoot во всех параметрах-путях, но и унифицировать поведение методов "ЗагрузитьНастройкиНаСервере" и "ПреобразоватьПутьСТочкамиКНомальномуПути" , в идеале оставив один метод, отвечающий за обработку путей.

Замена точки в начале пути на путь к каталогу проекта - это удобно. Но тогда нужно это и в методе "ПреобразоватьПутьСТочкамиКНомальномуПути" сделать.

@Pr-Mex
Copy link
Owner

Pr-Mex commented Feb 11, 2020

@VladimirLitvinenko84
Можете проверить на версии из develop?
Я вернул обработку $workspaceRoot. И добавил на это тест.

@vl-lit
Copy link
Contributor

vl-lit commented Feb 13, 2020

Да, в ветке develop работает как с точкой, так и с $workspaceRoot.
Точка действительно намного удобнее для указания пути к каталогу проекта, чем к каталогу инструментов. $instrumentsRoot по практике обычно указывается в файле настроек запуска только один раз - для загрузки стандартной библиотеки шагов.

@Pr-Mex
Copy link
Owner

Pr-Mex commented Mar 8, 2020

Решено.

@Pr-Mex Pr-Mex closed this as completed Mar 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants