ТЭ
А чем плох такой вариант?
переименуешь параметр и пизда твоему скипу
Size: a a a
2020 November 23
M
можно не переименовывать параметр
ТЭ
@pytest.fixture
def skip_by_env(env):
if not Config.env == Config.skipped_env:
pytest.skip()
yield env
Можно так как-нибудь попробовать
ТЭ
можно не переименовывать параметр
Изящно
BK
переименуешь параметр и пизда твоему скипу
такого точно никогда небудет) разве что только новый может добавится
BK
@pytest.fixture
def skip_by_env(env):
if not Config.env == Config.skipped_env:
pytest.skip()
yield env
Можно так как-нибудь попробовать
но идейка хорошая
ТЭ
такого точно никогда небудет) разве что только новый может добавится
А, ну раз точно никогда не будет, тогда можно оставить как есть👌
BK
всю эту задумку пришлось сделать) так как немного разные данные на енвах поэтому пока данные не актуализируются приходится такой костылёк со скипом городить
ТЭ
Зависит от масштаба проблемы, но я бы просто тест целиком вырубал и все
ТЭ
Либо если там очень дофига тестов не гонял бы на стенде без данных просто.
ТЭ
Либо наоборот, собрал смоук сьют, без которого смысла в тестах нет, и только его на проблемных стендах гонял
BK
Зависит от масштаба проблемы, но я бы просто тест целиком вырубал и все
если я просто выключу, то когда пойдёт прогон на beta енве, он там скипнется тоже, а там то есть же данные)
BK
а так пока на альфе данных нету то скипать
BK
как появятся уберу скип да и всё
ТЭ
Короче, любой вариант проще и надежнее, чем вот этот микроменеджемент скипов.
BK
в случае если и там и там херня, да я просто скипаю
СС
@pytest.fixture
def skip_by_env(env):
if not Config.env == Config.skipped_env:
pytest.skip()
yield env
Можно так как-нибудь попробовать
кстати, у меня сделано вообще по другому, только что вспомнил и глянул. похожая ситуация, только аргумент другой. я в таком случае выкидываю тест из коллекции и его как будто бы и не было (не маркается как skipped).
СС
ТЭ
Да, это хорошо. Не замусоривать отчет лишний раз
AM
Вообщем сделал просто таким способом)))
@pytest.mark.skipif('--env=alpha' in sys.argv, reason='Test')ну это есть в документации, да. 0 строк и классов и модулей - мне нра!