diff options
| author | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-05-11 04:56:15 +0000 |
|---|---|---|
| committer | Johannes Erdfelt <johannes.erdfelt@rackspace.com> | 2012-05-11 04:56:15 +0000 |
| commit | 9ca55d8e50761ee2a11d855b2f55ccd80318451d (patch) | |
| tree | c34796ef43a668d8443143f9cc9095ce9b76067c /nova/virt | |
| parent | bf6673a5952ed59ed55d504938d195083e23a2ce (diff) | |
Actually use xenapi fake setter
It was implemented (with a small bug), but never used. Also, lean on
the xenapi fake over using stubs in the migration tests to ensure we
test more code.
Change-Id: I96a7222a017302332bd6a3600fa645a74c502944
Diffstat (limited to 'nova/virt')
| -rw-r--r-- | nova/virt/xenapi/fake.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/nova/virt/xenapi/fake.py b/nova/virt/xenapi/fake.py index 1d07f209c..26f1c0847 100644 --- a/nova/virt/xenapi/fake.py +++ b/nova/virt/xenapi/fake.py @@ -610,6 +610,9 @@ class SessionBase(object): if self._is_gettersetter(name, True): LOG.debug(_('Calling getter %s'), name) return lambda *params: self._getter(name, params) + elif self._is_gettersetter(name, False): + LOG.debug(_('Calling setter %s'), name) + return lambda *params: self._setter(name, params) elif self._is_create(name): return lambda *params: self._create(name, params) elif self._is_destroy(name): @@ -683,6 +686,7 @@ class SessionBase(object): if (ref in _db_content[cls] and field in _db_content[cls][ref]): _db_content[cls][ref][field] = val + return LOG.debug(_('Raising NotImplemented')) raise NotImplementedError( |