Skip to content

[3.8] bpo-21478: Record calls to parent when autospecced objects are used as child with attach_mock (GH 14688)#14902

Merged
cjw296 merged 1 commit into
python:3.8from
miss-islington:backport-7397cda-3.8
Jul 22, 2019
Merged

[3.8] bpo-21478: Record calls to parent when autospecced objects are used as child with attach_mock (GH 14688)#14902
cjw296 merged 1 commit into
python:3.8from
miss-islington:backport-7397cda-3.8

Conversation

@miss-islington

@miss-islington miss-islington commented Jul 22, 2019

Copy link
Copy Markdown
Contributor
  • Clear name and parent of mock in autospecced objects used with attach_mock

  • Add NEWS entry

  • Fix reversed order of comparison

  • Test child and standalone function calls

  • Use a helper function extracting mock to avoid code duplication and refactor tests.
    (cherry picked from commit 7397cda)

Co-authored-by: Xtreak tir.karthi@gmail.com

https://bugs.python.org/issue21478

…s child with attach_mock (GH 14688)

* Clear name and parent of mock in autospecced objects used with attach_mock

* Add NEWS entry

* Fix reversed order of comparison

* Test child and standalone function calls

* Use a helper function extracting mock to avoid code duplication and refactor tests.
(cherry picked from commit 7397cda)

Co-authored-by: Xtreak <tir.karthi@gmail.com>
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @cjw296: Status check is done, and it's a success ✅ .

@cjw296
cjw296 merged commit 22fd679 into python:3.8 Jul 22, 2019
@miss-islington
miss-islington deleted the backport-7397cda-3.8 branch July 22, 2019 07:59
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @cjw296: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington

Copy link
Copy Markdown
Contributor Author

@tirkarthi and @cjw296: Status check is done, and it's a success ✅ .

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.

5 participants