<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 06/16/2016 03:23 PM, Lenka Doudova
      wrote:<br>
    </div>
    <blockquote
      cite="mid:766fd34a-b83b-1105-57d0-bd51420720a7@redhat.com"
      type="cite">Hi,
      <br>
      <br>
      attached are tests for authentication indicators. Please note:
      <br>
      <br>
      1. newly created service tracker is not exactly complete, list of
      unimplemented methods is in doc. These methods can be filled in
      when existing declarative tests are refactored.
      <br>
      <br>
      2. patch 0015 depends on 0014, so it should not be pushed without
      it.
      <br>
      <br>
      <br>
      Lenka
      <br>
      <br>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
    </blockquote>
    <br>
    <p>patch 0014:</p>
    <p>In the update method, what happens when the updated attributes
      contain addattr? It is not clear to me. Is it necessary?</p>
    <p>patch 0015:</p>
    <p>host1 and service2 do not tell anything about the purpose of the
      fixture. Please assign more descriptive names to them.<br>
      Why do the fixtures have 'function' scope? Does the service entry
      exist during the second and third test case?</p>
    <p>patch 0016:</p>
    <p>Per offline discussion, admin user has no special privileges
      here, LGTM.<br>
    </p>
    <pre class="moz-signature" cols="72">-- 
Milan Kubik</pre>
  </body>
</html>