Skip to content

Fix #168: 1) respect user case that provides specific event loop policy, 2) add a new unitary test to check it #172

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

Closed
wants to merge 2 commits into from

Conversation

alblasco
Copy link
Contributor

Hi @Tinche
Let me know if this PR can be helpful to support the project.
Please let me know your thoughts or share the solution you are working on.

Don't hesitate to let me know what's the best way to help

alblasco added 2 commits June 28, 2020 11:52
To take into account user case that sets a specific event loop policy before runnins pytest asyncio tests:
- Rollover changes in V0.14 (that clears event loop policy in pytest_fixture_post_finalizer)
- include a new unit test that checks if policy has been changed (fails with V0.14 and pass with this PR).
@alblasco
Copy link
Contributor Author

alblasco commented Jun 28, 2020

Hi @Tinche
After submitting PR, I have just realised you made a policies branch.

Apologise. Please close this PR, we would proceed discussions in policies branch.

@Tinche
Copy link
Member

Tinche commented Jun 28, 2020

Let's move the discussion to #174

@Tinche Tinche closed this Jun 28, 2020
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.

2 participants