Skip to content

Commit

Permalink
bpo-30266: support "= None" pattern in AbstractContextManager (GH-1448)…
Browse files Browse the repository at this point in the history
… (GH-2054)

contextlib.AbstractContextManager now supports anti-registration
by setting __enter__ = None or __exit__ = None, following the pattern
introduced in bpo-25958..
(cherry picked from commit 57161aa)
  • Loading branch information
Mariatta committed Jun 10, 2017
1 parent e380c19 commit 753422f
Show file tree
Hide file tree
Showing 3 changed files with 16 additions and 3 deletions.
5 changes: 2 additions & 3 deletions Lib/contextlib.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
"""Utilities for with-statement contexts. See PEP 343."""
import abc
import sys
import _collections_abc
from collections import deque
from functools import wraps

Expand All @@ -25,9 +26,7 @@ def __exit__(self, exc_type, exc_value, traceback):
@classmethod
def __subclasshook__(cls, C):
if cls is AbstractContextManager:
if (any("__enter__" in B.__dict__ for B in C.__mro__) and
any("__exit__" in B.__dict__ for B in C.__mro__)):
return True
return _collections_abc._check_methods(C, "__enter__", "__exit__")
return NotImplemented


Expand Down
10 changes: 10 additions & 0 deletions Lib/test/test_contextlib.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,16 @@ def __exit__(self, *args):

self.assertTrue(issubclass(DefaultEnter, AbstractContextManager))

class NoEnter(ManagerFromScratch):
__enter__ = None

self.assertFalse(issubclass(NoEnter, AbstractContextManager))

class NoExit(ManagerFromScratch):
__exit__ = None

self.assertFalse(issubclass(NoExit, AbstractContextManager))


class ContextManagerTestCase(unittest.TestCase):

Expand Down
4 changes: 4 additions & 0 deletions Misc/NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,10 @@ Library
- bpo-30048: Fixed ``Task.cancel()`` can be ignored when the task is
running coroutine and the coroutine returned without any more ``await``.

- bpo-30266: contextlib.AbstractContextManager now supports anti-registration
by setting __enter__ = None or __exit__ = None, following the pattern
introduced in bpo-25958. Patch by Jelle Zijlstra.

- bpo-30298: Weaken the condition of deprecation warnings for inline modifiers.
Now allowed several subsequential inline modifiers at the start of the
pattern (e.g. ``'(?i)(?s)...'``). In verbose mode whitespaces and comments
Expand Down

0 comments on commit 753422f

Please sign in to comment.