<div dir="ltr"><div>Hello guys,</div><div>I was reviewing the patch <a href="https://review.opendev.org/c/openstack/cloudkitty/+/793790">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">https://review.opendev.org/c/openstack/cloudkitty/+/793973</a>). <br></div><div><br></div><div>Could you guys review it?</div><div><br></div><div><span id="gmail-output">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.</span></div><br>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Rafael Weingärtner</div></div></div>