Skip to content

Move conftest to top-level #54

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Move conftest to top-level #54

wants to merge 1 commit into from

Conversation

jtojnar
Copy link

@jtojnar jtojnar commented Oct 18, 2022

Same as smarie/python-pytest-harvest#62

This fixes the following error:

________________________ ERROR collecting test session _________________________
Defining 'pytest_plugins' in a non-top-level conftest is no longer supported:
It affects the entire test suite instead of just below the conftest as expected.
  /build/pytest-steps-1.8.0/pytest_steps/conftest.py
Please move it to a top level conftest file at the rootdir:
  /build/pytest-steps-1.8.0
For more information, visit:
  https://docs.pytest.org/en/stable/deprecations.html#pytest-plugins-in-non-top-level-conftest-files

This fixes the following error:

	________________________ ERROR collecting test session _________________________
	Defining 'pytest_plugins' in a non-top-level conftest is no longer supported:
	It affects the entire test suite instead of just below the conftest as expected.
	  /build/pytest-steps-1.8.0/pytest_steps/conftest.py
	Please move it to a top level conftest file at the rootdir:
	  /build/pytest-steps-1.8.0
	For more information, visit:
	  https://docs.pytest.org/en/stable/deprecations.html#pytest-plugins-in-non-top-level-conftest-files
@smarie
Copy link
Owner

smarie commented Feb 26, 2023

Thanks @jtojnar !

Closing/Reopening PR to see the result of the CI

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants