Fix test_net.py testcase fail
This commit is contained in:
parent
14f4e65da4
commit
a5d59c38e9
@ -0,0 +1,40 @@
|
|||||||
|
From d8f39d79fcdb548252823949a4b24f49a00e8c27 Mon Sep 17 00:00:00 2001
|
||||||
|
From: yangzz-97 <70520104+yangzz-97@users.noreply.github.com>
|
||||||
|
Date: Thu, 31 Mar 2022 22:00:33 +0800
|
||||||
|
Subject: [PATCH] testing: add additional mocks to test_net tests (#1356)
|
||||||
|
|
||||||
|
Tests in tests/unittests/test_net.py::TestGetInterfaces were missing
|
||||||
|
mocks for bond and failover calls.
|
||||||
|
|
||||||
|
Signed-off-by: yangzz-97 <yangzz-97@1162011203@qq.com>
|
||||||
|
---
|
||||||
|
tests/unittests/test_net.py | 8 ++++++++
|
||||||
|
1 file changed, 8 insertions(+)
|
||||||
|
|
||||||
|
diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py
|
||||||
|
index 6294f4b..ce19498 100644
|
||||||
|
--- a/tests/unittests/test_net.py
|
||||||
|
+++ b/tests/unittests/test_net.py
|
||||||
|
@@ -5404,12 +5404,18 @@ class TestGetInterfaces(CiTestCase):
|
||||||
|
def _se_interface_has_own_mac(self, name):
|
||||||
|
return name in self.data['own_macs']
|
||||||
|
|
||||||
|
+ def _se_is_bond(self, name):
|
||||||
|
+ return name in self.data["bonds"]
|
||||||
|
+
|
||||||
|
+ def _se_is_netfailover(self, name):
|
||||||
|
+ return False
|
||||||
|
+
|
||||||
|
def _mock_setup(self):
|
||||||
|
self.data = copy.deepcopy(self._data)
|
||||||
|
self.data['devices'] = set(list(self.data['macs'].keys()))
|
||||||
|
mocks = ('get_devicelist', 'get_interface_mac', 'is_bridge',
|
||||||
|
'interface_has_own_mac', 'is_vlan', 'device_driver',
|
||||||
|
- 'device_devid')
|
||||||
|
+ 'device_devid', 'is_bond', 'is_netfailover')
|
||||||
|
self.mocks = {}
|
||||||
|
for n in mocks:
|
||||||
|
m = mock.patch('cloudinit.net.' + n,
|
||||||
|
--
|
||||||
|
1.8.3.1
|
||||||
|
|
||||||
@ -1,6 +1,6 @@
|
|||||||
Name: cloud-init
|
Name: cloud-init
|
||||||
Version: 21.4
|
Version: 21.4
|
||||||
Release: 2
|
Release: 3
|
||||||
Summary: the defacto multi-distribution package that handles early initialization of a cloud instance.
|
Summary: the defacto multi-distribution package that handles early initialization of a cloud instance.
|
||||||
License: ASL 2.0 or GPLv3
|
License: ASL 2.0 or GPLv3
|
||||||
URL: http://launchpad.net/cloud-init
|
URL: http://launchpad.net/cloud-init
|
||||||
@ -14,6 +14,7 @@ Patch2: bugfix-cloud-init-add-openEuler-os.patch
|
|||||||
Patch3: bugfix-sort-requirements.patch
|
Patch3: bugfix-sort-requirements.patch
|
||||||
Patch4: add-variable-to-forbid-tmp-dir.patch
|
Patch4: add-variable-to-forbid-tmp-dir.patch
|
||||||
Patch5: backport-add-Requires-cloud-init-hotplugd.socket-in-cloud-init-hotplugd.service-file.patch
|
Patch5: backport-add-Requires-cloud-init-hotplugd.socket-in-cloud-init-hotplugd.service-file.patch
|
||||||
|
Patch6: backport-testing-add-additional-mocks-to-test_net-tests-1356.patch
|
||||||
|
|
||||||
Patch9000: Fix-the-error-level-logs-displayed-for-the-cloud-init-local-service.patch
|
Patch9000: Fix-the-error-level-logs-displayed-for-the-cloud-init-local-service.patch
|
||||||
|
|
||||||
@ -125,6 +126,12 @@ fi
|
|||||||
%exclude /usr/share/doc/*
|
%exclude /usr/share/doc/*
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Fri Apr 1 2022 yangzhuangzhuang <yangzhuangzhuang1@h-partners.com> - 21.4-3
|
||||||
|
- Type:bugfix
|
||||||
|
- ID:NA
|
||||||
|
- SUG:NA
|
||||||
|
- DESC:Fix test_net.py testcase fail
|
||||||
|
|
||||||
* Fri Mar 18 2022 yangzhuangzhuang <yangzhuangzhuang1@h-partners.com> - 21.4-2
|
* Fri Mar 18 2022 yangzhuangzhuang <yangzhuangzhuang1@h-partners.com> - 21.4-2
|
||||||
- Type:update
|
- Type:update
|
||||||
- ID:NA
|
- ID:NA
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user