<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 21.06.2016 15:39, Lenka Doudova
wrote:<br>
</div>
<blockquote
cite="mid:6ff234ed-6cf3-7742-b388-d192f27e40a7@redhat.com"
type="cite">Hi,
<br>
<br>
attaching patch for failing location tests
(ipatests/test_xmlrpc/test_location_plugin.py).
<br>
<br>
Lenka
<br>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<br>
Hello,<br>
<br>
1)<br>
+ expected_updates={u'ipalocation_location':
[location.idnsname_obj],<br>
+ u'enabled_role_servrole': (<br>
+ u'CA server', u'DNS server',
u'NTP server')},<br>
<br>
This depends on services installed on server, so server without DNS
will cause test failures. We probably should skip test id DNS isn't
installed.<br>
Without DNS installed you get much more different warnings<br>
<br>
<br>
2)<br>
+ def update(self, updates, expected_updates=None,
messages=None):<br>
....<br>
+ self.messages = messages<br>
<br>
Why is this needed? I'm puzzled by this<br>
<br>
It is defined outside __init__ what is wrong and it is never used.<br>
<br>
</body>
</html>