On 18.08.2016 10:56, Petr Spacek wrote:
On 18.8.2016 10:08, Jan Cholasta wrote:
Could you add one sentence or a link to a ticket which forced this change?

When reading the patch, I have no way to say why the change is necessary - so
it is impossible to verify correctness. (Sure, the test will pass, but I have
no way to distinguish incorrect test passing on incorrect implementation vs.
correct test passing on correct implementation.)


and add there this link also https://fedorahosted.org/freeipa/ticket/6188

Manage your subscription for the Freeipa-devel mailing list:
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to