Skip to content

add backwards compatibility for pytest hook wrapper #23781

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.

Sign up for GitHub

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

Merged
merged 12 commits into from
Jul 15, 2024
Merged

add backwards compatibility for pytest hook wrapper #23781

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
f5d4ae2
add backwards compatibility for pytest hook wrapper
eleanorjboyd Jul 1, 2024
66dc81c
Add compatibility for pluggy version 1.1.0 in pytest
eleanorjboyd Jul 10, 2024
3e0bf38
remove print
eleanorjboyd Jul 10, 2024
44cb53f
attempt w/o test
eleanorjboyd Jul 10, 2024
2eb5efe
attempt with hookwrapper instead
eleanorjboyd Jul 11, 2024
f7e70ff
remove accidental subprocess call
eleanorjboyd Jul 11, 2024
fcc0ea1
re-add test for old versioning
eleanorjboyd Jul 11, 2024
bb25877
remove failing test
eleanorjboyd Jul 15, 2024
dfc00b9
remove unneeded import
eleanorjboyd Jul 15, 2024
c5df335
remove import
eleanorjboyd Jul 15, 2024
65dc5ef
Merge branch 'main' into hook-wrapper
eleanorjboyd Jul 15, 2024
a517713
org imports
eleanorjboyd Jul 15, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion python_files/vscode_pytest/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -888,7 +888,7 @@ def send_post_request(


class DeferPlugin:
@pytest.hookimpl(wrapper=True)
@pytest.hookimpl(hookwrapper=True)
def pytest_xdist_auto_num_workers(self, config: pytest.Config) -> Generator[None, int, int]:
"""Determine how many workers to use based on how many tests were selected in the test explorer."""
return min((yield), len(config.option.file_or_dir))
Expand Down
Loading