Re: CVS commit: src/tests/ipf

["Cherry G. Mathew" <cherry.g.mathew%gmail.com@localhost>]

On 27 March 2012 22:38, Alan Barrett <apb%cequrux.com@localhost> wrote:
> On Tue, 27 Mar 2012, Jukka Ruohonen wrote:
>>
>> Modified Files:
>>        src/tests/ipf: t_filter_exec.sh t_filter_parse.sh t_nat_exec.sh
>>            t_nat_ipf_exec.sh
>>
>> Log Message:
>> Mark the failing tests as broken. XXX: If no one is willing to maintain
>> the ipf tests, these should be removed.
>
>
> I object to this.  If ipf fails its tests, then the fact should be made
> clear in the test reports, not hidden by disabling the tests.
> I don't know whether the bugs are in ipf or in the tests, but either way,
> removing or disabling the tests seems to me to be counter-productive.
>
> --apb (Alan Barrett)

+1

-- 
~Cherry