Bump to pytest 7.2.0 #438

seifertm · 2022-11-01T08:45:50Z

Pytest 7.2 introduces some deprecations which cause warnings when running the pytest-asyncio test suite.

Pytest now warns about the use of deprecated @pytest.mark.tryfirst and about tests that return a value different from None. The warnings were addressed accordingly so that the tests pass again.

Signed-off-by: Michael Seifert <[email protected]>

Return values in tests are deprecated starting from pytest 7.2 and will result in a warning. Signed-off-by: Michael Seifert <[email protected]>

Signed-off-by: Michael Seifert <[email protected]>

codecov-commenter · 2022-11-01T08:49:05Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.56%. Comparing base (28ba705) to head (2b696fa).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #438   +/-   ##
=======================================
  Coverage   93.56%   93.56%           
=======================================
  Files           3        3           
  Lines         264      264           
  Branches       36       36           
=======================================
  Hits          247      247           
  Misses         10       10           
  Partials        7        7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Bumps [pytest-asyncio](https://github.com/pytest-dev/pytest-asyncio) from 0.20.1 to 0.20.2. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/pytest-dev/pytest-asyncio/releases">pytest-asyncio's releases</a>.</em></p> <blockquote> <h2>pytest-asyncio 0.20.2</h2> <hr /> <h2>title: 'pytest-asyncio: pytest support for asyncio'</h2> <p><a href="https://pypi.python.org/pypi/pytest-asyncio"><img src="https://img.shields.io/pypi/v/pytest-asyncio.svg" alt="image" /></a></p> <p><a href="https://github.com/pytest-dev/pytest-asyncio/actions?workflow=CI"><img src="https://github.com/pytest-dev/pytest-asyncio/workflows/CI/badge.svg" alt="image" /></a></p> <p><a href="https://codecov.io/gh/pytest-dev/pytest-asyncio"><img src="https://codecov.io/gh/pytest-dev/pytest-asyncio/branch/master/graph/badge.svg" alt="image" /></a></p> <p><a href="https://github.com/pytest-dev/pytest-asyncio"><img src="https://img.shields.io/pypi/pyversions/pytest-asyncio.svg" alt="Supported Python versions" /></a></p> <p><a href="https://github.com/ambv/black"><img src="https://img.shields.io/badge/code%20style-black-000000.svg" alt="image" /></a></p> <p>pytest-asyncio is an Apache2 licensed library, written in Python, for testing asyncio code with pytest.</p> <p>asyncio code is usually written in the form of coroutines, which makes it slightly more difficult to test using normal testing tools. pytest-asyncio provides useful fixtures and markers to make testing easier.</p> <pre lang="{.sourceCode" data-meta=".python}"><code>@pytest.mark.asyncio async def test_some_asyncio_code(): res = await library.do_something() assert b"expected result" == res </code></pre> <p>pytest-asyncio has been strongly influenced by <a href="https://github.com/eugeniy/pytest-tornado">pytest-tornado</a>.</p> <h1>Features</h1> <ul> <li>fixtures for creating and injecting versions of the asyncio event loop</li> <li>fixtures for injecting unused tcp/udp ports</li> <li>pytest markers for treating tests as asyncio coroutines</li> <li>easy testing with non-default event loops</li> <li>support for [async def]{.title-ref} fixtures and async generator fixtures</li> <li>support <em>auto</em> mode to handle all async fixtures and tests automatically by asyncio; provide <em>strict</em> mode if a test suite should work with different async frameworks simultaneously, e.g. <code>asyncio</code> and <code>trio</code>.</li> </ul> <h1>Installation</h1>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/pytest-dev/pytest-asyncio/blob/master/CHANGELOG.rst">pytest-asyncio's changelog</a>.</em></p> <blockquote> <h1>0.20.2 (22-11-11)</h1> <ul> <li>Fixes an issue with async fixtures that are defined as methods on a test class not being rebound to the actual test instance. <code>[#197](pytest-dev/pytest-asyncio#197) <https://github.com/pytest-dev/pytest-asyncio/issues/197></code>_</li> <li>Replaced usage of deprecated <code>@pytest.mark.tryfirst</code> with <code>@pytest.hookimpl(tryfirst=True)</code> <code>[#438](pytest-dev/pytest-asyncio#438) <https://github.com/pytest-dev/pytest-asyncio/pull/438></code>_</li> </ul> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/07a1416c2fe15d85fc149b3caa35b057de0b3d6e"><code>07a1416</code></a> Prepare release of v0.20.2.</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/dc3ad211d160006b4a30996c0a2a2c29754ef1fc"><code>dc3ad21</code></a> Build(deps): Bump pytest-trio in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/441">#441</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/d9faba85890334f0548732d35f1b1d54a850a69f"><code>d9faba8</code></a> Build(deps): Bump mypy from 0.982 to 0.990 in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/440">#440</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/fe63e346154b61bbfe767e585b0b3b55fb37463e"><code>fe63e34</code></a> Handle bound fixture methods correctly (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/439">#439</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/38fc0320c39e24a473240303fbc780213354e64d"><code>38fc032</code></a> Bump to pytest 7.2.0 (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/438">#438</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/28ba705a81d041bd3b5487eb53ded447676dad37"><code>28ba705</code></a> Build(deps): Bump hypothesis in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/437">#437</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/91e723a373952640e08d69adaff1957a8cbe8c8e"><code>91e723a</code></a> Build(deps): Bump zipp from 3.9.0 to 3.10.0 in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/434">#434</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/0ca201b09a8ce2ff3ddc912ad434d9db34ef5078"><code>0ca201b</code></a> Fix setuptools deprecation warning for license_file (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/432">#432</a>)</li> <li>See full diff in <a href="https://github.com/pytest-dev/pytest-asyncio/compare/v0.20.1...v0.20.2">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=pytest-asyncio&package-manager=pip&previous-version=0.20.1&new-version=0.20.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details>

[//]: # (dependabot-start) ⚠️ **Dependabot is rebasing this PR** ⚠️ Rebasing might not happen immediately, so don't worry if this takes some time. Note: if you make any changes to this PR yourself, they will take precedence over the rebase. --- [//]: # (dependabot-end) Bumps [pytest-asyncio](https://github.com/pytest-dev/pytest-asyncio) from 0.20.1 to 0.20.3. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/pytest-dev/pytest-asyncio/releases">pytest-asyncio's releases</a>.</em></p> <blockquote> <h2>pytest-asyncio 0.20.3</h2> <hr /> <h2>title: 'pytest-asyncio'</h2> <p><a href="https://pypi.python.org/pypi/pytest-asyncio"><img src="https://img.shields.io/pypi/v/pytest-asyncio.svg" alt="image" /></a></p> <p><a href="https://github.com/pytest-dev/pytest-asyncio/actions?workflow=CI"><img src="https://github.com/pytest-dev/pytest-asyncio/workflows/CI/badge.svg" alt="image" /></a></p> <p><a href="https://codecov.io/gh/pytest-dev/pytest-asyncio"><img src="https://codecov.io/gh/pytest-dev/pytest-asyncio/branch/master/graph/badge.svg" alt="image" /></a></p> <p><a href="https://github.com/pytest-dev/pytest-asyncio"><img src="https://img.shields.io/pypi/pyversions/pytest-asyncio.svg" alt="Supported Python versions" /></a></p> <p><a href="https://github.com/ambv/black"><img src="https://img.shields.io/badge/code%20style-black-000000.svg" alt="image" /></a></p> <p>pytest-asyncio is a <a href="https://docs.pytest.org/en/latest/contents.html">pytest</a> plugin. It facilitates testing of code that uses the <a href="https://docs.python.org/3/library/asyncio.html">asyncio</a> library.</p> <p>Specifically, pytest-asyncio provides support for coroutines as test functions. This allows users to <em>await</em> code inside their tests. For example, the following code is executed as a test item by pytest:</p> <pre lang="{.python}"><code>@pytest.mark.asyncio async def test_some_asyncio_code(): res = await library.do_something() assert b"expected result" == res </code></pre> <p>Note that test classes subclassing the standard <a href="https://docs.python.org/3/library/unittest.html">unittest</a> library are not supported. Users are advised to use <a href="https://docs.python.org/3/library/unittest.html#unittest.IsolatedAsyncioTestCase">unittest.IsolatedAsyncioTestCase</a> or an async framework such as <a href="https://asynctest.readthedocs.io/en/latest">asynctest</a>.</p> <p>pytest-asyncio is available under the <a href="https://github.com/pytest-dev/pytest-asyncio/blob/master/LICENSE">Apache License 2.0</a>.</p> <h1>Installation</h1> <p>To install pytest-asyncio, simply:</p> <pre lang="{.bash}"><code>$ pip install pytest-asyncio </code></pre>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/pytest-dev/pytest-asyncio/blob/master/CHANGELOG.rst">pytest-asyncio's changelog</a>.</em></p> <blockquote> <h1>0.20.3 (22-12-08)</h1> <ul> <li>Prevent DeprecationWarning to bubble up on CPython 3.10.9 and 3.11.1. <code>[#460](pytest-dev/pytest-asyncio#460) <https://github.com/pytest-dev/pytest-asyncio/issues/460></code>_</li> </ul> <h1>0.20.2 (22-11-11)</h1> <ul> <li>Fixes an issue with async fixtures that are defined as methods on a test class not being rebound to the actual test instance. <code>[#197](pytest-dev/pytest-asyncio#197) <https://github.com/pytest-dev/pytest-asyncio/issues/197></code>_</li> <li>Replaced usage of deprecated <code>@pytest.mark.tryfirst</code> with <code>@pytest.hookimpl(tryfirst=True)</code> <code>[#438](pytest-dev/pytest-asyncio#438) <https://github.com/pytest-dev/pytest-asyncio/pull/438></code>_</li> </ul> </blockquote> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/007e8ec12662ffd896c6151239dc7ed1402dc710"><code>007e8ec</code></a> [fix] Prevent DeprecationWarning about existing event loops to bubble up into...</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/44ca3da2ad68ea672d07a76ccc065922e13b5d5b"><code>44ca3da</code></a> Build(deps): Bump zipp from 3.10.0 to 3.11.0 in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/455">#455</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/c3c601cfd9a59e52b555cfd8313d16dbc15fb704"><code>c3c601c</code></a> Build(deps): Bump pypa/gh-action-pypi-publish from 1.5.1 to 1.5.2 (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/456">#456</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/a962e2bc89e1181de77e486b1d7cbd7815662350"><code>a962e2b</code></a> Build(deps): Bump importlib-metadata in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/454">#454</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/56a393abec9b60d4e061b053dfdf8ce6985c8b6b"><code>56a393a</code></a> Simplify README, move most content to a separate user documentation. (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/448">#448</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/3c78732497e02cfb4463fafd7c5b17bf1c88ce95"><code>3c78732</code></a> Build(deps): Bump hypothesis in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/453">#453</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/d6a9a72ef1749a864e64ac6222a8d0da99e67de5"><code>d6a9a72</code></a> Build(deps): Bump exceptiongroup in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/451">#451</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/42da7a0fea2b2bf0846dbbed5d1abcf56c7fa38b"><code>42da7a0</code></a> Build(deps): Bump hypothesis in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/450">#450</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/0b281b1b76b93c29894519e0750a4f8634786741"><code>0b281b1</code></a> Build(deps): Bump mypy from 0.990 to 0.991 in /dependencies/default (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/446">#446</a>)</li> <li><a href="https://github.com/pytest-dev/pytest-asyncio/commit/d39589c0353657ee6d75d38db779cc4ecb2491c4"><code>d39589c</code></a> Update pre-commit hooks (<a href="https://github-redirect.dependabot.com/pytest-dev/pytest-asyncio/issues/449">#449</a>)</li> <li>Additional commits viewable in <a href="https://github.com/pytest-dev/pytest-asyncio/compare/v0.20.1...v0.20.3">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=pytest-asyncio&package-manager=pip&previous-version=0.20.1&new-version=0.20.3)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

seifertm added 3 commits November 1, 2022 09:41

refactor: Address deprecation of @pytest.mark.tryfirst.

ed17190

Signed-off-by: Michael Seifert <[email protected]>

test: Remove return value from test case.

a8b947a

Return values in tests are deprecated starting from pytest 7.2 and will result in a warning. Signed-off-by: Michael Seifert <[email protected]>

build: Bump pytest to v7.2.0.

2b696fa

Signed-off-by: Michael Seifert <[email protected]>

seifertm requested review from asvetlov and Tinche as code owners November 1, 2022 08:45

seifertm merged commit 38fc032 into pytest-dev:master Nov 10, 2022

seifertm mentioned this pull request Nov 10, 2022

Handle bound fixture methods correctly #439

Merged

seifertm deleted the bump-to-pytest-7.2.0 branch October 23, 2023 06:16

seifertm restored the bump-to-pytest-7.2.0 branch October 23, 2023 08:17

seifertm deleted the bump-to-pytest-7.2.0 branch October 23, 2023 08:31

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bump to pytest 7.2.0 #438

Bump to pytest 7.2.0 #438

seifertm commented Nov 1, 2022

codecov-commenter commented Nov 1, 2022 •

edited

Loading

Bump to pytest 7.2.0 #438

Bump to pytest 7.2.0 #438

Conversation

seifertm commented Nov 1, 2022

codecov-commenter commented Nov 1, 2022 • edited Loading

Codecov Report

codecov-commenter commented Nov 1, 2022 •

edited

Loading