octavia 14.0.0__py3-none-any.whl → 15.0.0.0rc1__py3-none-any.whl

octavia/tests/functional/api/v2/test_member.py

@@ -89,7 +89,7 @@ class TestMember(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -194,7 +194,7 @@ class TestMember(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -352,10 +352,10 @@ class TestMember(base.BaseAPITest):
         members = self.get(self.members_path, params={
             'fields': ['id', 'address']}).json
         for member in members['members']:
-            self.assertIn(u'id', member)
-            self.assertIn(u'address', member)
-            self.assertNotIn(u'name', member)
-            self.assertNotIn(u'monitor_address', member)
+            self.assertIn('id', member)
+            self.assertIn('address', member)
+            self.assertNotIn('name', member)
+            self.assertNotIn('monitor_address', member)
 
     def test_get_one_fields_filter(self):
         member1 = self.create_member(
@@ -365,10 +365,10 @@ class TestMember(base.BaseAPITest):
         member = self.get(
             self.member_path.format(member_id=member1.get('id')),
             params={'fields': ['id', 'address']}).json.get(self.root_tag)
-        self.assertIn(u'id', member)
-        self.assertIn(u'address', member)
-        self.assertNotIn(u'name', member)
-        self.assertNotIn(u'monitor_address', member)
+        self.assertIn('id', member)
+        self.assertIn('address', member)
+        self.assertNotIn('name', member)
+        self.assertNotIn('monitor_address', member)
 
     def test_get_all_filter(self):
         mem1 = self.create_member(self.pool_id,
@@ -529,7 +529,7 @@ class TestMember(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -591,7 +591,7 @@ class TestMember(base.BaseAPITest):
         self.set_object_status(self.pool_repo, pool1_id,
                                provisioning_status=constants.ERROR)
         api_member = self.create_member(pool1_id, '192.0.2.1', 80, status=409)
-        ref_msg = 'Pool %s is immutable and cannot be updated.' % pool1_id
+        ref_msg = f'Pool {pool1_id} is immutable and cannot be updated.'
         self.assertEqual(ref_msg, api_member.get('faultstring'))
 
     # TODO(rm_work) Remove after deprecation of project_id in POST (R series)
@@ -689,7 +689,7 @@ class TestMember(base.BaseAPITest):
         # Order matters here
         provider_creates += provider_updates
 
-        mock_provider.assert_called_once_with(u'noop_driver',
+        mock_provider.assert_called_once_with('noop_driver',
                                               mock_driver.member_batch_update,
                                               self.pool_id, provider_creates)
 
@@ -732,7 +732,7 @@ class TestMember(base.BaseAPITest):
             provider_dict['name'] = None
             provider_members.append(driver_dm.Member(**provider_dict))
 
-        mock_provider.assert_called_once_with(u'noop_driver',
+        mock_provider.assert_called_once_with('noop_driver',
                                               mock_driver.member_batch_update,
                                               self.pool_id, provider_members)
 
@@ -792,7 +792,7 @@ class TestMember(base.BaseAPITest):
         req_dict = [member1, member2, member5, member6]
         body = {self.root_tag_list: req_dict}
         path = self.MEMBERS_PATH.format(pool_id=self.pool_id)
-        path = "{}?additive_only=True".format(path)
+        path = f"{path}?additive_only=True"
         self.put(path, body, status=202)
         returned_members = self.get(
             self.MEMBERS_PATH.format(pool_id=self.pool_id)
@@ -834,7 +834,7 @@ class TestMember(base.BaseAPITest):
         provider_creates += provider_updates
         provider_creates += provider_ignored
 
-        mock_provider.assert_called_once_with(u'noop_driver',
+        mock_provider.assert_called_once_with('noop_driver',
                                               mock_driver.member_batch_update,
                                               self.pool_id, provider_creates)
 
@@ -885,7 +885,7 @@ class TestMember(base.BaseAPITest):
             del provider_dict['subnet_id']
             provider_members.append(driver_dm.Member(**provider_dict))
 
-        mock_provider.assert_called_once_with(u'noop_driver',
+        mock_provider.assert_called_once_with('noop_driver',
                                               mock_driver.member_batch_update,
                                               self.pool_id, provider_members)
 
@@ -913,6 +913,38 @@ class TestMember(base.BaseAPITest):
             m_subnet_exists.assert_called_once_with(
                 member1['subnet_id'], context=mock.ANY)
 
+    @mock.patch('octavia.api.drivers.driver_factory.get_driver')
+    @mock.patch('octavia.api.drivers.utils.call_provider')
+    def test_update_members_member_duplicate(
+            self, mock_provider, mock_get_driver):
+        mock_driver = mock.MagicMock()
+        mock_driver.name = 'noop_driver'
+        mock_get_driver.return_value = mock_driver
+        subnet_id = uuidutils.generate_uuid()
+
+        member1 = {'address': '192.0.2.1', 'protocol_port': 80,
+                   'project_id': self.project_id, 'subnet_id': subnet_id}
+
+        req_dict = [member1]
+        body = {self.root_tag_list: req_dict}
+        path = self.MEMBERS_PATH.format(pool_id=self.pool_id)
+        self.put(path, body, status=202)
+
+        self.set_lb_status(self.lb_id)
+
+        # Same member (same address and protocol_port) updated twice in the
+        # same PUT request
+        member1 = {'address': '192.0.2.1', 'protocol_port': 80,
+                   'project_id': self.project_id, 'subnet_id': subnet_id,
+                   'name': 'member1'}
+        member2 = {'address': '192.0.2.1', 'protocol_port': 80,
+                   'project_id': self.project_id, 'subnet_id': subnet_id,
+                   'name': 'member2'}
+
+        req_dict = [member1, member2]
+        body = {self.root_tag_list: req_dict}
+        self.put(path, body, status=400)
+
     @mock.patch('octavia.api.drivers.driver_factory.get_driver')
     @mock.patch('octavia.api.drivers.utils.call_provider')
     def test_update_members_subnet_not_found(
@@ -971,7 +1003,7 @@ class TestMember(base.BaseAPITest):
                  rm['protocol_port'],
                  rm['provisioning_status']), expected_members)
 
-        mock_provider.assert_called_once_with(u'noop_driver',
+        mock_provider.assert_called_once_with('noop_driver',
                                               mock_driver.member_batch_update,
                                               self.pool_id, provider_members)
 
@@ -1146,7 +1178,7 @@ class TestMember(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -1328,7 +1360,7 @@ class TestMember(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -1402,7 +1434,7 @@ class TestMember(base.BaseAPITest):
         member_path = self.MEMBERS_PATH.format(
             pool_id=bad_pool_id) + '/' + api_member['id']
         result = self.delete(member_path, status=404).json
-        ref_msg = 'Member %s not found.' % api_member['id']
+        ref_msg = f"Member {api_member['id']} not found."
         self.assertEqual(ref_msg, result.get('faultstring'))
 
     @mock.patch('octavia.api.drivers.utils.call_provider')

octavia/tests/functional/api/v2/test_pool.py

@@ -105,7 +105,7 @@ class TestPool(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -247,7 +247,7 @@ class TestPool(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -553,9 +553,9 @@ class TestPool(base.BaseAPITest):
         pools = self.get(self.POOLS_PATH, params={
             'fields': ['id', 'project_id']}).json
         for pool in pools['pools']:
-            self.assertIn(u'id', pool)
-            self.assertIn(u'project_id', pool)
-            self.assertNotIn(u'description', pool)
+            self.assertIn('id', pool)
+            self.assertIn('project_id', pool)
+            self.assertNotIn('description', pool)
 
     def test_get_one_fields_filter(self):
         pool1 = self.create_pool(
@@ -568,9 +568,9 @@ class TestPool(base.BaseAPITest):
         pool = self.get(
             self.POOL_PATH.format(pool_id=pool1.get('id')),
             params={'fields': ['id', 'project_id']}).json.get(self.root_tag)
-        self.assertIn(u'id', pool)
-        self.assertIn(u'project_id', pool)
-        self.assertNotIn(u'description', pool)
+        self.assertIn('id', pool)
+        self.assertIn('project_id', pool)
+        self.assertNotIn('description', pool)
 
     def test_get_all_filter(self):
         po1 = self.create_pool(
@@ -742,7 +742,7 @@ class TestPool(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -1274,7 +1274,7 @@ class TestPool(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -2102,7 +2102,7 @@ class TestPool(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,

octavia/tests/functional/api/v2/test_provider.py

@@ -33,10 +33,10 @@ class TestProvider(base.BaseAPITest):
         super().setUp()
 
     def test_get_all_providers(self):
-        octavia_dict = {u'description': u'Octavia driver.',
-                        u'name': u'octavia'}
-        amphora_dict = {u'description': u'Amp driver.', u'name': u'amphora'}
-        noop_dict = {u'description': u'NoOp driver.', u'name': u'noop_driver'}
+        octavia_dict = {'description': 'Octavia driver.',
+                        'name': 'octavia'}
+        amphora_dict = {'description': 'Amp driver.', 'name': 'amphora'}
+        noop_dict = {'description': 'NoOp driver.', 'name': 'noop_driver'}
         providers = self.get(self.PROVIDERS_PATH).json.get(self.root_tag_list)
         self.assertEqual(4, len(providers))
         self.assertIn(octavia_dict, providers)
@@ -44,9 +44,9 @@ class TestProvider(base.BaseAPITest):
         self.assertIn(noop_dict, providers)
 
     def test_get_all_providers_fields(self):
-        octavia_dict = {u'name': u'octavia'}
-        amphora_dict = {u'name': u'amphora'}
-        noop_dict = {u'name': u'noop_driver'}
+        octavia_dict = {'name': 'octavia'}
+        amphora_dict = {'name': 'amphora'}
+        noop_dict = {'name': 'noop_driver'}
         providers = self.get(self.PROVIDERS_PATH, params={'fields': ['name']})
         providers_list = providers.json.get(self.root_tag_list)
         self.assertEqual(4, len(providers_list))

octavia/tests/functional/api/v2/test_quotas.py

@@ -394,7 +394,7 @@ class TestQuotas(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -440,7 +440,7 @@ class TestQuotas(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_observer'],
+                'roles': ['load-balancer_observer', 'reader'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -480,18 +480,18 @@ class TestQuotas(base.BaseAPITest):
         self._assert_quotas_equal(quotas, quota2)
 
     def test_get_Authorized_member(self):
-        self._test_get_Authorized('load-balancer_member')
+        self._test_get_Authorized(['load-balancer_member', 'member'])
 
     def test_get_Authorized_observer(self):
-        self._test_get_Authorized('load-balancer_observer')
+        self._test_get_Authorized(['load-balancer_observer', 'reader'])
 
     def test_get_Authorized_global_observer(self):
-        self._test_get_Authorized('load-balancer_global_observer')
+        self._test_get_Authorized(['load-balancer_global_observer'])
 
     def test_get_Authorized_quota_admin(self):
-        self._test_get_Authorized('load-balancer_quota_admin')
+        self._test_get_Authorized(['load-balancer_quota_admin'])
 
-    def _test_get_Authorized(self, role):
+    def _test_get_Authorized(self, roles):
         project1_id = uuidutils.generate_uuid()
         quota1 = self.create_quota(
             project_id=project1_id, lb_quota=1, member_quota=1
@@ -509,7 +509,7 @@ class TestQuotas(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': [role],
+                'roles': roles,
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,
@@ -704,7 +704,7 @@ class TestQuotas(base.BaseAPITest):
                 'is_admin_project': True,
                 'service_project_domain_id': None,
                 'service_project_id': None,
-                'roles': ['load-balancer_member'],
+                'roles': ['load-balancer_member', 'member'],
                 'user_id': None,
                 'is_admin': False,
                 'service_user_domain_id': None,

octavia/tests/functional/db/test_models.py

@@ -24,7 +24,7 @@ from octavia.tests.functional.db import base
 from sqlalchemy.orm import collections
 
 
-class ModelTestMixin(object):
+class ModelTestMixin:
 
     FAKE_IP = '10.0.0.1'
     FAKE_UUID_1 = uuidutils.generate_uuid()

octavia/tests/functional/db/test_repositories.py

@@ -128,8 +128,8 @@ class AllRepositoriesTest(base.OctaviaDBTestBase):
                            'additional_vip')
         for repo_attr in repo_attr_names:
             single_repo = getattr(self.repos, repo_attr, None)
-            message = ("Class Repositories should have %s instance"
-                       " variable.") % repo_attr
+            message = (f"Class Repositories should have {repo_attr} instance "
+                       f"variable.")
             self.assertIsNotNone(single_repo, message=message)
             message = (("instance variable, %(repo_name)s, of class "
                         "Repositories should be an instance of %(base)s") %

octavia/tests/unit/amphorae/backends/agent/api_server/test_amphora_info.py

@@ -155,36 +155,36 @@ class TestAmphoraInfo(base.TestCase):
         m_count.return_value = 5
         original_version = api_server.VERSION
         api_server.VERSION = self.API_VERSION
-        expected_dict = {u'active': True,
+        expected_dict = {'active': True,
                          'active_tuned_profiles': '',
-                         u'api_version': self.API_VERSION,
-                         u'cpu': {u'soft_irq': u'8336',
-                                  u'system': u'52554',
-                                  u'total': 7503411,
-                                  u'user': u'252551'},
+                         'api_version': self.API_VERSION,
+                         'cpu': {'soft_irq': '8336',
+                                 'system': '52554',
+                                 'total': 7503411,
+                                 'user': '252551'},
                          'cpu_count': os.cpu_count(),
-                         u'disk': {u'available': 109079126016,
-                                   u'used': 25718685696},
-                         u'haproxy_count': 5,
-                         u'haproxy_version': self.HAPROXY_VERSION,
-                         u'hostname': u'FAKE_HOST',
-                         u'listeners': sorted([self.FAKE_LISTENER_ID_1,
-                                               self.FAKE_LISTENER_ID_2]),
-                         u'load': [u'0.09', u'0.11', u'0.10'],
-                         u'memory': {u'buffers': 344792,
-                                     u'cached': 4271856,
-                                     u'free': 12685624,
-                                     u'shared': 9520,
-                                     u'slab': 534384,
-                                     u'swap_used': 0,
-                                     u'total': 21692784},
-                         u'networks': {u'eth1': {u'network_rx': 996,
-                                                 u'network_tx': 418},
-                                       u'eth2': {u'network_rx': 848,
-                                                 u'network_tx': 578}},
-                         u'packages': {},
-                         u'topology': u'SINGLE',
-                         u'topology_status': u'OK'}
+                         'disk': {'available': 109079126016,
+                                  'used': 25718685696},
+                         'haproxy_count': 5,
+                         'haproxy_version': self.HAPROXY_VERSION,
+                         'hostname': 'FAKE_HOST',
+                         'listeners': sorted([self.FAKE_LISTENER_ID_1,
+                                              self.FAKE_LISTENER_ID_2]),
+                         'load': ['0.09', '0.11', '0.10'],
+                         'memory': {'buffers': 344792,
+                                    'cached': 4271856,
+                                    'free': 12685624,
+                                    'shared': 9520,
+                                    'slab': 534384,
+                                    'swap_used': 0,
+                                    'total': 21692784},
+                         'networks': {'eth1': {'network_rx': 996,
+                                               'network_tx': 418},
+                                      'eth2': {'network_rx': 848,
+                                               'network_tx': 578}},
+                         'packages': {},
+                         'topology': 'SINGLE',
+                         'topology_status': 'OK'}
         self.useFixture(test_utils.OpenFixture('/etc/tuned/active_profile'))
         actual = self.amp_info.compile_amphora_details()
         self.assertEqual(expected_dict, actual.json)
@@ -243,41 +243,40 @@ class TestAmphoraInfo(base.TestCase):
         mock_get_lb.return_value = [self.LB_ID_1]
         original_version = api_server.VERSION
         api_server.VERSION = self.API_VERSION
-        expected_dict = {u'active': True,
+        expected_dict = {'active': True,
                          'active_tuned_profiles': '',
-                         u'api_version': self.API_VERSION,
-                         u'cpu': {u'soft_irq': u'8336',
-                                  u'system': u'52554',
-                                  u'total': 7503411,
-                                  u'user': u'252551'},
+                         'api_version': self.API_VERSION,
+                         'cpu': {'soft_irq': '8336',
+                                 'system': '52554',
+                                 'total': 7503411,
+                                 'user': '252551'},
                          'cpu_count': os.cpu_count(),
-                         u'disk': {u'available': 109079126016,
-                                   u'used': 25718685696},
-                         u'haproxy_count': 5,
-                         u'haproxy_version': self.HAPROXY_VERSION,
-                         u'keepalived_version': self.KEEPALIVED_VERSION,
-                         u'ipvsadm_version': self.IPVSADM_VERSION,
-                         u'lvs_listener_process_count': 1,
-                         u'hostname': u'FAKE_HOST',
-                         u'listeners': sorted(list(set(
-                             [self.FAKE_LISTENER_ID_3,
-                              self.FAKE_LISTENER_ID_4,
-                              'sample_listener_id_1']))),
-                         u'load': [u'0.09', u'0.11', u'0.10'],
-                         u'memory': {u'buffers': 344792,
-                                     u'cached': 4271856,
-                                     u'free': 12685624,
-                                     u'shared': 9520,
-                                     u'slab': 534384,
-                                     u'swap_used': 0,
-                                     u'total': 21692784},
-                         u'networks': {u'eth1': {u'network_rx': 996,
-                                                 u'network_tx': 418},
-                                       u'eth2': {u'network_rx': 848,
-                                                 u'network_tx': 578}},
-                         u'packages': {},
-                         u'topology': u'SINGLE',
-                         u'topology_status': u'OK'}
+                         'disk': {'available': 109079126016,
+                                  'used': 25718685696},
+                         'haproxy_count': 5,
+                         'haproxy_version': self.HAPROXY_VERSION,
+                         'keepalived_version': self.KEEPALIVED_VERSION,
+                         'ipvsadm_version': self.IPVSADM_VERSION,
+                         'lvs_listener_process_count': 1,
+                         'hostname': 'FAKE_HOST',
+                         'listeners': sorted(list({self.FAKE_LISTENER_ID_3,
+                                                   self.FAKE_LISTENER_ID_4,
+                                                   'sample_listener_id_1'})),
+                         'load': ['0.09', '0.11', '0.10'],
+                         'memory': {'buffers': 344792,
+                                    'cached': 4271856,
+                                    'free': 12685624,
+                                    'shared': 9520,
+                                    'slab': 534384,
+                                    'swap_used': 0,
+                                    'total': 21692784},
+                         'networks': {'eth1': {'network_rx': 996,
+                                               'network_tx': 418},
+                                      'eth2': {'network_rx': 848,
+                                               'network_tx': 578}},
+                         'packages': {},
+                         'topology': 'SINGLE',
+                         'topology_status': 'OK'}
         self.useFixture(test_utils.OpenFixture('/etc/tuned/active_profile'))
         actual = self.amp_info.compile_amphora_details(self.lvs_driver)
         self.assertEqual(expected_dict, actual.json)

octavia/tests/unit/amphorae/backends/agent/api_server/test_keepalived.py

@@ -32,10 +32,10 @@ class KeepalivedTestCase(base.TestCase):
     @mock.patch('subprocess.check_output')
     def test_manager_keepalived_service(self, mock_check_output):
         res = self.test_keepalived.manager_keepalived_service('start')
-        cmd = ("/usr/sbin/service octavia-keepalived {action}".format(
-            action='start'))
+        cmd = 'systemctl start octavia-keepalived.service'
         mock_check_output.assert_called_once_with(cmd.split(),
-                                                  stderr=subprocess.STDOUT)
+                                                  stderr=subprocess.STDOUT,
+                                                  encoding='utf-8')
         self.assertEqual(202, res.status_code)
 
         res = self.test_keepalived.manager_keepalived_service('restart')

octavia/tests/unit/amphorae/backends/agent/api_server/test_keepalivedlvs.py

@@ -16,7 +16,6 @@ from unittest import mock
 from oslo_utils import uuidutils
 
 from octavia.amphorae.backends.agent.api_server import keepalivedlvs
-from octavia.amphorae.backends.agent.api_server import util
 from octavia.tests.unit import base
 
 
@@ -36,24 +35,7 @@ class KeepalivedLvsTestCase(base.TestCase):
             mock.call(
                 json=dict(message='UDP Listener Not Found',
                           details="No UDP listener with UUID: "
-                                  "{0}".format(self.FAKE_ID)), status=404),
+                                  "{}".format(self.FAKE_ID)), status=404),
             mock.call(json={'message': 'OK'})
         ]
         m_webob.Response.assert_has_calls(calls)
-
-    @mock.patch('octavia.amphorae.backends.agent.api_server.util.'
-                'get_os_init_system')
-    @mock.patch('octavia.amphorae.backends.agent.api_server.util.'
-                'get_keepalivedlvs_pid')
-    @mock.patch('subprocess.check_output')
-    @mock.patch('os.remove')
-    @mock.patch('os.path.exists')
-    def test_delete_lvs_listener_unsupported_sysinit(self, m_exist, m_remove,
-                                                     m_check_output, mget_pid,
-                                                     m_init_sys):
-        m_exist.return_value = True
-        mget_pid.return_value = '0'
-        self.assertRaises(
-            util.UnknownInitError,
-            self.test_keepalivedlvs.delete_lvs_listener,
-            self.FAKE_ID)