On 07/09/2015 01:49 PM, Veronika Kabatova wrote:
> The attached patch solves the <https://fedorahosted.org/freeipa/ticket/3959> 
> ticket.
> 
> Veronika Kabatova
> 
> 
> 

Hello,

thanks for the patch.

Actually, the doctest does not pass:

$ ipa-run-tests /usr/lib/python2.7/site-packages/ipalib/errors.py
--doctest-modules
=============================================================================================================
test session starts
=============================================================================================================
platform linux2 -- Python 2.7.10 -- py-1.4.28 -- pytest-2.6.4
plugins: multihost, sourceorder
collected 85 items

../ipalib/errors.py
...................F..............F..................................................

==================================================================================================================
FAILURES
===================================================================================================================
_________________________________________________________________________________________________
[doctest] ipalib.errors.DNSNotARecordError
__________________________________________________________________________________________________
1137     """
1138     **4019** Raised when a hostname is not a DNS A/AAAA record
1139
1140     For example:
1141
1142     >>> raise DNSNotARecordError()
Differences (unified diff with -expected +actual):
    @@ -1,4 +1,6 @@
     Traceback (most recent call last):
    -  ...
    -DNSNotARecordError: Host does not have corresponding DNS A/AAAA record,
    -use --force to continue anyway
    +  File "/usr/lib64/python2.7/doctest.py", line 1315, in __run
    +    compileflags, 1) in test.globs
    +  File "<doctest ipalib.errors.DNSNotARecordError[0]>", line 1, in
<module>
    +    raise DNSNotARecordError()
    +DNSNotARecordError: Host does not have corresponding DNS A/AAAA
record, use --force to continue anyway

/usr/lib/python2.7/site-packages/ipalib/errors.py:1142: DocTestFailure

The reason for the mismatch here is that you wrapped the line - in this
case, we need to violate the PEP8, and allow the length of the line
exceed 80 characters.

HTH,

Tomas

-- 
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