[SSSD] [PATCH][ding_libs] INI: run also negative_test

Lukas Slebodnik lslebodn at redhat.com
Mon Jan 13 14:32:51 UTC 2014


On (13/01/14 08:55), Dmitri Pal wrote:
>On 01/13/2014 05:46 AM, Lukas Slebodnik wrote:
>> ehlo,
>>
>> Test negative_test was defined, but it has never been executed as part of
>> ini_config test suite.
>>
>> Simple patch is attached.
>>
>> LS
>>
>This unit test tests old interfaces that is being phased out.
>I do not think we need to improve UNIT test for the API that is
>deprecated. IMO sends a wrong message so I would rather drop this.
>
Yes,
it is testing old interface, but old interface is still part of library.
Unit test passes without any problem. Do we really need to remove this unit
test? Personally, I don't have problem with it.

LS



More information about the sssd-devel mailing list