On 18.08.2016 10:56, Petr Spacek wrote:
On 18.8.2016 10:08, Jan Cholasta wrote:
SSIA
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.)

+1

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

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

Reply via email to