[Freeipa-devel] [PATCH] 0078-82: webui tests: tests for new certificate widget
Lenka Doudova
ldoudova at redhat.com
Mon Aug 1 14:13:19 UTC 2016
On 07/29/2016 03:00 PM, Pavel Vomacka wrote:
>
>
>
> On 07/28/2016 08:16 AM, Lenka Doudova wrote:
>>
>>
>>
>> On 07/20/2016 04:51 PM, Pavel Vomacka wrote:
>>> Please review attached patches, which add tests for new certificate
>>> widget in WebUI.
>>>
>>> https://fedorahosted.org/freeipa/ticket/6064
>>>
>>>
>>>
>> Hi,
>> thanks for patches.
>> Functionally ok, but you have lots of PEP8 errors in patches 78, 80,
>> 81 and 82 -> NACK.
>> Also in patch 82, method test_arbitrary_certificate, comment says
>> user needs to have "arbitrary_cert" configured, but the property in
>> config file is correctly "arbitrary_cert_path", so it's a bit misleading.
>>
>> Patch 79 is OK, ACK.
>>
>> Lenka
>>
>>
> Thank you for review. Attaching patches which have fixed all pep8
> erros. Bad property of config file was also mentioned in patch 81.
> These are also fixed.
>
> --
> Pavel^3 Vomacka
Hi,
all is fine now, ACK for all patches.
Lenka
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20160801/8dfcabf7/attachment.htm>
More information about the Freeipa-devel
mailing list