Martin Polednik has posted comments on this change.
Change subject: tests: vdsProxy remove need to duplicate api ......................................................................
Patch Set 1:
(1 comment)
.................................................... File tests/functional/utils.py Line 139: return result['status']['code'], result['status']['message'] Line 140: """ Line 141: if hasattr(self.vdscli, func): Line 142: def wrapper(*args, **kw): Line 143: result = getattr(self.vdscli, func)(*args, **kw) I kind of like that because it would eliminate even more code, but on the other hand it would negatively affect code - api calls from tests would have nonstandard arguments compared to calls from raw code, adding to confusion.
It would be great to hear more opinions on this idea Line 144: return result['status']['code'], result['status']['message'] Line 145: return wrapper Line 146: raise AttributeError(func) Line 147: