<div dir="ltr"><div>I totally support this idea, in fact if auto generated bindings could be used to replace pulp-smash (partially or fully) that would save a lot of effort on maintaining it.</div><div><br></div><div>There are some reasons I tracked on why refactoring pulp-smash would be a good idea: <a href="https://gist.github.com/rochacbruno/96e19af2eb2a5c8d83cb7396cd27df64">https://gist.github.com/rochacbruno/96e19af2eb2a5c8d83cb7396cd27df64</a></div><div><br></div><div>@Tanya one of the things we wanted to do in pulp-smash was removing the "unittest" class based style in favor of pytest functional style, why haven't you used pytest and its fixtures for 2to3-migration tests? have you considered it?<br></div><div><br></div><div>Bruno<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 2, 2020 at 3:47 PM Tatiana Tereshchenko <<a href="mailto:ttereshc@redhat.com">ttereshc@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">+1 to use bindings to test Pulp 3.<br><div><br></div><div>We've already started using them (thanks to @dkliban) for pulp-2to3-migration initial testing.</div><div><a href="https://github.com/pulp/pulp-2to3-migration/blob/master/pulp_2to3_migration/tests/functional/test_file_migration.py" target="_blank">https://github.com/pulp/pulp-2to3-migration/blob/master/pulp_2to3_migration/tests/functional/test_file_migration.py</a></div><div><br></div><div>Tanya</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 16, 2019 at 11:32 PM Bruno Rocha <<a href="mailto:brocha@redhat.com" target="_blank">brocha@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi,</div><div><br></div><div>+1 to this idea, we have discussed that a little bit in QE some time ago in this issue[1]</div><div><br></div><div>This can solve the constants files dependency the current pulp-smash testing has.<br></div><div><br></div><div>[1] <a href="https://github.com/PulpQE/pulp-smash/issues/1182" target="_blank">https://github.com/PulpQE/pulp-smash/issues/1182</a></div><div><br></div><div><div><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>
<p style="font-weight:bold;margin:0px;padding:0px;font-size:14px;text-transform:capitalize;font-family:RedHatText,sans-serif">
<span>Bruno</span> <span>Rocha</span><span style="color:rgb(170,170,170);margin:0px"></span>
</p>
<p style="font-weight:normal;margin:0px;font-size:12px;font-family:RedHatText,sans-serif">
<span>Senior Quality Engineer - Management QE - Pulp Project<br><b>irc</b>: <span>rochacbruno</span></span>
</p>
<p style="font-size:12px;margin:0px;font-family:RedHatText,sans-serif"><span><b>social</b>: <a href="http://about.me/rochacbruno" target="_blank">http://about.me/rochacbruno</a><br></span></p><p style="font-size:12px;margin:0px;font-family:RedHatText,sans-serif"><span>“Progress is the realization of utopia.”</span><br><span></span>
</p>
<div style="margin-top:12px">
<table border="0">
<tbody><tr>
<td width="100px"><a href="https://red.ht/sig" target="_blank"> <img src="https://static.redhat.com/libs/redhat/brand-assets/latest/corp/logo.png" width="90" height="auto"></a> </td>
</tr>
</tbody></table>
</div>
</div></div></div></div></div></div></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 16, 2019 at 6:03 PM Brian Bouterse <<a href="mailto:bmbouter@redhat.com" target="_blank">bmbouter@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>I believe we have an opportunity to improve Pulp's testing through the use of bindings tests. I wrote up an epic to gather ideas and discussion. <a href="https://pulp.plan.io/issues/5888" target="_blank">https://pulp.plan.io/issues/5888</a></div><div><br></div><div>Initially this would be for pulpcore and pulp_file, but the idea is we would enable this automatically via the plugin_tempalte. Comments are welcome! With more input, I hope we can prioritize this work.<br></div><div><br></div><div>Thanks,</div><div>Brian</div><div><br></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>