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

All plugins loaded using pytest_plugins must be optional hooks #2

Closed
goodboy opened this issue Feb 27, 2017 · 1 comment
Closed

All plugins loaded using pytest_plugins must be optional hooks #2

goodboy opened this issue Feb 27, 2017 · 1 comment

Comments

@goodboy
Copy link
Contributor

goodboy commented Feb 27, 2017

Due to pytest-dev/pytest#1821 we are currently stuck annotating all external plugins which use the pytest_plugins attribute with:

@pytest.hookimpl(optionalhook=True)
def pytest_lab_addroles(config, rolemanager):
     ...

It would be much preferable to have the pluggy.PluginManager.check_pending() call in pytest be invoked after the historic call to pytest_addhooks() to avoid this early validation error. It's that or we have to use our own plugin manager instance.

@goodboy
Copy link
Contributor Author

goodboy commented Feb 28, 2017

Misfire on this one.
Fixed by using entry points in ba12575.

@goodboy goodboy closed this as completed Feb 28, 2017
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

No branches or pull requests

1 participant