]> granicus.if.org Git - python/commit
[3.6] bpo-29403: Fix mock's broken autospec behavior on method-bound builtin function...
authorBerker Peksag <berker.peksag@gmail.com>
Fri, 21 Jul 2017 22:31:04 +0000 (01:31 +0300)
committerGitHub <noreply@github.com>
Fri, 21 Jul 2017 22:31:04 +0000 (01:31 +0300)
commit64b9a15886840df422c5203fad25c9801b4cf61e
tree5e445252e7029768307f0519f54390412723479f
parentbb323b261d0f41ad6998d3a16c41b29b3bb8e982
[3.6] bpo-29403: Fix mock's broken autospec behavior on method-bound builtin functions (GH-3)

Cython will, in the right circumstances, offer a MethodType instance
where im_func is a builtin function. Any instance of MethodType is
automatically assumed to be a Python-defined function (more
specifically, a function that has an inspectable signature), but
_set_signature was still conservative in its assumptions. As a result
_set_signature would return early with None instead of a mock since
the im_func had no inspectable signature. This causes problems
deeper inside mock, as _set_signature is assumed to _always_
return a mock, and nothing checked its return value.

In similar corner cases, autospec will simply not check the spec of the
function, so _set_signature is amended to now return early with the
original, not-wrapped mock object.

Patch by Aaron Gallagher.

(cherry picked from commit 856cbcc12f2e4cca93af5dc7ed6bcea4dd942f10)
Lib/unittest/mock.py
Lib/unittest/test/testmock/testhelpers.py
Misc/NEWS.d/next/Library/2017-07-20-02-29-49.bpo-29403.3RinCV.rst [new file with mode: 0644]