bugfix/fix-flaky-aiohttp-unit-tests

[chopkinsmade]

Description

• Amend the aio_client_factory fixture to use the asyncio loop created by the unit test inside the aiohttp test client. When this code is missing, the aiohttp code might use a different task loop than the test which causes flaky tests.
• Skip the broken test for now while a fix is investigated. Even after setting the aiohttp test client to use the same task loop, an internal function that closes the connection is using a different task loop causing errors
• Use the config setting asyncio_mode = "auto" to auto run async tests without needing to use a pytest.mark statement

Contributors

Let's acknowledge the people who contributed to the work.

Type of change

• Refactoring (made code better without changing its behaviour)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

How this has been tested

Please describe the tests that you ran to verify your changes.

If they are not automated tests please explain why and provide screenshots and/or instructions so they can reproduced.

Checklist

• I have performed a self-review of my code
• I have commented my code in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings

Reviewer Checklist

• I have reviewed the PR and ensured no secret values are present

[github-actions]

Dependency Review

✅ No vulnerabilities or OpenSSF Scorecard issues found.

Scanned Files

None