Open Stack

Hi Jay,

Sure, that is understood. In fact such refactoring could be a big change so
I'd split it to two or more patches.
Hope this will not overlap with ongoing neutron API tests development.

Thanks,
Eugene.



On Sat, Dec 14, 2013 at 7:01 PM, Jay Pipes <jaypipes at gmail.com> wrote:

> On Sat, 2013-12-14 at 15:26 +0400, Eugene Nikanorov wrote:
> > Hi tempest folks,
> >
> > While adding new API tests for Neutron I found out that network client
> > used in tempest could be improved in at least two ways:
> > 1) code duplication between xml and json versions could be removed
> > 2) client API could be made more flexible and similar to real neutron
> > client API.
> > E.g. instead of accepting fixed subset of properties of the resource,
> > methods could accept full dict with resource description.
> >
> > Implementing this two points will also require some changes in the
> > tests: for example, instead of calling
> >  network_client.create_network(name)
> > test should call:
> >  network_data = {'name': name, 'property2': property, ...}
> >  network_client.create_network(network_data)
> >
> > Those both items require quite a bit of work and I'm working on such
> > improvement right now.
> > My personal motivation for this is that I hate to add new client
> > methods by copy-pasting existing and
> > hardcoding stuff in them (and btw, I need to do it for both xml and
> > json versions)
> >
> > So before committing more than two hours on this, I wanted to ask if
> > such kind of change is desirable?
> > If so, I'm going to present a sequence of patches that will implement
> > this plan.
>
> Hi Eugene!
>
> It's best to keep refactoring patches separate from any patches that add
> functionality or new tests to Tempest. I think the above improvements
> would be a welcome change, but it's easier for reviewers if that
> refactoring patch is all in one patch and not mixed in with new tests or
> added functionality.
>
> All the best,
> -jay
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20131214/552de63f/attachment.html>