<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 11, 2016 at 5:40 AM, Ian Cordasco <span dir="ltr"><<a href="mailto:sigmavirus24@gmail.com" target="_blank">sigmavirus24@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5"><span style="color:rgb(34,34,34)">So, as a core developer</span><br></div></div>
of Requests, I would endorse requests-mock for this category of<br>
dependency.<br></blockquote><div><br></div><div>Excellent; exactly the kind of feedback I was hoping to solicit.  If you're looking to add this catagory of dependency - requests-mock is best-of-breed.</div><div><br></div><div>Although, honestly, I was also hoping to get some input from the perspective of maintaining a client library specifically to address the question of if this category of dependency is something that makes sense?</div><div><br></div><div>e.g.</div><div><br></div><div>Ian, you've worked on glance - so I went looking how glanceclient uses requests_mock and found another change from Jamie:</div><div><br></div><div><a href="https://review.openstack.org/#/c/141992/">https://review.openstack.org/#/c/141992/</a><br></div><div><br></div><div>In this change, it *seems* like instead of glanceclient maintaining some stubs that are used to set explicit expectations about the external behavior of the other system - glaceclient has *outsourced* it's expectations to implicitly couple with whatever keystoneclient provides.  On one hand this seems like it might reduce some maintenance burden.  OTOH, the burden of maintaining some of the expectations about the behavior of the external system you depend on seems *related* to maintaining the glance client?  I'm not sure if this is a great tradeoff?  Maybe?  I'm not sure if this gets into what you were talking about wrt integration tests?  The change I'm currently evaluating doesn't import an external fixture I don't think... I'm not really sure what a fixture is this context?</div><div><br></div><div><a href="http://requests-mock.readthedocs.io/en/latest/api/requests_mock.html?highlight=fixture#requests-mock-fixture-module">http://requests-mock.readthedocs.io/en/latest/api/requests_mock.html?highlight=fixture#requests-mock-fixture-module</a><br></div><div><br></div><div>-Clay</div></div></div></div>