On 04/16/2014 04:21 PM, Misnyovszki Adam wrote:
On Wed, 16 Apr 2014 07:59:39 +0200
Martin Kosek <mko...@redhat.com> wrote:

On 04/15/2014 05:36 PM, Misnyovszki Adam wrote:
On Tue, 15 Apr 2014 12:51:47 +0200
Petr Viktorin <pvikt...@redhat.com> wrote:

On 04/15/2014 12:41 PM, Misnyovszki Adam wrote:
Hi,
this patch fixes FreeIPA Jenkins CI test
freeipa-integration-forced_client_reenrollment-f19, by turning
sshfp records into a set, and sorting them before assertion.

https://fedorahosted.org/freeipa/ticket/4298

Greets
Adam

The list.sort() method sorts in-place and returns None, so now the
test would not really test anything. Use the sorted() function.

You might want to log the value before returning it.


My mistake, see the attached, corrected patch.
Thanks
Adam

Adam, Petr - why can't we use a set as I already proposed in the
ticket description?

  >>> set(['i', 'p', 'a']) == set(['a', 'p', 'i'])
True

Martin

Hi,
see the attached patch.
Thanks
Adam


ACK, pushed to:
master: f85fe1e8513327396ef2f9732810abdffc88abba
ipa-3-3: 1d16370dc536434e63097115150afab26febd5b4

--
PetrĀ³

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to