Antoni Segura Puimedon has uploaded a new change for review. Change subject: Fix configNetworkTests.ConfigWriter monkeypatch. ......................................................................
Fix configNetworkTests.ConfigWriter monkeypatch. ConfigWriter creates NET_CONF_PREF from netinfo's NET_CONF_PREF at import time, it being a class variable, so the current scope Monkey patch of just netinfo was not sufficient. Change-Id: I6925c59650aaa81fa925a4be3bcd72dcbebbb415 Signed-off-by: Antoni S. Puimedon <[email protected]> --- M tests/configNetworkTests.py 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/61/10161/1 diff --git a/tests/configNetworkTests.py b/tests/configNetworkTests.py index 07291db..eb89df9 100644 --- a/tests/configNetworkTests.py +++ b/tests/configNetworkTests.py @@ -355,6 +355,8 @@ (netinfo, 'NET_CONF_BACK_DIR', os.path.join(self._tempdir, 'netback')), (netinfo, 'NET_CONF_DIR', self._tempdir), + (configNetwork.ConfigWriter, 'NET_CONF_PREF', + os.path.join(self._tempdir, 'ifcfg-')) ]): #after vdsm package is installed, the 'vdsm' account will be #created if no 'vdsm' account, we should skip this test -- To view, visit http://gerrit.ovirt.org/10161 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I6925c59650aaa81fa925a4be3bcd72dcbebbb415 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Antoni Segura Puimedon <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
