starred/certbot-certbot
1
0
Fork 0
mirror of https://github.com/certbot/certbot.git synced 2026-08-04 12:21:51 +02:00
Code Issues Packages Projects Releases Wiki Activity

fix fix_test_non_systemd_os_info (#8539)

Browse Source
This commit is contained in:
Brad Warren
2020-12-21 13:31:37 -08:00
committed by GitHub
parent 8e7353900c
commit 421e8b6270
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
certbot/tests/util_test.py
+2 -2
View File
@@ -547,8 +547,7 @@ class OsInfoTest(unittest.TestCase):
m_distro.linux_distribution.return_value = ("something", "else") m_distro.linux_distribution.return_value = ("something", "else")
self.assertEqual(cbutil.get_os_info(), ("something", "else")) self.assertEqual(cbutil.get_os_info(), ("something", "else"))
@mock.patch("certbot.util.subprocess.Popen") def test_non_systemd_os_info(self):
def test_non_systemd_os_info(self, popen_mock):
import certbot.util as cbutil import certbot.util as cbutil
with mock.patch('certbot.util._USE_DISTRO', False): with mock.patch('certbot.util._USE_DISTRO', False):
with mock.patch('platform.system_alias', with mock.patch('platform.system_alias',
@@ -557,6 +556,7 @@ class OsInfoTest(unittest.TestCase):
with mock.patch('platform.system_alias', with mock.patch('platform.system_alias',
return_value=('darwin', '', '')): return_value=('darwin', '', '')):
with mock.patch("subprocess.Popen") as popen_mock:
comm_mock = mock.Mock() comm_mock = mock.Mock()
comm_attrs = {'communicate.return_value': comm_attrs = {'communicate.return_value':
('42.42.42', 'error')} ('42.42.42', 'error')}