On Wed, 15 Jul 2015 07:40:57 +1200 Robert Collins <robertc at robertcollins.net> wrote: > On 15 July 2015 at 07:39, Paul Moore <p.f.moore at gmail.com> wrote: > > On 14 July 2015 at 20:27, Robert Collins <robertc at robertcollins.net> wrote: > > >>> In effect, this patch is "reserving" all attributes starting with > >>> "assert" or "assret" as actual methods of the mock object, and not > >>> mocked attributes. > >> > >> Yes, and thats ugly. OTOH it caught hundreds of useless tests in > >> OpenStack when this got ported into mock 1.1.0. > > > > ... which I guess counts as strong evidence that this *is* a common > > typo, at least in certain contexts. > > For clarity: None of the caught failures were assret as far as I know. > They were things like assert_called_onec_with, or assert_called. By the way, I've also been bitten by this several times, so I appreciate the desire to at least warn users (or raise an exception, or whatever). Regards Antoine.
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4