<p dir="ltr">+1 </p>
<p dir="ltr">keeping the body as a ~dict will help with all existing asserts comparing dicts in tests.</p>
<p dir="ltr">Andrea </p>
<div class="gmail_quote">On 30 Aug 2014 06:45, "Christopher Yeoh" <<a href="mailto:cbkyeoh@gmail.com">cbkyeoh@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Fri, 29 Aug 2014 11:13:39 -0400<br>
David Kranz <<a href="mailto:dkranz@redhat.com">dkranz@redhat.com</a>> wrote:<br>
<br>
> On 08/29/2014 10:56 AM, Sean Dague wrote:<br>
> > On 08/29/2014 10:19 AM, David Kranz wrote:<br>
> >> While reviewing patches for moving response checking to the<br>
> >> clients, I noticed that there are places where client methods do<br>
> >> not return any value. This is usually, but not always, a delete<br>
> >> method. IMO, every rest client method should return at least the<br>
> >> response. Some services return just the response for delete<br>
> >> methods and others return (resp, body). Does any one object to<br>
> >> cleaning this up by just making all client methods return resp,<br>
> >> body? This is mostly a change to the clients. There were only a<br>
> >> few places where a non-delete  method was returning just a body<br>
> >> that was used in test code.<br>
> > Yair and I were discussing this yesterday. As the response<br>
> > correctness checking is happening deeper in the code (and you are<br>
> > seeing more and more people assigning the response object to _ ) my<br>
> > feeling is Tempest clients should probably return a body obj that's<br>
> > basically.<br>
> ><br>
> > class ResponseBody(dict):<br>
> >      def __init__(self, body={}, resp=None):<br>
> >          self.update(body)<br>
> >     self.resp = resp<br>
> ><br>
> > Then all the clients would have single return values, the body<br>
> > would be the default thing you were accessing (which is usually<br>
> > what you want), and the response object is accessible if needed to<br>
> > examine headers.<br>
> ><br>
> >     -Sean<br>
> ><br>
> Heh. I agree with that and it is along a similar line to what I<br>
> proposed here <a href="https://review.openstack.org/#/c/106916/" target="_blank">https://review.openstack.org/#/c/106916/</a> but using a<br>
> dict rather than an attribute dict. I did not propose this since it<br>
> is such a big change. All the test code would have to be changed to<br>
> remove the resp or _ that is now receiving the response. But I think<br>
> we should do this before the client code is moved to tempest-lib.<br>
<br>
+1. this would be a nice cleanup.<br>
<br>
Chris<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>