<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<div class="moz-cite-prefix">On 01/14/2015 06:13 PM, Gabe Alford
wrote:<br>
</div>
<blockquote
cite="mid:CAGLxfGxkxNfMFoEQMjX_47cC0jXoP5MzBgzaCnazkF7JAB+5kQ@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">On Wed, Jan 14, 2015 at 10:05 AM, Tomas
Babej <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:tbabej@redhat.com" target="_blank">tbabej@redhat.com</a>></span>
wrote:<br>
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF"> <br>
<div>On 01/14/2015 06:00 PM, Tomas Babej wrote:<br>
</div>
<blockquote type="cite">
<div>
<div class="h5"> <br>
<div>On 01/14/2015 05:37 PM, Tomas Babej wrote:<br>
</div>
<blockquote type="cite"> <br>
<div>On 01/14/2015 02:55 PM, Gabe Alford wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">
<div>
<div>
<div>Hello,<br>
<br>
</div>
In looking into <a
moz-do-not-send="true"
href="https://fedorahosted.org/freeipa/ticket/4029"
target="_blank">https://fedorahosted.org/freeipa/ticket/4029</a>
I am wondering if there should be
separate ipa-advise test, Yes/No? Could
be handy in the future to test more
ipa-advise output? Or should this test
be added to the test_legacy_clients.py?<br>
<br>
</div>
Thanks,<br>
<br>
</div>
Gabe </div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Tue, Dec 2, 2014
at 9:21 PM, Gabe Alford <span dir="ltr"><<a
moz-do-not-send="true"
href="mailto:redhatrises@gmail.com"
target="_blank">redhatrises@gmail.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote"
style="margin:0 0 0 .8ex;border-left:1px
#ccc solid;padding-left:1ex">
<div dir="ltr">
<div>
<div>Hello,<br>
<br>
I was going to try my hand at
attempting a patch for ipa-tests.
However in wanting to test my
patch, I am not sure how to run
ipa-tests to check if it works or
not. Documentation is not really
clear on what needs to be done to
start a test and run a test. This
is for <a moz-do-not-send="true"
href="https://fedorahosted.org/freeipa/ticket/4029" target="_blank">https://fedorahosted.org/freeipa/ticket/4029</a>
<br>
<br>
</div>
I have attached the patch that I
have yet to really test with
ipa-test. Any help on how to test
the patch running ipa-tests would be
great. Of course, if one of the
reviewers looks at the patch and
looks good, then I would be happy
with that as well.<br>
<br>
</div>
Thanks,<br>
<br>
Gabe<br>
</div>
</blockquote>
</div>
<br>
</div>
<br>
<fieldset></fieldset>
<br>
<pre>_______________________________________________
Freeipa-devel mailing list
<a moz-do-not-send="true" href="mailto:Freeipa-devel@redhat.com" target="_blank">Freeipa-devel@redhat.com</a>
<a moz-do-not-send="true" href="https://www.redhat.com/mailman/listinfo/freeipa-devel" target="_blank">https://www.redhat.com/mailman/listinfo/freeipa-devel</a></pre>
</blockquote>
<br>
Hello,<br>
<br>
TL;DR: feel free to create a separate ipa-advise
test file. Test requested in this ticket really
does not belong to the legacy clients feature
test.<br>
<br>
As for the any new tests that might come: I
think tests for ipa-advise that are specific to
that particular feature should be tested with
that feature, more so, if they contain parts
that are supposed to work copy-pasted. If a
tests, however, tests a general behaviour of
ipa-advise, it should live in the ipa-advise
namespace, hence separate test file.<br>
<br>
HTH,<br>
<br>
<pre cols="72">--
Tomas Babej
Associate Software Engineer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | <a moz-do-not-send="true" href="http://freeipa.org" target="_blank">freeipa.org</a> </pre>
</blockquote>
<br>
</div>
</div>
The attached patch looks fine, although, please also
test for a non-zero return code number.<br>
<br>
</blockquote>
<br>
Upon hitting send I noticed you did not include
raiseonerr=False into the run_command call. You need to
do that, otherwise a exception will be raised, since
ipa-advise exited with non-zero return code.<span
class=""><br>
</span></div>
</blockquote>
<div>Thanks Tomas.<br>
<br>
</div>
<div>Which do you prefer: a test_advise.py or an update to
the existing patch? <br>
</div>
</div>
</div>
</div>
</blockquote>
<br>
A new test file, as I pointed out in the second email :) sorry for
splitting.<br>
<br>
However, it would be the best if you could spin up a positive test
as well (maybe listing out available advices), not just this
negative one, to justify the overhead reinstalling IPA for testing
this feature.<br>
<br>
<br>
<blockquote
cite="mid:CAGLxfGxkxNfMFoEQMjX_47cC0jXoP5MzBgzaCnazkF7JAB+5kQ@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF"><span class="">
<blockquote type="cite"> <br>
<pre cols="72">--
Tomas Babej
Associate Software Engineer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | <a moz-do-not-send="true" href="http://freeipa.org" target="_blank">freeipa.org</a> </pre>
</blockquote>
<br>
<pre cols="72">--
Tomas Babej
Associate Software Engineer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | <a moz-do-not-send="true" href="http://freeipa.org" target="_blank">freeipa.org</a> </pre>
</span></div>
</blockquote>
</div>
<br>
</div>
</div>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Tomas Babej
Associate Software Engineer | Red Hat | Identity Management
RHCE | Brno Site | IRC: tbabej | freeipa.org </pre>
</body>
</html>