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

Add positional arguments to the synchronous test function wrapper. #303

Conversation

seifertm
Copy link
Contributor

@seifertm seifertm commented Mar 3, 2022

This fixes a compatibility issue with Hypothesis 6.39.0.

No new test case was added, because the existing unit tests were already failing (see https://github.com/pytest-dev/pytest-asyncio/runs/5380149531)

Closes #302

This fixes a compatibility issue with Hypothesis 6.39.0.

Closes pytest-dev#302

Signed-off-by: Michael Seifert <m.seifert@digitalernachschub.de>
@codecov-commenter
Copy link

Codecov Report

Merging #303 (724be81) into master (357cddb) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #303   +/-   ##
=======================================
  Coverage   92.67%   92.67%           
=======================================
  Files           3        3           
  Lines         273      273           
  Branches       40       40           
=======================================
  Hits          253      253           
  Misses         12       12           
  Partials        8        8           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 357cddb...724be81. Read the comment docs.

@seifertm
Copy link
Contributor Author

seifertm commented Mar 3, 2022

I hope you don't mind that I'm merging this without an additional review. It seems like a trivial change and fixes the broken CI (because CI always installs the newest version of Hypothesis, which is 6.39.0 at the moment).

@seifertm seifertm merged commit cad1b94 into pytest-dev:master Mar 3, 2022
@seifertm seifertm deleted the add-positional-args-to-test-function-wrapper branch October 23, 2023 06:15
@seifertm seifertm restored the add-positional-args-to-test-function-wrapper branch October 23, 2023 08:17
@seifertm seifertm deleted the add-positional-args-to-test-function-wrapper branch October 23, 2023 08:35
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 this pull request may close these issues.

Tests fail against hypothesis 6.39.0
2 participants