-
Notifications
You must be signed in to change notification settings - Fork 237
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
[BUG] Latest pytest release (8.1.0) breaks allure_listener plugin #794
Comments
|
This is the related change: https://docs.pytest.org/en/stable/changelog.html#trivial-internal-changes So it should a matter of changing: fixturedefs_pytest = fixturemanager.getfixturedefs(name, item.nodeid) To: fixturedefs_pytest = fixturemanager.getfixturedefs(name, item) |
Our CI/CD pipeline fails with pytest==8.1.0. removing |
pytest 已经发布新的版本 8.1.1。于是重新引发 EN |
The fix is included in allure-pytest 2.13.3 |
So,guys!
when i follow nicoddemus, it works!
I just have upgraded pytest version from 8.1.1 to 8.2.0, |
I'm submitting a ...
What is the current behavior?
After
pytest
released its latest version8.1.0
it suddenly started to breakallure_listener
plugin on fixtures initilization/teardown thus preventing from running any tests withpytest
andallure-pytest
installed.If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
pytest=8.1.0
installedallure-pytest
installed (as of the moment of writing it is2.13.0
)pytest
configured@pytest.fixture
decoratorThe following is a minimal example the I could reproduce from our tests that caught this issue:
What is the expected behavior?
It is expected for
allure-pytest
not to break whole project when updating topytest==8.1.0
What is the motivation / use case for changing the behavior?
Not to break tests
Please tell us about your environment:
Other information
When running the test the stack trace gives the following error:
As it can be seen from the error message the following line is causing the issue:
allure-python/allure-pytest/src/listener.py
Line 102 in 058a6af
I believe this can be due to following change recently introduced in the
pytest
framework, because right after that, we upgraded to the latestpytest
and our tests started to fail without even running (the error thrown during the initialization of the fixtures I believe). And downgrading topytest==8.0.0
fixed this issues. However, I think, it would be nice to fix this to make sure further compatability betweenallure-pytest
andpytest
framework.Commit from the
pytest
: pytest-dev/pytest@434282eP.S I am not exactly sure who should be responsible for handling this issue whether
allure
team orpytest
team.The text was updated successfully, but these errors were encountered: