Skip to content

bpo-40275: Avoid importing asyncio in test.support - #19600

Merged
serhiy-storchaka merged 5 commits into
python:masterfrom
serhiy-storchaka:test-support-asyncio
Apr 25, 2020
Merged

bpo-40275: Avoid importing asyncio in test.support#19600
serhiy-storchaka merged 5 commits into
python:masterfrom
serhiy-storchaka:test-support-asyncio

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Apr 19, 2020

Copy link
Copy Markdown
Member
  • Import IsolatedAsyncioTestCase lazily in unittest.
  • Import asyncio.events lazily in test.support.

https://bugs.python.org/issue40275

* Import IsolatedAsyncioTestCase lazily in unittest.
* Import asyncio.events lazily in test.support.

@shihai1991 shihai1991 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Comment thread Lib/unittest/__init__.py
Comment thread Lib/unittest/__init__.py

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@vstinner

Copy link
Copy Markdown
Member

Even if unittest.async_case is only new in 3.9, you might add a NEWS entry to mention that you made the import lazy. It's up to you, I already approved your PR ;-)

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. (I reviewed the NEWS entry.)

@serhiy-storchaka
serhiy-storchaka merged commit 3c8a5b4 into python:master Apr 25, 2020
@serhiy-storchaka
serhiy-storchaka deleted the test-support-asyncio branch April 25, 2020 07:04
CuriousLearner added a commit to CuriousLearner/cpython that referenced this pull request Apr 27, 2020
* 'master' of github.com:python/cpython: (2949 commits)
  Add files in tests/test_peg_generator to the install target lists (pythonGH-19723)
  bpo-40398: Fix typing.get_args() for special generic aliases. (pythonGH-19720)
  bpo-40348: Fix typos in the programming FAQ (pythonGH-19729)
  bpo-38387: Formally document PyDoc_STRVAR and PyDoc_STR macros (pythonGH-16607)
  bpo-40401: Remove duplicate pyhash.h include from pythoncore.vcxproj (pythonGH-19725)
  bpo-40387: Improve queue join() example. (pythonGH-19724)
  bpo-40396: Support GenericAlias in the typing functions. (pythonGH-19718)
  Fix typo in Lib/typing.py (pythonGH-19717)
  Fix typo in object.__format__ docs (pythonGH-19504)
  bpo-40275: Avoid importing logging in test.support (pythonGH-19601)
  bpo-40275: Avoid importing socket in test.support (pythonGH-19603)
  bpo-40275: Avoid importing asyncio in test.support (pythonGH-19600)
  bpo-40279: Add some error-handling to the module initialisation docs example (pythonGH-19705)
  closes bpo-40385: Remove Tools/scripts/checkpyc.py (pythonGH-19709)
  bpo-40334: Add What's New sections for PEP 617 and PEP 585 (pythonGH-19704)
  bpo-40340: Separate examples more clearly in the programming FAQ (pythonGH-19688)
  bpo-40360: Deprecate lib2to3 module in light of PEP 617 (pythonGH-19663)
  bpo-40334: Rewrite test_c_parser to avoid memory leaks (pythonGH-19694)
  bpo-38061: subprocess uses closefrom() on FreeBSD (pythonGH-19697)
  bpo-38061: os.closerange() uses closefrom() on FreeBSD (pythonGH-19696)
  ...

@MadGreg-UK MadGreg-UK left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@MadGreg-UK MadGreg-UK left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on it

@MadGreg-UK

This comment was marked as off-topic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

6 participants