patch for subunithelper

Matthieu Patou mat at samba.org
Sat Nov 20 03:15:50 MST 2010


On 15/11/2010 02:29, Jelmer Vernooij wrote:
> Hi Matthieu,
>
> On Sun, 2010-11-14 at 14:59 +0300, Matthieu Patou wrote:
>> can you review this patch it's design to make filter-subunit.py to exit
>> with a non zero error code if we have only 1 testsuite who has failed
>> but without any failed test (happens if the initialization of the
>> testsuite has failed, ie. because a module is missing).
>>
>> An example of subunit output that shows this combination is also
>> attached to the email.
> I think in this case we should just change the error code if there was a
> failed testsuite without failed tests, rather than making up nonexistant
> tests. Creating tests will cause havoc for test runners, since they will
> not be able to request this test be run manually.
>
Is this version of the patch better for you ?
I tried not to pollute the test output with fake/non existing errors.

Matthieu.

-- 
Matthieu Patou
Samba Team        http://samba.org
Private repo      http://git.samba.org/?p=mat/samba.git;a=summary


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-selftest-fix-tap2subunit-detection-on-system-with-ta.patch
Type: text/x-patch
Size: 1299 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20101120/eb6eb7aa/attachment.bin>


More information about the samba-technical mailing list