On 01/29/2014 04:37 PM, Crístian Viana wrote:
Am 29-01-2014 16:31, schrieb Aline Manera:
>> + def softwareupdate_lookup(self, *name):
>> + return self.host_swupdate.getUpdates()
>> +
>> + def softwareupdate_update(self, args=None):
>> + kimchi_log.info('Host is going to be updated.')
>> + self.host_swupdate.doUpdate()
>
> The mockmodel is a fake model.
> Which means we should not touch the system.
> You need to return default values there.
But this is a patch to src/kimchi/model.py, isn't it?
Sorry, I commented in the
model.py patch
But the same code is in mockmodel.py so the comment is still valid.