Issue 33667: Mock calls on mutable objects (original) (raw)

Issue33667

Created on 2018-05-28 13:12 by pawelj, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
test.py pawelj,2018-05-28 13:12
unintuitive_side_effect.py mcscope@gmail.com,2018-05-28 16:32
Messages (3)
msg317849 - (view) Author: Pawel (pawelj) Date: 2018-05-28 13:12
When method of mocked object is called multiple times with mutable object as parameter and attribute of this object has been updated between calls, mock_calls is only aware of the last value. I think it unexpected behaviour, that mock doesn't track value changes. I've attached example code
msg317884 - (view) Author: Lady Red (mcscope@gmail.com) * Date: 2018-05-28 16:32
To have the behavior that you are expecting, I believe the mock would have to store a deep copy of every mutable object that is passed in to any of it's calls, in case that object later mutates. That would really expand the memory and time footprint of working with a Mock object. The mock object would grow with every call to. (This is already true, as it's adding to it's mock_calls list... but it would grow much more. ) I'm also not sure about the safety of trying to deepcopy all objects passed in. I think that in addition to the safety and memory/performance issues that I've highlighted, deepcopying all objects would lead to it's own unintuitive behaviors. Objects you get back from mock_call wouldn't be the objects that it was actually called with, they would be copies. I am attaching a file that illustrates this last point.
msg317902 - (view) Author: Pawel (pawelj) Date: 2018-05-28 19:28
Ok, fair enough. Thank You for explanation. I was just surprised with this behaviour. On the other hand, I don't like mutable objects and I'm trying to avoid them.
History
Date User Action Args
2022-04-11 14:59:00 admin set github: 77848
2018-05-28 20:11:38 r.david.murray set status: open -> closedsuperseder: Python unittest.mock.mock_calls stores references to arguments instead of their valuesresolution: duplicatestage: resolved
2018-05-28 19:28:47 pawelj set messages: +
2018-05-28 16:32:35 mcscope@gmail.com set files: + unintuitive_side_effect.pynosy: + mcscope@gmail.commessages: +
2018-05-28 13:12:45 pawelj create