[TIP] failing tests after upgrading Mock 0.7 to 0.8

Martin Pool mbp at sourcefrog.net
Tue Nov 20 14:03:52 PST 2012


On 21 November 2012 08:47, Michael <mock at webhippo.net> wrote:

> I would like to respectfully answer this:
>

It doesn't look like you succeeded in answering respectfully.


>

This is a super un-useful answer.
>
> 1. Yes it worked in 0.7.2
> 2. No the docs did not mention the __init__/super was call was required.
> For all we knew it was not required.
> 3. The docs also do not mention like all kinds of things, like that one
> should step on one foot while using mock ... that does not mean we should
> do them.
>

That's true; nor does it mention you must have Python installed, or your
computer must be switched on.  I don



>
>
>  Did Mock's documentation ever tell you not to call the super init?
>> Generally speaking this is something subclasses need to do in Python, and
>> not doing it is a latent bug.
>>
>> -m
>>
>
>  On 20 November 2012 11:53, Eric Rasmussen <ericrasmussen at gmail.com>
>> wrote:
>>
>>  Hello,
>>>
>>> I am upgrading from Mock 0.7 and it looks like there was a backwards
>>> incompatible change in 0.8: it's apparently no longer possible to create
>>> a
>>> class inheriting from Mock unless you call the super method. Can someone
>>> explain why the below works in 0.7 but not in 0.8 and above?
>>>
>>> class MyMock(Mock):
>>>     def __init__(self, name):
>>>         self.name = name
>>>
>>> Now I have to write:
>>>
>>> class MyMock(Mock):
>>>     def __init__(self, name):
>>>         Mock.__init__(self)
>>>         self.name = name
>>>
>>> I did find an issue here:
>>> http://code.google.com/p/mock/**issues/detail?id=161<http://code.google.com/p/mock/issues/detail?id=161>
>>>
>>> But it does not mention the reason for the change. I was unable to find
>>> any mention of a backwards incompatible change of this nature in the
>>> changelog, which is concerning. I just want to make sure I fully
>>> understand
>>> the scope of the change. If you can point me to any documentation or pull
>>> requests that show when and why this change was made that would be an
>>> enormous help.
>>>
>>> It's really important for a project I'm working on that we document this
>>> new requirement for creating a subclass of Mock, so I appreciate any
>>> insight you can offer.
>>>
>>> Thank you,
>>> Eric
>>>
>>> ______________________________**_________________
>>> testing-in-python mailing list
>>> testing-in-python at lists.idyll.org
>>> http://lists.idyll.org/**listinfo/testing-in-python<http://lists.idyll.org/listinfo/testing-in-python>
>>>
>>>
>>>
>>
>> --
>> Martin
>>
> -------------- next part --------------
> An HTML attachment was scrubbed...
> URL: <http://lists.idyll.org/**pipermail/testing-in-python/**
> attachments/20121121/0dc24a9d/**attachment.html<http://lists.idyll.org/pipermail/testing-in-python/attachments/20121121/0dc24a9d/attachment.html>
> >
>
>
> ______________________________**_________________
> testing-in-python mailing list
> testing-in-python at lists.idyll.**org <testing-in-python at lists.idyll.org>
> http://lists.idyll.org/**listinfo/testing-in-python<http://lists.idyll.org/listinfo/testing-in-python>
>



-- 
Martin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.idyll.org/pipermail/testing-in-python/attachments/20121121/f3a26f90/attachment.htm>


More information about the testing-in-python mailing list