Skip to content

Add pytest CLI validators#235

Merged
taipoxinous merged 1 commit intotestit-tms:mainfrom
shinbatsu:main
Feb 16, 2026
Merged

Add pytest CLI validators#235
taipoxinous merged 1 commit intotestit-tms:mainfrom
shinbatsu:main

Conversation

@shinbatsu
Copy link
Contributor

@shinbatsu shinbatsu commented Feb 12, 2026

Что делает PR

В рамках Issue #212 была подсвечена проблема с валидаторами - они реализованы на уровне, который общий для всех адаптеров, но pytest это не нравится. Для корректной работы с этим фреймворком, нужно валидировать на этапе передачи аргументов. Данный PR этот фукнционал добавляет.

P.S. Был добавлен соответствующий набор тестов, который подтвержает - ничего не поломали.

Нюансы

Адаптер - это абстракция на уровне TestIT, поэтому с добавлением валидатора для него на уровень pytest, нужно будет дополнительно следить, что при потенциальном добавлении другого режима адаптера, нужно обновлять и валидатор.

Как можно и это победить

Можно для адаптера не добавлять валидатор на уровень с pytest, но тогда нужно будет в app_propertyes импортировать pytest, например, через try -except или importlib + sys для проверки источника запуска. и если импортирован успешно, значит кидаем pytest.exit(...), а не SystemExit.

@taipoxinous
Copy link
Collaborator

Спасибо за MR, @shinbatsu, мерджу и проверяю. Если ок - исправление появится в новом релизе :)

@taipoxinous taipoxinous merged commit 44a4e9f into testit-tms:main Feb 16, 2026
30 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants