<div dir="ltr">I disagree to moving this logic to "tempest/services/*". The idea of these modules - assemble requests and return responses. Testing and verification should be wrapped over it. Either base class or tests, it depends on situation...<div>
<br></div><div>-- <br><div dir="ltr">Kind Regards<br>Valeriy Ponomaryov</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 13, 2014 at 6:55 AM, Christopher Yeoh <span dir="ltr"><<a href="mailto:cbkyeoh@gmail.com" target="_blank">cbkyeoh@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi,<br>
<br>
The new tempest body response validation is being added to individual<br>
testcases. See this as an example:<br>
<br>
<a href="https://review.openstack.org/#/c/78149" target="_blank">https://review.openstack.org/#/c/78149</a><br>
<br>
After having a look at <a href="https://review.openstack.org/#/c/80174/" target="_blank">https://review.openstack.org/#/c/80174/</a><br>
I'm now thinking that perhaps we should be doing the response validation<br>
in the tempest/services/compute classes. And only check the<br>
response body if the status code is a success code (and then check that<br>
it is an appropriate success code).<br>
<br>
I think this will lead to fewer changes needed in the end as the<br>
response body checking will not needed to be added to individual tests.<br>
<br>
There may be some complications with handling extensions, but I think<br>
they are all implement backwards compatible behaviour so should be ok.<br>
<br>
Anyone have any thoughts about this alternative approach?<br>
<br>
Regards,<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></blockquote></div>
</div></div></div>