diff options
| author | Ewan Mellor <ewan.mellor@citrix.com> | 2011-11-26 11:40:31 -0800 |
|---|---|---|
| committer | Ewan Mellor <ewan.mellor@citrix.com> | 2011-11-26 14:22:30 -0800 |
| commit | a0baa5c125c0a7ef5bbc8684184ebf2c130bb466 (patch) | |
| tree | c15c3058832c367e75161c4b0eac4811e13197f4 /nova/api | |
| parent | efc639ff4e0846535e1689a04537021d34c4c77d (diff) | |
| download | nova-a0baa5c125c0a7ef5bbc8684184ebf2c130bb466.tar.gz nova-a0baa5c125c0a7ef5bbc8684184ebf2c130bb466.tar.xz nova-a0baa5c125c0a7ef5bbc8684184ebf2c130bb466.zip | |
Refactor a few things inside the xenapi unit tests.
There were a couple of places where basically the same code was being
used to create a simulated VDI record -- these have been brought together
into _make_fake_vdi.
fake_fetch_image was stubbing out parse_xmlrpc_value, which looks like it
was a workaround for the fact that the task result wasn't being populated
properly. I've fixed the latter, and removed the former. This moved the
JSON handling into xenapi.fake.
There were a couple of implementations of host_call_plugin, which contained
a lot of duplicated code. In particular, they both made a simulated VDI
record, even when the plugin function being called didn't require that.
I have brought the two implementations together into fake.SessionBase, with
overrides in the subclasses for those things that are specific to a given
test. I have also made the baseclass strict about flagging unsimulated
plugin calls, and added explicit handling for agent.version, and a couple
of methods from glance and migration.
Change-Id: Idc3a872870ae15165747a04ecd1b48e889bd90fd
Diffstat (limited to 'nova/api')
0 files changed, 0 insertions, 0 deletions
