From 9ca55d8e50761ee2a11d855b2f55ccd80318451d Mon Sep 17 00:00:00 2001 From: Johannes Erdfelt Date: Fri, 11 May 2012 04:56:15 +0000 Subject: 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 --- nova/virt/xenapi/fake.py | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'nova/virt') 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( -- cgit