[Checkins] Re: SVN: zope.sendmail/trunk/src/zope/sendmail/ merging

[Andreas Jung]

--On 20. August 2007 15:57:51 +0200 Philipp von Weitershausen 
<philipp at weitershausen.de> wrote:

> On 20 Aug 2007, at 14:26 , Andreas Jung wrote:
>>> Using the mock test at least we can improve test coverage,  is it?
>>
>> Real world tests against the variety of different mailservers is
>> _much_ more
>> important than test coverage and testing against a non-real-world
>> environment. Such artificial tests test the code against "how the
>> world should" be but it don't tell you if your code actually works
>> in the real world.
>
> I agree that real world testing is important in this particular case, but
> Baiju does have a point: the code paths that throw the RuntimeErrors
> could and *should* be tested with mock objects.

ACK

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 186 bytes
Desc: not available
Url : http://mail.zope.org/pipermail/checkins/attachments/20070820/d1e2df2c/attachment.bin