From 386777598fc4f451218eae7e15d7d74614a8d082 Mon Sep 17 00:00:00 2001 From: Andrey Kurilin Date: Wed, 28 Feb 2018 09:18:10 +0200 Subject: [PATCH] Remove redundant mock We do not use method "rally.task.utils.check_service_status" from users@opoenstack context. Actually, it had not used from any code for quite a long time that is why it was removed from Rally code. Change-Id: Ie7d68f0d29b2e0c18546f1d586ca4be5753f8a11 --- tests/unit/contexts/keystone/test_users.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tests/unit/contexts/keystone/test_users.py b/tests/unit/contexts/keystone/test_users.py index 0f91a024..fcbe1c8f 100644 --- a/tests/unit/contexts/keystone/test_users.py +++ b/tests/unit/contexts/keystone/test_users.py @@ -292,11 +292,8 @@ class UserGeneratorForNewUsersTestCase(test.ScenarioTestCase): @mock.patch("rally.common.utils.iterate_per_tenants") @mock.patch("%s.network" % CTX) - @mock.patch("rally.task.utils.check_service_status", - return_value=False) def test__remove_default_security_group( - self, mock_check_service_status, mock_network, - mock_iterate_per_tenants): + self, mock_network, mock_iterate_per_tenants): net_wrapper = mock.Mock(SERVICE_IMPL=consts.Service.NEUTRON) net_wrapper.supports_extension.return_value = (True, None) mock_network.wrap.return_value = net_wrapper