mirror of
https://github.com/python/cpython.git
synced 2025-10-22 09:23:54 +00:00
bpo-30541: Add new method to seal mocks (GH61923)
The new method allows the developer to control when to stop the feature of mocks that automagically creates new mocks when accessing an attribute that was not declared before Signed-off-by: Mario Corchero <mariocj89@gmail.com>
This commit is contained in:
parent
2bd37c227e
commit
552be9d7e6
5 changed files with 249 additions and 2 deletions
|
@ -18,6 +18,7 @@
|
|||
'NonCallableMagicMock',
|
||||
'mock_open',
|
||||
'PropertyMock',
|
||||
'seal',
|
||||
)
|
||||
|
||||
|
||||
|
@ -382,6 +383,7 @@ def __init__(
|
|||
__dict__['_mock_name'] = name
|
||||
__dict__['_mock_new_name'] = _new_name
|
||||
__dict__['_mock_new_parent'] = _new_parent
|
||||
__dict__['_mock_sealed'] = False
|
||||
|
||||
if spec_set is not None:
|
||||
spec = spec_set
|
||||
|
@ -608,7 +610,7 @@ def __getattr__(self, name):
|
|||
return result
|
||||
|
||||
|
||||
def __repr__(self):
|
||||
def _extract_mock_name(self):
|
||||
_name_list = [self._mock_new_name]
|
||||
_parent = self._mock_new_parent
|
||||
last = self
|
||||
|
@ -638,7 +640,10 @@ def __repr__(self):
|
|||
if _name_list[1] not in ('()', '().'):
|
||||
_first += '.'
|
||||
_name_list[0] = _first
|
||||
name = ''.join(_name_list)
|
||||
return ''.join(_name_list)
|
||||
|
||||
def __repr__(self):
|
||||
name = self._extract_mock_name()
|
||||
|
||||
name_string = ''
|
||||
if name not in ('mock', 'mock.'):
|
||||
|
@ -705,6 +710,11 @@ def __setattr__(self, name, value):
|
|||
else:
|
||||
if _check_and_set_parent(self, value, name, name):
|
||||
self._mock_children[name] = value
|
||||
|
||||
if self._mock_sealed and not hasattr(self, name):
|
||||
mock_name = f'{self._extract_mock_name()}.{name}'
|
||||
raise AttributeError(f'Cannot set {mock_name}')
|
||||
|
||||
return object.__setattr__(self, name, value)
|
||||
|
||||
|
||||
|
@ -888,6 +898,12 @@ def _get_child_mock(self, **kw):
|
|||
klass = Mock
|
||||
else:
|
||||
klass = _type.__mro__[1]
|
||||
|
||||
if self._mock_sealed:
|
||||
attribute = "." + kw["name"] if "name" in kw else "()"
|
||||
mock_name = self._extract_mock_name() + attribute
|
||||
raise AttributeError(mock_name)
|
||||
|
||||
return klass(**kw)
|
||||
|
||||
|
||||
|
@ -2401,3 +2417,26 @@ def __get__(self, obj, obj_type):
|
|||
return self()
|
||||
def __set__(self, obj, val):
|
||||
self(val)
|
||||
|
||||
|
||||
def seal(mock):
|
||||
"""Disable the automatic generation of "submocks"
|
||||
|
||||
Given an input Mock, seals it to ensure no further mocks will be generated
|
||||
when accessing an attribute that was not already defined.
|
||||
|
||||
Submocks are defined as all mocks which were created DIRECTLY from the
|
||||
parent. If a mock is assigned to an attribute of an existing mock,
|
||||
it is not considered a submock.
|
||||
|
||||
"""
|
||||
mock._mock_sealed = True
|
||||
for attr in dir(mock):
|
||||
try:
|
||||
m = getattr(mock, attr)
|
||||
except AttributeError:
|
||||
continue
|
||||
if not isinstance(m, NonCallableMock):
|
||||
continue
|
||||
if m._mock_new_parent is mock:
|
||||
seal(m)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue