<div dir="ltr">Glad to help! <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 8, 2021 at 5:19 AM Pierre Riteau <<a href="mailto:pierre@stackhpc.com">pierre@stackhpc.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">Thanks a lot Rafael for fixing this gate blocker!<br>
<br>
On Tue, 1 Jun 2021 at 15:55, Rafael Weingärtner<br>
<<a href="mailto:rafaelweingartner@gmail.com" target="_blank">rafaelweingartner@gmail.com</a>> wrote:<br>
><br>
> Hello guys,<br>
> I was reviewing the patch <a href="https://review.opendev.org/c/openstack/cloudkitty/+/793790" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/cloudkitty/+/793790</a>, and decided to propose an alternative patch (<a href="https://review.opendev.org/c/openstack/cloudkitty/+/793973" rel="noreferrer" target="_blank">https://review.opendev.org/c/openstack/cloudkitty/+/793973</a>).<br>
><br>
> Could you guys review it?<br>
><br>
> The idea I am proposing is that, instead of mocking the root object ("flask.request"), we address the issue by mocking only the needed methods and attributes. This facilitates the understanding of the unit test, and also helps people to pin-point problems right away as the mocked attributes/methods are clearly seen in the unit test.<br>
><br>
> --<br>
> Rafael Weingärtner<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Rafael Weingärtner</div></div>