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

Project name based collection triggers regression #99

Closed
goodboy opened this issue Nov 13, 2017 · 1 comment · Fixed by #102
Closed

Project name based collection triggers regression #99

goodboy opened this issue Nov 13, 2017 · 1 comment · Fixed by #102
Assignees

Comments

@goodboy
Copy link
Contributor

goodboy commented Nov 13, 2017

I'll put up a test which demonstrates this shortly.
We also don't seem to have any tests which don't also include adding a hookspec the lack of which is the source of the problem.

@goodboy goodboy self-assigned this Nov 13, 2017
goodboy pushed a commit to goodboy/pluggy that referenced this issue Nov 13, 2017
Triggers a regression introduced during refactoring of the multicall
loop. Demonstrates the issue described in pytest-dev#99.
@goodboy
Copy link
Contributor Author

goodboy commented Nov 13, 2017

To fix this we can bring in the spec-as-object wrapping stuff from #57 which should resolve this.

goodboy pushed a commit to goodboy/pluggy that referenced this issue Nov 13, 2017
Triggers a regression introduced during refactoring of the multicall
loop. Demonstrates the issue described in pytest-dev#99.
goodboy pushed a commit to goodboy/pluggy that referenced this issue Nov 18, 2017
Multi-call loop(s) should not care about spec opts - only whether
or not the first result should be returned.
Fix issue where a collected but unmarked hook not having
`_HookCaller.spec_opts` defined results in an attr error.

Resolves pytest-dev#99
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 a pull request may close this issue.

1 participant