fix compatibility with pytest ^8

[akeeman]

fixes #737

pytest-asyncio is incompatible with pytest 8.0.0rc1 and 8.0.0rc2, but not with pytest 8.0.0. Although 2 tests had to be updated, as suggested here by @mgorny.

• test_asyncio_mark_provides_session_scoped_loop_strict_mode: order of module collection has changed from files first to mixed files and folders (source)
• test_returns_false_for_unmarked_coroutine_item_in_strict_mode: raises a failure now (again)

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (e92efad) 94.33% compared to head (d53c129) 94.96%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #776      +/-   ##
==========================================
+ Coverage   94.33%   94.96%   +0.62%     
==========================================
  Files           2        2              
  Lines         477      477              
  Branches       95       95              
==========================================
+ Hits          450      453       +3     
+ Misses         17       15       -2     
+ Partials       10        9       -1     

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

[mgorny]

Thanks for doing this!

[seifertm]

Thanks for the patch!

I think some of the version changes are problematic. Other than that, the PR looks good.

If you don't mind, I'll adjust the version numbers myself and add a changelog entry. Maybe this allows us to get out a pre-release for pytest-asyncio with pytest 8 support today.

[seifertm]

I ran your patch successfully against pytest-8.0.0rc1 and pytest-8.0.0rc2. This means there's not reason to exclude them in the compatible versions. It's more likely that #764 reduced the number of issues with the collection changes in pytest.

More importantly, we should try to keep the upper version bound at all costs, both in requirements.txt and especially in setup.py. Otherwise we assume that pytest-asyncio is compatible with potentially breaking changes in pytest 9.

[seifertm]

The intention of the pytest-min environment is to test against an older version of pytest. This way, pytest-asyncio doesn't accidentally use any of the newer pytest features without a proper bump of the minimum supported version.

I think we should continue to support pytest 7 for at least a short period of time, in order to allow users to migrate pytest independently from pytest-asyncio.