<div dir="ltr">Hi tempest folks,<div><br></div><div>While adding new API tests for Neutron I found out that network client used in tempest could be improved in at least two ways:</div><div>1) code duplication between xml and json versions could be removed</div>
<div>2) client API could be made more flexible and similar to real neutron client API. </div><div>E.g. instead of accepting fixed subset of properties of the resource, methods could accept full dict with resource description.</div>
<div><br></div><div>Implementing this two points will also require some changes in the tests: for example, instead of calling</div><div> network_client.create_network(name)</div><div>test should call:</div><div> network_data = {'name': name, 'property2': property, ...}</div>
<div> network_client.create_network(network_data)</div><div><br></div><div>Those both items require quite a bit of work and I'm working on such improvement right now.</div><div>My personal motivation for this is that I hate to add new client methods by copy-pasting existing and</div>
<div>hardcoding stuff in them (and btw, I need to do it for both xml and json versions)</div><div><br></div><div>So before committing more than two hours on this, I wanted to ask if such kind of change is desirable?</div>
<div>If so, I'm going to present a sequence of patches that will implement this plan.</div><div><br></div><div>Thanks,</div><div>Eugene.</div></div>